Merge lp:~thomir-deactivatedaccount/adt-continuous-deployer/trunk-fix-warning-message into lp:adt-continuous-deployer

Proposed by Thomi Richards
Status: Merged
Approved by: Thomi Richards
Approved revision: 62
Merged at revision: 62
Proposed branch: lp:~thomir-deactivatedaccount/adt-continuous-deployer/trunk-fix-warning-message
Merge into: lp:adt-continuous-deployer
Diff against target: 42 lines (+16/-16)
1 file modified
mojo.py (+16/-16)
To merge this branch: bzr merge lp:~thomir-deactivatedaccount/adt-continuous-deployer/trunk-fix-warning-message
Reviewer Review Type Date Requested Status
Thomi Richards (community) Approve
Review via email: mp+261332@code.launchpad.net

Commit message

Fix warning message about using both config-on-disk and config-from-cli.

Description of the change

Fix a bug where the warning message about using config on disk and supplying config arguments was skipping any config argument with a '-' in the name.

To post a comment you must log in.
Revision history for this message
Thomi Richards (thomir-deactivatedaccount) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'mojo.py'
2--- mojo.py 2015-06-02 02:36:49 +0000
3+++ mojo.py 2015-06-07 20:43:31 +0000
4@@ -166,22 +166,22 @@
5 # config on the command line, as they might get confused otherwise.
6 deprecated_args = (
7 'network',
8- 'amqp-uris',
9- 'logstash-host',
10- 'swift-public-url',
11- 'image-mapper-url',
12- 'os-username',
13- 'os-tenant-name',
14- 'os-password',
15- 'os-auth-url',
16- 'os-region-name',
17- 'adt-os-username',
18- 'adt-os-tenant-name',
19- 'adt-os-password',
20- 'adt-os-auth-url',
21- 'adt-os-region-name',
22- 'adt-extra-args',
23- 'adt-http-proxy',
24+ 'amqp_uris',
25+ 'logstash_host',
26+ 'swift_public_url',
27+ 'image_mapper_url',
28+ 'os_username',
29+ 'os_tenant_name',
30+ 'os_password',
31+ 'os_auth_url',
32+ 'os_region_name',
33+ 'adt_os_username',
34+ 'adt_os_tenant_name',
35+ 'adt_os_password',
36+ 'adt_os_auth_url',
37+ 'adt_os_region_name',
38+ 'adt_extra_args',
39+ 'adt_http_proxy',
40 )
41 used_deprepcated = [a for a in deprecated_args if a in args]
42 if used_deprepcated:

Subscribers

People subscribed via source and target branches

to all changes: