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

Remove backup slot number, 0 or 1, from Velero Backup request name #872

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

hatfieldbrian
Copy link
Collaborator

A VRG writes to one backup slot number, 0 or 1, at a time per s3 profile. Velero Backup name need not include the backup number. This patch removes it.

Note that a BackupStorageLocation may result in a Backup request in an object store being recreated in Kube API server. However, a VRG's Backups and BackupStorageLocations for an s3 store are all deleted prior to a backup starting.

- Use jq instead of json_to_yaml to format json

Signed-off-by: hatfieldbrian <[email protected]>
@hatfieldbrian hatfieldbrian requested a review from tjanssen3 May 5, 2023 03:04
@tjanssen3
Copy link
Contributor

It'd be useful if the Recipe e2e scripts were updated for this change, as well. I believe the only places required are here and here for the Protect step. Specifically, $BACKUP_START_INDEX can be removed from the backup name. Once this is merged, it should be reflected in the Recipe documentation as well (#826), but that can be deferred.

@hatfieldbrian hatfieldbrian marked this pull request as draft May 30, 2023 17:22
@hatfieldbrian hatfieldbrian added the low Low priority issue and would be attended to in spare time label Sep 25, 2023
@ShyamsundarR
Copy link
Member

@raghavendra-talur Do we still need this change? We may want to track it in an issue and open a new PR as required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low Low priority issue and would be attended to in spare time
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants