Skip to content

rhel: update backup content-manifest path#1790

Merged
github-actions[bot] merged 1 commit intoquay:mainfrom
crozzy:correct-backup-filepath
Mar 25, 2026
Merged

rhel: update backup content-manifest path#1790
github-actions[bot] merged 1 commit intoquay:mainfrom
crozzy:correct-backup-filepath

Conversation

@crozzy
Copy link
Contributor

@crozzy crozzy commented Mar 25, 2026

The backup content-manifest path was incorrect as it didn't include the content_manifests directory.

@crozzy crozzy requested a review from a team as a code owner March 25, 2026 15:48
@crozzy crozzy requested review from hdonnay and removed request for a team March 25, 2026 15:48
@crozzy crozzy force-pushed the correct-backup-filepath branch from 3ff0b45 to 8fb186c Compare March 25, 2026 18:25
@crozzy crozzy requested a review from hdonnay March 25, 2026 18:26
hdonnay
hdonnay previously approved these changes Mar 25, 2026
Copy link
Member

@hdonnay hdonnay left a comment

Choose a reason for hiding this comment

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

One suggestion on naming.

LGTM

This patch adds another glob to check for the content_manifest file on a
filesystem, namely: usr/share/buildinfo/content_manifests/*.json.

Signed-off-by: crozzy <joseph.crosland@gmail.com>
@crozzy crozzy force-pushed the correct-backup-filepath branch from 43e2b4b to 3223b9d Compare March 25, 2026 19:22
@crozzy
Copy link
Contributor Author

crozzy commented Mar 25, 2026

/fast-forward

@github-actions github-actions bot merged commit 3223b9d into quay:main Mar 25, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants