summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | Merge "Warning for empty password set for user create/set"Jenkins2016-10-252-0/+20
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | Warning for empty password set for user create/setqtang2016-10-192-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Raise warning when empty password set for user Change-Id: If03516f3f1290e4c329fe3d1277dee0512de0410 Closes-Bug: #1607959
* | | | | | | | Improve a network testcaseYan Xing'an2016-10-201-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix a typo networ to network, and add assertEqual lines. Change-Id: I13c0c775c1d5f7a8f579d387cd9bea1bdd74aebc
* | | | | | | | Correct help string of the subnet pool list optionsYan Xing'an2016-10-201-8/+10
| |_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Correct help string of subnet pool list options, from "subnets" to "subnet pools". Change-Id: Ic6dd6649f411748cd0c6e6b08f0451c8a0fa220f Partially-Implements: blueprint network-commands-options
* | | | | | | Reset allocation pools to [] instead of ''Carl Baldwin2016-10-191-1/+1
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the sdk expects subnet allocation pools to be reset to an empty array, not an empty string. Currently this results in an error message: "Invalid input for allocation_pools. Reason: Invalid data format for IP pool" Change-Id: I7cc84b9c8e4abdbd2c91e5d591ad31f0849c1a83 Closes-Bug: #1634672
* | | | | | Merge "Fix --shared/block-migration options in server migrate command"Jenkins2016-10-171-8/+8
|\ \ \ \ \ \
| * | | | | | Fix --shared/block-migration options in server migrate commandCedric Brandily2016-10-111-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, --shared-migration and --block-migration options effects are reversed: --block-migration requests a migration with share, --shared-migration a block-migration. This change corrects OSC implementation and clarifies arguments passed to novaclient (the root cause of the bug). Change-Id: Ib682cff0c44d3b1304670f8606907b1762d8b1e7 Closes-Bug: #1518059
* | | | | | | Merge "properly format error messages for volume resizing"Jenkins2016-10-171-2/+3
|\ \ \ \ \ \ \
| * | | | | | | properly format error messages for volume resizingSteve Martinelli2016-10-151-2/+3
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The error messages are tuples and not substituting variables. Before: $ openstack volume set vol1 --size 1 Failed to set volume size: (u'New size must be greater than %s GB', 2) $ openstack volume set vol1 --size 1 Failed to set volume size: (u'Volume is in %s state, it must be available before size can be extended', u'error') After: $ openstack volume set vol2 --size 3 Failed to set volume size: New size must be greater than 4 GB $ openstack volume set vol2 --size 3 Failed to set volume size: Volume is in error state, it must be available before size can be extended Change-Id: Ide6e644b1c6d1c11a9dc2f3f53c1a1837380b8d5
* | | | | | | Merge "Add necessary info to Floating IP list"Jenkins2016-10-172-0/+8
|\ \ \ \ \ \ \
| * | | | | | | Add necessary info to Floating IP listReedip2016-10-162-0/+8
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds the Network ID and Project ID for the listed floating IPs when Neutron is enabled for OpenstackClient. Change-Id: I823090f2e6d30acd72247d30956f48f4d7672a50 Closes-Bug:#1566090
* | | | | | | Merge "Add security groups options to "port create/set/unset""Jenkins2016-10-172-1/+292
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | |
| * | | | | | Add security groups options to "port create/set/unset"Nguyen Phuong An2016-10-122-1/+292
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds '--security-group' and '--no-security-group' options to "port create", "port set" and "port unset" commands. Change-Id: Iff60d8f29227017b0a3966efca6cdecba69abcea Partial-Bug: #1612136 Partially-Implements: blueprint network-commands-options
* | | | | | Add support make a router HAReedip2016-10-162-5/+23
| |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently router set CLI does not provide the support make a router highly available. The following patch enables the same. Checking for setting a router as HA is left on the neutron server itself. Partially-Implements: blueprint network-commands-options Change-Id: I0d0548cf037a14e5ccb2f732918ee9d1f63f43b4 Closes-Bug:#1631492
* | | | | Merge "Add a new column and a new option the 'os port list' cmd"Jenkins2016-10-122-0/+52
|\ \ \ \ \
| * | | | | Add a new column and a new option the 'os port list' cmdNam Nguyen Hoai2016-10-122-0/+52
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch will add a new column called status to the result of the 'os port list' command and --long option to 'os port list' command. Co-Authored-By: Ha Van Tu <tuhv@vn.fujitsu.com> Change-Id: I4f942414e969687304b578ed7f003dd219c0f2f8 Closes-Bug: #1613995 Closes-Bug: #1614321 Partially-Implements: blueprint network-commands-options
* | | | | Merge "Add translation markers for object commands"Jenkins2016-10-123-48/+50
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Add translation markers for object commandsRajasi Kulkarni2016-09-263-48/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | None of the help messages for the object commands are marked for translation. This patch adds the necessary support. Co-Authored-By: Steve Martinelli <s.martinelli@gmail.com> Change-Id: Ibf472d8f7d5ab6c876f60cddcab8833b28f042e0
* | | | | Merge "Add network support for Network QoS policies"Jenkins2016-10-114-0/+816
|\ \ \ \ \
| * | | | | Add network support for Network QoS policiesRodolfo Alonso Hernandez2016-10-114-0/+816
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added following commands: - network qos policy create - network qos policy delete - network qos policy set - network qos policy show - network qos policy list Closes-Bug: 1609037 Depends-On: I33bafeca979410d329ae10a82772ccdb48c10daa Change-Id: I63a8f63702514ff5814481bb021e2aa9d5f3d4b1
* | | | | | Merge "Refactor image v1 unit tests with FakeImage class"Jenkins2016-10-112-95/+185
|\ \ \ \ \ \
| * | | | | | Refactor image v1 unit tests with FakeImage classHuanxuan Ao2016-10-092-95/+185
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add FakeImage class, and refactor the unit tests with it in image v1. Change-Id: I9024ca5eca5c604e7588c1d905562bf6838309f1 Implements: bp improve-image-unittest-framework
* | | | | | Fix router unset --route optionChoe, Cheng-Dae2016-10-102-4/+3
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the "--route" option one the "os route unset" command. The option did not convert gateway to nexthop which results "Router does not contain route" error. Change-Id: Ia57bc7ea77ad7c6030535180a6ce42b4928c9e56 Closes-Bug: 1631471
* | | | | Merge "Improve output of supported API versions"Jenkins2016-10-071-3/+5
|\ \ \ \ \
| * | | | | Improve output of supported API versionsKyrylo Romanenko2016-10-071-3/+5
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Sort supported versions properly for better look. Change-Id: I6c2f5ecc04cf14ea5bf1b214cb303fcc9783af3f Closes-Bug: #1630962
* | | | | Mask passwords in debug logs for auth_config_hookMatt Riedemann2016-10-051-1/+5
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The auth config hook can have credentials in it so we have to mask the config before logging it. To avoid doing the work of masking the password if we aren't going to log it, there is a conditional put around the actual debug statement. Change-Id: I8e626672ec94fc837610216bccb4354dbdedca17 Closes-Bug: #1630822
* | | | Refactor qos spec unit test in volume v1Huanxuan Ao2016-10-042-159/+124
| |/ / |/| | | | | | | | | | | | | | | | | Refactor qos spec unit test with FakeQos class in volume v1. Change-Id: Ia76bf41a911457282b293dfaf0af311b1a9194ce
* | | Merge "Add --ha option to os router create command"Jenkins2016-09-302-2/+33
|\ \ \
| * | | Add --ha option to os router create commandNam Nguyen Hoai2016-10-012-2/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch added --ha option which the 'os router create' command was missed. Change-Id: I77635fb17af32beb0d8ed9aa080ef79285719fdc Closes-Bug: #1610161
* | | | Merge "Replace 'MagicMock' with 'Mock'"Jenkins2016-09-302-13/+13
|\ \ \ \ | |/ / / |/| | |
| * | | Replace 'MagicMock' with 'Mock'Nguyen Phuong An2016-09-272-13/+13
| | | | | | | | | | | | | | | | | | | | Change-Id: I35ebb2f0c83ec4300e26a72d9217ed0208f526eb Closes-Bug: #1475722
* | | | Merge "Support "--no" option in aggregate set"Jenkins2016-09-293-15/+117
|\ \ \ \
| * | | | Support "--no" option in aggregate setRui Chen2016-09-243-15/+117
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Supporting "--no-property" option will apply user a convenience way to clean all properties of aggregate in a short command, and this kind of behavior is the recommended way to devref. The patch add "--no-property" option in "aggregate set" command, and update related test cases and devref document. Change-Id: I7614a23c0db05144562330dc600dbab7d003d5d8 Implements: blueprint support-no-property-in-aggregate
* | | | | Merge "Remove beta label for network segment resource"Jenkins2016-09-295-137/+18
|\ \ \ \ \
| * | | | | Remove beta label for network segment resourceRichard Theis2016-09-285-137/+18
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With Ocata development opened, neutron network segments are no longer beta resources. Change-Id: I25be51c30df17c746136471b88840f64a03187b9 Partially-Implements: blueprint routed-networks
* | | | | Merge "Add and modify options for "volume create" command"Jenkins2016-09-294-24/+266
|\ \ \ \ \
| * | | | | Add and modify options for "volume create" commandHuanxuan Ao2016-09-284-24/+266
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1.Add mutually exclusive options into a mutually exclusive group. 2.Add "--source-replicated", "--consistency-group", "--hint" and "multi-attach" options 3.Make --size option to be optional under some cases Closes-Bug: #1568005 Closes-Bug: #1627913 Implements: bp implement-cinder-features Co-Authored-By: Roman Vasilets <rvasilets@mirantis.com> Change-Id: I2c4c3073195d33774e477f4d7f22e383b14b41dd
* | | | | | Merge "Implement "consistency group list" command"Jenkins2016-09-293-0/+234
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Implement "consistency group list" commandHuanxuan Ao2016-09-283-0/+234
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add "consistency group" object in volume v2 (v2 only) and implement "consistency group list" command Change-Id: Ifa90d97f4b223f9a5b00708cff07fa2c5e2635f0 Implements: bp cinder-command-support Partial-Bug: #1613964 Co-Authored-By: Sheel Rana <ranasheel2000@gmail.com>
* | | | | Merge "Update default nova api version to 2.1"Jenkins2016-09-282-2/+3
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Update default nova api version to 2.1qtang2016-09-282-2/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ref:https://review.openstack.org/#/c/311653/ We should use 2.1 as the default nova version now The API_MIN_VERSION of novaclient has already changed to 2.1 Change-Id: I9ff16cf052556e5d3756f81e02a8e76e8f315df5 Closes-bug: #1588171
* | | | Add option "--name" to command "openstack object create"Rajasi Kulkarni2016-09-264-3/+46
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Option "--name" can be used to set as the object name of the file to be uploaded in the container. Similar to option "--object-name" in command "swift upload". Added unit test case to ensure an exception is raised when using option "--name" for uploading multiple objects. Change-Id: Ied7827841f6ca1cf9d4b48e304cbe5d62eda38ab Closes-Bug: #1607972
* | | Merge "Add default limit for container/object"Jenkins2016-09-272-0/+21
|\ \ \
| * | | Add default limit for container/objectzheng yin2016-09-262-0/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Default container name length less than or equal to 256 in link[1], as the same time,default object name length less than or equal to 1024 in link[2]. Thereforce, I check the length of container and object in take_action. and if it's greater than 256/1024 I warn the user. [1] https://github.com/openstack/swift/blob/master/swift/common/constraints.py#L39 [2] https://github.com/openstack/swift/blob/master/swift/common/constraints.py#L35 Change-Id: I304b77cbc464eaba041321654cc29248cbe4b9a6
* | | | Add warning message for --state option of set command in volumeHuanxuan Ao2016-09-263-4/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are some set commands can set object state, it maybe a danger behavor for users, so add explanation and warning in the help message of the "--state" option to talk users be caution when using (cinderclient have done this too) Change-Id: I6a902887ea98879999c9972f36b1b7ef332173c3
* | | | Add functional test for snapshot in volume v1Huanxuan Ao2016-09-252-3/+92
| |/ / |/| | | | | | | | | | | | | | | | | Add functional test for snapshot commands in volume v1. Tests can always help to find or avoid bugs. Change-Id: Ieb0ab9c763d381a6343b4c4a8a5874f3e682f24f
* | | Merge "Add unit tests for backup commands in volume v1"Jenkins2016-09-232-0/+435
|\ \ \
| * | | Add unit tests for backup commands in volume v1Huanxuan Ao2016-09-232-0/+435
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was not any unit tests for backup commands in volume v1 so that sometimes some small bugs maybe ignored, this patch add unit tests for them. Change-Id: Ic67c1b80243f7b3d15dabd25e4e4a1b1517a8b59
* | | | Merge "Fix unset commands to pass normally when nothing specified"Jenkins2016-09-237-5/+571
|\ \ \ \ | |/ / /
| * | | Fix unset commands to pass normally when nothing specifiedHuanxuan Ao2016-09-237-5/+571
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After I found this problem appear in "volume unset", I checked all the volume command and also found some same problems. This patch fix them all. The main reason of we ignored this problem before is there was not any tests for it. So I add tests for "nothing unset" for them all to test and aviod this problem. Also, I add unit tests for all snapshot commands in volume v1 by the way in this patch. We will need more tests to avoid some ignored problem. Change-Id: I46775f24643d715e168b30785b8b531c0431a55b Partial-bug: #1588588
View Lorry Controller Status