Skip to content

feat(workloads): don't instrument workloads w. low storage limits#1014

Merged
basti1302 merged 1 commit into
mainfrom
ope-331-detect-ephemeral-storage-limit-do-not-instrument-with
May 18, 2026
Merged

feat(workloads): don't instrument workloads w. low storage limits#1014
basti1302 merged 1 commit into
mainfrom
ope-331-detect-ephemeral-storage-limit-do-not-instrument-with

Conversation

@basti1302
Copy link
Copy Markdown
Member

Adding the dash0-instrumentation volume to workloads with tight ephemeral storage limits will cause pod eviction.

@basti1302 basti1302 requested a review from a team as a code owner May 7, 2026 15:42
@basti1302
Copy link
Copy Markdown
Member Author

/run-e2e

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

🚀 E2E tests triggered

  • Branch: ope-331-detect-ephemeral-storage-limit-do-not-instrument-with
  • Triggered by: @basti1302

Results will be posted here once the tests complete. You can also view the workflow runs directly.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

✅ E2E tests passed

  • Branch: ope-331-detect-ephemeral-storage-limit-do-not-instrument-with
  • Run: #25506251471

@basti1302
Copy link
Copy Markdown
Member Author

I think I'd like to merge this one after #1009 actually, and then only reject instrumenting a container with low storage limits if the instrumentation delivery is != image volume.

@basti1302
Copy link
Copy Markdown
Member Author

I think I'd like to merge this one after #1009 actually, and then only reject instrumenting a container with low storage limits if the instrumentation delivery is != image volume.

Added a blocked label to make it more obvious that this one is currently on hold.

Adding the dash0-instrumentation emptyDir volume to workloads with tight
ephemeral storage limits will cause pod eviction.
This check is only applied for instrumentation delivery method
init-container, not for image-volume delivery.
@basti1302 basti1302 force-pushed the ope-331-detect-ephemeral-storage-limit-do-not-instrument-with branch from 745c969 to b6626f6 Compare May 18, 2026 11:02
@sonarqubecloud
Copy link
Copy Markdown

@basti1302 basti1302 removed the blocked label May 18, 2026
@basti1302 basti1302 merged commit a20d623 into main May 18, 2026
16 checks passed
@basti1302 basti1302 deleted the ope-331-detect-ephemeral-storage-limit-do-not-instrument-with branch May 18, 2026 11:28
@github-actions github-actions Bot locked and limited conversation to collaborators May 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants