summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Handle multiple ports in AddFloatingIP3.16.2melanie witt2018-11-083-15/+164
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AddFloatingIP refers to an old nova proxy API to neutron that was deprecated in nova. The neutron API for floating IP associate requires a port to be specified. Currently, the code is selecting the first port if the server has multiple ports. But, an attempt to associate the first port with a floating IP can fail if the first port is not on a network that is attached to an external gateway. In order to make the command work better for users who have a server with multiple ports, we can: 1. Select the port corresponding to the fixed_ip_address, if one was specified 2. Try to associate the floating IP with each port until one of the attempts succeeds, else re-raise the last exception. (404 ExternalGatewayForFloatingIPNotFound from neutron) This also fixes incorrect FakeFloatingIP attributes that were being set in the TestServerAddFloatingIPNetwork unit tests, which were causing the tests to use None as parsed args for ip-address and --fixed-ip-address and thus bypassing code in the 'if parsed_args.fixed_ip_address:' block. Task: 27800 Story: 2004263 Change-Id: I11fbcebf6b00f12a030b000c84dcf1d6b5e86250 (cherry picked from commit 013c9a4f3a44cb0b81fc7affe9b933e701cb5dba)
* Merge "Default --nic to 'auto' if creating a server with >= 2.37" into ↵Zuul2018-11-024-7/+66
|\ | | | | | | stable/rocky
| * Default --nic to 'auto' if creating a server with >= 2.37Matt Riedemann2018-10-254-7/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Compute API version >= 2.37 requires a 'networks' value in the server create request. The novaclient CLI defaults this to 'auto' if not specified, but the novaclient ServerManager.create python API binding code does not, as it wants clients to be explicit. For the purposes of the OSC CLI, we should follow suit and if the user is requesting OS_COMPUTE_API_VERSION>=2.37 without specific nics, we should just default to 'auto'. Change-Id: Ib760c55e31209223338a4086ff1f4fee88dc6959 Closes-Bug: #1750395 (cherry picked from commit 1008544882fbdae16b045abca05cf3e2e8a14787)
* | Allow endpoint filtering on both project and project-domainJulie Pichon2018-10-152-1/+42
|/ | | | | | | | | | | | The --project and --project-domain flags are currently mutually exclusive for listing endpoints, however the --project-domain argument is supposed to help with filtering projects with colliding names. They should be allowed together. Story: 2004018 Task: 27004 Change-Id: I7340e01f509e3515f07cb46f175fb603f1ce8b67 (cherry picked from commit 91a2d888625488da3f65ad372b4248e9747b9a3e)
* Fix 'project purge' deleting wrong project's servers and volumesJulie Pichon2018-09-172-14/+20
| | | | | | | | | | | | | | | Project purge would delete the servers and volumes for the project the user is currently authenticated for, regardless of the --project flag. Note: This change means that no server at all will be deleted if the logged in user doesn't have the get_all_tenants permission set in the Nova policy (default: admin_api). This doesn't appear to be an issue with Cinder as the default rule appears to be admin_or_owner. Change-Id: If1c54e24e1482438b81c3c32fd5fc9fdd7a7be04 Story: 1747988 Task: 26494 (cherry picked from commit 1b66ad9067cc404ebfdc8569822d226d5bffddd6)
* Fix broken gate jobs3.16.1Fan Zhang2018-09-039-176/+160
| | | | | | | | | | | | | | | | | | | | | | | | This patch aims at fixing the broken gate jobs because of cinder and glance patches [1], [2], [3] and [4]. * Remove parameter `--source-replicated` to drop volume replication v1 support * Address some timing issues with volume transfer requests * Only run Image v1 tests when the test cloud has v1 available * Get tolerant of unexpected additional attributes being returned in Image data [1].https://review.openstack.org/#/c/586293/ [2].https://review.openstack.org/#/c/532503/ [3].https://review.openstack.org/#/c/533564/ [4].https://review.openstack.org/#/c/578755/ Co-Authored-By: Dean Troyer <dtroyer@gmail.com> Co-Authored-By: Monty Taylor <mordred@inaugust.com> Depends-on: https://review.openstack.org/588664 Change-Id: I2a785750e92155185d3344e6116c7f5c6fdd3cbe Signed-off-by: Fan Zhang <zh.f@outlook.com> (cherry picked from commit def83a0e94de2f98e3bd68ee412c0f0a2c316f32)
* Merge "Update UPPER_CONSTRAINTS_FILE for stable/rocky" into stable/rockyZuul2018-08-221-4/+4
|\
| * Update UPPER_CONSTRAINTS_FILE for stable/rockyOpenStack Release Bot2018-07-261-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new stable upper-constraints file is only available after the openstack/requirements repository is branched. This will happen around the RC1 timeframe. Recheck and merge this change once the requirements repository has been branched. The CI system will work with this patch before the requirements repository is branched because zuul configues the job to run with a local copy of the file and defaults to the master branch. However, accepting the patch will break the test configuration on developers' local systems, so please wait until after the requirements repository is branched to merge the patch. Change-Id: I623cd9b65afae23f74e777f8db75ca286c5111a3
* | import zuul job settings from project-configqingszhao2018-08-211-0/+9
|/ | | | | | | | | | | | | | | | | | | | This is a mechanically generated patch to complete step 1 of moving the zuul job settings out of project-config and into each project repository. Because there will be a separate patch on each branch, the branch specifiers for branch-specific jobs have been removed. Because this patch is generated by a script, there may be some cosmetic changes to the layout of the YAML file(s) as the contents are normalized. See the python3-first goal document for details: https://governance.openstack.org/tc/goals/stein/python3-first.html Change-Id: I9347eb52b24e7cc9dd115685017629f85881ca2e Story: #2002586 Task: #24320
* Update .gitreview for stable/rockyOpenStack Release Bot2018-07-261-0/+1
| | | | Change-Id: I47f90ef4ced52d458f9e4ca89853def63b3160d5
* Merge "Release note cleanup for 3.16.0 release"3.16.0Zuul2018-07-257-17/+21
|\
| * Release note cleanup for 3.16.0 releaseDean Troyer2018-07-257-17/+21
| | | | | | | | | | Change-Id: I64efea562117706529c4b474085f5c12939ba4d6 Signed-off-by: Dean Troyer <dtroyer@gmail.com>
* | Merge "Don't sent disk_over_commit if nova api > 2.24"Zuul2018-07-252-5/+43
|\ \
| * | Don't sent disk_over_commit if nova api > 2.24Artom Lifshitz2018-07-242-5/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In API microversion 2.25 Nova removed the disk_over_commit parameter to the live migration server action. This patch makes sure that we don't include it in our request if we're running with 2.25 or higher. Story: #2002963 Task: #22966 Change-Id: I1bbdd33be96d82422a05982508e370237c3560f3
* | | Merge "Add command to show all service versions"Zuul2018-07-255-0/+194
|\ \ \ | |_|/ |/| |
| * | Add command to show all service versionsMonty Taylor2018-07-235-0/+194
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Knowing what services and what versions of those services exist on a cloud isn't always a spectacular experience. Add a command that will use get_all_version_data from keystoneauth to produce a report of the available services and the version info for each service. Depends-On: https://review.openstack.org/584944 Change-Id: I84751c175d0c5f6d857a5473d2db6d5f1b41f946
* | | Merge "Support --community in openstack image list"Zuul2018-07-255-7/+60
|\ \ \
| * | | Support --community in openstack image listNobuto Murata2018-07-245-7/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "--community" was added to "image create" and "image set" previously, but was missed in "image list". Change-Id: I959fdd7f67ae62c8326659ce52389228152ec019 Story: 2001925 Task: 14453
* | | | Merge "Implement support for project limits"Zuul2018-07-258-0/+1021
|\ \ \ \
| * | | | Implement support for project limitsLance Bragstad2018-07-238-0/+1021
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit let's users manage limits via the command line. bp unified-limits Change-Id: I7c44bbb60557378b66c5c43a7ba917f40dc2b633
* | | | | Merge "Implement support for registered limits"Zuul2018-07-2510-2/+1169
|\ \ \ \ \ | |/ / / /
| * | | | Implement support for registered limitsLance Bragstad2018-07-2310-2/+1169
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds support for users to manage registered limits via the command line. bp unified-limits Depends-On: https://review.openstack.org/#/c/574391/ Change-Id: Id8377363f7a3248b45aeeba21d2acc02684a0305
* | | | | Merge "compute: host: expand kwargs in host_set() call"Zuul2018-07-252-5/+4
|\ \ \ \ \
| * | | | | compute: host: expand kwargs in host_set() callBenoît Knecht2018-07-202-5/+4
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `host_set()` expects `status` and `maintenance_mode` as keyword arguments, but in `SetHost.take_action()`, it is called without expanding the keyword arguments. So it's called as ``` host_set(host, {'status': 'enable'}) ``` instead of ``` host_set(host, status='enable') ``` Change-Id: If0b37ac60091161a892bfc694fce31a988f66005 Task: 23023
* | | | | Merge "Support filtering port with IP address substring"Zuul2018-07-254-7/+38
|\ \ \ \ \
| * | | | | Support filtering port with IP address substringHongbin Lu2018-06-294-7/+38
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | Change-Id: I9559f1c0a6db943705bd32aefb60d7ea7054dd1b Related-Bug: #1718605
* | | | | Merge "Fix error with image show when image name is None"Zuul2018-07-244-4/+5
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Fix error with image show when image name is NoneJake Yip2018-07-234-4/+5
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Need to bump osc-lib to 1.10.0 Closes-Bug: #1736696 Depends-On: I2aab5cc1f550848bda2b90ef7ef9a60f07b88996 Change-Id: I7420204f28d36529354e5671bd88587d9b15bb06
* | | | Merge "Do not require port argument when updating floating IP"Zuul2018-07-243-14/+60
|\ \ \ \
| * | | | Do not require port argument when updating floating IPyanpuqing2018-06-133-14/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When setting floating ip other properties, port argument is force to use. The patch modifies the command, when setting floating ip other properties, like tags, no need port argument. Change-Id: I908712c8913f32d3dd5fdfefe7347277d72f66de Story: 1751431 Task: 13865
* | | | | Pass volume snapshot size to volume createMonty Taylor2018-07-232-4/+14
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When creating a volume from a snapshot, the size parameter is required and type is checked. Since we have to pass something and it needs to be a valid data type (None is not acceptable) grab the size from the snapshot object and pass it. Change-Id: Ie23e3d23828919234e40336b5c65b22e140d337c
* | | | Merge "Slow down and retry aggregate create/delete to lessen race"Zuul2018-07-131-0/+18
|\ \ \ \
| * | | | Slow down and retry aggregate create/delete to lessen raceDean Troyer2018-07-061-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I676894c1bfd3156313d88e6457250b9ff226118b Signed-off-by: Dean Troyer <dtroyer@gmail.com>
* | | | | Merge "Fix docs from I0dc80bee3ba6ff4ec8cc3fc113b6de7807e0bf2a"Zuul2018-07-132-4/+8
|\ \ \ \ \
| * | | | | Fix docs from I0dc80bee3ba6ff4ec8cc3fc113b6de7807e0bf2aMatt Riedemann2018-06-282-4/+8
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The CLI usage docs should mention the microversion restriction for creating a flavor with a description and setting a description on a flavor. Also, the release note talks about setting the server description on the flavor commands (wrong resource). Change-Id: I93af3da9a0be62395a1c719fd1dcbfd2b669580d Story: 2002196 Task: 22607
* | | | | Merge "Skip calls to glance and nova when got no servers"Zuul2018-07-112-2/+21
|\ \ \ \ \
| * | | | | Skip calls to glance and nova when got no serversPavlo Shchelokovskyy2018-07-092-2/+21
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | save (potentially many) HTTP calls to Glance API for image list and a call to Nova API for flavor list when the server list actually returned no servers. Change-Id: I93a56138c50b82fb4dce67a2f788107f71c5f423 Story: #2002039 Task: #19681
* | | | | Merge "Replace pbr autodoc with sphinxcontrib-apidoc"Zuul2018-07-114-27/+16
|\ \ \ \ \
| * | | | | Replace pbr autodoc with sphinxcontrib-apidocStephen Finucane2018-07-104-27/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes local building of the documentation using tox, and allows the gate to stop relying on pbr and move completely to the new docs PTI. http://lists.openstack.org/pipermail/openstack-dev/2018-March/128594.html Change-Id: I485acda07098a435753e91c1ca45e586de199c35
* | | | | | Fix lower-constraints.txtTuan Do Anh2018-07-111-1/+1
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | During the change https://review.openstack.org/#/c/573216/ neutron-vpnaas lower-constraints.txt looks out-of-date. This commit fixes lower-constraints.txt. Change-Id: Id9cfb463a98bdcc3c45505d8701c515549ecaa55
* | | | | Merge "Make Profile fallback go bye-bye"Zuul2018-07-101-41/+11
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Make Profile fallback go bye-byeMonty Taylor2018-03-141-41/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's time to move on with life. (dtroyer) This requires sdk >= 0.10.0 (0.9.19 is in global-requirements.txt) and osc-lib >= 1.8.0 (1.8.0 is already in g-r). Once we have sdk bumped make_client() becomes just a pass-through for the plugin interface and existing code compatibility. Change-Id: Ie3d7c442da4257b11140c109e9df69b629336f42
* | | | | Merge "Retry floating IP tests"Zuul2018-07-061-4/+36
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Retry floating IP testsDean Troyer2018-07-061-4/+36
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | test_server_attach_detach_floating_ip() has a test for server add/remove floating IP that seems to be racy, add a retry loop to let neutron and nova do their thing before calling it bad. Change-Id: I999a0d7dae1706d746053bafb7ab4e3b791d0042
* | | | Merge "Adding api_version to FakeApp"Zuul2018-06-291-0/+1
|\ \ \ \
| * | | | Adding api_version to FakeAppTelles Nobrega2018-06-241-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to use app.api_version['<some_service>'] we need to have the api_version on FakeApp otherwise the test will fail. Patch that will benefit from this: https://review.openstack.org/#/c/572536/ Change-Id: Ibb3c548daf2a62b6c4aefb5e257372dd5c56521e
* | | | | Merge "Display private flavors in server list"Zuul2018-06-292-1/+7
|\ \ \ \ \
| * | | | | Display private flavors in server listnpraveen352018-03-212-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update the code so that "openstack server list --all" also displays the names of private flavors. Change-Id: I4804fcd905eaf67b1ad9b461084eaf0caa820d2f Closes-Bug: #1742453
* | | | | | Merge "Add ability to filter image list by tag"Zuul2018-06-295-7/+55
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | Add ability to filter image list by tagAdam Harwell2018-06-285-7/+55
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I2e222d3e69df9d8d7cd472663caaee31bedd848c
View Lorry Controller Status