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

velero: bugfix: BackOffLimitExceeded GLIBC not found #117

Merged
merged 4 commits into from
Apr 30, 2024

Conversation

jdaln
Copy link
Contributor

@jdaln jdaln commented Apr 8, 2024

Description of the change

Fixes BackfOffLimit Exceeded due to libc.so.6: version `GLIBC_2.33' not found, as mentioned upstream at vmware-tanzu/helm-charts#550

Benefits

One will be able to deploy RADAR with Velero because it is currently not possible. This was an unnoticed regression.

Possible drawbacks

Uses a specific version of Debian but, because this is in an init container, it does not involve security issues in the long term.

Applicable issues

Additional information

Checklist

  • Chart version bumped in Chart.yaml according to semver.
  • Variables are documented in the README.md
  • Title of the PR starts with chart name (e.g. [<name_of_the_chart>])

Fixes BackfOffLimit Exceeded due to libc.so.6: version `GLIBC_2.33' not found, as mentioned upstream at vmware-tanzu/helm-charts#550
velero: bugfix: BackOffLimitExceeded GLIBC not found Velero
Copy link
Member

@baixiac baixiac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM and thanks for the PR @jdaln. Is that possible to make the helm-docs job green?

@baixiac
Copy link
Member

baixiac commented Apr 9, 2024

... by bumping up versions in README to 0.2.2?

![Version: 0.2.1](https://img.shields.io/badge/Version-0.2.1-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 1.0](https://img.shields.io/badge/AppVersion-1.0-informational?style=flat-square)

Copy link
Member

@baixiac baixiac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍
Please consider opening an issue to document this version pindown.

@jdaln
Copy link
Contributor Author

jdaln commented Apr 11, 2024

@baixiac is the opened issue fine?

@baixiac
Copy link
Member

baixiac commented Apr 12, 2024

Yes @jdaln, and feel free to resolve it when the upstream gets fixed.

@jdaln
Copy link
Contributor Author

jdaln commented Apr 25, 2024

Hi @Bdegraaf1234 @keyvaann @blootsvoets .
Is there anything preventing this merge?

@Bdegraaf1234 Bdegraaf1234 merged commit bd0b133 into RADAR-base:main Apr 30, 2024
4 checks passed
keyvaann added a commit to RADAR-base/RADAR-Kubernetes that referenced this pull request May 2, 2024
pvannierop pushed a commit to RADAR-base/RADAR-Kubernetes that referenced this pull request Oct 16, 2024
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.

3 participants