summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Updated from global requirementsOpenStack Proposal Bot2016-04-072-7/+7
| | | | Change-Id: I2af3dc17e6f2c4cada96f1ff08627a3b41196f4d
* Merge "Correct addCleanup use in functests"Jenkins2016-04-076-88/+89
|\
| * Correct addCleanup use in functestsCedric Brandily2016-04-066-88/+89
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change replaces in many identity functests the pattern: raw_resource = create_resource(...) check(raw_resource) self.addCleanup(delete_resource, ...) ... by the pattern: raw_resource = create_resource(...) self.addCleanup(delete_resource, ...) check(raw_resource) ... which ensures that cleanup is defined and called after the test even if check(resource) fails. Change-Id: I4da541b7552d06eaffafda446e389bb552422cda
* | Merge "Make snapshot and backup name optional"Jenkins2016-04-077-4/+52
|\ \
| * | Make snapshot and backup name optionalIvan Kolodyazhny2016-04-067-4/+52
| | | | | | | | | | | | | | | | | | | | | Cinder does not require snapshot and backup name. These arguments are optional. Change-Id: I05d59efc9642205a25684bf0b77758328296d959
* | | TrivialFix: Fix help messages for port setreedip2016-04-071-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Certain messages in ``port set`` were incorrect as mentioned in [1] which are fixed in this patch. [1]:https://review.openstack.org/#/c/300309/6/doc/source/command-objects/port.rst TrivialFix Change-Id: Icb8e91664fea590cd4202f80d61a82eb50b22403
* | | Add option to clear information from portsreedip2016-04-074-17/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds the option of "no-fixed-ip" and "no-binding-profile" which is used to clear the fixed-ip and binding:profile information from the ports. Change-Id: I946301eaf6c647bae55e4f416aa0d98e5f06e699
* | | Merge "Support client certificate/key"Jenkins2016-04-066-0/+80
|\ \ \
| * | | Support client certificate/keyCedric Brandily2016-04-066-0/+80
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change enables to specify a client certificate/key with: * usual CLI options (--os-cert/--os-key) * usual environment variables ($OS_CERT/$OS_KEY) * os-client-config Change-Id: Ibeaaa5897ae37b37c1e91f3e47076e4e8e4a8ded Closes-Bug: #1565112
* | | | Merge "Devref: Document OSC interfaces available to plugins"Jenkins2016-04-061-1/+63
|\ \ \ \
| * | | | Devref: Document OSC interfaces available to pluginsRichard Theis2016-03-181-1/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This devref documents the OSC interfaces are officially available for plugins to implement commands and related unit tests. It also covers requirements for plugins. Change-Id: I68caa188e389e400fa9f5fd38f32c76cdd3e0986
* | | | | Merge "Add provider network options to osc network create"Jenkins2016-04-064-0/+63
|\ \ \ \ \
| * | | | | Add provider network options to osc network createreedip2016-04-064-0/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following patch adds the provider network options to OSC "network create". Change-Id: Ib8449c00ee4b4285889588f03ddd7a686ce8f987 Partial-Bug: #1545537
* | | | | | Merge "Add external network options to osc network create"Jenkins2016-04-065-1/+79
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Add external network options to osc network createreedip2016-04-065-1/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following patch adds the options "--external" & "--internal" and the suboptions to "external": "--default" & "--no-default", to "osc network create" CLI to provide the user an option to create a network as an external network. Change-Id: Idf73714bb94c0610ea164131140a51848908b00b Partial-Bug: #1545537
* | | | | | Merge "Log hint when --enable present with --disable-reason"Jenkins2016-04-063-13/+74
|\ \ \ \ \ \
| * | | | | | Log hint when --enable present with --disable-reasonRui Chen2016-04-053-13/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | --enable and --disable-reason should be mutually exclusive in "compute service set" command, but now when they are present at the same time, --disable-reason would be ignored silently. Fix these and add some hints about --disable-reason argument is ignored in this situation. Change-Id: I43254b6bc40fcae4fd0dc3457f26fad84c267072 Closes-Bug: #1556801
* | | | | | | Merge "Improve tmpfile cleanup in functests"Jenkins2016-04-062-27/+26
|\ \ \ \ \ \ \
| * | | | | | | Improve tmpfile cleanup in functestsCedric Brandily2016-04-062-27/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change replaces when possible homemade temporary file management by tempfile.NamedTemporaryFile[1][2] and defines only when needed a cleanup for a temporary file[2]. [1] functional/tests/compute/v2/test_keypair.py [2] functional/tests/object/v1/test_object.py Change-Id: I728ab96381ca9f3fd1f899dd50e5ceb5e97b9397
* | | | | | | | Merge "Use fixtures and addCleanup instead of tearDown"Jenkins2016-04-061-34/+25
|\ \ \ \ \ \ \ \
| * | | | | | | | Use fixtures and addCleanup instead of tearDownCedric Brandily2016-04-051-34/+25
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Nothing ensures tearDown call as tearDown is called only if test succeeds. This change replaces tearDown use with: * addCleanup use to stop mocks * EnvFixture which ensures to unmock environment thanks to useFixture. Change-Id: I1ff422e6a7585bc48b04b8f5c4cc1e7e9ddab1bc
* | | | | | | | Merge "Remove unused method cleanup_tmpfile"Jenkins2016-04-061-7/+0
|\ \ \ \ \ \ \ \ | | |/ / / / / / | |/| | | | | |
| * | | | | | | Remove unused method cleanup_tmpfileCedric Brandily2016-04-061-7/+0
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change removes unused cleanup_tmpfile method from functests code. Change-Id: I5d8e7edb0e50e94a5a469ce393d411b390b4db34
* | | | | | | Merge "Fix typos in docstrings and comments"Jenkins2016-04-063-3/+3
|\ \ \ \ \ \ \
| * | | | | | | Fix typos in docstrings and commentsDao Cong Tien2016-04-063-3/+3
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | Change-Id: Ic2dc057dca87212f715970f8325956c42f62ea9f
* | | | | | | Merge "Add name option to 'port set'"Jenkins2016-04-064-2/+19
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Add name option to 'port set'reedip2016-04-054-2/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Port's name can be updated in NeutronClient and it is a very good feature for a user to have the ability to rename a port. This was missing in the openstackclient, and the same has been added in this patch. Change-Id: I6e712ef08ab1c0a23786c4bb6972d3e0f8f0f999 Implements: blueprint neutron-client
* | | | | | | Trivial: Rename FakehypervisorStats to FakeHypervisorStatsTang Chen2016-04-052-3/+3
| |/ / / / / |/| | | | | | | | | | | | | | | | | Change-Id: I138b1b8a3327947b8cd032d8d0c32d98548ce2ad
* | | | | | rxtx factor should be a floatSean Dague2016-04-042-4/+4
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | The rxtx factor on compute flavors is a float in the API. OSC is currently blocking float values. Change-Id: Ifa9c14825f388d2821ff4b63ab8ae83fa9c8d88b
* | | | | Don't mask authorization errorsHidekazu Nakamura2016-04-012-16/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Project show with name argument returns 'Could not find resource' error when the user is not authorized. It should report the authorization error instead. This patch makes that change. Change-Id: Iac3521f8a411060b0ec9ef46c8f0e1f3551e56ae Closes-Bug: #1511625
* | | | | Merge "Add default value to pool-prefix in Subnet-pool"Jenkins2016-04-012-0/+4
|\ \ \ \ \
| * | | | | Add default value to pool-prefix in Subnet-poolreedip2016-04-012-0/+4
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If user tries to create a subnet pool without --pool-prefix, it fails at the NeutronServer [1]. This patch tries to add a default value to the --pool-prefix to try and resolve it. Closes-Bug: #1564271 [1] : http://paste.openstack.org/show/492537/ Change-Id: I6cf324a5a8037048602e59c0bbfc93b40e73a74e
* | | | | Merge "Devref: Options with Multiple Values"Jenkins2016-04-011-0/+81
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Devref: Options with Multiple ValuesRichard Theis2016-03-311-0/+81
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a devref for options with multiple values. Change-Id: Ic90c2317eb6c0445d234964c5243ecc689d5f4c7
* | | | | Merge "Follow Boolean Option rule"Jenkins2016-04-016-66/+57
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Follow Boolean Option rulereedip2016-03-316-66/+57
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | OSC has a specific rule for handling boolean based options in [1]. This patch modifies the exisiting code so that it matches the criteria that have been specified in [1]. [1]: http://docs.openstack.org/developer/python-openstackclient/command-options.html#boolean-options Closes-Bug: #1559418 Change-Id: I182381d5579efbc75a5d3e8a91f412398abf5c3c
* | | | | Add --project to "subnet pool create"Tang Chen2016-03-314-4/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds --project and --project-domain options to "subnet pool create" command. Change-Id: I2fe006013a194861299a9c77234a7cf988a8dad8 Partial-Bug: #1544586
* | | | | Merge "Add fixed keypair create functional test"Jenkins2016-03-311-0/+26
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Add fixed keypair create functional testDean Troyer2016-03-301-0/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds a test case to exercise the --public-key option of the 'keypair create' command. It is a follow-on to I7a299a542d9df543bff43d3ea1e7907fc8c5f640 that fixed a key file read bug. Change-Id: Id78c1c7ece02f619aca69dc397185fc426b92306
* | | | | Merge "Add unit tests for compute v2 aggregate"Jenkins2016-03-301-5/+331
|\ \ \ \ \
| * | | | | Add unit tests for compute v2 aggregateRui Chen2016-03-261-5/+331
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch add the complete unit tests to cover compute v2 aggregate object. Change-Id: Ifcf9248eabb5ca3ee5ca01c5843e96ba681c3b5a Closes-Bug: #1560832
* | | | | | Merge "Add "aggregate unset" to osc"Jenkins2016-03-306-0/+157
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Add "aggregate unset" to oscRui Chen2016-03-246-0/+157
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support "aggregate unset" command in order to remove the property of aggregate object in OSC. Change-Id: I49645135586362f0fd251f5e4a4c03eff273d9e9 Closes-Bug: #1559866
* | | | | Merge "Fix subnet pool prefix length option"Jenkins2016-03-302-9/+9
|\ \ \ \ \
| * | | | | Fix subnet pool prefix length optionRichard Theis2016-03-302-9/+9
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the --min-prefix-length, --max-prefix-length and --default-prefix-length options for the "os subnet pool create" and "os subnet pool set" commands. Using these options caused the commands to fail with "HttpException: Bad Request" because unrecognized attributes were passed to the REST API. No release note was added since these commands are new and haven't been released. Change-Id: I2f765750f79dc91d6c1f5962f699fbf2ee38657a Closes-Bug: #1564004
* | | | | Merge "Aggregate object should be "grouping of compute hosts""Jenkins2016-03-302-2/+2
|\ \ \ \ \
| * | | | | Aggregate object should be "grouping of compute hosts"Rui Chen2016-03-292-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The describe of Compute "aggregate" in command-objects and commands documents use "a grouping of servers", but exactly that should be a grouping of compute hosts. "server" object in OSC is a virtual machine instance, and server group should be "grouping of servers". Change-Id: Ib034fed15f11fc3e756985b3131a9922129ed6bf Closes-Bug: #1563172 Related-Bug: #1542171
* | | | | | Wrong param type in compute-service.rstSheel Rana2016-03-302-7/+7
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | compute-service.rst file uses 'describe' for below optional parameters 1. --long 2. --service <service> 3. --host <host> This patchset changes type of these argument to option. Change-Id: Ifd57bdf058efaeaa6cb43a7d4a60ee61ddb8de14 Closes-bug:#1563700
* | | | | Merge "Remove unused method 'from_response'"Jenkins2016-03-291-25/+0
|\ \ \ \ \
| * | | | | Remove unused method 'from_response'Wenzhi Yu2016-03-291-25/+0
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'openstackclient.common.exceptions.from_response' method is never called in openstackclient code base, so we should remove it. Change-Id: I04254a4e66863942e6c273d77bbd66ce2ce7804c Related-Bug: #1559072
View Lorry Controller Status