summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Update .gitreview for bugfix/6.5bugfix/6.5-eolbugfix/6.5OpenStack Release Bot2020-12-301-0/+1
| | | | Change-Id: I9ff56b6c8a1c534c8962fc56335d416108196d6c
* Merge "Handle situation when a configdrive is already mounted"6.5.0Zuul2020-12-212-24/+92
|\
| * Handle situation when a configdrive is already mountedDmitry Tantsur2020-12-162-24/+92
| | | | | | | | | | | | | | Glean mounts the configdrive and does not unmount it afterwards. If a mount point already exists, just use it. Change-Id: Ia62279afbb9fd9770864942dc40629b69ae8f4ae
* | Fix boot mode detection for partition imagesJulia Kreger2020-12-193-3/+10
| | | | | | | | | | | | | | | | | | | | | | | | Previously, partition images were hard coded to be bios based as opposed to consulting all of the values AND the node itself before making the most appropriate determination. Now the agent utilises the internal helper to properly determine the boot mode when calling ironic-lib. Story: 2008070 Task: 41265 Change-Id: Id5eeda69d5b9de2b393af414472d57b0d4380c43
* | Change default ironic_lib invocation to flag local bootingJulia Kreger2020-12-191-1/+1
| | | | | | | | | | | | | | | | | | | | The partition image support has been telling ironic-lib that the machine will be local booted. While this is likely harmless, and doesn't seem to break anythign, we should have it match moving forward just to be on the safe side so we don't accidently break things down the road. Change-Id: I33e5d583964ef8c21aa04d7427bcd3957b89d449
* | Merge "Add fstab pointer to EFI partition"Zuul2020-12-173-7/+271
|\ \
| * | Add fstab pointer to EFI partitionJulia Kreger2020-12-173-7/+271
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds support for the EFI partition to be appended to fstab so the filesystem can be automounted and EFI loader updated should the deployed operating system need to do so. This should enable bootloaders to be upgraded by linux based operating systems after the instance has been deployed when a partition image was utilized for the initial deployment. Change-Id: Iec28a8841cc01ec8b01a3f5cca070c934c7a2531 Story: 2008070 Task: 40754
* | | Merge "Prevent broken partition image UEFI deploys"Zuul2020-12-173-68/+764
|\ \ \ | |/ /
| * | Prevent broken partition image UEFI deploysJulia Kreger2020-12-143-68/+764
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Partition images can sometimes contain a /boot folder structure event he assets for EFI booting on that filesystem. Which is a good thing. The conundrum is that Ironic does not handle this properly and potentially replaces the bootloader in this sequence such that grub2-install is used instead of signed bootloader assets. As such, we should be preserving the assets and using them from a partition image much like we do when we have a wholedisk image and can identify the assets. Now we will preserve the EFI boot assets, copy them to the new EFI boot partition, and call the EFI setup methods to manage the EFI nvram. Note, this change also splits the logic path out that performs the end call of the EFI boot manager into a reusable method but does not retool all of the testing as it is intertwined in the install_grub2 testing. Also adds some additional debug logging, as much of the bootloader installation code has multiple fallback/cleanup points which makes it difficult to debug from logs. Story: 2008070 Task: 40753 Change-Id: If17d4b4c06df5504987e61a1fde6662e9acd6989
* | | Merge "Fix default disk label with partition images"Zuul2020-12-174-1/+74
|\ \ \ | |/ / | | / | |/ |/|
| * Fix default disk label with partition imagesJulia Kreger2020-12-144-1/+74
| | | | | | | | | | | | | | | | | | | | | | | | Partition images through the agent have the unfortunate side effect of being executed without full node context by default. Luckilly we've had a similar problem and cache the node. This patch changes the lookup from a default of msdos partitions to use the cached node object. Change-Id: I002816c9372fdf1cc32f3c67f420073551479fd9
* | Merge "Correctly decode error messages from ironic API"Zuul2020-12-163-12/+89
|\ \
| * | Correctly decode error messages from ironic APIDmitry Tantsur2020-12-073-12/+89
| | | | | | | | | | | | | | | | | | Knowing a status code is simply not enough for debugging. Change-Id: If1d3f182ab028948ff05aea7e8024d4e7bc3d53c
* | | Merge "Remove lower-constraints job"Zuul2020-12-162-87/+0
|\ \ \
| * | | Remove lower-constraints jobRiccardo Pittau2020-12-152-87/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | As discussed during the upstream ironic community meeting on Monday Dec 14 2020, the lower-constraints job is being removed. Change-Id: Ide0f6c38a59ae6486fa33cfb19b383d022e57d5a
* | | | Merge "Generate TLS certificates with validity time in the past"Zuul2020-12-154-3/+32
|\ \ \ \
| * | | | Generate TLS certificates with validity time in the pastDmitry Tantsur2020-12-104-3/+32
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | Otherwise a slight clock skew may prevent them from working, see e.g. https://bugzilla.redhat.com/show_bug.cgi?id=1906448. Change-Id: Icea103af06edef16c0dc4578877dc04cd6ec3b0c
* | | | Merge "Avoid a full install in tox environments that do not need it"Zuul2020-12-151-5/+12
|\ \ \ \ | |_|/ / |/| | |
| * | | Avoid a full install in tox environments that do not need itDmitry Tantsur2020-12-141-5/+12
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | Usage of usedevelop=True results in all requirements.txt dependencies always installed (without constraints) even when requirements.txt is not explicitly added. Override usedevelop for these environments. Add missing constraints to the bandit environment. Change-Id: I037dc04ffc4f426c828667df010ebe4e357e7fda
* | | Merge "Copy any configuration from the virtual media"Zuul2020-12-144-103/+221
|\ \ \
| * | | Copy any configuration from the virtual mediaDmitry Tantsur2020-11-234-103/+221
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For ramdisk TLS (and other potential future enhancements) we need to be able to inject configuration and certificates into the ramdisk. Since we cannot pass files through kernel parameters, we need to put them on the generated ISO or (in the future) config drive. This change detects IPA configuration and copies it into the ramdisk early enough for any configuration files to get picked. Changed /dev/disk/by-label to blkid since the former may not exist on all ramdisks (e.g. tinyIPA). Change-Id: Ic64d7842a59795bbf02f194221dedc07c6b56e8c
* | | | Upgrade version of doc8Riccardo Pittau2020-12-142-3/+2
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | The doc8 module explicitl supports Python 3.6 starting from version 0.8.1 Also remove it from test-requirements as we already have it in tox.ini Change-Id: Idc66758dc350d9a7a09b4295b08f4423cec288dc
* | | Merge "Option to enable bootloader config failure bypass"Zuul2020-12-144-9/+178
|\ \ \
| * | | Option to enable bootloader config failure bypassJulia Kreger2020-12-104-9/+178
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some hardware is very well intentioned. However this intention can result in the UEFI NVRAM table being full which prevents us from adding new records to the table. We can't be sure what to delete, so in this case some operators just need the ability to tell ironic "it is okay if this fails, it will still work." The added ``ignore_bootloader_failure`` option adds this capability which can be set per-node either in the agent configuation via the ramdisk image, or in the pxe_append_params configuration parameter for the node itself with a ``ipa-ignore-bootloader-failure`` option in order to prevent the failure from being raised. Change-Id: If3c83fb2ea2025fce092d495a64f32077c70d2d6 Story: 2008386 Task: 41309
* | | Fix lower-constraints with the new pip resolverIury Gregory Melo Ferreira2020-12-093-4/+7
|/ / | | | | | | | | | | | | | | | | * Remove pep8 dependencies from test-requirements, they're not needed there and are hard to constraint properly. * Add cliff to l-c (same version we have on ironic) Change-Id: I8e7582492276db2308217460af37f691fb99c0ce
* | Merge "Bring up VLAN interfaces and include in introspection report"Zuul2020-12-025-1/+344
|\ \
| * | Bring up VLAN interfaces and include in introspection reportBob Fournier2020-11-205-1/+344
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the ability to bring up VLAN interfaces and include them in the introspection report. A new configuration field is added - ``ipa-enable-vlan-interfaces``, which defines either the VLAN interface to enable, the interface to use, or 'all' - which indicates all interfaces. If the particular VLAN is not provided, IPA will use the lldp info for the interface to determine which VLANs should be enabled. Change-Id: Icb4f66a02b298b4d165ebb58134cd31029e535cc Story: 2008298 Task: 41183
* | | Merge "Reuse the docs deps to benefit from constraints"Zuul2020-11-251-0/+2
|\ \ \
| * | | Reuse the docs deps to benefit from constraintslikui2020-11-181-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tox trying to install latest versions for building docs, which may not be supported by stable and lower branches, so should be restricted by respective version's upper-constraints.txt Change-Id: I1e134bb72b6fef3fba8f20d3b3d65520c66bbbf1
* | | | Merge "Make mdadm a soft requirement"Zuul2020-11-232-0/+12
|\ \ \ \ | |_|_|/ |/| | |
| * | | Make mdadm a soft requirementDmitry Tantsur2020-11-202-0/+12
| |/ / | | | | | | | | | | | | | | | No point in requiring it for deployments that don't use software RAID. Change-Id: I8b40f02cc81d3154f98fa3f2cbb4d3c7319291b8
* | | Merge "Remove the unused coding style modules"Zuul2020-11-231-8/+0
|\ \ \
| * | | Remove the unused coding style moduleslikui2020-11-121-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Python modules related to coding style checks (listed in blacklist.txt in openstack/requirements repo) are dropped from lower-constraints.txt they are not needed during installation. Change-Id: I941c2247692a74e0fd0fb350135f8610af11d7bb
* | | | Merge "Add clean step 'erase_pstore'"Zuul2020-11-184-0/+117
|\ \ \ \
| * | | | Add clean step 'erase_pstore'Arne Wiebalck2020-11-174-0/+117
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add an automatic clean step to clean the Linux kernel's pstore. The step is disabled by default. Story: #2008317 Task: #41214 Change-Id: Ie1a42dfff4c7e1c7abeaf39feca956bb9e2ea497
* | | | | Fix vendor info retrieval for some versions of lshwVladyslav Drok2020-11-162-5/+14
| |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is one more place that relies on lshw json output being a dict, so let's fix the function that gets the dict rather than places it is being used in. Change-Id: Ia1c2c2e6a32c76ac0249e6a46e4cced18d6093a9 Task: 39527 Story: 2007588
* | | | Merge "Log configuration options on start-up"Zuul2020-11-123-2/+10
|\ \ \ \
| * | | | Log configuration options on start-upDmitry Tantsur2020-11-113-2/+10
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is very convenient for debugging and is something ironic and ironic-inspector already do. Register SSL options earlier so that they're accounted for. Change-Id: I56aca8eec1dfeb065ac657452a7076a9e3d17cc3
* | | | Merge "Add UUID to BlockDevice object"Zuul2020-11-114-41/+49
|\ \ \ \ | |/ / / |/| | |
| * | | Add UUID to BlockDevice objectVladyslav Drok2020-11-114-41/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It'd allow for example custom ansible playbooks to use UUIDs of the introspected node's disks. In future it might also enable agent to use UUID (or by_path value) to refer to a device instead of name, as it happens currently. Change-Id: Id00437d2295c39fb12f3c25a92b30b56a58eef13
* | | | Merge "Support using LABEL as identifier for rootfs"Zuul2020-11-103-6/+37
|\ \ \ \
| * | | | Support using LABEL as identifier for rootfsFedor Tarasenko2020-11-033-6/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add possibility to use disk LABEL to identify rootfs uuid for Software RAID deployment Change-Id: I77f36e70ddc539af0190db1c1abe0fb2c66f34b4 Story: 2008303 Task: 41188
* | | | | Fix physical memory calculation with new lshwVladyslav Drok2020-11-062-2/+14
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It seems that fix Id5a30028b139c51cae6232cac73a50b917fea233 was dealing with a different issue. According to the description in the story, and the linked commit there, the problem is the fact that output is changed from dictionary to a list (with just one value supposedly?). This commit changes the isinstance call to check if an output of lshw is a list, and if so, we just use the first element of the list. Story: 2007588 Task: 39527 Change-Id: I87d87fd035701303e7d530a47b682db84e72ccb9
* | | | Merge "Updated Implementation of string interpolation delay on LOG messages"Zuul2020-11-041-67/+72
|\ \ \ \ | |_|_|/ |/| | |
| * | | Updated Implementation of string interpolation delay on LOG messagesebagakis2020-11-031-67/+72
| | | | | | | | | | | | | | | | | | | | | | | | This is a follow up to https://review.opendev.org/#/c/756300/ Change-Id: Ifba8a57b58d61ede169c60f6d51f224d134c7708
* | | | Merge "Add example for custom disk erasure"Zuul2020-11-034-0/+94
|\ \ \ \
| * | | | Add example for custom disk erasureJay Faulkner2020-11-024-0/+94
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's a common use case for operators to need to use vendor utilities to erase block devices. This adds an example that specifically addresses this use case. Change-Id: I20dfc37e04466dc0ded9571637818e8f6fb10216
* | | | | Merge "Use TOX_CONSTRAINTS_FILE"Zuul2020-11-031-1/+1
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Use TOX_CONSTRAINTS_FILElikui2020-11-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | UPPER_CONSTRAINTS_FILE is old name and deprecated -https://zuul-ci.org/docs/zuul-jobs/python-roles.html#rolevar-tox.tox_constraints_file This allows to use uppper-constraints file as more readable way instead of UPPER_CONSTRAINTS_FILE=<lower-constraints file>. [1] https://review.opendev.org/#/c/722814/ Change-Id: Ibd7895fe3856a179b9f90104015057a38a062aa9
* | | | | Merge "Follow-up to API version setting"Zuul2020-11-022-7/+10
|\ \ \ \ \
View Lorry Controller Status