environs/config: derive schema from table of all attributes #2487

Merged
merged 1 commit into from Jun 15, 2015

Conversation

Projects
None yet
2 participants
Owner

rogpeppe commented Jun 3, 2015

This is the first step on the way to having environment config information
that can be introspected. In this PR we just make the table and leave
all other semantics unchanged, leaving that for a subsequent PR.

Note that some of the error messages have changed because the new
scheme makes it natural to specify sets of possible values in the schema.

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

Owner

rogpeppe commented Jun 15, 2015

$$merge$$

Contributor

jujubot commented Jun 15, 2015

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

Contributor

jujubot commented Jun 15, 2015

Build failed: Generating tarball failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/3651

Owner

rogpeppe commented Jun 15, 2015

$$merge$$

Contributor

jujubot commented Jun 15, 2015

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

Contributor

jujubot commented Jun 15, 2015

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

Owner

rogpeppe commented Jun 15, 2015

$$merge$$

Contributor

jujubot commented Jun 15, 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 15, 2015

Merge pull request #2487 from rogpeppe/027-environs-config-table
environs/config: derive schema from table of all attributes

This is the first step on the way to having environment config information
that can be introspected. In this PR we just make the table and leave
all other semantics unchanged, leaving that for a subsequent PR.

Note that some of the error messages have changed because the new
scheme makes it natural to specify sets of possible values in the schema.


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

@jujubot jujubot merged commit 7986d4e into juju:master Jun 15, 2015

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