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

Reverts #1541: do not attempt to mount /usr/lib/sysimage/rpm #1559

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

tbavelier
Copy link
Member

What this PR does / why we need it:

  • Reverts Mount /usr/lib/sysimage/rpm when using host SBOM feature #1541: the SBOM feature is not compatible with Bottlerocket nodes. When enabling this feature, kubelet tries to create /usr/lib/sysimage/rpm on the node if the directory does not exist, but /usr is read-only, thus causing the pod to not be scheduled
  Warning  Failed     12m                  kubelet            Error: failed to generate container "9ccab0256589e0b5c0aae5d8f0f32dad3912f67ebf1595883a02b473a32e06b1" spec: failed to apply OCI options: failed to mkdir "/usr/lib/sysimage/rpm": mkdir /usr/lib/sysimage: read-only file system

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

  • fixes #

Special notes for your reviewer:

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Chart Version bumped
  • Documentation has been updated with helm-docs (run: .github/helm-docs.sh)
  • CHANGELOG.md has been updated
  • Variables are documented in the README.md
  • For Datadog Operator chart or value changes update the test baselines (run: make update-test-baselines)

@tbavelier tbavelier added bug Something isn't working not-merge chart/datadog This issue or pull request is related to the datadog chart labels Oct 15, 2024
@tbavelier tbavelier changed the title Reverts #1541: do not attempt to Reverts #1541: do not attempt to mount /usr/lib/sysimage/rpm Oct 15, 2024
@tbavelier tbavelier marked this pull request as ready for review October 15, 2024 12:01
@tbavelier tbavelier requested a review from a team as a code owner October 15, 2024 12:01
@tbavelier tbavelier merged commit a4e8dd2 into main Oct 15, 2024
17 checks passed
@tbavelier tbavelier deleted the tbavelier/revert_1541 branch October 15, 2024 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working chart/datadog This issue or pull request is related to the datadog chart
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants