Skip to content

[adoption_osp_deploy] Allow overriding scenario stack CLI args#3750

Draft
imatza-rh wants to merge 1 commit intoopenstack-k8s-operators:stablefrom
shiftstack:adoption-osp-deploy-stack-args-override
Draft

[adoption_osp_deploy] Allow overriding scenario stack CLI args#3750
imatza-rh wants to merge 1 commit intoopenstack-k8s-operators:stablefrom
shiftstack:adoption-osp-deploy-stack-args-override

Conversation

@imatza-rh
Copy link

Summary

  • Add cifmw_adoption_osp_deploy_stack_args_remove and cifmw_adoption_osp_deploy_stack_args_add variables to the adoption_osp_deploy role
  • These allow callers to modify the TripleO overcloud deploy CLI arguments inherited from the DPA scenario file

Problem

TripleO CLI arguments like --libvirt-type generate internal heat environments that are applied after all user-provided -e files. This means cifmw_adoption_osp_deploy_overcloud_extra_args (which creates internal-configuration.yaml as the last -e file) cannot override them.

For shiftstack adoption jobs, we need --libvirt-type kvm instead of --libvirt-type qemu (nested virt requires KVM), but the existing override mechanism has no effect.

Solution

Use Ansible's difference filter to remove unwanted args and append replacements:

# In Zuul job vars:
cifmw_adoption_osp_deploy_stack_args_remove:
  - '--libvirt-type qemu'
cifmw_adoption_osp_deploy_stack_args_add:
  - '--libvirt-type kvm'

The defaults are empty lists, so this is fully backwards compatible.

Test plan

  • ci-framework-jobs MR will consume this via Depends-On
  • Verify shiftstack adoption deploy-osp produces --libvirt-type kvm in the overcloud deploy command

@imatza-rh imatza-rh marked this pull request as draft March 7, 2026 10:11
@imatza-rh imatza-rh changed the base branch from main to stable March 7, 2026 15:36
Add cifmw_adoption_osp_deploy_stack_args_remove and
cifmw_adoption_osp_deploy_stack_args_add variables to allow
callers to modify the TripleO overcloud deploy CLI arguments
from the DPA scenario.

TripleO CLI arguments like --libvirt-type generate internal
heat environments applied after all user-provided -e files,
so they cannot be overridden via the existing
cifmw_adoption_osp_deploy_overcloud_extra_args mechanism.
The new variables use Ansible's difference filter to remove
unwanted args and append replacements.

Signed-off-by: Itay Matza <imatza@redhat.com>
@imatza-rh imatza-rh force-pushed the adoption-osp-deploy-stack-args-override branch from c4f40bc to aea7374 Compare March 11, 2026 21:58
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 11, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign sathlan for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant