Skip to content

Conversation

AAraKKe
Copy link
Contributor

@AAraKKe AAraKKe commented Sep 5, 2025

What does this PR do?

Modify the repo used by the velero helm chart from the bitnami one to bitnami legacy. This is not a final solution but VMware Tanzu is still discussing a final alternative to the bitnami image.

See: vmware-tanzu/helm-charts#698

Motivation

Do not rely on bitnami images for testing.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

@AAraKKe AAraKKe added the qa/skip-qa Automatically skip this PR for the next QA label Sep 5, 2025
Copy link

github-actions bot commented Sep 5, 2025

⚠️ The qa/skip-qa label has been added with shippable changes

The following files, which will be shipped with the agent, were modified in this PR and
the qa/skip-qa label has been added.

You can ignore this if you are sure the changes in this PR do not require QA. Otherwise, consider removing the label.

List of modified files that will be shipped with the agent
velero/changelog.d/21274.fixed
velero/pyproject.toml

@AAraKKe AAraKKe marked this pull request as ready for review September 5, 2025 13:39
@AAraKKe AAraKKe requested a review from a team as a code owner September 5, 2025 13:39
Copy link

codecov bot commented Sep 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.00%. Comparing base (598ba54) to head (bb64ff6).

Additional details and impacted files
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@AAraKKe
Copy link
Contributor Author

AAraKKe commented Sep 5, 2025

I took ages for the test to finish and eventually it failed. But it can run properly locally. I will keep looking into this on Monday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integration/velero qa/skip-qa Automatically skip this PR for the next QA team/agent-integrations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants