provider/{ec2,local}: use environschema #2597

Merged
merged 1 commit into from Jun 18, 2015

Conversation

Projects
None yet
2 participants
Owner

rogpeppe commented Jun 18, 2015

We add Schema methods to the providers to make the schema available
directly. In a future PR we will add the Schema method to the Provider
interface, but for the moment it's optional to avoid the need to change
all the providers at once.

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

Owner

rogpeppe commented Jun 18, 2015

$$merge$$

Contributor

jujubot commented Jun 18, 2015

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

Contributor

jujubot commented Jun 18, 2015

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/3719

Owner

rogpeppe commented Jun 18, 2015

$$merge$$

Contributor

jujubot commented Jun 18, 2015

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

jujubot added a commit that referenced this pull request Jun 18, 2015

Merge pull request #2597 from rogpeppe/030-provider-schema
provider/{ec2,local}: use environschema

We add Schema methods to the providers to make the schema available
directly. In a future PR we will add the Schema method to the Provider
interface, but for the moment it's optional to avoid the need to change
all the providers at once.


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

@jujubot jujubot merged commit 662232b into juju:master Jun 18, 2015

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