delta/openstack/python-openstackclient.git - opendev.org: openstack/python-openstackclient

	Commit message (Collapse)	Author	Age	Files	Lines
...
* \| \| \| \| \| \| \| \| \| \| \|	Merge "Use fixtures and addCleanup instead of tearDown"	Jenkins	2016-04-06	1	-34/+25
\|\ \ \ \ \ \ \ \ \ \ \ \
\| * \| \| \| \| \| \| \| \| \| \| \|	Use fixtures and addCleanup instead of tearDown	Cedric Brandily	2016-04-05	1	-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"	Jenkins	2016-04-06	1	-7/+0
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \|/ / / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Remove unused method cleanup_tmpfile	Cedric Brandily	2016-04-06	1	-7/+0
\| \|/ / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This change removes unused cleanup_tmpfile method from functests code. Change-Id: I5d8e7edb0e50e94a5a469ce393d411b390b4db34
* \| \| \| \| \| \| \| \| \| \|	Merge "Fix typos in docstrings and comments"	Jenkins	2016-04-06	3	-3/+3
\|\ \ \ \ \ \ \ \ \ \ \
\| * \| \| \| \| \| \| \| \| \| \|	Fix typos in docstrings and comments	Dao Cong Tien	2016-04-06	3	-3/+3
\| \| \|_\|_\|_\|_\|/ / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Change-Id: Ic2dc057dca87212f715970f8325956c42f62ea9f
* \| \| \| \| \| \| \| \| \| \|	Merge "Add name option to 'port set'"	Jenkins	2016-04-06	4	-2/+19
\|\ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \|	Add name option to 'port set'	reedip	2016-04-05	4	-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 FakeHypervisorStats	Tang Chen	2016-04-05	2	-3/+3
\| \|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Change-Id: I138b1b8a3327947b8cd032d8d0c32d98548ce2ad
* \| \| \| \| \| \| \| \| \|	rxtx factor should be a float	Sean Dague	2016-04-04	2	-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 errors	Hidekazu Nakamura	2016-04-01	2	-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"	Jenkins	2016-04-01	2	-0/+4
\|\ \ \ \ \ \ \ \ \
\| * \| \| \| \| \| \| \| \|	Add default value to pool-prefix in Subnet-pool	reedip	2016-04-01	2	-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"	Jenkins	2016-04-01	1	-0/+81
\|\ \ \ \ \ \ \ \ \ \| \|_\|/ / / / / / / \|/\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Devref: Options with Multiple Values	Richard Theis	2016-03-31	1	-0/+81
\| \| \|_\|_\|_\|_\|_\|/ \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add a devref for options with multiple values. Change-Id: Ic90c2317eb6c0445d234964c5243ecc689d5f4c7
* \| \| \| \| \| \| \|	Merge "Follow Boolean Option rule"	Jenkins	2016-04-01	6	-66/+57
\|\ \ \ \ \ \ \ \ \| \|_\|/ / / / / / \|/\| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Follow Boolean Option rule	reedip	2016-03-31	6	-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 Chen	2016-03-31	4	-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"	Jenkins	2016-03-31	1	-0/+26
\|\ \ \ \ \ \ \ \| \|/ / / / / / \|/\| \| \| \| \| \|
\| * \| \| \| \| \|	Add fixed keypair create functional test	Dean Troyer	2016-03-30	1	-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"	Jenkins	2016-03-30	1	-5/+331
\|\ \ \ \ \ \ \
\| * \| \| \| \| \| \|	Add unit tests for compute v2 aggregate	Rui Chen	2016-03-26	1	-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"	Jenkins	2016-03-30	6	-0/+157
\|\ \ \ \ \ \ \ \ \| \|/ / / / / / /
\| * \| \| \| \| \| \|	Add "aggregate unset" to osc	Rui Chen	2016-03-24	6	-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"	Jenkins	2016-03-30	2	-9/+9
\|\ \ \ \ \ \ \
\| * \| \| \| \| \| \|	Fix subnet pool prefix length option	Richard Theis	2016-03-30	2	-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""	Jenkins	2016-03-30	2	-2/+2
\|\ \ \ \ \ \ \
\| * \| \| \| \| \| \|	Aggregate object should be "grouping of compute hosts"	Rui Chen	2016-03-29	2	-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.rst	Sheel Rana	2016-03-30	2	-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'"	Jenkins	2016-03-29	1	-25/+0
\|\ \ \ \ \ \ \
\| * \| \| \| \| \| \|	Remove unused method 'from_response'	Wenzhi Yu	2016-03-29	1	-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
* \| \| \| \| \| \|	Merge "Docs cleanup: volume type"	Jenkins	2016-03-29	3	-33/+54
\|\ \ \ \ \ \ \ \| \|/ / / / / / \|/\| \| \| \| \| \|
\| * \| \| \| \| \|	Docs cleanup: volume type	Dean Troyer	2016-03-28	3	-33/+54
\| \| \|_\|_\|_\|/ \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Clean up volume type command help, add volume type show to doc, sort command classes in v1/volume_type.py. Change-Id: I1f8e5c047d9c08f5704fc23cfb694f23d32e3caf
* \| \| \| \| \|	Merge "Docs cleanup: volume command help"	Jenkins	2016-03-29	3	-88/+106
\|\ \ \ \ \ \
\| * \| \| \| \| \|	Docs cleanup: volume command help	Dean Troyer	2016-03-28	3	-88/+106
\| \|/ / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This formats the volume command help text consistent with the rest of OSC, adds some reference targets and some explanation text to some commands. No functional changes have been made, only cosmetic/help output. Change-Id: Ib86ec3ca58bdea5f33078ced3ec3583b2be0e89a
* \| \| \| \| \|	Merge "Added functional tests for 'service provider' v3 commands"	Jenkins	2016-03-28	2	-0/+78
\|\ \ \ \ \ \
\| * \| \| \| \| \|	Added functional tests for 'service provider' v3 commands	Kristi Nikolla	2016-03-28	2	-0/+78
\| \| \|/ / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	'identity provider' commands had functional tests but 'service provider' commands did not. Added the tests in a similar way to how it is done it test_idp. Change-Id: Id4b24ef7d34db65c6b0260c89327ec9be683284d
* \| \| \| \| \|	Docs cleanup: sort subnet commands	Dean Troyer	2016-03-28	1	-16/+16
\| \|/ / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \|	Change-Id: I43c0052ec5ed2e94b203a0befc4b39d3522d4e9c
* \| \| \| \|	Merge "Support security group name for --src-group"	Jenkins	2016-03-28	4	-8/+21
\|\ \ \ \ \
\| * \| \| \| \|	Support security group name for --src-group	Richard Theis	2016-03-23	4	-8/+21
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Support security group name for the "--src-group" option on the "os security group rule create" command. Change-Id: Ic23d0671dad77566269c9a588644c8d774368733 Closes-Bug: #1540656
* \| \| \| \| \|	Merge "Refactor security group rule create to use SDK"	Jenkins	2016-03-28	9	-345/+434
\|\ \ \ \ \ \ \| \|/ / / / /
\| * \| \| \| \|	Refactor security group rule create to use SDK	Richard Theis	2016-03-23	9	-345/+434
\| \|/ / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Refactored the 'os security group rule create' command to use the SDK when neutron is enabled, but continue to use the nova client when nova network is enabled. Added a release note for the change in security group rules output due to Network v2. Change-Id: I8c6c99d5272ff5d410a449f73d198d834c5cd96e Partial-Bug: #1519512 Implements: blueprint neutron-client
* \| \| \| \|	Merge "Subnet: Add "subnet set" command using SDK"	Jenkins	2016-03-25	6	-84/+275
\|\ \ \ \ \
\| * \| \| \| \|	Subnet: Add "subnet set" command using SDK	reedip	2016-03-24	6	-84/+275
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This patch adds "subnet set" command to osc using sdk. Implements: blueprint neutron-client Closes-bug: #1542363 Change-Id: Id3b7f4b9190b4d73ca3ae423321a65f94a6da62e
* \| \| \| \| \|	[Floating IP] Neutron support for "ip floating create" command	Tang Chen	2016-03-24	6	-27/+281
\| \|/ / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This patch adds --subnet, --port, --floating-ip-address and --fixed-ip-address options only. Project related options will be added in another patch because it relates to identity v2 and v3, which will make the unit tests too complicated in one single patch. Change-Id: I3cce4404a114ff128b74e4596f0e847be2846b17 Partial-Bug: 1519502 Related-to: blueprint neutron-client
* \| \| \| \|	Merge "Fix "server unset" document issue"	Jenkins	2016-03-23	2	-2/+2
\|\ \ \ \ \
\| * \| \| \| \|	Fix "server unset" document issue	Rui Chen	2016-03-23	2	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix "--property" option describe issue in "server unset" document, and update the help message to keep consistent. Change-Id: I68022a187e83fad6320365400ad2a1b0c8cf9a61
* \| \| \| \| \|	Trivial: Fix typo in common/limits.py	Tang Chen	2016-03-23	2	-3/+3
\|/ / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Change-Id: I3040cc69512df50459960f96b862de55804068ac
* \| \| \| \|	Use assert_called_once_with() instead of assert_called_with()	Tang Chen	2016-03-22	6	-58/+65
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	If one API is only called once, we'd better use assert_called_once_with() to check the call. Change-Id: Id05055bb90592d218826732259ed84fc366734ce
* \| \| \| \|	Merge "Remove superfluous variable assignment statements"	Jenkins	2016-03-21	1	-2/+0
\|\ \ \ \ \