Repository navigation
Synchronise master with upstream - #644
Merged
Merged
Conversation
When inspecting nodes using the agent inspect interface, the node may boot either via ironic_dnsmasq (discovery) or via neutron-dhcp-agent (conductor-managed boot). When conductor prepares the PXE configuration for managed inspection, [inspector] extra_kernel_params was not configured, causing IPA to boot without inspection collectors (ipa-inspection-collectors). IPA then sent an empty data payload to ironic-api, failing schema validation with HTTP 400 and causing inspection to time out. This change: - Configures [inspector] extra_kernel_params in ironic.conf from inspector_extra_kernel_options so conductor-managed inspection boot supplies the required IPA collectors. Depends-On: https://review.opendev.org/c/openstack/kayobe-config-dev/+/1003925 Change-Id: I44773ec8a17314b745f286edf20217a84eeee69b Signed-off-by: Will Szumski <will@stackhpc.com>
This reverts commit 372e505. Inspection failures in CI should be resolved by the preceding fix to configure the Ironic inspection kernel options. Change-Id: I69051bfb43567458ffd701010a65a83d8534b17e Signed-off-by: Will Szumski <will@stackhpc.com>
These jobs are not failing anymore, so there is no reason to keep them as non-voting. There is no impact anyway since this is in the experimental pipeline. Change-Id: Ib0fc67c51dc0974b6031993c5f4a165d9b1c8618 Signed-off-by: Pierre Riteau <pierre@stackhpc.com>
This reverts commit 9e55ada. Reason for revert: IPv6 is fixed in ca-ymq-1 region Change-Id: Ie91a072a05c9345483c03b6038a2547ead8d519c Signed-off-by: Pierre Riteau <pierre@stackhpc.com>
`bifrost_limit` is a list containing bifrost hosts. Ansible no longer allows non-booleans to be implicitly converted to booleans in conditionals. Instead, we check if the length is above 0. Change-Id: Icd6f8525e07f696d5f45c7506e50421326726d81 Signed-off-by: MurrayGroves <murray@stackhpc.com>
Leaving `cpu_mode` as none caused IPA to never boot during inspection when following https://docs.openstack.org/kayobe/latest/contributor/automated.html#seed. When left as none, the hypervisor (libvirt) is allowed to choose which cpu model to use. It chooses qemu64 which is a very basic model and not recommended for use: https://qemu-project.gitlab.io/qemu/system/qemu-cpu-models.html#default-x86-cpu-models Instead we now set it to maximum, which is used in the other tenks dev configs, and ensures all CPU features are passed through. The memory allocation was also too small, causing an out of space error when downloading the deployment image during provisioning. Change-Id: I12baa91022d3c444d67f13b817429d5471c91ebe Signed-off-by: MurrayGroves <murray@stackhpc.com>
Since the automated setup docs are part of the new contributor experience, it is likely the reader lacks context about different ways in which the seed can be provisioned. This adds some details in the seed provisioning explaining the choice between running the seed directly on the host, or using it as a hypervisor. The setup is also unclear that at the end of the seed setup the overcloud has not been setup. Added a brief section with some next steps. Change-Id: Ic4c1c492766886ba4a3abf306d35ff28e8f77395 Signed-off-by: MurrayGroves <murray@stackhpc.com>
Closes-Bug: #2164718 Change-Id: I9a7b12c660978f51b6102d58cc64de0d8dc30dd4 Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
github-actions
Bot
added
automated
Alex-Welsh
approved these changes
Oct 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains a snapshot of master from upstream master.