summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Move keys() methods in each resource class to FakeResource"Jenkins2016-04-114-166/+63
|\
| * Move keys() methods in each resource class to FakeResourceTang Chen2016-04-114-166/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | FakeXXX classes in network don't need to fake any method, except keys(). But keys() can be put in FakeResource since it just returns all attributes in _info. This patch moves removes all unnecessary fake methods code, moves keys() method to FakeResource. This patch also finds out some missing attributes in network and router tests. Change-Id: I799822c8715b9ac4f95b98f8350f196757d79d3e
* | Merge "Doc: Fix network command documentation issues"Jenkins2016-04-111-9/+8
|\ \ | |/ |/|
| * Doc: Fix network command documentation issuesRichard Theis2016-04-081-9/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | Fix the following network command documentation issues: - Fix format of subnet pool positional arguments - Update port set options to match help - Fix network command documentation for Network v2 versus Compute v2 options and commands - Fix subnet command documentation errors and formatting problems in help text Change-Id: I808c2a70ca62eafc9e42d6873539cdd142ffe50c Related-Bug: #1558677
* | Merge "Refactor security group rule list to use SDK"Jenkins2016-04-114-332/+301
|\ \
| * | Refactor security group rule list to use SDKRichard Theis2016-03-284-332/+301
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactored the 'os security group rule list' command to use the SDK when neutron is enabled, but continue to use the nova client when nova network is enabled. In addition, a release note was added to document the features and fixes resulting from this refactor. Change-Id: I24d04b720102ed1c60776e1ca67d4ca20e31b663 Partial-Bug: #1519512 Implements: blueprint neutron-client
* | | Merge "Add Subnet add/remove support to router"Jenkins2016-04-092-0/+133
|\ \ \
| * | | Add Subnet add/remove support to routerreedip2016-03-222-0/+133
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following patch adds the support for "router add subnet" and "router remove subnet" to the OSC as a part of migration of Neutron's CLI commands. Partial-Bug: #1546849 Implements: blueprint neutron-client-advanced-router Change-Id: Ia3770c41026194bdb1543d4e67446f81936d44d1
* | | | Merge "Add "router remove port" to osc"Jenkins2016-04-092-0/+64
|\ \ \ \ | |/ / /
| * | | Add "router remove port" to oscreedip2016-03-212-1/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-082-1/+66
|\ \ \ \ | |/ / / | | | / | |_|/ |/| |
| * | Add "router add port" to oscreedip2016-03-212-1/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-082-4/+4
|\ \ \
| * | | TrivialFix: Rename provider segment optionreedip2016-04-082-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | Merge "Make snapshot and backup name optional"Jenkins2016-04-076-4/+48
|\ \ \ \
| * | | | Make snapshot and backup name optionalIvan Kolodyazhny2016-04-066-4/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cinder does not require snapshot and backup name. These arguments are optional. Change-Id: I05d59efc9642205a25684bf0b77758328296d959
* | | | | Add option to clear information from portsreedip2016-04-072-15/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-064-0/+62
|\ \ \ \ \
| * | | | | Support client certificate/keyCedric Brandily2016-04-064-0/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Add provider network options to osc network create"Jenkins2016-04-062-0/+35
|\ \ \ \ \ \ | | |_|/ / / | |/| | | |
| * | | | | Add provider network options to osc network createreedip2016-04-062-0/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-063-1/+46
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Add external network options to osc network createreedip2016-04-063-1/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-062-11/+71
|\ \ \ \ \ \
| * | | | | | Log hint when --enable present with --disable-reasonRui Chen2016-04-052-11/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | --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 "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 "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-062-2/+10
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | Add name option to 'port set'reedip2016-04-052-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-041-3/+3
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "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-312-4/+73
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds --project and --project-domain options to "subnet pool create" command. Change-Id: I2fe006013a194861299a9c77234a7cf988a8dad8 Partial-Bug: #1544586
* | | | 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-303-0/+129
|\ \ \ \ \ | |/ / / /
| * | | | Add "aggregate unset" to oscRui Chen2016-03-243-0/+129
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | | Wrong param type in compute-service.rstSheel Rana2016-03-301-2/+2
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | | Merge "Docs cleanup: volume type"Jenkins2016-03-292-30/+30
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
View Lorry Controller Status