summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
* Add 'openstack server migration list --type' optionStephen Finucane2020-10-132-4/+17
| | | | | | | Another gap with novaclient closed. Change-Id: Id3ca95ceda6f438fa72496ab9ab15ac09bb64fa5 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Cleanup of 'server migration list' commandStephen Finucane2020-10-132-167/+211
| | | | | | | Address some post merge nits. Change-Id: Ie59521d81fab191194f6c1a114b007fa17f5299f Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
* Add 'server migration force complete' commandjay2020-10-132-0/+89
| | | | | | | | | This is equivalent to novaclient's 'live-migration-force-complete' command. Change-Id: Ic4dc639afa16cdf8c5a46774895e850d92985292 Story: 2007513 Task: 39293
* Add 'server migration abort' commandhackertron2020-10-133-0/+97
| | | | | | | | This is equivalent to nova client's 'live-migration-abort' command. Change-Id: I0ff520ccfdf2de52c427affad7bef4554c86a06f Story: 2007489 Task: 39210
* Merge "Add source_ip_prefix and destination_ip_prefix to metering label rules"Zuul2020-10-133-1/+35
|\
| * Add source_ip_prefix and destination_ip_prefix to metering label rulesRafael Weingärtner2020-09-023-1/+35
| | | | | | | | | | | | | | | | | | | | | | | | As proposed in the RFE and then approved in the spec, we are adding to the neutron metering rules two new parameters. The source IP prefix, and destination IP prefix. Partially-Implements: https://bugs.launchpad.net/neutron/+bug/1889431 RFE: https://bugs.launchpad.net/neutron/+bug/1889431 Depends-On: https://review.opendev.org/#/c/746586/ Change-Id: Ic44d88fabea0fffef2279f2f2c3d2b1da6426d4d
* | Merge "Remove oslo.utils"Zuul2020-10-135-16/+18
|\ \
| * | Remove oslo.utilsMonty Taylor2020-10-125-16/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Oslo things are really server-side oriented and are heavy-weight for client things. Remove oslo.utils and just use iso8601 and importlib directly. It's not actually a bad library, but pulling it and its other deps in just for a couple of wrapper methods is a bit much here. oslo.i18n, fwiw, is lightweight and helpful. Change-Id: I463993170c03a1d98c47ab6a3c19131b7fca1099
* | | Merge "Add an error message when server bind floating IP"Zuul2020-10-132-8/+44
|\ \ \
| * | | Add an error message when server bind floating IPyanpuqing2020-10-122-8/+44
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | If we add a floating IP for the server with no fixed IP, CLI doesn't report an error and nothing happens. The patch adds an error message when the server which don't have fixed IP bind floating IP. Change-Id: I400f2bab08521bb7fa443d87c7f45cc79eb80694 Task: 27941 Story: 2004346
* | | Merge "Compute: Add tag support for server add network"Zuul2020-10-132-1/+81
|\ \ \
| * | | Compute: Add tag support for server add networktianhui2020-10-122-1/+81
| |/ / | | | | | | | | | | | | | | | Change-Id: I31a66b2d4dac44052a71f43a5a67836247ccac64 Story: 2002195 Task: 21678
* | | Merge "Compute: Add tag support for server add port"Zuul2020-10-132-1/+77
|\ \ \
| * | | Compute: Add tag support for server add porttianhui2020-10-122-1/+77
| |/ / | | | | | | | | | | | | | | | Change-Id: Ice6bf5fb57afeb10862c870b42732dcf166772d1 Story: 2002195 Task: 21676
* | | Merge "Add server migration list CLI"Zuul2020-10-123-0/+735
|\ \ \ | |/ / |/| |
| * | Add server migration list CLIzhangbailin2020-09-243-0/+735
| | | | | | | | | | | | | | | | | | | | | | | | Add ``openstack server migration list`` to fetch server migrations. Part of blueprint add-user-id-field-to-the-migrations-table Change-Id: I15b4a5aca8d0dee59dd293e7b1c7272cdfbeea20
* | | Merge "Remove usage of six"Zuul2020-10-0912-48/+30
|\ \ \
| * | | Remove usage of sixsongwenping2020-10-0712-48/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With python3.x, classes can use 'metaclass=' instead of 'six.add_metaclass', 'six.iteritems' and 'six.iterkeys' can be replaced by 'items' and 'keys', 'six.moves.urllib.parse' can be replaced by 'urllib.parse', 'six.StringIO' and 'six.moves.cStringIO' can be replaced by 'io.StringIO', 'six.text_type' and 'six.string_type' are just 'str'. Change-Id: I84848c0bf8ab3c36dd821141191e2725e4e3b58b
* | | | Merge "Add 'openstack server create --use-config-drive'"Zuul2020-10-092-14/+36
|\ \ \ \
| * | | | Add 'openstack server create --use-config-drive'Stephen Finucane2020-07-222-14/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Despite what the help text for this options says, the nova API only accepts boolean values for this value and has done so since at least the introduction of the 2.1 microversioned API. While it would be nice to convert '--config-drive' to a boolean flag, we'd need to be able to retain temporary support for people passing arguments. 'nargs=?' [1] looks promising but it has an annoying tendency to swallow a positional argument following it [2]. Since that is not an option, we have to live with a new config option, '--use-config-drive' and a '--no-config-drive' counterpart. [1] https://docs.python.org/3/library/argparse.html#nargs [2] https://bugs.python.org/issue9338 Change-Id: If9cce0ad4094cc9cef1c9136b80c3b0f35a82c7a Signed-off-by: Stephen Finucane <sfinucan@redhat.com> Story: #2005468 Task: #30547
* | | | | Fix: port attribute name propagate_uplink_statuspedh2020-10-062-5/+5
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | Change the incorrect port attribute name "uplink_status_propagation" to "propagate_uplink_status". Change-Id: Icd7c49af8d988a6e3a52a58c784bd701b2d36faf Closes-Bug: #1891873
* | | | Merge "Fix --image-property option in 'create server'"Zuul2020-10-012-3/+70
|\ \ \ \
| * | | | Fix --image-property option in 'create server'Myeongchul Chae2020-09-102-3/+70
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was a problem that the '-image-property' option, which can be used to create an instance, did not work as intended. I found that there were two problems with this option. First, I cannot select an image as its metadata. The second is that when there are multiple images available, the desired image may not be selected depending on the situation. This patch solves these two problems. I wrote the test case with these two problems considered together. Change-Id: Ib2745d7e067056ff4ca8bfaf6cff492d0dacb73a story: #2007860
* | | | Merge "Add API check for server_groups.create"Zuul2020-09-302-2/+28
|\ \ \ \
| * | | | Add API check for server_groups.createLewis Denny2020-08-262-2/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The policies field has been replaced with the policy field since Nova API version 2.64[1] This commit adds a check to make sure the correct field is used. [1]https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id59 Change-Id: I06d3211937d822c26070b7f8ad757c365dcbb1bb Story: #2007822 Task: #40101
* | | | | Merge "Add API check for server_groups.list"Zuul2020-09-303-4/+138
|\ \ \ \ \
| * | | | | Add API check for server_groups.listLewis Denny2020-08-213-4/+138
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The policies parameter has been replaced with the policy parameter since Nova API version 2.64[1] This commit adds a check to make sure the correct parameter is used. [1]https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id59 Change-Id: Ia37beb7790884d6d15bec45074f446e64af1a2aa Story: #2008041 Task: #40703
* | | | | Merge "Show words indicating booted from volume for server image"Zuul2020-09-303-8/+35
|\ \ \ \ \
| * | | | | Show words indicating booted from volume for server imagemelanie witt2020-08-273-8/+35
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For a server booted from a volume, nova API does not store an image_id and instead returns an empty string. Currently, openstackclient similarly shows an empty string for Image Name and Image ID for servers booted from volumes. To aid CLI users in understanding the meaning of no image_id, we can display the string "N/A (booted from volume)" in the image field if the server was booted from a volume. Change-Id: I9c62cf6fe23b2e934dcbf5ebbf706b2705d2e424
* | | | | Merge "Output correct json for security groups in 'openstack server show'"Zuul2020-09-253-21/+38
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Output correct json for security groups in 'openstack server show'jay2020-09-143-21/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes incorrect json output for 'openstack server show -f json'. The security group json output groups all the json as one for e.g. "security_groups": "name='group1'\nname='group2'" The correct output should be "security_groups" : [{"name" : "group1"}, {"name" : "group2"}] properties and volumes_attached fields also has similar issue. Story: 2007755 Change-Id: I1b1cac716329e0530400aff782c08000b21d8e1d
* | | | | Merge "Support tagging Neutron ports on creation"Zuul2020-09-132-15/+58
|\ \ \ \ \
| * | | | | Support tagging Neutron ports on creationMiguel Lavalle2020-09-042-15/+58
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change adds support for tagging ports on creation Co-Authored-By: Slawek Kaplonski <skaplons@redhat.com> Change-Id: I3148a568664588eb2f529138f984859570c0fca1 Related-Bug: #1815933
* | | | | Merge "Add NUMA affinity policy parameter to "port""Zuul2020-09-073-0/+129
|\ \ \ \ \
| * | | | | Add NUMA affinity policy parameter to "port"Rodolfo Alonso Hernandez2020-07-313-0/+129
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added port NUMA affinity policy parameter to "port create", "port set" and "port unset" commands. Change-Id: I48cacab275856af2911829f9b7176bb87fd039b3 Related-Bug: #1886798
* | | | | | Merge "Allow openstack flavor set to update flavor description using name"Zuul2020-09-032-1/+37
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Allow openstack flavor set to update flavor description using nameGabriel Ramirez2020-06-102-1/+37
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Modified take_action() method for SetFlavor to use flavor id instead of flavor name when setting description Closes-Bug: #1844708 Story: #2007781 Task: #40019 Change-Id: If6798c89fef4c9feb4ebb460722b891f5655037d
* | | | | Merge "Bypass user and group verification in RemoveRole"Zuul2020-08-132-32/+278
|\ \ \ \ \
| * | | | | Bypass user and group verification in RemoveRoleLance Bragstad2020-08-122-32/+278
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Keystone let's users remove role assignments that reference non-existent users and groups. This is nice when keystone backs to an identity store like LDAP and users or groups are removed. Previously, openstackclient would validate the user and group existed in keystone before sending the request to delete the role assignment. This commit updates the code to bypass that validation so that users can use IDs to forcibly cleanup role assignments. Change-Id: I102b41677736bbe37a82abaa3c5b3e1faf2475d5 Story: 2006635 Task: 36848
* | | | | | Merge "Add system role assignment tests for users and groups"Zuul2020-08-061-0/+137
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Add system role assignment tests for users and groupsLance Bragstad2020-07-101-0/+137
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I was writing some additional functionality and noticed these tests were missing. This commit adds tests for adding and removing system role assignments for users and groups. Change-Id: I30fdc6ec55e1eb1cfa55f4cbf92c3f001d89865f
* | | | | | Merge "Fix reverted osc-lib interface change"Zuul2020-08-031-1/+1
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | Fix reverted osc-lib interface changeRoger Luethi2020-07-231-1/+1
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The patch https://review.opendev.org/#/c/673389/ introduced a regression by changing the osc-lib interface. Two conflicting attempts to fix the regression were launched: 1) Reverting the patch. 2) The patch https://review.opendev.org/683119 changes the exception from the generic CommandError back to a specific Forbidden exception. The patch https://review.opendev.org/683118 catches this exception and passes on, i.e. re-implements the same behavior as before. The first idea was implemented, the initial patch reverted. The second idea was partially implemented. The change in python-openstackclient (683118) was merged. The change in osc-lib was approved but failed to merge because the initial change had been reverted. Now we have again a situation where the exception produced in osc-lib does not match the exception expected by the caller. It is unclear if the osc-lib interface will ever get a rebased version of https://review.opendev.org/683119 merged, so the safest way to address the issue is to also catch the exception that used to be thrown before the inital change and is again thrown after the inital change has been reverted. Change-Id: I2ea2def607ec5be112e42d53a1e660fef0cdd69c
* | | | | Delete the testcases that arent needed anymoremb711d2020-07-233-32/+0
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | The file test_examples.py has never worked since its written and and cli in the example directory are covered by other functional tests for container, flavor and object lists and they have better asserts. So, deleting the file Change-Id: Ib9af40da96e66354fe878e79a80048a58f8dd6fe
* | | | Fix compatibility issue in 5.35.3.1Felix Yan2020-07-141-1/+4
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | The offending entry point object looks like: EntryPoint(name='compute', value='openstackclient.compute.client', group='openstack.cli.base') Story: 2007917 Task: 40323 Change-Id: I0f3cc62e23efdc14203ce6645581d5ba5dbf7fa0
* | | Merge "Fix uploading an signed image does not work if private signing key is ↵5.3.0Zuul2020-07-081-0/+6
|\ \ \ | | | | | | | | | | | | encrypted"
| * | | Fix uploading an signed image does not work if private signing key is encryptedAlexander Gräb2020-07-071-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ia7c84aa7b840bf92aeb7db7246d14119eb727b03 Story: 2007890 Task: 40269
* | | | Merge "Expose flag for forcing use of import for images"Zuul2020-07-082-0/+33
|\ \ \ \
| * | | | Expose flag for forcing use of import for imagesMonty Taylor2020-06-302-0/+33
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | openstacksdk added support for using image import as a fallback which is transparently supported here, but also provides an override flag to allow a user to force use of import. Expose that here. Depends-On: https://review.opendev.org/737608 Change-Id: Ied938a8f63f305305a20ace42e9f4c84b0a5c00e
* | | | Merge "switch to stevedore for entry points"Zuul2020-07-071-7/+16
|\ \ \ \
View Lorry Controller Status