juju-ci-tools

Merge lp://staging/~sinzui/juju-ci-tools/model-config into lp://staging/juju-ci-tools

  1. model-config
  2. Merge into trunk
Proposed by Curtis Hovey
Status: Merged
Merged at revision: 1584
Proposed branch: lp://staging/~sinzui/juju-ci-tools/model-config
Merge into: lp://staging/juju-ci-tools
Diff against target: 281 lines (+92/-30)
6 files modified
assess_log_forward.py (+1/-3)
assess_user_grant_revoke.py (+5/-7)
fakejuju.py (+2/-2)
jujupy.py (+23/-5)
tests/test_deploy_stack.py (+6/-6)
tests/test_jujupy.py (+55/-7)
To merge this branch: bzr merge lp://staging/~sinzui/juju-ci-tools/model-config
Reviewer Review Type Date Requested Status
Seman (community) Approve
Review via email: mp+305156@code.staging.launchpad.net

Description of the change

This branch updates ci scripts to use model-config instead of set-/get-/unset-.

Commit https://github.com/juju/juju/commit/164ca7fed7ad4b2f7d16161fc19818148badde8e
renamed the get/set/unset model-config command. they are now one command where intent is understood by the arguments.

Get all options
    get-model-config -m mycontroller:mymodel => model-config -m mycontroller:mymodel

Get one option:
    get-model-config default-series => model-config default-series

Set options:
    set-model-config ftp-proxy=10.0.0.1:8000 => model-config ftp-proxy=10.0.0.1:8000

Unset options to their default
    unset-model-config default-series => model-config --reset default-series

To post a comment you must log in.
Revision history for this message
Seman (sseman) wrote :

Thank you.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches