Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport release/1.7] CASMTRIAGE-7627 ignore non-csm failed backups #658

Merged
merged 1 commit into from
Jan 9, 2025

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jan 8, 2025

Backport of #655

@jacobsalmela jacobsalmela force-pushed the backport/655-to-release/1.7 branch from 00b28cb to 1f77747 Compare January 8, 2025 16:38
`velero_backups_check.sh` causes a test failure intermittently on
in-house sytems, often from other products such as PBS or Slingshot.
While they may be valid failures, the CSM product is not at fault, so
those should not trigger a test failure.

for this commit, I modified it to check only for failed vault backups
since that is more pertinent to CSM itself.

I moved the code that was previously run with a pipe and checking the
exit code to instead make a temp file, use a single jq command to query
it, and then fail if there are any failed vault backups.

if we determine other backups should be checked, we can add those to
    that jq command.

Signed-off-by: Jacob Salmela <[email protected]>
(cherry picked from commit 164e86d)
Signed-off-by: Jacob Salmela <[email protected]>
@mharding-hpe mharding-hpe force-pushed the backport/655-to-release/1.7 branch from 1f77747 to 76d19f8 Compare January 8, 2025 16:44
@jacobsalmela jacobsalmela merged commit c153d0f into release/1.7 Jan 9, 2025
3 checks passed
@jacobsalmela jacobsalmela deleted the backport/655-to-release/1.7 branch January 9, 2025 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants