~rogpeppe/juju-core/438-local-instance-Addresses

Viewing all changes in revision 285.

  • Committer: William Reade
  • Author(s): William Reade
  • Date: 2012-07-05 23:12:11 UTC
  • mfrom: (284.1.2 juju-core)
  • Revision ID: fwereade@gmail.com-20120705231211-n2fj0du3r9yk4cui
more forgiving parsing of environments.yaml

The only errors that should now occur when environments.yaml is first read
are (1) invalid yaml and (2) a "default" environment that doesn't exist. All
other errors are deferred until the erroneous environ is loaded, at which
point they are returned.

This enables a small but IMO significant cleanup in the ec2 environ: the
providerConfig no longer has an authorizedKeysPath field, because this is
not in fact part of the environment's configuration at all: it's a distinct
piece of information that assists in the contruction of an environment
config in a certain context, and even allowing it to exist on the server
side is profoundly misleading.

PS why is it called providerConfig?

R=niemeyer
CC=
https://codereview.appspot.com/6354077

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: