HomePhorge

(PUP-3244) Allow autoload to handle missing environment
9b46778119abUnpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

(PUP-3244) Allow autoload to handle missing environment

When a previous commit changed from get() to get!() the agent stopped
being able to use an environment that is available on the master, but
not available locally. I had misread the code and missed a trailing
&&. This caused me to think that the environment lookup in this case
was required, where in fact it is still optional.

This changes the structure of the code to make it much clearer that the
environment is optional, changes from get!() to get(), and adds some
tests around these code paths.

Details

Provenance
Andrew Parker <andy@puppetlabs.com>Authored on
vanmeeuwenPushed on Jun 2 2015, 2:22 PM
Parents
rPUe84edcbdfbf0: Merge pull request #3182 from demophoon/fix/stable/pup-3244
Branches
Unknown
Tags
Unknown

Event Timeline

Andrew Parker <andy@puppetlabs.com> committed rPU9b46778119ab: (PUP-3244) Allow autoload to handle missing environment (authored by Andrew Parker <andy@puppetlabs.com>).Oct 13 2014, 8:51 PM