summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | Merge "Fix server show for microversion 2.47"Zuul2018-06-123-7/+52
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Fix server show for microversion 2.47Matt Riedemann2018-06-083-7/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Compute API version 2.47 embeds the server's internal flavor in the response. The original flavor id is not preserved since it could have changed if the flavor was deleted and re-created after the server was created, which was the dreaded Horizon "Edit Flavor" issue. So the flavor dict in the server response is a dict of information about the flavor representing the server "right now" excluding the id. The original flavor name is shown though along with the ram/disk/vcpu etc information. The server list command has a similar issue which will be fixed in a follow up change. Change-Id: I1a92999758006d02567c542b6be8902a049899cc Task: 13864 Story: 1751104
* | | | | | | | | | Merge "Optimize _prep_server_detail to avoid redundant find_resource"Zuul2018-06-122-10/+15
|\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / /
| * | | | | | | | | Optimize _prep_server_detail to avoid redundant find_resourceMatt Riedemann2018-06-082-10/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When showing a server or doing a rebuild, we already have the latest version of the server so _prep_server_detail getting the server again is an unnecessary performance hit. ShowServer is pretty obvious here. For RebuildServer, the compute API actually refreshes the server before returning it in the response, so the client already gets the latest when the rebuild call returns. The only other usage of _prep_server_detail that does require a refresh is CreateServer since the POST /servers response is a minimal version of the server object. This adds a new refresh kwarg, backward compatible by default, to _prep_server_detail but changes ShowServer and RebuildServer to no longer refresh. Change-Id: Ib1c9c424ed1cafc2dfd8be90af8de8a774bdfbf0
* | | | | | | | | | Merge "Use Server.to_dict() rather than Server._info"Zuul2018-06-121-4/+6
|\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / /
| * | | | | | | | | Use Server.to_dict() rather than Server._infoMatt Riedemann2018-06-081-4/+6
| | |_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is a to_dict() method on the Server object from python-novaclient which makes a deepcopy of the internal Server._info - use this instead of accessing the _info attribute directly. Also, while in here, fixed a typo in _prep_server_detail. Change-Id: I679b4489c815f8a54368ef6b23b9f77e75b4d0bc
* | | | | | | | | Merge "Remove deprecated ip floating commands"Zuul2018-06-096-275/+4
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Remove deprecated ip floating commandsDongcan Ye2018-04-086-275/+4
| | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We had already implemented floating ip(pool) commands more than two cycles, we can remove those deprecated commands. Change-Id: Ib98a7403a63bb0c48c03de4c79795737de2aa84c
* | | | | | | | | Merge "Add bgp commands to neutron decoder"Zuul2018-06-091-19/+19
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Add bgp commands to neutron decoderJens Harbott2018-04-091-19/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Help folks that need to migrate their use of the neutron CLI for neutron-dynamic-routing commands. Change-Id: I324608d7bbce8c55aaab122bfd8bff0ee1ccf9e3
* | | | | | | | | | Merge "Fix urls in README.rst"Zuul2018-06-091-8/+4
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Fix urls in README.rstChen2018-06-071-8/+4
| | |_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. bug tracking has been moved to storyboard 2. syntax issues 3. remove PyPI downloads since it is no longer maintained https://packaging.python.org/guides/analyzing-pypi-package-downloads/ Change-Id: Ic57c7032af0296d666841021c19e6e56f0b3e89a
* | | | | | | | | | Merge "Update role document to include system parameter"Zuul2018-06-091-2/+20
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Update role document to include system parameterLance Bragstad2018-06-071-2/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With the recent addition of system scope support in osc, we should add system docs to the role documentation. bp system-scope Change-Id: Id77511c52ff0c36ea845e0f0fbbe3ec14818ee58
* | | | | | | | | | | Merge "Add system role functionality"Zuul2018-06-097-16/+112
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / | | / / / / / / / / / | |/ / / / / / / / / |/| | | | | | | | |
| * | | | | | | | | Add system role functionalityLance Bragstad2018-06-047-16/+112
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds the necessary bits to expose system role assignments to openstackclient via python-keystoneclient. bp system-scope Depends-On: Iecbcbf020a15f2bec777334c648d4477f89f3b2c Change-Id: I261e84700b51e8715eaebdc3f8f8bc46b68542c2
* | | | | | | | | | Allow setting network-segment on subnet updateHarald Jensas2018-06-064-4/+48
| |_|_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To enable the possibility to migrate a non-routed network to a routed network allow updating the segment_id of a subnet. Change-Id: I3ebae2ff28d5d4e5373ebd1f52194f8c52071b88 Partial-Bug: bug/1692490 Depends-On: I1aee29dfb59e9769ec0f1cb1f5d2933bc5dc0dc5
* | | | | | | | | Merge "Prevent "server migrate --wait" from hanging"Zuul2018-06-011-0/+1
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Prevent "server migrate --wait" from hangingJulie Pichon2018-05-091-0/+1
| | |_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Migrate uses the same mechanism in the backend than Resize and so the steps and step names are similar. Currently when using the --wait option with 'migrate', we wait forever because the status won't get to active until the user performs an action. This makes it return on verify_resize status just like 'resize' does, so that the user can perform the next manual step. Change-Id: Ie1aeac52506bc8801f88fd6a6eb4f6094cf20050 Story: 2001994 Task: 19621
* | | | | | | | | Merge "Add cliff project link"Zuul2018-06-011-1/+1
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Add cliff project linkzhang.lei2018-05-231-1/+1
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I6a964781667aa92f4b8d364f63a25f1c76309dca
* | | | | | | | | Merge "Make max_burst_kbps option as optional for bw limit QoS rule"Zuul2018-06-013-11/+53
|\ \ \ \ \ \ \ \ \ | |_|/ / / / / / / |/| | | | | | | |
| * | | | | | | | Make max_burst_kbps option as optional for bw limit QoS ruleSławek Kapłoński2018-05-253-11/+53
| | |_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Attribute max_burst_kbps of QoS bandwidth limit rule in Neutron's is optional in API so it should be also optional on client's side. Change-Id: Ie085b73fa885ff12f9ac080666cf3ca6a09b632a Related-Bug:#1770622 Task: 19658 Story: 2002017
* | | | | | | | Fix lower-constraints.txtJens Harbott2018-05-311-7/+7
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was originally generated before the compliance test was installed, so it is now failing for every unrelated change to requirements. Replace lower-constraints.txt with the output of running openstack/requirements/tools/fix-lower-constraints.py . Change-Id: I492e663622db75994bdababfa6dc81589e3fb53b
* | | | | | | Update command test for volume.v3Monty Taylor2018-05-181-1/+1
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The default cinder version in devstack changed to v3 in https://review.openstack.org/#/c/566747/which breaks this test. Change the test to test what's going to happen. Change-Id: Iff4d8b47812a86d21bf5dbdddbd642b9d63ff8fe
* | | | | | Merge "Fix limits show command without Nova and Cinder"Zuul2018-04-236-11/+383
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | Fix limits show command without Nova and CinderJude Cross2018-04-036-11/+383
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch implements an endpoint lookup when showing limits. This addresses the issue when showing limits without both Nova and Cinder and will display limits if one is missing. Change-Id: I2214b281e0206f8fe117aae52de2bf4c4e2c6525 Closes-bug: #1707960
* | | | | | Merge "Network: Add tag support for floating ip"Zuul2018-04-235-4/+220
|\ \ \ \ \ \
| * | | | | | Network: Add tag support for floating ipDongcan Ye2018-04-175-4/+220
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | Change-Id: I7a500a4ff6cec2442b4050df26c0b017d9f71903 Closes-Bug: #1750985
* | | | | | Merge "Fix functional job failed"Zuul2018-04-231-3/+14
|\ \ \ \ \ \
| * | | | | | Fix functional job failedDongcan Ye2018-04-201-3/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After Neutron patch I3c93818002c2d7753454547231ba08544b6fa1c0 merged, the default value of segment description is an empty string. This patch will determine whether the Neuron shim extension standard-attr-segment supported. Change-Id: I68a3c018f03e5bb53bd637844ac9d7742a765db0
* | | | | | | Trivial: Update pypi url to new urlTovin Seven2018-04-202-4/+4
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pypi url changed from [1] to [2] [1] https://pypi.python.org/pypi/<package> [2] https://pypi.org/project/<package> Change-Id: I61f3d53737616dbbd6df725823b5335a57045dba
* | | | | | Merge "Clean up W503 and E402 pep8 errors"Zuul2018-04-1713-255/+232
|\ \ \ \ \ \
| * | | | | | Clean up W503 and E402 pep8 errorsSean McGinnis2018-04-1513-255/+232
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | pycodestyle 2.40 and later enforce these rules that were not previously enforced. Rather than just skipping them, this cleans up the trivial instances of these violations. This does also include some other updates that were not triggering errors in an attempt to keep some of the style consistent. Change-Id: Id7c0a6b8f1f835e69d844b000e3ed751852ada63 Closes-bug: #1762803
* | | | | | Add help for nova interface-list to decoderJens Harbott2018-04-091-2/+2
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Running "openstack port list --server <id>" is a good replacement for "nova interface-list <id>", add it to the decoder. Change-Id: I656e190f4da59d9c57e9ab97d068a8a0dbd77774
* | | | | Merge "Add support to list image members"Zuul2018-04-026-0/+101
|\ \ \ \ \
| * | | | | Add support to list image membersMohammed Naser2018-03-216-0/+101
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The OpenStack client presently has support to add or remove members from an image, but no way to list image members. This patch addreses this issue. Change-Id: Ie85c5de23c6beb21fd6b4c04c83ddf2a116606ef
* | | | | Merge "Add --image-property parameter in 'server create'"Zuul2018-03-313-0/+208
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Add --image-property parameter in 'server create'Chen Hanxiao2018-03-283-0/+208
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add --image-property option, just like --image-with of novaclient did. Change-Id: Ic1a8976559255529a8785b1b301a0307812433cb Signed-off-by: Chen Hanxiao <chenhx@certusnet.com.cn>
* | | | | Merge "Update help text for encryption provider"Zuul2018-03-293-12/+12
|\ \ \ \ \
| * | | | | Update help text for encryption providerBrianna Poulos2018-02-233-12/+12
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The volume encryption provider no longer uses class names. Instead, 'luks' and 'plain' are used. This patch updates the help text for the volume encryption provider to use the new encryption provider format constants. Change-Id: I2911098505a99658e04ac4008e5f3e857db81f95
* | | | | Merge "Updated from global requirements"3.15.0Zuul2018-03-292-5/+5
|\ \ \ \ \
| * | | | | Updated from global requirementsOpenStack Proposal Bot2018-03-282-5/+5
| | |/ / / | |/| | | | | | | | | | | | | Change-Id: I8c400b12269f00fcf9fcaac27098ca55cdc27ac4
* | | | | Merge "add lower-constraints job"Zuul2018-03-293-0/+153
|\ \ \ \ \
| * | | | | add lower-constraints jobDoug Hellmann2018-03-253-0/+153
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Create a tox environment for running the unit tests against the lower bounds of the dependencies. Create a lower-constraints.txt to be used to enforce the lower bounds in those tests. Add openstack-tox-lower-constraints job to the zuul configuration. See http://lists.openstack.org/pipermail/openstack-dev/2018-March/128352.html for more details. Change-Id: I4a4ca9726fab1d0cf9a33311201b7f65951a0942 Depends-On: https://review.openstack.org/555034 Signed-off-by: Doug Hellmann <doug@doughellmann.com>
* | | | | Merge "Fix additional output encoding issues"Zuul2018-03-297-54/+78
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Fix additional output encoding issuesDean Troyer2018-03-207-54/+78
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a followup to https://review.openstack.org/#/c/541609/ that changes most outstanding direct uses of sys.stdout to use the encoded stdout set up by cliff. Change-Id: I07cfc418385fc787d3b7d3c32d39676cf81bb91f
* | | | | Merge "Fix crashing "console log show""Zuul2018-03-231-3/+4
|\ \ \ \ \ | |/ / / /
| * | | | Fix crashing "console log show"Thomas Goirand2018-03-161-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because of encoding issue, the "openstack console show log" is prone to a stack dump, as explained in the bug report. Use the stdout handle that has already been set up by cliff's App class with a sane default encoding. Change-Id: I4d8b0df7f16ee0463e638bb11276220e5b92023b Closes-Bug: 1747862
* | | | | Merge "Rename python-openstacksdk to openstacksdk"Monty Taylor2018-03-232-7/+7
|\ \ \ \ \