Skip to content

Conversation

@msfrucht
Copy link
Contributor

@msfrucht msfrucht commented Sep 24, 2024

Please add a summary of your change

I hit an interesting issue with ephemeral-storage as of late with OCP 4.16.

OCP and similar Kubernetes variations add ephemeral-storage to kubernetes resourceAllocations as options besides CPU and memory. Ignored in most Kubernetes clusters.

Previous versions of OCP did not notice ephemeral-storage usage from plugins as they are finished setting up in the container at startup time. As a result, ephemeral-storage could be set as low as limits of 25Mi or less.

With new installs, ephemeral-storage violations on installation. And eventually tracked down the issue to the /plugins directory.

These estimates are based on current object storage providers. It is entirely possible some plugins will exceed the estimated 100MB per plugin, but this seems to a comfortable estimate at current for some of the more common plugins.

From our own testing of Velero 1.14.1 object storage plugins and a couple of item transformation plugins.

sh-5.1$ du -h plugins/*
63M     plugins/kubevirt-velero-plugin <--- kubevirt item transformation plugin
83M     plugins/velero-plugin-for-aws <--- Amazon AWS object storage and snapshotter plugin
62M     plugins/velero-plugin-for-gcp <--- Google Cloud object storage and snapshotter plugin
52M     plugins/velero-plugin-for-microsoft-azure <-- Microsoft Azure storage and snapshotter plugin
91M     plugins/velero-plugins <--- OpenShift item transformation plugin
sh-5.1$ du -h  plugins 
348M    plugins

Does your change fix a particular issue?

Documentation for resolving ephemeral-storage plugin requirements seen in OpenShift 4.16 and any other Kubernetes cluster or environment that monitors and restricts ephemeral-storage usage.

Fixes #(issue)

Please indicate you've done the following:

/kind changelog-not-required

@msfrucht
Copy link
Contributor Author

/kind changelog-not-required

@github-actions github-actions bot added the kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes label Sep 24, 2024
@kaovilai
Copy link
Collaborator

please force push and/or reopen pr to trigger changelog check due to issue.

@codecov
Copy link

codecov bot commented Sep 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.65%. Comparing base (2390bc8) to head (f1e1b7a).
Report is 9 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8244   +/-   ##
=======================================
  Coverage   59.65%   59.65%           
=======================================
  Files         374      374           
  Lines       40753    40753           
=======================================
  Hits        24311    24311           
  Misses      14960    14960           
  Partials     1482     1482           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

@msfrucht msfrucht closed this Sep 24, 2024
@msfrucht msfrucht reopened this Sep 24, 2024
@msfrucht
Copy link
Contributor Author

@kaovilai I've corrected the issue. Would appreciate a review or I will just close this in another couple of months.

@kaovilai
Copy link
Collaborator

Will take a look

@msfrucht
Copy link
Contributor Author

Closing due to inactivity.

@msfrucht msfrucht closed this Feb 13, 2025
@weshayutin
Copy link
Contributor

@kaovilai can we get this re-opened?

@kaovilai kaovilai reopened this Jun 5, 2025
Signed-off-by: MICHAEL S FRUCHTMAN <[email protected]>
@blackpiglet blackpiglet force-pushed the ephemeral-storage-docs branch from 913e1f6 to f1e1b7a Compare June 6, 2025 06:40
@kaovilai kaovilai merged commit 9a95743 into vmware-tanzu:main Jun 6, 2025
7 checks passed
@MaloLelandais MaloLelandais mentioned this pull request Jul 28, 2025
3 tasks
MaloLelandais pushed a commit to MaloLelandais/velero that referenced this pull request Jul 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants