Skip to content

[DCN] Add tests for cinderBackups#3963

Draft
gais-ameer-rh wants to merge 2 commits into
openstack-k8s-operators:mainfrom
gais-ameer-rh:OSPRH-28343
Draft

[DCN] Add tests for cinderBackups#3963
gais-ameer-rh wants to merge 2 commits into
openstack-k8s-operators:mainfrom
gais-ameer-rh:OSPRH-28343

Conversation

@gais-ameer-rh
Copy link
Copy Markdown

@gais-ameer-rh gais-ameer-rh commented May 26, 2026

spec.cinder.template.cinderBackup (singluar) in DCN DT is
replaced with cinderBackups (plural) to deploy multiple cinder backups per edge sites.
Invoking hooks/playbooks/cinder_backups.yaml playbook to validates the behaviour of cinderBackups in DCN scenario.

The playbook tests different scenarios of cinder backup creation
and restoring the backups across availability zones.

Jira: [OSPRH-28343]
Signed-off-by: Gais Ameer gameer@redhat.com

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 26, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 26, 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 bshewale 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

@centosinfra-prod-github-app
Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/99ff33f1bbf6461ab44e9a849ef1445c

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 35m 00s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 28m 05s
cifmw-crc-podified-edpm-baremetal RETRY_LIMIT Host unreachable in 1h 14m 55s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 2h 08m 28s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 43s
cifmw-pod-pre-commit FAILURE in 9m 25s

Comment thread playbooks/dcn.yml Outdated
Comment thread playbooks/dcn.yml Outdated
Comment thread playbooks/dcn.yml Outdated
ansible-playbook
{{ playbook_dir }}/../hooks/playbooks/dz_storage_cinder_backups.yaml
-i {{ inventory_file }}
-e cifmw_openshift_namespace={{ cifmw_openshift_namespace | default('openstack') }}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The invocation passes cifmw_openshift_namespace but the target playbook reads cifmw_openstack_namespace; different variables.

Comment thread playbooks/dcn.yml Outdated
Comment thread playbooks/dcn.yml Outdated
spec.cinder.template.cinderBackup (singluar) in DZ-Storage DT is
replaced with cinderBackups (plural) to deploy multiple cinder backups based on AZ topology.
hooks/playbooks/dz_storage_cinder_backups.yaml validates the behaviour of cinderBackups.

The playbook tests different scenarios of cinder backup creation
and restoring the backups across availability zones.

Jira: OSPRH-28342

Signed-off-by: Gais Ameer <gameer@redhat.com>
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
spec.cinder.template.cinderBackup (singluar) in DCN DT is
replaced with cinderBackups (plural) to deploy multiple cinder backups per edge sites.
Invoking hooks/playbooks/cinder_backups.yaml playbook to validates the behaviour of cinderBackups in DCN scenario.

The playbook tests different scenarios of cinder backup creation
and restoring the backups across availability zones.
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.

2 participants