summaryrefslogtreecommitdiff
path: root/heatclient
Commit message (Collapse)AuthorAgeFilesLines
* Remove white space between print and ()yuyafei2016-07-051-1/+1
| | | | | | TrivialFix Change-Id: Iea9913ec7934ec0879a724ad3f90d52fab8bca48
* Merge "base.Resource not define __ne__() built-in function"Jenkins2016-07-051-0/+3
|\
| * base.Resource not define __ne__() built-in functionyuyafei2016-07-041-0/+3
| | | | | | | | | | | | | | | | | | | | | | Class base.Resource defines __eq__() built-in function, but does not define __ne__() built-in function, so self.assertEqual works but self.assertNotEqual does not work at all in this test case in python2. This patch fixes it by defining __eq__() built-in function of class base.Resource. Change-Id: I13ed9ddd6ae6ef074232b33695916bc83930dcf0 Closes-Bug: #1586268
* | Merge "Show 'deletion_time' of stacks"Jenkins2016-07-053-3/+23
|\ \
| * | Show 'deletion_time' of stackshuangtianhua2016-06-253-3/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | Add 'deletion_time' column in following cases: 1. if specify --show-deleted when list stack 2. show stack Change-Id: I4348d03b3433c4962f51aa124ea27dc0c55c71c1 DependsOn: I8c55fe7f6a899ca66aa3fef15c93195c03c5aa11
* | | Merge "Fix typo"Jenkins2016-07-011-1/+1
|\ \ \
| * | | Fix typoPablo Iranzo Gómez2016-06-301-1/+1
| | | | | | | | | | | | | | | | Change-Id: I1614916518c20be9842a7cb7a60f996467b6e398
* | | | Pass correct arguments in osc pluginrabi2016-06-301-8/+11
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | Pass session or auth_token based on whether using a session or not. SessionClient can't accept auth_token in kwargs. Change-Id: Ib30aeac0ec0a12faf1ae874b0a97e936f12cd61a Closes-Bug: #1597691
* | | Use osc_lib instead of cliffTang Chen2016-06-2912-47/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | Base classes of commands are defined in cliff, but have been encapsulated again in osc-lib for all plugin clients. So use osc-lib instead of cliff. Change-Id: I0e430457af3033ebf56e47f85c0cfd71d2227cf7
* | | Use osc-lib instead of openstackclientTang Chen2016-06-2917-27/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | osc-lib is a package of common support modules for writing OSC plugins. All common functions, classes such as exceptions, utils, logs and so on have been moved from openstackclient to osc-lib. So use osc-lib instead of openstackclient. Change-Id: I14323d43d0e5b41ce0611590f5f6a375cdc3dc2d
* | | Imported Translations from ZanataOpenStack Proposal Bot2016-06-283-0/+75
| |/ |/| | | | | | | | | | | For more information about this automatic import see: https://wiki.openstack.org/wiki/Translations/Infrastructure Change-Id: I5a7ecb69e9624d37b88da7b7e5e53f8a9906cd0b
* | Replace dict.iteritems with six.iteritems to make PY3 compatibleyuyafei2016-06-251-2/+2
|/ | | | | | | | Python3 do not use dict.iteritems, which would raise AttributeError: 'dict' object has no attribute 'iteritems'. Change-Id: I75f5954ce0d5573f69a22799dacb74b918028115 Closes-Bug: #1596124
* Merge "Implement "openstack stack failures list""Jenkins2016-06-243-0/+399
|\
| * Implement "openstack stack failures list"Steve Baker2016-05-273-0/+399
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A command which prints failed resources in the stack to help debug stack issues. Failed resources are added by recursing from the top level stack into failed nested stack resources. A failed nested stack resource is only added to the failed list if it contains no failed resources so that the user only sees root-cause failed resources. As an added convenience to debugging deployment resources, deployment outputs deploy_stdout and deploy_stderr are printed as well (the full outputs are printed when --long is specified). The output is a yamlish format where the key is a dot-delimited resource name path and the value is a dict of resource information. Change-Id: Ifa44175d8dbab2f7b62691b839e1696e3af7f4f3 Closes-Bug: #1585820
* | Merge "Refactor test_shell"Jenkins2016-06-172-1681/+500
|\ \
| * | Refactor test_shellSteve Baker2016-06-102-1681/+500
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This file has become difficult to maintain with conditionals everywhere to handle the different http clients. This change replaces the http mocking code with calls to a handful of mock methods, and deletes what is left unused. Change-Id: Id2339b63a90f4805afe94f89f4c2715834588c61
* | | Merge "Serialize files when using SessionClient"Jenkins2016-06-092-5/+28
|\ \ \
| * | | Serialize files when using SessionClientrabi2016-06-082-5/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SessionClient can not json serialize TemplateFiles object. We should do them explictly. Change-Id: Ia67562ae9a13c0975cb5c39532c0dc413d82c4c3 Closes-Bug: #1589519
* | | | Merge "A resource list formatter for graphviz dot diagrams"Jenkins2016-06-064-0/+461
|\ \ \ \ | |_|/ / |/| | |
| * | | A resource list formatter for graphviz dot diagramsSteve Baker2016-05-314-0/+461
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change registers a "dot" formatter for the stack resource list command. It generates the text for a graphviz dot diagram which can be rendered as an image with this example usage: openstack stack resource list --format dot -n3 my_stack | dot -Tsvg -o my_stack.svg Nested resources are fully supported by creating nodes for top-level resources then linking stack resources to a subgraph representing the nested stack. Change-Id: I4b899287eaf818137d60cb278db2d86598aa6794
* | | Merge "Show 'parameter_defaults' for environment-show"Jenkins2016-05-312-29/+38
|\ \ \
| * | | Show 'parameter_defaults' for environment-showhuangtianhua2016-05-312-29/+38
| | |/ | |/| | | | | | | | | | | | | Closes-Bug: #1586884 Change-Id: If05100b9c7efd0570b28d8d80419c05de0c2d16b
* | | Merge "osc resource-type-show description option"Jenkins2016-05-303-5/+32
|\ \ \
| * | | osc resource-type-show description optionPeter Razumovsky2016-05-203-5/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add new option to osc resource-type-show, which allows to print description of specified resource type. Change-Id: I99347a1aa01cea3fe7e9686e06c7d8180e6551b1 Depends-on: #I148bb7764af4db45b75ce63d7fa58daa210b3ed4 Closes-bug: #1558049
* | | | Merge "osc resource-type-list description option"Jenkins2016-05-303-12/+56
|\ \ \ \ | |/ / / | | / / | |/ / |/| |
| * | osc resource-type-list description optionPeter Razumovsky2016-05-203-12/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | Add new option to osc resource-type-list, which allows to print description of each resource type. Change-Id: I34ca4531fb0c9f4896f55011ae234771e9ef965f Depends-on: Ia92699e5ca60a891e9704c468e447e1053be7609 Closes-bug: #1558049
* | | Clean up osc fakes and mocksSteve Baker2016-05-2612-387/+107
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | The original fakes included a lot of unneeded pasted code from openstackclient. This change replaces the client with a single MagicMock, and deletes unused fakes classes. Since the base client is a magic mock, a significant amount of mock object creation has been replaced with the magic equivalent. This change actually revealed a faulty test, test_deployment_config_delete_failed should be asserting that deleting the derived config is best-effort and does not raise an error. Change-Id: I11be97603b2d42ea70dde7d936e11d15e7f5709e
* | Fix typoThomas Herve2016-05-232-2/+2
| | | | | | | | | | | | Fix typo WRT physical_resource_id in help messages. Change-Id: Ica89b13cc9cd2b355cf65c7ccb329518e5c745bd
* | Merge "Register a log formatter for stack event list"Jenkins2016-05-232-11/+25
|\ \ | |/ |/|
| * Register a log formatter for stack event listSteve Baker2016-03-012-11/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently "stack event list" overrides the value formatter to use for the default log formatting style. This change creates a formatter namespace for event list so that it can have its own custom list of formatters. This means that the 'log' formatter is now registered and is the default. The formatter expects event objects rather than dicts, and so ignores the columns argument. The intention is that this will be the first of more custom formatters for event list, making the effort to establish this pattern worth it. Change-Id: I22f884f871d7467c8abadec477b3e732eb8c2150
* | Merge "Allow redirects to use location from response"Jenkins2016-05-203-33/+16
|\ \
| * | Allow redirects to use location from responserabi2016-05-173-33/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | This allows redirects to use fully qualified location from response header and removes the check to prohibit redirects to different project(required for super admin actions). Change-Id: I46876189e96375922c4f0bf30452b7c4980fe39c
* | | Remove redundant auth discoveryrabi2016-05-181-114/+23
| | | | | | | | | | | | | | | | | | | | | This removes the redundant auth discovery and use generic auth plugins. Change-Id: Ibf96a188363467e74879018e31c0d8729c6cdeee
* | | switch to keystoneauthSteve Martinelli2016-05-184-13/+10
|/ / | | | | | | | | | | | | | | | | move heatclient to keystoneauth as keystoneclient's auth session, plugins and adapter code has been deprecated [1]. [1] http://docs.openstack.org/releasenotes/python-keystoneclient/mitaka.html#deprecation-notes Change-Id: Iedd7ceb97d39ab3ecccf8786aad8b5f45d15175e
* | Merge "Moved required parameter for visibility"Jenkins2016-05-113-18/+18
|\ \
| * | Moved required parameter for visibilityxiaolihope2016-04-263-18/+18
| | | | | | | | | | | | | | | | | | | | | This change moved the required parameters so that it is more easily visible to the user what is required. Change-Id: I888960d2c83c96af80414c61dfd5faf38478f3fa
* | | Merge "use thread safe fnmatch"Jenkins2016-05-111-1/+2
|\ \ \
| * | | use thread safe fnmatchZhiQiang Fan2016-04-211-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fnmatch is not thread safe in python <2.7.10, let's use the safe one in oslo.utils Change-Id: I489271f8fca94b1a1cb3ad30790042fe7468338c ref: https://hg.python.org/cpython/rev/fe12c34c39eb
* | | | Merge "Fix obscure error message when no template given to OSC"Jenkins2016-05-102-5/+28
|\ \ \ \
| * | | | Fix obscure error message when no template given to OSCPaul Breaux2016-05-012-5/+28
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is an internal-only fix, as it does not change any external APIs. A bit of obfuscation was occurring in the 'template_utils.process_template_path' function call. First we try to retrieve the template from a urllib call. If that fails, then we try to make a request with the custom request_object function. If the request_object function failed to retrieve the template as well, it would report back 'ERROR: None'. I left both of the calls in this function, but I made the first exception bubble up back to the caller, since it is more helpful to the user. The new error message should report back the following when a template cannot be found: <urlopen error [Errno 2] No such file or directory: '/root/python-heatclient/test.yaml'> Closed Bug: #1570229 Change-Id: I29e19172322c18559aa476744f7ff3530e689bd0
* | | | Merge "Fixed fragile assertion in test_template_utils"Jenkins2016-05-041-2/+3
|\ \ \ \
| * | | | Fixed fragile assertion in test_template_utilsJay Dobies2016-04-141-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | An assertion was added that compared the JSON dumped environment against the string version of an environment file that was read in. The problem is that there is no guarantee that dumped string will retain the same order as the original string. This was seen in a failing CI run on an unrelated change [1], manifesting itself as: 2016-03-30 20:12:57.278 | File "/home/jenkins/workspace/gate-python-heatclient-python34/heatclient/tests/unit/test_template_utils.py", line 413, in test_process_multiple_environments_and_files_tracker 2016-03-30 20:12:57.278 | files['file:///home/my/dir/env1.yaml']) 2016-03-30 20:12:57.278 | File "/home/jenkins/workspace/gate-python-heatclient-python34/.tox/py34/lib/python3.4/site-packages/testtools/testcase.py", line 362, in assertEqual 2016-03-30 20:12:57.278 | self.assertThat(observed, matcher, message) 2016-03-30 20:12:57.279 | File "/home/jenkins/workspace/gate-python-heatclient-python34/.tox/py34/lib/python3.4/site-packages/testtools/testcase.py", line 447, in assertThat 2016-03-30 20:12:57.279 | raise mismatch_error 2016-03-30 20:12:57.279 | testtools.matchers._impl.MismatchError: !=: 2016-03-30 20:12:57.279 | reference = '{"resource_registry": {"OS::Thingy1": "file:///home/b/a.yaml"}, "parameters": {"param1": "value1"}}' 2016-03-30 20:12:57.279 | actual = '{"parameters": {"param1": "value1"}, "resource_registry": {"OS::Thingy1": "file:///home/b/a.yaml"}}' This patch changes the assertion to parse the files version of the environment into a dict for the comparison. [1] http://logs.openstack.org/34/270934/7/check/gate-python-heatclient-python34/deed567/console.html Change-Id: I23fa0fddc9de933587e3b66c579dfd31ea49a2bd
* | | | | Merge "Replace tempest-lib with tempest.lib"Jenkins2016-05-035-6/+6
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Replace tempest-lib with tempest.libDaniel Gonzalez2016-04-285-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tempest-lib is deprecated, replace it with tempest.lib. Closes-Bug: #1553047 Change-Id: Icc17203a83513a1770dc845978550ff856dd8594
* | | | | OpenStack client plugin for environment showJay Dobies2016-04-182-0/+93
| |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | Adds the "openstack stack environment show" command. Depends-On: I7e3577dfc854018245d79afdfee45a9d250d73a7 Change-Id: I7ef4858738964198bc4614122be4bab8f49b1b5f Implements: blueprint environment-show
* | | | Add client library support for retrieving a stack's environmentJay Dobies2016-04-182-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I84aa5ed9dc2b8aee089de3b1e1810552b8f2dd96 Depends-On: I7e3577dfc854018245d79afdfee45a9d250d73a7 Implements: blueprint environment-show
* | | | Support "--pre-delete" argument for hook-clearhuangtianhua2016-04-167-5/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changes: 1. Add "--pre-delete" argument for cmd "heat hook-clear" to clear pre-delete hooks. 2. Add "--pre-delete" argument for osc cmd "openstack stack hook clear" to clear pre-delete hooks. 3. Support to poll pre-delete hooks for "heat hook-poll" and "openstack stack hook poll". Change-Id: Ic5a6e1b7588e3f9ad49eaaf8085f18ec74fc629e Closes-Bug: #1567814
* | | | Merge "Moved -t parameter for visibility"Jenkins2016-04-151-6/+6
|\ \ \ \
| * | | | Moved -t parameter for visibilityJay Dobies2016-03-291-6/+6
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When called without the required arguments, the allowed arguments are output to the screen. The order of those arguments is dictated by when they are added to the parser. Before this change, -t (a required parameter) was at the start of the list. This obfuscated the required parameter list when the help output was displayed: usage: openstack stack create [-h] [-f {html,json,json,shell,table,value,yaml,yaml}] [-c COLUMN] [--max-width <integer>] [--noindent] [--prefix PREFIX] -t <template> [-e <environment>] [--timeout <timeout>] [--pre-create <resource>] [--enable-rollback] [--parameter <key=value>] [--parameter-file <key=file>] [--wait] [--tags <tag1,tag2...>] [--dry-run] <stack-name> Notice how the -t appears in the middle of the optional arguments. This change moves it to the end so that it is more easily visible to the user what is required: usage: openstack stack create [-h] [-f {html,json,json,shell,table,value,yaml,yaml}] [-c COLUMN] [--max-width <integer>] [--noindent] [--prefix PREFIX] [-e <environment>] [--timeout <timeout>] [--pre-create <resource>] [--enable-rollback] [--parameter <key=value>] [--parameter-file <key=file>] [--wait] [--tags <tag1,tag2...>] [--dry-run] -t <template> <stack-name> Change-Id: I4e17650622dc8e206dc1aa39e4102f16429ac353
* | | | Merge "Add basic read-only function test for template validate"Jenkins2016-04-141-0/+20
|\ \ \ \
View Lorry Controller Status