summaryrefslogtreecommitdiff
path: root/heatclient
Commit message (Collapse)AuthorAgeFilesLines
* Set Content-Type header explictly for SessionClient1.16.2Rabi Mishra2019-03-182-2/+9
| | | | | | | | | | | SessionClient is subclassed from keystoneauth LegacyJsonAdapter. LegacyJsonAdapter does not set the Content-Type explicitly and results in decodeing issues in some cases. Change-Id: Idf23022b394607c332490331fc4b216de6ff1313 Story: 2005237 Task: 30027 (cherry picked from commit bd33e310e09339a21097c66d4022538f6bcbad2f)
* Merge "Enable some off-by-default checks"Zuul2018-07-201-2/+1
|\
| * Enable some off-by-default checksblue552017-06-221-2/+1
| | | | | | | | | | | | | | | | Some of the available checks are disabled by default, like: [H106] Don’t put vim configuration in source files [H203] Use assertIs(Not)None to check for None Change-Id: I815f189258356cb911d1e232b6f53c864e53c602
* | Merge "Add heat CLI test for stack-show"Zuul2018-07-132-0/+12
|\ \
| * | Add heat CLI test for stack-showGeorgy Dyuldin2018-03-262-0/+12
| | | | | | | | | | | | Change-Id: I35b17be14438e6de6fffd0396bab9989ef812a2a
* | | Merge "Fix log_http_request function in http module"Zuul2018-07-051-1/+4
|\ \ \
| * | | Fix log_http_request function in http moduleliyi2018-02-011-1/+4
| | | | | | | | | | | | | | | | | | | | Change-Id: I043f90541c8d2d604f3edb4e7ea082ec8adb4b25 Closes-Bug: #1746669
* | | | Merge "Update the documentation link for doc migration"Zuul2018-06-291-1/+1
|\ \ \ \
| * | | | Update the documentation link for doc migrationlingyongxu2018-06-091-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch is proposed according to the Direction 10 of doc migration. https://etherpad.openstack.org/p/doc-migration-tracking Co-Authored-By: Nguyen Hai <nguyentrihai93@gmail.com> Change-Id: I9675af92149ab126534f541534b50a02688e5337
* | | | | Handle token_endpoint auth type in osc pluginrabi2018-06-111-1/+3
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There can be situations where the auth_type=token_endpoint, we should use the appropriate attributes from the auth plugin when creating heatclient. Change-Id: Ic929f6ceaa92002eb2b3443934d1f541758e4dda Closes-Bug: #1632341
* | | | Mark stacks returned by get loadedThomas Herve2018-06-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When calling stacks.get, mark the returning stacks as loaded, so that attribute retrieval works right away. If not, any getattr on the Stack will call get() which will request the stack again (and ignore things like resolve_outputs). Change-Id: I3a3f25aa0c6f88e7e4274107c0578fd30dc9f155 Related-Bug: #1774958
* | | | Use assertRegex instead of assertRegexpMatchesVu Cong Tuan2018-04-111-17/+17
| |/ / |/| | | | | | | | | | | | | | | | | In Python3, assertRegexpMatches & assertNotRegexpMatches are deprecated in favor of assertRegex and assertNotRegex Change-Id: Ied56eda65800d72105ef2e2b683c6ce71cda6c4c
* | | Merge "Document osc CLI"Zuul2018-03-011-9/+10
|\ \ \
| * | | Document osc CLIZane Bitter2018-01-251-9/+10
| |/ / | | | | | | | | | | | | Change-Id: Idafe730021f272360ac0588f8b35e7d392c96d4b Closes-Bug: #1735019
* | | Merge "[policy in code] Change policy description"Zuul2018-02-071-1/+2
|\ \ \ | |/ / |/| |
| * | [policy in code] Change policy descriptionricolin2017-10-201-1/+2
| | | | | | | | | | | | | | | | | | | | | Now we also support get policy from policies in code. Depends-On: I610115dc1974b2182ce673bb086a1da15b022de3 Change-Id: I46d03c3303b3116e749effeecefe48949473daeb
* | | Merge "Add `--converge` argument for osc stack update"Zuul2017-12-012-0/+18
|\ \ \
| * | | Add `--converge` argument for osc stack updatericolin2017-10-202-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch add `--converge` argument for osc stack update. This flag will be used as the trigger for observe on reality. implements bp get-reality-for-resources depends-on: I151b575b714dcc9a5971a1573c126152ecd7ea93 Change-Id: If35e1b9234cab173b3c46601bbd331aceebcbcb3
* | | | Merge "Add plug-in summary for osc doc"Zuul2017-11-291-0/+2
|\ \ \ \
| * | | | Add plug-in summary for osc docKATO Tomoyuki2017-06-011-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Stevedore Sphinx extension handles this comment. http://docs.openstack.org/developer/python-openstackclient/plugin-commands.html Change-Id: I4958ec1dfda96f54b75963a04b5a4fc9eec09d84
* | | | | Don't preformat stack output show list/map valuesSteven Hardy2017-11-211-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This breaks the built-in --format json and --format yaml options Change-Id: I01d46ace380756ae57c8c7c9f65c5a686a06818f Closes-Bug: #1733664
* | | | | Allow cancelling create_in_progress stacks with --no-rollbackrabi2017-11-082-5/+22
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ib107c82f341f4d271859ca7681d65f7ce4c5d0b1 Related-Bug: #1709041
* | | | | Add --no-rollback option for stack cancelrabi2017-11-084-4/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds a new --no-rollback option for stack cancel. Usage: openstack stack cancel --no-rollback <stack_name> Change-Id: I46252823701779e3a3c6867bc470d5532cf9d7a7 Related-Bug: #1709041
* | | | | Modify error message encountered during stack updatePriyaDuggirala2017-10-272-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Parameters of an existing stack can be updated with openstack stack update <stack> --existing --parameter <parameter>=<newvalue> Example : openstack stack update stack1 --parameter p1=v1 If the --existing option is skipped, it leads to the below error Need to specify exactly one of --template-file, --template-url or --template-object This error is misleading as the user might think that stack update cannot be performed without specifying a new template. Modify the error message with --existing option. Change-Id: Idce88bde848378e00b1c873245600ba205939668 Closes-Bug: #1723864
* | | | | Merge "Use generic user for both zuul v2 and v3"1.12.0Zuul2017-10-231-3/+3
|\ \ \ \ \
| * | | | | Use generic user for both zuul v2 and v3Nam Nguyen Hoai2017-10-171-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Zuul v2 uses 'jenkins' as user, but Zuul v3 uses 'zuul'. Using $USER solves it for both cases. Change-Id: If6b023e3be4a4555b656660a8522271668496ead
* | | | | | Merge "No longer use oslotest.mockpatch"Zuul2017-10-171-2/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | No longer use oslotest.mockpatchThomas Bechtold2017-10-091-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is deprecated[1] and will be removed soon. [1] I3efb3c64ebce396e756a47d110a74fe8604b6b72 Closes-Bug: #1712017 Change-Id: If6406da4c940686cbda696f760d06d45690912a1
* | | | | | Don't override sections in deep_updateThomas Herve2017-10-052-0/+48
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When you comment all the elements in a YAML mapping, you end up with None instead of an empty mapping, which can have bad side effect when handling multiple environemnents. Let's handle that by ignoring the latest None value. Change-Id: I77ffabeb8d4cd2886ef4f41351e42ebe487b5d4b
* | | | | Support --show-nested in openstack stack update --dry-runRoberto Polli2017-09-282-0/+18
| |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | Added --show-nested support in `openstack stack update`. The work is based on `heat stack-update` implementation. Closes-Bug: #1720085 Change-Id: I50c8f5ac9ba4cb7e478af03ff4c1f099270ae226
* | | | Imported Translations from ZanataOpenStack Proposal Bot2017-08-233-75/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | For more information about this automatic import see: https://docs.openstack.org/i18n/latest/reviewing-translation-import.html Change-Id: I8109cffba95bedacc33d24cf8c10ddaf4eb85488
* | | | Merge "Decode content before checking"Jenkins2017-08-092-3/+3
|\ \ \ \
| * | | | Decode content before checkingrabi2017-07-262-3/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Decode before checking for py3 compatibility. Change-Id: Ic62ebc385146ab1406dd64384044d92be8171052 Related-Bug: #1695144
* | | | Fixing read before prompt bugJoe Borg2017-07-263-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Due to Python buffering sys.stdin and sys.stdout, the read can be shown before text prompt. Change-Id: I86d9ec62f7791da276585ce74a48162315effa7b Signed-off-by: Joe Borg <joe@josephb.org>
* | | | Merge "Add functinoal tests for 'openstack stack snapshot delete'"Jenkins2017-07-191-0/+9
|\ \ \ \ | |_|/ / |/| | |
| * | | Add functinoal tests for 'openstack stack snapshot delete'shizhihui2016-11-141-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For now, there are several heat osc CLIs which do not have own functional tests in heatclient/tests/functional/osc/v1. I think it is time to add some for heatclient. Add functional test for 'openstack stack snapshot delete' Change-Id: I21257b4f6126b9693256c953ed9335edad53a844 Implements: bp add-functional
* | | | Merge "Add optional arguments '-y' in CLI:snapshot-delete"Jenkins2017-07-052-2/+85
|\ \ \ \ | |_|_|/ |/| | |
| * | | Add optional arguments '-y' in CLI:snapshot-deletericolin2017-05-172-2/+85
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is no judgement before use the cli: heat snapshot-delete. So I add it. Partial-Bug: #1642490 Depends-On: If7b515dff64a18f56046b890279c2c59b0ab9dc7 Change-Id: I6e82630816e54aa2d98c3653ab43b865f445e881
* | | | Merge "Replace six.iteritems() with .items()"Jenkins2017-06-128-15/+11
|\ \ \ \
| * | | | Replace six.iteritems() with .items()M V P Nitesh2017-03-318-15/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1.As mentioned in [1], we should avoid using six.iteritems to achieve iterators. We can use dict.items instead, as it will return iterators in PY3 as well. And dict.items/keys will more readable. 2.In py2, the performance about list should be negligible, see the link [2]. [1] https://wiki.openstack.org/wiki/Python3 [2] http://lists.openstack.org/pipermail/openstack-dev/2015-June/066391.html Change-Id: I3913e8876b34e112140788d7a405da6eedfb5f29
* | | | | Merge "Don't encode path separators"1.10.0Jenkins2017-06-058-48/+48
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Don't encode path separatorsrabi2017-05-098-48/+48
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Encoding the path separators results in urls refused by by apache with 404 unless AllowEncodedSlashes is turned on, which is not the default. The existing encoding of path separators seems unnecessary. Change-Id: I3b77aafc6104119503363e5db36b0379964daba5
* | | | Merge "Replace mox3 with mock"Jenkins2017-05-271-278/+230
|\ \ \ \
| * | | | Replace mox3 with mockPanFengyun2016-12-201-278/+230
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In heatclient, some unit tests still use mox3, which will make the test speed much slow, so I decide to remove it. The file: heatclient/tests/unit/test_template_utils.py Implements: bp remove-mox3-in-heatclient Change-Id: I1aa556630ff88b4708834549ce9dc31967e0bb32 Co-author-by: PanFengyun <fengyun.pan@easystack.cn>
* | | | Merge "Pass event filters to the server side"Jenkins2017-05-192-9/+23
|\ \ \ \ | |_|_|/ |/| | |
| * | | Pass event filters to the server sideThomas Herve2017-03-282-9/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This passes event filters to the server side. It only matters for event_time with limit now, but this fixes that case, and the case for resource_type as well. Change-Id: Ide8f0481cf031cc61d4bb43d1d92109d42326e95
* | | | Merge "Add '--yes' for openstack stack snapshot delete"Jenkins2017-05-172-1/+51
|\ \ \ \ | |_|_|/ |/| | |
| * | | Add '--yes' for openstack stack snapshot deletericolin2017-04-262-1/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add optional arguments '--yes' or '-y' in osc cli `openstack stack snapshot delete`. There is no judgement before use that cli So propose to add it in case that we delete the snapshot unintentionally. Closes-Bug: #1642874 Change-Id: If7b515dff64a18f56046b890279c2c59b0ab9dc7
* | | | Build event stack_name from linksThomas Herve2017-03-282-3/+14
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | When retrieving events from nested stacks, we used to do a query per stack and thus get the stack that way. We changed it to do only one API call, but didn't fix the way the stack name was set. We need to use the event links to retrieve the stack name, instead of the root stack. Closes-Bug: #1676896 Change-Id: I09ab1cc2c6aaab02036b124b1dfa561dd6132083
* | | Merge "Fix error message when deployment not found"Jenkins2017-03-271-1/+3
|\ \ \