summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Remove reno job and cap tox to version lower than 4bugfix/8.1Riccardo Pittau2023-01-272-1/+2
| | | | | | | It does not make sense to run it on bugfix branches. Tox is capped in all stable branches, let's cap it here as well. Change-Id: I202317cacefdb69e6c32750c5b0e11cf07636bf4
* Merge "Fixes for tox 4.0" into bugfix/8.1Zuul2022-12-191-6/+7
|\
| * Fixes for tox 4.0Jay Faulkner2022-12-151-6/+7
| | | | | | | | | | | | | | - passenv needed better structure - usedevelop no longer allowed alongside skipsdist Change-Id: I22dae3b1e4cca2dc328c21e889acf7fe98a80e0b
* | Fix CI for bugfix/8.1Riccardo Pittau2022-12-162-0/+13
|/ | | | | | | All jobs should run on ubuntu focal, except python3.6 that still needs bionic. Change-Id: Ia42bf96ebb5083f6ec874eb97c7478776cc18753
* CI: Zuul no longer respects queue paramJay Faulkner2022-10-142-5/+8
| | | | | | | | | | | | | Also disable use of swift tempurls as swift seems to sporatically fail on accessing assets, and the thing being tested here is not swift. Disabling the redfish virutal media job, as it is failing due to swift issues, and it is unlikely for them to be fixed. Besides, there is no local override to bypass swift on older versions. Change-Id: I1e7a3b3c9ded13b10002bb47e98d4a7b486e1dd4
* Gather details about bond interfaces if presentDerek Higgins2022-07-133-11/+66
| | | | | | | | | | If present gather information about bonded interfaces. Story: #2010093 Task: #45637 Change-Id: I394187640b4788ebec21c3391d33ed728fb72ffa (cherry picked from commit 7e4fe3bf6a2ae41656b7923796f9c2d056a2ed04)
* Fix discovering WWN/serial for devicemapper devicesDmitry Tantsur2022-06-223-16/+31
| | | | | | | | UDev prefix is DM_ not ID_ for them. On top of that, they don't have short serials (or at least don't always have). Change-Id: I5b6075fbff72201a2fd620f789978acceafc417b (cherry picked from commit 69e22545033f544d628f9c4ecd5a665ba0b5b85e)
* Multipath Hardware path handlingJulia Kreger2022-06-088-54/+748
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removes multipath base devices from consideration by default, and instead allows the device-mapper device managed by multipath to be picked up and utilized instead. In effect, allowing us to ignore standby paths *and* leverage multiple concurrent IO paths if so offered via ALUA. In reality, anyone who has previously built IPA with multipath tooling might not have encountered issues previously because they used Active/Active SAN storage environments. They would have worked because the IO lock would have been exchanged between controllers and paths. However, Active/Passive environments will block passive paths from access, ultimately preventing new locks from being established without proper negotiation. Ultimately requiring multipathing *and* the agent to be smart enough to know to disqualify underlying paths to backend storage volumes. An additional benefit of this is active/active MPIO devices will, as long as ``multipath`` is present inside the ramdisk, no longer possibly result in duplicate IO wipes occuring accross numerous devices. Story: #2010003 Task: #45108 Resolves: rhbz#2076622 Resolves: rhbz#2070519 Change-Id: I0fd6356f036d5ff17510fb838eaf418164cdfc92 (cherry picked from commit 014d37743a3b5694e0e2a3cabfafe885417172d5) (cherry picked from commit 2c95ee45339dc910a6b29cb5e0f24a1f48898165) (cherry picked from commit 3c9b1131320fb97e81af38fc7471f5dbc968ca12)
* Do not try to guess EFI partition path by its numberDmitry Tantsur2022-06-084-47/+112
| | | | | | | | | | | | | | | The logic of adding a partition number to the device path does not work for devicemapper devices (e.g. a multipath storage device). Conflicts: ironic_python_agent/efi_utils.py ironic_python_agent/extensions/image.py ironic_python_agent/tests/unit/extensions/test_image.py ironic_python_agent/tests/unit/test_efi_utils.py Change-Id: I9a445e847d282c50adfa4bad5e7136776861005d (cherry picked from commit f09f6c9f1a09c7062d0450b3e0a4d3164fd53f7f) (cherry picked from commit 12e03698874d3d51884fd0de3174fa7b56076852)
* Add `mount` and `parted -l` to the collected commandsDmitry Tantsur2022-06-072-2/+12
| | | | | | | | | Conflicts: ironic_python_agent/tests/unit/test_utils.py Change-Id: I1c759552220291890704d0002a62ea3f51701691 (cherry picked from commit f1ee454a0ee9a8f18fbfd504d081ce3aeeb0ffa3) (cherry picked from commit 54cbfa1f3fe0b8935e39044baee8c4996b2a3ced)
* Collect a full lsblk output in the ramdisk logsDmitry Tantsur2022-06-073-6/+8
| | | | | | | | | | | The existing lsblk call is very handy for an overview, but there a lot more useful pairs to collect. Collect them in a machine-readable format to be able to use in debugging and further development. Change-Id: Ib27843524421944ee93de975d275e93276a5597a (cherry picked from commit 424e649bed3db5d1129b18b7ea4dfba88d552537) (cherry picked from commit bc74df8bfe89f703880f8a545ce939e9b1cd8651) (cherry picked from commit c674e567fc3fbf0976b1310763be88ad5a11f7e4)
* Create fstab entry with appropriate labelJulia Kreger2022-06-076-10/+135
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Depending on the how the stars align with partition images being written to a remote system, we *may* end up with *either* a Partition UUID value, or a Partition's UUID value. Which are distinctly different. This is becasue the value, when collected as a result of writing an image to disk *falls* back and passes the value to enable partition discovery and matching. Later on, when we realized we ought to create an fstab entry, we blindly re-used the value thinking it was, indeed, always a Partition's UUID and not the Partition UUID. Obviously, the label type is quite explicit, either UUID or PARTUUID respectively, when initial ramdisk utilities such as dracut are searching and mounting filesystems. Adds capability to identify the correct label to utilize based upon the current state of the block devices on disk. Granted, we are likely only exposed to this because of IO race conditions under high concurrecy load operations. Normally this would only be seen on test VMs, but systems being backed by a Storage Area Network *can* exibit the same IO race conditions as virtual machines. Change-Id: I953c936cbf8fad889108cbf4e50b1a15f511b38c Resolves: rhbz#2058717 Story: #2009881 Task: 44623 (cherry picked from commit 99ca1086dbfc7b6e41cf800b0bd899565e2e8922) (cherry picked from commit c69ea032fe8ab81e459fb44f846f440e7a2c8922)
* Merge "Make the standalone job voting again" into bugfix/8.1Zuul2022-06-011-7/+2
|\
| * Make the standalone job voting againDmitry Tantsur2022-02-011-7/+2
| | | | | | | | | | | | | | | | | | We forgot to revert it. This job covers software RAID and manual cleaning, so it's very important to avoid regressions, even if it costs us some rechecks from time to time. Change-Id: I2446afeaca866ffc3131b5e9f266526f35fc5ed7 (cherry picked from commit b921b761679df06afe19ac6f570f66fa1fdcf371)
* | Fix 8.1 CIIury Gregory Melo Ferreira2022-05-311-2/+4
| | | | | | | | | | Depends-On: https://review.opendev.org/c/openstack/ironic/+/844077 Change-Id: I9d7254239c6476a03350735c36fa90daba8ddc33
* | Merge "Use a pre-defined partition UUID to detect configdrive on GPT" into ↵Zuul2022-05-193-39/+59
|\ \ | | | | | | | | | bugfix/8.1
| * | Use a pre-defined partition UUID to detect configdrive on GPTDmitry Tantsur2022-05-103-39/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using partition numbers is currently broken for devicemapper devices. Fortunately, GPT has partition UUIDs, so we can just generate one and use it for lookup. NOTE on backport: _get_partition is imported from the image extension. In Yoga it was moved to partition_utils. Change-Id: I41ffe4f8e4c6e43182090b5aa2a2b4b34f32efd5 (cherry picked from commit 65c4de903a2d8059b1520b0102235b6700287f87)
* | | Use Xena constraints for bugfix/8.1Dmitry Tantsur2022-05-183-23/+77
|/ / | | | | | | Change-Id: I3afef0bdea7b4289b55d0a04789e2e32b2bfb9e6
* | Merge "Re-read the partition table with partx -a, part 2" into bugfix/8.1Zuul2021-11-233-43/+50
|\ \
| * | Re-read the partition table with partx -a, part 2Riccardo Pittau2021-11-103-43/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use add instead of update to re-read the partition table with partx. See [1] for more details. Co-authored-by: Arne Wiebalck <arne.wiebalck@cern.ch> [1] https: //opendev.org/openstack/ironic-python-agent/commit/dc8c1f16f9a00e2bff21612d1a9cf0ea0f3addf0 Change-Id: I2336e22dadc790cfbde87904612fcaa3b8c501db (cherry picked from commit 23e67b5fea58295d0320f99d4513a66d720f09ee)
* | | Fix UEFI record regexJulia Kreger2021-11-163-6/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I accidently put colons on the test data and remembered taking the colon character out of the regex I was working on, but apparently left it in, and accounted for the active entry indicator flag which appears to have inconsistent support across vendors. The regex has been fixed, and a test added from a Lenovo SR650 which has some additional string entry data in the UEFI output which may separate entries. Change-Id: I1f67b0fb1f645fa82e98bd7c7bba3ffc7755cc74 (cherry picked from commit e10f052c06c03016b0ff4d9c1f3191c79fc50a1a)
* | | Merge "Delete EFI boot entry duplicate labels first" into bugfix/8.1Zuul2021-11-153-24/+51
|\ \ \
| * | | Delete EFI boot entry duplicate labels firstJulia Kreger2021-11-103-24/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some firmware seems to take an objection with EFI nvram entries being deleted after one is added, resulting in the entire entry table being reset to the last known good state. This is problematic, as ultimately deployments can time out if we previously booted with Networking, and the machine, while commanded to do other wise, reboots back to networking regardless. We will now delete entries first, before proceeding. Additionally, for general use, this pattern may serve the community better by avoiding cases where we would have previously just relied upon efibootmgr[0] to warn us of duplicate entries. [0]: https://github.com/rhboot/efibootmgr/blob/103aa22ece98f09fe3ea2a0c83988f0ee2d0e5a8/src/efibootmgr.c#L228 Change-Id: Ib61a7100a059e79a8b0901fd8f46b9bc41d657dc Story: 2009649 Task: 43808 (cherry picked from commit 67eddfa7e3fedbb530045f5b43a2c89db832fa2a) (cherry picked from commit 33b39705a50513c5af411216b48e2a6f6ac9ab14)
* | | | Merge "Output verbose info from efibootmgr" into bugfix/8.1Zuul2021-11-152-21/+21
|\ \ \ \ | |/ / / | | / / | |/ / |/| |
| * | Output verbose info from efibootmgrDerek Higgins2021-11-102-21/+21
| | | | | | | | | | | | | | | | | | | | | | | | When debugging boot manager problems it can be advantageous to see all the full entries rather then just their labels. Change-Id: I6a1bb78acaf5a4284727bdf533d4be6db2099f50 (cherry picked from commit caf695f70ab366498b46cb6f07f6751369c67e30)
* | | Re-read the partition table with partx -aArne Wiebalck2021-11-093-16/+21
|/ / | | | | | | | | | | | | | | | | | | | | | | Re-read the partition table with 'partx -a', rather than 'partx -u'. This should fix an timing issue where the bootloader installation fails to mount the EFI partition from a whole disk image since it is not yet aware of the new partitions (observed with both, the iscsi and the direct deploy interface). Change-Id: If5da3075e813ae01df3decf8f0647aba111b0515 (cherry picked from commit dc8c1f16f9a00e2bff21612d1a9cf0ea0f3addf0)
* | Assert EFI part UUID is not None before editing fstabArne Wiebalck2021-10-212-1/+7
| | | | | | | | | | | | | | | | | | | | | | The EFI partition UUID may be None and this will break the fstab editing. While this is not necessarily fatal when instantiating a node, it creates an exception at the end of bootloader installation, so only attempt to add a line to fstab when the UUID is not None. Change-Id: I68799980e67c05afe4ca68ca9733605dd166d54d (cherry picked from commit 333ed70c94e366f16d8f2633f74a5ef05aa5fadb)
* | Software RAID: Call udev_settle before creationArne Wiebalck2021-10-132-0/+14
|/ | | | | | | | | | | | | This patch fixes a race during software RAID creation: we create the partition with parted, the kernel then notifies udev, but we need to wait for udevd to create the device files before calling mdadm to create the md device. Credits to jcosmao for finding this. Change-Id: I642f28acc351cf50263e37dfbc8468bf59de2cc5 (cherry picked from commit 9d707e9f4bab40109b7e29df2136e86d65325ea3)
* Configure CI for bugfix/8.1Dmitry Tantsur2021-07-221-1/+20
| | | | Change-Id: I62f55afac8b56166b7336231a669fad2ebb3d6f5
* Update .gitreview for bugfix/8.1OpenStack Release Bot2021-07-221-0/+1
| | | | Change-Id: Ia26c96d1f0fef4fb6684b6f4c63057ea767703a9
* Merge "Force immediate NTP time sync with chronyd at IPA startup"8.1.0Zuul2021-07-204-46/+19
|\
| * Force immediate NTP time sync with chronyd at IPA startupArne Wiebalck2021-07-164-46/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to make sure we have the correct time early, e.g. by the time we create a TLS certificate, this patch proposes to force an immediate NTP update when using chronyd. While the previous approach uses the passed NTP server as well, the update may happen only after chronyd has performed measurements (which may be too late). Story: #2009058 Task: #42843 Change-Id: I6edafe8edeb8549f324959e7a1ec175c3049a515
* | Merge "Catch ismount not being handled"Zuul2021-07-162-9/+153
|\ \ | |/ |/|
| * Catch ismount not being handledJulia Kreger2021-06-292-9/+153
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While investigating another grub issue, I was confused by the path taken in the logs reported, and noticed that on a ramdisk, we might not actually have a valid response to os.path.ismount, I'm guessing depending on what in memory filesystem is in use while also coupled with attempting to check a filesystem. Adds a test to validate that exceptions raised on these commands where this issue can be encountered, are properly bypassed, and also adds additional logging to make it easier to figure out what is going on in the entire bootloader setup sequence. Change-Id: Ibd3060bef2e56468ada6b1a5c1cc1632a42803c3
* | Burn-in: Add network stepArne Wiebalck2021-07-136-0/+179
|/ | | | | | | | | | Add a clean step for network burn-in via fio. Get basic run parameters from the node's driver_info. Story: #2007523 Task: #42385 Change-Id: I2861696740b2de9ec38f7e9fc2c5e448c009d0bf
* Merge "Only mount the ESP if not yet mounted"Zuul2021-06-253-7/+20
|\
| * Only mount the ESP if not yet mountedArne Wiebalck2021-06-213-7/+20
| | | | | | | | | | | | | | Check if the ESP is already mounted before attempting to mount it for the bootloader installation. Change-Id: Ifd738b2c5663f1a211d7e13b5ba386be631d8db1
* | Coalesce heartbeatsDmitry Tantsur2021-06-183-9/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The IPA sends heartbeats to the conductor periodically and when requested, e.g. at the end of asynchronous commands. In order to avoid to send such notifications in too quick succession, e.g. when two asynchronous commands finish at the same time or when the periodic heartbeat was just sent right before a command ended, this patch proposes to coalesce heartbeats which are close together timewise and send only one for all of them in a time interval of 5 seconds. Co-Authored-By: Arne Wiebalck <arne.wiebalck@cern.ch> Story: #2008983 Task: 42633 Change-Id: Idfbce44065e1e5a8b730b94741b2604c51f0ab14
* | Merge "Reduce logging verbosity when collecting logs"Zuul2021-06-141-2/+1
|\ \ | |/ |/|
| * Reduce logging verbosity when collecting logsDmitry Tantsur2021-06-111-2/+1
| | | | | | | | | | | | | | | | It's not uncommon that some commands fail when collecting logs. We already log all failures in utils.execute, no need to duplicate them with a non-fatal ERROR logging. Change-Id: If151b3a3be979bd2b3ce01030e5d6242ad74eaa3
* | Merge "Remove lower-constraints job"Zuul2021-06-111-7/+0
|\ \ | |/ |/|
| * Remove lower-constraints joblikui2021-06-111-7/+0
| | | | | | | | | | | | | | As discussed during the upstream ironic community meeting on Monday Dec 14 2020, the lower-constraints job is being removed. Change-Id: I9392117ccdef8fc6ff3d95d667107d752459cb24
* | Utilize CSV file for EFI loader selectionJulia Kreger2021-06-103-14/+137
|/ | | | | | | | | | | | | | | | | | | | | | Adds support to identify and utilize a CSV file to signal which bootloader to utilize, and set it when the OS is running as opposed to when EFI is running. This works around EFI loader potentially crashing some vendors hardware types when entry stored in the image does not match the EFI loader record which was utilzied to boot. Grub2+shim specifically specifically needs the CSV file name and entry label to match what the system was booted with in order to prevent the machine from potentially crashing. See https://storyboard.openstack.org/#!/story/2008962 and https://bugzilla.redhat.com/show_bug.cgi?id=1966129#c37 for more information. Change-Id: Ibf1ef4fe0764c0a6f1a39cb7eebc23ecc0ee177d Story: 2008962 Task: 42598 Co-Authored-By: Bob Fournier <bfournie@redhat.com>
* Merge "Burn-in: Add disk step"Zuul2021-06-076-0/+122
|\
| * Burn-in: Add disk stepArne Wiebalck2021-05-216-0/+122
| | | | | | | | | | | | | | | | | | | | Add a clean step for disk burn-in via fio. Get basic run parameters from the node's driver_info. Story: #2007523 Task: #42384 Change-Id: I5f5e336bd629846b3d779fd0fc7a2060b385b035
* | Merge "Ignore efi grub2-install failure"Zuul2021-06-073-29/+61
|\ \
| * | Ignore efi grub2-install failureSteve Baker2021-06-043-29/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Recent releases of redhat grub2 will always fail when installing to EFI paths, to encourage a transition to the signed shim bootloader. Partition image deploys avoid calling grub2-install with the preserve-efi-assets functions. Deploying whole disk images doesn't require grub2-install. This leaves whole disk images installed onto softraid devices, which still attempts to call grub2-install. This change will still attempt to run grub2-install in this one remaining case, but will ignore any failure. A future enhancement can avoid calling grub2-install entirely so that non-redhat secure-boot capable images can keep their signed bootloaders. Story: 2008923 Task: 42521 Change-Id: If432ef795d64d76442d739eb4f7d155ff847041e
* | | Merge "Refactor: use convert_image from ironic_lib"Zuul2021-06-043-28/+14
|\ \ \
| * | | Refactor: use convert_image from ironic_libDmitry Tantsur2021-06-013-28/+14
| |/ / | | | | | | | | | Change-Id: If890baf3545cff6cef7c645c42e7f9d9038c9aa7
* | | Changed minversion in tox to 3.18.0likui2021-06-031-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | The patch bumps min version of tox to 3.18.0 in order to replace tox's whitelist_externals by allowlist_externals option: https://github.com/tox-dev/tox/blob/master/docs/changelog.rst#v3180-2020-07-23 Change-Id: I434a6330defbc5cc65788f4d64b7f2d4d2b37db7