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

Add support for defining additional volumes #114

Merged
merged 1 commit into from
Feb 19, 2024

Conversation

javanthropus
Copy link

Our clusters apply restricted PSPs that make the root filesystem read-only. In such cases, the application crashes when trying to write to /tmp. The feature enabled by this PR allows us to mount an emptyDir volume at /tmp to work around the restriction.

@github-actions
Copy link

github-actions bot commented Jan 1, 2022

This pull request has been inactive for 60 days. If the pull request is still relevant please comment to re-activate the pull request. If no action is taken within 7 days, the pull request will be marked closed.

@github-actions github-actions bot added the Stale label Jan 1, 2022
@javanthropus
Copy link
Author

This PR is still needed. Please reactivate it.

@github-actions github-actions bot closed this Jan 9, 2022
@javanthropus
Copy link
Author

This is still needed. Please reopen it.

@javanthropus
Copy link
Author

Should I open a new PR with this same branch? Is this project still alive?

@toelke toelke reopened this Feb 12, 2024
@toelke toelke removed the Stale label Feb 12, 2024
* This allows use of emptyDir volumes for /tmp when the root filesystme is read-only, for instance
@toelke toelke force-pushed the support-custom-volumes branch from 974126e to 2e63a34 Compare February 19, 2024 09:26
@javanthropus javanthropus requested a review from a team as a code owner February 19, 2024 09:26
@toelke toelke merged commit a753e9c into wave-k8s:master Feb 19, 2024
3 checks passed
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.

2 participants