summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Fixes the issue that instance bond port can't get IP address" into ↵16.0.2Zuul2020-11-236-7/+116
|\ | | | | | | stable/victoria
| * Fixes the issue that instance bond port can't get IP addressKaifeng Wang2020-11-136-7/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The issue is that when a port group doesn't have a mac address assigned by operators, and during provisioning we unbind/bind tenant port with None which causes the mac address to be regenerated twice and differs from the originally one allocated by nova or users which was packed into config drive. The end result is that, bond port has different mac address configured and can't the IP address from neutron. Change-Id: I92ed5d17239216324d6a69e0ed8771fd6948d6ec Story: 2008300 Task: 41185 (cherry picked from commit fe01ddb2bce12dc1693d64a30e5b1b822e7b839e)
* | Merge "CI: update bindep for centos-8 py36 job changes" into stable/victoriaZuul2020-11-232-1/+24
|\ \
| * | CI: update bindep for centos-8 py36 job changesJulia Kreger2020-10-042-1/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Opendev infra changed py36 defaults to run on centos8 and of course, there is no python-devel mapping on centos. Removed the entry and adjusted the test-setup.sh script so the databases are started locally as they are not auto started upon installation. Change-Id: I826757ad73b0b14f119f9205475379b85f111383 (cherry picked from commit eac7226ce29bc291b4b9bdc34775b22b147f9f72)
* | | Merge "Limit the default value of [api]api_workers to 4" into stable/victoriaZuul2020-11-194-8/+32
|\ \ \
| * | | Limit the default value of [api]api_workers to 4Dmitry Tantsur2020-11-184-8/+32
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | Each ironic-api process consumes non-negligible amount of RAM, defaulting to CPU core count may result in many hundres of megabytes occupied by ironic-api processes. Limit the default value to 4 and let people who actually need more than that pick their value. Change-Id: I5aefa8c6c7aadc56aea151647e1c0a5af54ada4c (cherry picked from commit c9c492725ed581db1ba49a1a5985bfcf4952e82d)
* | | Fix incorrect network_data.json locationDmitry Tantsur2020-11-132-2/+5
|/ / | | | | | | | | | | | | There is no metadata subdirectory, it goes right into openstack/latest. Change-Id: I576c3c85515970262b5e7480913ff7daefa1b539 (cherry picked from commit 3fd513ee1c5d9846b7d44ccd7f591e42a4b88f97)
* | Merge "Fix DHCP-less operations with the noop network interface" into ↵Zuul2020-11-125-33/+8
|\ \ | | | | | | | | | stable/victoria
| * | Fix DHCP-less operations with the noop network interfaceDmitry Tantsur2020-11-125-33/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | The base implementation of get_node_network_data returns {} and is not overridden in the noop network. Update the base implementation to use task.node.network_data and remove the excessive logging. Change-Id: Ie50dcd1c2a151f5dd09794467792527032249809 (cherry picked from commit 2e5d01d48dc3a231140a1680a7082be436f5dd55)
* | | Merge "Move the multinode grenade job to the experimental pipeline" into ↵Zuul2020-11-121-4/+4
|\ \ \ | | | | | | | | | | | | stable/victoria
| * | | Move the multinode grenade job to the experimental pipelineDmitry Tantsur2020-10-281-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's hopelessly broken, let's not waste resources on it until we get back to making it work. Change-Id: I171fa566e36ad5ac8659ecb0578029df270497d6 (cherry picked from commit 2e2b07bb91a95dcafbd85bc6c2c92016bd027c0d)
* | | | Fix redfish BIOS apply config error handlingAija Jauntēva2020-11-113-11/+77
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of using process_event('fail') use error_handlers, otherwise in case of failure node gets stuck and fails because of timeout, instead of failing earlier due to step failure. Besides adding new unit tests, also update related unit tests to test for success correctly and have realistic data. Story: 2008307 Task: 41196 Change-Id: If28ccb252a87610e3fd3dc78e1ed75bb8ca1cdcf (cherry picked from commit 70b7ca345f39b5126b5e68aa507298b623932dd5)
* | | json-rpc: surround IPv6 address with [] in conductor URLZane Bitter2020-11-103-1/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the conductor's host option is configured to be an IPv6 address, we need to surround it with [] when incorporating it in a URL to contact the conductor over json-rpc. Change-Id: Ib3bc4c570ec0f2e5c73e3ce15b05684b8e4c1ff9 Story: 2008288 Task: 41166 (cherry picked from commit 85b489288669af342bbbd4b0a875580a346c2441)
* | | Merge "Use centos as base element for dib images" into stable/victoriaZuul2020-11-051-1/+1
|\ \ \
| * | | Use centos as base element for dib imagesRiccardo Pittau2020-10-291-1/+1
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | At the moment, it's not possible to use diskimage-builder to create images based on centos-minimal on ubuntu focal. To be able to use ubuntu focal as nodeset, we start using the centos dib element. Change-Id: Ibe363729038da92c630c4d8c4deedb2507515999 (cherry picked from commit 86bee227c96be02077a47c41a6cdd92357e4fff0)
* | | Merge "Make redfish-virtual-media respect default_boot_mode" into ↵Zuul2020-11-033-3/+29
|\ \ \ | | | | | | | | | | | | stable/victoria
| * | | Make redfish-virtual-media respect default_boot_modeDmitry Tantsur2020-11-023-3/+29
| | | | | | | | | | | | | | | | | | | | Change-Id: I46c865ba1cc05a60aa9703f0b35247b62ad4235a (cherry picked from commit 2dfb3f5ecaecfcbf86b36a007a0dc742cfefe960)
* | | | Prevent timeouts when using fast-track with redfish-virtual-mediaDmitry Tantsur2020-11-023-0/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Calling prepare_ramdisk may break fast-track, as it's the case with redfish-virtual-media (it powers nodes off unconditionally). To avoid timeouts, check fast-track status again after prepare_ramdisk. Change-Id: Iad2d6f4827bd7e8b2a02005fe18d31ec8d37db97 (cherry picked from commit 551ca9c8f7556b162afe28d1b1ba020a06251555)
* | | | Merge "CI: increase cleaning timeout and tie it to PXE boot timeout" into ↵Zuul2020-11-022-5/+8
|\ \ \ \ | |/ / / |/| | | | | | | stable/victoria
| * | | CI: increase cleaning timeout and tie it to PXE boot timeoutDmitry Tantsur2020-10-312-5/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're seeing cases where cleaning barely manages to finish after a 2nd PXE retry, failing a job. Also make the PXE retry timeout consistent between the CI and local devstack installations. Change-Id: I6dc7a91d1a482008cf4ec855a60a95ec0a1abe28 (cherry picked from commit 87e634dee1dfaec80b428cad459d174d65ce1b68)
* | | | Merge "Sync boot mode when changing the boot device via Redfish" into ↵Zuul2020-11-023-11/+30
|\ \ \ \ | | | | | | | | | | | | | | | stable/victoria
| * | | | Sync boot mode when changing the boot device via RedfishBob Fournier2020-10-283-11/+30
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After changing the boot device via Redfish, check that the boot mode being reported matches what is configured and, if not, set it to the configured value. Some BMCs change the boot mode when the device is set via Redfish, this will ensure the mode is set properly. Change-Id: Ib077f7f32de029833e6bd936853c382305bce36e Story: 2008252 Task: 41103 (cherry picked from commit 685131fd3658b2ff0b131d19d32fd82d74f3bb17)
* | | | Merge "Do not pass BOOTIF=None if no BOOTIF can be guessed" into stable/victoriaZuul2020-11-015-7/+13
|\ \ \ \ | |_|/ / |/| | |
| * | | Do not pass BOOTIF=None if no BOOTIF can be guessedDmitry Tantsur2020-10-145-7/+13
| |/ / | | | | | | | | | | | | | | | | | | It breaks inspection with the default add_ports=pxe. Change-Id: I730b4bbd48e7188148669670fdb742b88a62f820 (cherry picked from commit 7a89ddcf0cb3a6499cd00fc25c092147210c1fa2)
* | | migrate testing to ubuntu focalRiccardo Pittau2020-10-292-7/+21
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | As per victoria cycle testing runtime and community goal we need to migrate upstream CI/CD to Ubuntu Focal(20.04). keeping few jobs running on bionic nodeset till https://storyboard.openstack.org/#!/story/2008185 is fixed otherwise base devstack jobs switching to Focal will block the gate. Change-Id: I1106c5c2b400e7db899959550eb1dc92577b319d Story: #2007865 Task: #40188 (cherry picked from commit 4775288b936cc60f8808ee4af6c5a9133d9d4cfb)
* | Merge "Fix inspection for idrac" into stable/victoria16.0.1Zuul2020-10-122-0/+12
|\ \
| * | Fix inspection for idracIury Gregory Melo Ferreira2020-10-042-0/+12
| |/ | | | | | | | | | | | | | | | | | | | | This commit fix an issue to inspect nodes using idrac when using redfish virtual media. We are using the redfish configuration so it can be backported. Change-Id: I478c25fac13b49867349c2d9fc8d206c9994c398 Story: #2008221 Task: #41010 (cherry picked from commit db557003848f416c252dd8a80ae5c338d4973c09)
* | Merge "Wiping agent tokens on reboot via API - take 2" into stable/victoriaZuul2020-10-083-3/+11
|\ \
| * | Wiping agent tokens on reboot via API - take 2Dmitry Tantsur2020-10-073-3/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because of using an incorrect variable, reboot was treated as power on, and the token was not wiped. Change-Id: I656450c2bedc3dc0d20a70de78cc29bf64d5fe85 Story: #2008097 Task: #40799 (cherry picked from commit e39858dd8c4e4fa2d2779280090b034dd594601c)
* | | Trivial: use the correct error handler for do_next_clean_stepDmitry Tantsur2020-10-071-1/+1
|/ / | | | | | | | | Change-Id: Id02f8557d3a34c0ec0f8c04ecd37a170e5cc029c (cherry picked from commit c9c605d8b83270edbdb56c6e77d2a2ec339bba3e)
* | Update TOX_CONSTRAINTS_FILE for stable/victoriaOpenStack Release Bot2020-10-011-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | Update the URL to the upper-constraints file to point to the redirect rule on releases.openstack.org so that anyone working on this branch will switch to the correct upper-constraints list automatically when the requirements repository branches. Until the requirements repository has as stable/victoria branch, tests will continue to use the upper-constraints list on master. Change-Id: I2de0389458c89441c4bd142052f681a52caf6b4b
* | Update .gitreview for stable/victoriaOpenStack Release Bot2020-10-011-0/+1
|/ | | | Change-Id: I9429a142208a63232062f78cd4907d9deac69c1a
* Merge "Feat: add ibmc hardware info support for devstack"16.0.0Zuul2020-10-011-0/+19
|\
| * Feat: add ibmc hardware info support for devstackQianbiao.NG2020-09-011-0/+19
| | | | | | | | | | | | | | Add ibmc hardware type support when generate testing nodes for ironic devstack Change-Id: I6f9f129256151f0a721f2b1962477ddef0ab453e
* | Add GPU reporting to idrac-wsman inspect interfaceMudit2020-09-305-5/+145
| | | | | | | | | | | | | | | | | | | | | | This patch implements reporting number of NVIDIA Tesla T4 devices connected to a system by discovering such devices and reporting them through capability 'pci_gpu_devices'. Change-Id: If713895f05f08a9827c4c085108abb3e388b2a2e Story: 2008118 Task: 40839 Depends-On: https://review.opendev.org/#/c/750364/
* | Merge "Handle conductor_affinity earlier in the deployment process"Zuul2020-09-304-30/+28
|\ \
| * | Handle conductor_affinity earlier in the deployment processDmitry Tantsur2020-09-294-30/+28
| | | | | | | | | | | | | | | | | | | | | Not sure why we do it only after the first deploy step, seems more logical to it before running steps. Will simplify further refactoring. Change-Id: Ia102c8697a9d23253c8b86fef71fc13ab9d68753
* | | Merge "Handle patching node /protected value with None"Zuul2020-09-302-0/+8
|\ \ \
| * | | Handle patching node /protected value with NoneSteve Baker2020-09-292-0/+8
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Calling baremetal node unset --protected will attempt to set the protected attribute to None, which raises a 500 error because the object attribute is not nullable[1]. This change checks for this case in the patch call and sets protected to False. This appears to be the only boolean attribute affected by this issue (node, port, and portgroup boolean attributes were checked). [1] https://opendev.org/openstack/ironic/src/branch/master/ironic/objects/node.py#L146 Change-Id: I561e059218a99154e77df075aeadd68f56b86267 Story: 2008205 Task: 40989
* | | Merge "Don't migrate away from iscsi if it is the default"Zuul2020-09-302-0/+14
|\ \ \
| * | | Don't migrate away from iscsi if it is the defaultMark Goddard2020-09-302-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We hit an issue in Kayobe CI where we have the following config: enabled_deploy_interfaces: iscsi,direct default_deploy_interface: iscsi In practice, direct does not work out of the box in Kayobe currently. The online migration added recently to move nodes from iscsi to direct caused issues in our upgrade jobs, since it migrated nodes to direct, without the config in place to make this work. This change avoids migrating nodes when iscsi is the default deploy interface. Story: 2008114 Task: 40996 Change-Id: I5cf0684dc53a2d89fac12e578b28e73e7697a97e
* | | | Merge "Update 'idrac' driver requirements for Victoria"Zuul2020-09-301-2/+2
|\ \ \ \
| * | | | Update 'idrac' driver requirements for VictoriaRichard Pioso2020-09-281-2/+2
| | |/ / | |/| | | | | | | | | | Change-Id: Ib3ad5a0f41e8a5c5eda3fa43aac3ea74624528fd
* | | | Merge "Trivial: fix formatting in the prelude"Zuul2020-09-301-9/+10
|\ \ \ \
| * | | | Trivial: fix formatting in the preludeDmitry Tantsur2020-09-291-9/+10
| | | | | | | | | | | | | | | | | | | | Change-Id: I967af0e280650156c3a9794a80e3af1177a5ffd1
* | | | | Merge "Minor agent version code cleanup"Zuul2020-09-305-32/+32
|\ \ \ \ \
| * | | | | Minor agent version code cleanupJulia Kreger2020-09-295-32/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Internal to the conductor, we have long determiend that if no agent version is submitted by the client, that the client is a version 3.0.0 agent. With agent token, we're effectively requiring 6.1.0 or later to be leveraged, so all agents should be sending versions. An agent not sending versions is thus unsupported and would no longer work without agent token support. Redudnant code has thus been removed. Additionally the conductor utility is_agent_token_supported has been removed since it is now redundant. Change-Id: Id6c8d1df08c3ac7af61ed7d05d274f3099003582
* | | | | | Merge "Fix handling OctetString for pysnmp"Zuul2020-09-305-6/+25
|\ \ \ \ \ \
| * | | | | | Fix handling OctetString for pysnmpYushiro FURUKAWA2020-09-295-6/+25
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit fixes getting MAC address while inspection for iRMC driver. To remove ord() in retrieving MAC address, it can avoid the following exception: TypeError: ord() expected string of length 1, but int found Also, this commit sets the following type of mock for UT. So, I add 'pysnmp'[1] into test-requirements.txt and lower-constraints.txt. * pysnmp.proto.rfc1902.OctetString * pysnmp.proto.rfc1902.Integer32 [1] https://bit.ly/3bDeTCH Change-Id: I1e013a93854e01a7060e1fc48aac091e7e9b74cb
* | | | | | Merge "Fix a race condition in the hash ring code"Zuul2020-09-303-16/+24
|\ \ \ \ \ \
View Lorry Controller Status