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

CASMINST-7168 adjust velero_backups_check.sh to ignore failed #677

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

jacobsalmela
Copy link
Contributor

backups if a newer successful one exists

during nightly CT runs, velero backups for vault have failed, but newer successful backups existed. the original issue was transient and due to a timeout error, per CASMTRIAGE-7762, but the ct tests continued to generate auto-triage tickets since a failure was detected.

by extending this script to check for the existence of a newer successful backup and not failing in that case, we will prevent additional, irrelevant auto-triage tickets from being generated.

@jacobsalmela
Copy link
Contributor Author

/backport release/1.7

Copy link
Contributor

github-actions bot commented Feb 5, 2025

Backporting into branch release/1.7 was successful. New PR: #678

backups if a newer successful one exists

during nightly CT runs, velero backups for vault have failed, but
newer successful backups existed.  the original issue was transient
and due to a timeout error, per CASMTRIAGE-7762, but the ct tests
continued to generate auto-triage tickets since a failure was detected.

by extending this script to check for the existence of a newer
successful backup and not failing in that case, we will prevent
additional, irrelevant auto-triage tickets from being generated.

Signed-off-by: Jacob Salmela <[email protected]>
@jacobsalmela jacobsalmela merged commit 5514807 into release/1.6 Feb 12, 2025
3 checks passed
@jacobsalmela jacobsalmela deleted the CASMINST-7168-1.6 branch February 12, 2025 15:04
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