Feature/collapse app config #6205

Merged
merged 5 commits into from Sep 13, 2016

Conversation

Projects
None yet
3 participants
Contributor

reedobrien commented Sep 9, 2016

This is pretty much a forklift copy paste of the existing code with some modifications to make the UI analogous to the model-config and model-defaults commands.

Also, this is not ready to land until the LP issue referenced below is fixed.

Refs: http://pad.lv/1621576
Refs: spec https://goo.gl/yqrrPI

QA:

  1. Run all unit tests and they all pass.
  2. juju bootstrap reed/aws/us-west-1 --build-agent # bootstrap a controller with settings similar to http://paste.ubuntu.com/23080517/
  3. juju deploy haproxy && juju deploy wordpress && juju deploy mysql && juju add-relation haproxy wordpress && juju add-relation wordpress mysql && juju expose haproxy
  4. juju config wordpress # verify output is like: http://paste.ubuntu.com/23156363/
  5. juju config wordpress debug=yes
  6. juju config wordpress debug # verify it is yes
  7. juju config wordpress --reset debug
  8. save the contents of the mysql settings to a file with `juju config mysql --format=yaml > mysql.yaml
  9. It should look like http://paste.ubuntu.com/23156375/
  10. Edit file like http://paste.ubuntu.com/23156406/
  11. juju config mysql --file mysql.yaml ## verify the settings are there "ON" and 1.2345679e+08
  12. juju config mysql --reset query-cache-size query-cache-type
  13. juju config mysql query-cache-size=123456789 query-cache-type=DEMAND
  14. juju config mysql ## verify the settings
  15. juju config mysql query-cache-size ## verify a single setting -> 1.2345679e+08
  16. juju config mysql --reset query-cache-size # reset a single
  17. juju config mysql --reset query-cache-size # -> 0
    18 juju config mysql query-cache-type # -> DEMAND
  18. juju config mysql --reset query-cache-type
  19. juju config mysql --reset # -> error: no configuration options specified
  20. uju config mysql foo bar # -> error: can only retrieve a single value, or all values
  21. juju destroy-controller reed/aws/us-west-1 -y --destroy-all-models

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

Contributor

reedobrien commented Sep 13, 2016

$$merge$$

Contributor

jujubot commented Sep 13, 2016

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

Contributor

jujubot commented Sep 13, 2016

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

Contributor

reedobrien commented Sep 13, 2016

$$merge$$

Contributor

jujubot commented Sep 13, 2016

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

Contributor

jujubot commented Sep 13, 2016

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

Contributor

reedobrien commented Sep 13, 2016

$$seriously$$

Contributor

mjs commented Sep 13, 2016

$$merge$$

Contributor

jujubot commented Sep 13, 2016

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

@jujubot jujubot merged commit ca551b8 into juju:master Sep 13, 2016

@reedobrien reedobrien deleted the reedobrien:feature/collapse-app-config branch Sep 13, 2016

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