summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Add "router remove port" to osc"Jenkins2016-04-095-1/+95
|\
| * Add "router remove port" to oscreedip2016-03-215-2/+96
| | | | | | | | | | | | | | | | | | | | This patch proposes the migration of port deletion of Neutron's "router-interface-delete" to OSC's "router remove port". Change-Id: Ifceabce080aacac9f6410c809fcc097760e0c5ee Partial-Bug: #1546849 Implements: blueprint neutron-client-advanced-router
* | Fix pep8 fail that crept inDean Troyer2016-04-081-1/+1
| | | | | | | | | | | | Not sure how, but this crept in via https://review.openstack.org/#/c/281691/ Change-Id: I919276ee5204277dbc5e96f4de4b98381142423a
* | Merge "Add "router add port" to osc"Jenkins2016-04-085-3/+95
|\ \ | |/
| * Add "router add port" to oscreedip2016-03-215-3/+95
| | | | | | | | | | | | | | | | | | | | This patch proposes the migration of port addition of Neutron's "router-interface-add" to OSC's "router add port". Change-Id: I3a8a76d384caa32975f8e77f50f3cf3c3be13786 Closes-Bug: #1546849 Depends-On: I9783bc4ccceae3d361dce52d51483ef2187920a9 Implements: blueprint neutron-client-advanced-router
* | Merge "TrivialFix: Rename provider segment option"Jenkins2016-04-083-7/+7
|\ \
| * | TrivialFix: Rename provider segment optionreedip2016-04-083-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | As per [1], provider-segmentation-id is changed to provider-segment as segments may soon have a "name" support as well. [1]: https://review.openstack.org/#/c/294422/12/doc/source/command-objects/network.rst Change-Id: I74d4366301b41fa181faccbc12cff9530df60353
* | | Updated from global requirementsOpenStack Proposal Bot2016-04-081-1/+1
| | | | | | | | | | | | Change-Id: I9fe06c9739c3625abd7cb03ac97c58bb9cdbc5bd
* | | Merge "Updated from global requirements"Jenkins2016-04-072-7/+7
|\ \ \
| * | | Updated from global requirementsOpenStack Proposal Bot2016-04-072-7/+7
| | | | | | | | | | | | | | | | Change-Id: I2af3dc17e6f2c4cada96f1ff08627a3b41196f4d
* | | | Merge "Clean up release notes since 2.2.0 release"Jenkins2016-04-0720-74/+34
|\ \ \ \ | |/ / / |/| | |
| * | | Clean up release notes since 2.2.0 releaseDean Troyer2016-04-0720-74/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove release notes for changes to commands that have not been released yet * Combine notes for common command sets (ie, new port commands) * Rename notes with bugs to the bug-NNNNN-XXXXXXX form Change-Id: I6216aad443056ada2fe28ba533bf20a09f118e4d
* | | | 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