summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Merge "[microversions] Add support for 2.15"Jenkins2016-01-213-2/+3
|\ \ \ \ \ | |/ / / /
| * | | | [microversions] Add support for 2.15Balazs Gibizer2016-01-123-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With I3f156d5e5df4d9642bb4b0ffac30a6288459ce61 nova introduced microversion 2.15 which adds two new policies soft-affinty and soft-anti-affinity for the server-group api. This patch bumps the nova client microversion support to 2.15. The novaclient is transparent regarding the policy value of the server-group api so no further changes are needed. However the help text of the policies parameter mentioned the possible policies. This was removed as the current framework does not support providing different help text for a parameter depending on the microversion. Implements: blueprint soft-affinity-for-server-group Change-Id: I739ed1dd3e4c15e28a269c4f980a12a74fb1def0
* | | | | Merge "[microversions] Add support for 2.14"Jenkins2016-01-215-9/+56
|\ \ \ \ \ | |/ / / /
| * | | | [microversions] Add support for 2.14Balazs Gibizer2015-12-315-9/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In I54bfa1275e188573c1b95d770d89160a86cdf52c the onSharedStorage flag is removed from the evacuate API. This patch removes it from the novaclient as well. Implements: bp remove-shared-storage-flag-in-evacuate-api Change-Id: I5ae75fdac226f0246f22a4d5245c1e4952571fc1
* | | | | Merge "Fix URLs for CLI Reference and API"Jenkins2016-01-212-3/+3
|\ \ \ \ \
| * | | | | Fix URLs for CLI Reference and APIAndreas Jaeger2016-01-162-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The CLI Reference files has been moved, update the link. Replace link to API Quick Start with link to API reference for nova. Change-Id: Ic933d44cddc24702648c69498e632efefda7bf88
* | | | | | Merge "Functional tests for flavors with public and non-public access"Jenkins2016-01-214-29/+139
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | Functional tests for flavors with public and non-public accessAnna Babich2016-01-194-29/+139
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The added tests check granting access actions for public and non-public flavors The test for microversion 2.7 checks that the appropriate error is returned when attempting to grant an access to a public flavor for the given tenant The base TenantTest class has been added to avoid code duplication in test_keypairs and test_flavor_access tests Also, test_keypairs code has been modified in accordance with TenantTestBase class usage Change-Id: Icf21d8aceaa191aee8b8ffc7c52e97ebe9509b3d
* | | | | | Updated from global requirementsOpenStack Proposal Bot2016-01-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I552f35171e15991a1f57c0d09fa20fe5042cc8e7
* | | | | | Updated from global requirementsOpenStack Proposal Bot2016-01-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Id51d3606a6d29f909e8c40edfeed0505d8144250
* | | | | | Updated from global requirementsOpenStack Proposal Bot2016-01-172-21/+21
| |/ / / / |/| | | | | | | | | | | | | | Change-Id: I38b118679fae02c3e37a8f145b28dd7340eb1bc7
* | | | | Merge "Add functional tests launcher for py3 env"Jenkins2016-01-163-2/+9
|\ \ \ \ \
| * | | | | Add functional tests launcher for py3 envAndrey Kurilin2016-01-153-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It would be nice to check not only unit test on py3 env, but functional tests too. Also, this patch fixes usage of xrange in functional tests. Change-Id: I15ef204022583a40a02b3f2a48771347ddc95f3e
* | | | | | Merge "[microversions] Add support for API microversion 2.13"Jenkins2016-01-134-5/+115
|\ \ \ \ \ \ | | |_|/ / / | |/| | | |
| * | | | | [microversions] Add support for API microversion 2.13Kevin_Zheng2015-12-234-5/+115
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Nova now supports API microversion 2.13 to allow return project_id and user_id for os-server-groups API. Sync this to the client side. Change-Id: Ia09ab4bd5c693ed95b0f5dd9bc709b7597f7034e Closes-Bug: #1526143
* | | | | | Merge "Test: Clean v2 client userwarning"Jenkins2016-01-137-45/+55
|\ \ \ \ \ \
| * | | | | | Test: Clean v2 client userwarningChaozhe.Chen2016-01-127-45/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: When we run nova client unit test, there are lots of UserWarnings printed on the screen. These warnings mean to remind users not to use v2.client directly. Solution: 1. In top level tests such as some tests in test_auth_plugins.py test_client.py and fixture_data/client.py, we use updated usage novaclient.client instead of using novaclient.v2.client directly. 2. In v2 unit tests, we clean those warnings with setting direct_use False. Change-Id: I70682e54874860f1d67d29325811c9da616bb705 Closes-Bug: #1532711
* | | | | | | Merge "Add a mechanism to call out deprecated options"Jenkins2016-01-133-11/+558
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | |
| * | | | | | Add a mechanism to call out deprecated optionsKevin L. Mitchell2016-01-073-11/+558
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have had deprecated command line options in novaclient for a long time, but no warnings about the deprecations were ever emitted, despite the help text being suppressed. It would be nice to finally get rid of those deprecated options. This change sets up the precondition to accomplish this, by emitting appropriate warnings when a deprecated option is used. Note: The "use" text for the deprecated options has been deliberately chosen to reduce the number of translations required. Change-Id: Ibe13faa56c5abca97f85f9c5172ef5a5591b5f71
* | | | | | Updated from global requirementsOpenStack Proposal Bot2016-01-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ief924564960d7733f129fd02d0e9898527393151
* | | | | | Merge "Fix W503 line break before binary operator"Jenkins2016-01-122-5/+5
|\ \ \ \ \ \
| * | | | | | Fix W503 line break before binary operatorDirk Mueller2016-01-092-5/+5
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | According to PEP8 the line break should be always after a binary operator. Change-Id: I9d0f6909d0e649fe740b748b831894ccb4e80b15
* | | | | | Fix broken link in documentationJake Yip2016-01-121-1/+1
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | Documentation on developer.openstack.org points to API 2.1 now Change-Id: Ic49279bf2ad66b2467ae49fe87243011e3a8c362
* | | | | Updated from global requirementsOpenStack Proposal Bot2016-01-071-1/+1
|/ / / / | | | | | | | | | | | | Change-Id: I19f98a76b8e0d595fe75842db7b23453ef7e102a
* | | | Replace assertTrue(isinstance()) by optimal assertLiuNanke2016-01-072-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | assertTrue(isinstance(A, B)) or assertEqual(type(A), B) in tests should be replaced by assertIsInstance(A, B) provided by testtools. Change-Id: Ie4fe7b40e3f4350db94b350395269340ee6acf11 Related-bug: #1268480
* | | | Merge "Use assertTrue/False instead of assertEqual(T/F)"3.2.0Jenkins2016-01-062-3/+2
|\ \ \ \
| * | | | Use assertTrue/False instead of assertEqual(T/F)yangyapeng2016-01-042-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The usage of assertEqual(True/False, ***) should be changed to a meaningful format of assertTrue/False(***). Change-Id: I66110e5fc1afdf9dd6824b1f7bcb7b2c091a272d Closes-Bug: #1512207
* | | | | Merge "[microversions] share one object for shell arguments"Jenkins2016-01-052-22/+19
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | [microversions] share one object for shell argumentsAndrey Kurilin2015-12-222-22/+19
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I8d599b712b17dcfc0be940a61c537d2dfe1b715b change provides a wrong fix for an issue with cli arguments. _find_actions method is designed to find all action methods(do_server_list, do_server_show and etc). Since one module can include several versions of one action(after microversion implementation) and only the last one will be registrated in module, _find_actions method was adopted to handle versioned methods. Now it checks that discovered method is related to versioning stuff and replace it by appropriate(in terms of microversions) functions. In this case, the substituation is used only to determine function name and that it relates to versioning methods. That is why the change(see a change-id above) is not help at all. We should share list object named "arguments"(it used by all action methods to store cli arguments) with substitution and original method(which will be used by _find_action). It will allow to put api_versions.wraps and cliutils.arg decorators in any order. Change-Id: Ief316a8597555db6cb02c9f23406b9f1f09f8313
* | | | Merge "Validate the fixed ip address passed with --nic"Jenkins2016-01-042-0/+19
|\ \ \ \
| * | | | Validate the fixed ip address passed with --nicZhihai Song2015-12-232-0/+19
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Currently fixed ip address passed with --nic is not validated. This patch add the validation to the fixed address. Change-Id: I032cc9ce9333b723d37e94b81d699cc0d78d36bf Closes-Bug: #1528455
* | | | Merge "document search_opts parameter"Jenkins2016-01-041-1/+15
|\ \ \ \
| * | | | document search_opts parameterSean Dague2015-12-181-1/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Based on irc conversation, the value to be passed to search_opts isn't super clear. Document it for posterity. Change-Id: I58a9b8f09056f16f58be4ef7b5b78e3ea3132faa
* | | | | Merge "Fixed test_shell which can't test microversions>=2.4"Jenkins2016-01-042-4/+9
|\ \ \ \ \
| * | | | | Fixed test_shell which can't test microversions>=2.4Sergey Nikitin2015-12-222-4/+9
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Method run_command() gets max and min API verisons from FakeHTTPClient.get_versions(). That's why max version which could be tested by run)command is 2.3 (because it's hardcoded in get_versions()). get_versions() should return max and min versions from novaclient.__init__.py Closes-Bug: #1528527 Change-Id: I489c4714cacb345d3b45d5987113093c038cf151
* | | | | Merge "Add python 2.7 comment"Jenkins2016-01-041-0/+1
|\ \ \ \ \
| * | | | | Add python 2.7 commentSean Dague2016-01-041-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ie4183da314bd7ca0719f45f0d9b3c26f8707e266
* | | | | | Merge "Put py34 first in the env order of tox"Jenkins2016-01-041-1/+1
|\ \ \ \ \ \
| * | | | | | Put py34 first in the env order of toxJanonymous2015-12-221-1/+1
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To solve the problem of "db type could not be determined" on py34 we have to run first the py34 env to, then, run py27. This patch puts py34 first on the tox.ini list of envs to avoid this problem to happen. Change-Id: I06c6a276b75eb2a3d69e94f467f26122d856d17c Closes-bug: #1489059
* | | | | | Merge "Fix extension loading from python path on Python 2.7"Jenkins2016-01-041-0/+2
|\ \ \ \ \ \ | | |/ / / / | |/| | | |
| * | | | | Fix extension loading from python path on Python 2.7Jim Rollenhagen2015-12-311-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 81f8fa655ccecd409fe6dcda0d3763592c053e57 broke extension loading from pythonpath on python 2.7 due to the removal of code that was erroneously marked for 2.6 compatibility. Put it back. Change-Id: Ic04f4d57953967fde9817ff5119ce0182453a86d Closes-Bug: #1530230
* | | | | | Merge "Wrong usage of "a/an""Jenkins2016-01-044-5/+5
|\ \ \ \ \ \
| * | | | | | Wrong usage of "a/an"hgangwx2015-12-304-5/+5
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Wrong usage of "a/an" in the messages: "It will be deprecated after an suitable deprecation" "Delete metadata from an server" "Get a list of actions performed on an server" "based on an requests response" "Get a aggregate by name or ID" Should be: "It will be deprecated after a suitable deprecation" "Delete metadata from a server" "Get a list of actions performed on a server" "based on a requests response" "Get an aggregate by name or ID" Totally 5 occurrences in python-novaclient base code. Change-Id: Iabe764f8b547b39431deb34221266695e0e79fa9
* | | | | | Merge "Fix a Typo in Docstring"Jenkins2016-01-041-1/+1
|\ \ \ \ \ \
| * | | | | | Fix a Typo in Docstringlei zhang2015-12-011-1/+1
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fix a typo in def remove_security_group(self, server, security_group) Change-Id: I2dc4250496707a6213797cb120f36b3415a774c5
* | | | | | Merge "Fix help strings"Jenkins2016-01-044-16/+16
|\ \ \ \ \ \
| * | | | | | Fix help stringsAndreas Jaeger2015-12-284-16/+16
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reviewing the import of novaclient 3.1.0 into the CLI Reference (change Ib39cdfdd563a2a53f0e6de1f8d2dc4f8c4678c15), a few inconsistencies where found and fixed: * Missing "." at end of help string * Inconsistent capitalization of "DEPRECATED, use" * Inconsistent capitalization of "Set or delete" * It's IPv4 and IPv6 * It's ID Change-Id: I425cd6575ca31efd92fc7f0be6192031886fcc41
* | | | | | Merge "Cleanup needless code from oslo-incubator"Jenkins2016-01-0416-2276/+255
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Cleanup needless code from oslo-incubatorAndrey Kurilin2015-12-1816-2276/+255
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | oslo-incubator was deprecated and all modules used by novaclient were removed from oslo-incubator master branch. This patch removes all unused code from novaclient's repo. Change-Id: Ie3c542d19da6ed4d25ec490faf6a86a82f48cd34
* | | | | | Updated from global requirementsOpenStack Proposal Bot2015-12-211-1/+1
| |/ / / / |/| | | | | | | | | | | | | | Change-Id: I59d6b3f54c21b1f4cc86bb58f58ec1ef5f11dee4