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

Add E2E cases for BackupRepository configuration #8390

Open
blackpiglet opened this issue Nov 11, 2024 · 3 comments
Open

Add E2E cases for BackupRepository configuration #8390

blackpiglet opened this issue Nov 11, 2024 · 3 comments

Comments

@blackpiglet
Copy link
Contributor

Describe the problem/challenge you have

Should add new E2E automation cases to verify the feature of the BackupRepository configuration.

The feature's original issue is #7620.
The feature's design is https://github.com/vmware-tanzu/velero/blob/main/design/Implemented/backup-repo-config.md.

Describe the solution you'd like

Anything else you would like to add:

Environment:

  • Velero version (use velero version):
  • Kubernetes version (use kubectl version):
  • Kubernetes installer & version:
  • Cloud provider or hardware configuration:
  • OS (e.g. from /etc/os-release):

Vote on this issue!

This is an invitation to the Velero community to vote on issues, you can see the project's top voted issues listed here.
Use the "reaction smiley face" up to the right of this comment to vote.

  • 👍 for "The project would be better with this feature added"
  • 👎 for "This feature will not enhance the project in a meaningful way"
@reasonerjt
Copy link
Contributor

Is it a good way to verify it in E2E?

@reasonerjt reasonerjt added the Needs triage We need discussion to understand problem and decide the priority label Dec 6, 2024
@blackpiglet
Copy link
Contributor Author

By far, only the cache size is configurable in BackupRepository, and there is no good way to check its value from the Velero CLI side. It's more Kopia-related, then it's good to close this issue.
Let's add the E2E cases when there is a concrete scenario.

@reasonerjt reasonerjt added this to the v1.16 milestone Dec 18, 2024
@reasonerjt reasonerjt removed Needs triage We need discussion to understand problem and decide the priority 1.16-candidate labels Dec 18, 2024
@reasonerjt
Copy link
Contributor

The configuration setting may be hard to verify via E2E but let's make sure that the configuration is set in the regular regression test.

@vmware-tanzu vmware-tanzu deleted a comment from blackpiglet Dec 18, 2024
@reasonerjt reasonerjt reopened this Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants