Bootstrap with (public-)clouds/credentials.yaml #4237

Merged
merged 2 commits into from Jan 29, 2016

Conversation

Projects
None yet
2 participants
Member

axw commented Jan 29, 2016

Update bootstrap to use clouds.yaml and credentials.yaml.
The primary difference from the spec is that we will do
an auto-detection of credentials if none are specified
and there is no default.

Full support for cloud credentials has been added to the
following providers so far:

  • dummy (for unit tests)
  • lxd
  • ec2
  • gce
  • openstack

(Review request: http://reviews.vapour.ws/r/3677/)

axw added some commits Jan 28, 2016

Bootstrap with (public-)clouds/credentials.yaml
Update bootstrap to use clouds.yaml and credentials.yaml.
The primary difference from the spec is that we will do
an auto-detection of credentials if none are specified
and there is no default.

Full support for cloud credentials has been added to the
following providers so far:
 - dummy (for unit tests)
 - lxd
 - ec2
 - gce
 - openstack
Address review comments
- Add cleanup TODOs
- Move some code to cloud package
Member

axw commented Jan 29, 2016

$$merge$$

Contributor

jujubot commented Jan 29, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

jujubot added a commit that referenced this pull request Jan 29, 2016

Merge pull request #4237 from axw/cloud-credentials-bootstrap2
Bootstrap with (public-)clouds/credentials.yaml

Update bootstrap to use clouds.yaml and credentials.yaml.
The primary difference from the spec is that we will do
an auto-detection of credentials if none are specified
and there is no default.

Full support for cloud credentials has been added to the
following providers so far:
 - dummy (for unit tests)
 - lxd
 - ec2
 - gce
 - openstack

(Review request: http://reviews.vapour.ws/r/3677/)

@jujubot jujubot merged commit b033024 into juju:cloud-credentials Jan 29, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment