Skip to content

[b/r] Add backup/restore and OADP data collection#144

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
abays:backup_restore
Jun 29, 2026
Merged

[b/r] Add backup/restore and OADP data collection#144
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
abays:backup_restore

Conversation

@abays

@abays abays commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Collect OADP and backup/restore resources to support troubleshooting of OpenStack backup and restore operations. Add openshift-adp to the default namespace list for pod/log/event collection, and introduce a new gather_backup_restore script that collects Velero Backup/Restore CRs, DataUpload/DataDownload, BackupStorageLocation, VolumeSnapshots, VolumeSnapshotClasses, backup/restore label verification, and deployment-stage annotation status. OMC mode support is included.

Ref: OSPRH-32043

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

@abays abays requested review from fmount and stuggi June 29, 2026 13:53
@openshift-ci openshift-ci Bot requested a review from gibizer June 29, 2026 13:54
@fmount

fmount commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

I guess we need #142 to land and unblock CI, /lgtm otherwise

@fmount

fmount commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

#142 merged, and CI should be unblocked now. I guess by simply rebasing we should be able to have CI green, then we can land this patch. Thanks for adding support to both "legacy" and "omc" mode!

Collect OADP and backup/restore resources to support troubleshooting
of OpenStack backup and restore operations. Add openshift-adp to the
default namespace list for pod/log/event collection, and introduce a
new gather_backup_restore script that collects Velero Backup/Restore
CRs, DataUpload/DataDownload, BackupStorageLocation, VolumeSnapshots,
VolumeSnapshotClasses, backup/restore label verification, and
deployment-stage annotation status. OMC mode support is included.

Ref: OSPRH-32043

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

@fmount fmount left a comment

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.

/approve
/lgtm

@openshift-ci

openshift-ci Bot commented Jun 29, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fmount

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

The pull request process is described 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

@openshift-merge-bot openshift-merge-bot Bot merged commit 3f55b05 into openstack-k8s-operators:main Jun 29, 2026
4 checks passed
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