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

[Feature] Add --deployment.feature.init-containers-copy-resources (default enabled) - GT-480 #1404

Merged
merged 5 commits into from
Sep 19, 2023

Conversation

nikita-vanyasin
Copy link
Contributor

@nikita-vanyasin nikita-vanyasin commented Sep 8, 2023

Copy resource spec to init containers if they are not specified.

Fixes #1387

@cla-bot cla-bot bot added the cla-signed label Sep 8, 2023
@nikita-vanyasin nikita-vanyasin marked this pull request as draft September 8, 2023 10:18
@nikita-vanyasin nikita-vanyasin changed the title [Feature] Add --deployment.feature.init-containers-copy-limits (default enabled) - GT-480 [Feature] Add --deployment.feature.init-containers-copy-resources (default enabled) - GT-480 Sep 13, 2023
@nikita-vanyasin nikita-vanyasin requested a review from a team September 14, 2023 11:12
@nikita-vanyasin nikita-vanyasin marked this pull request as ready for review September 14, 2023 11:12
@nikita-vanyasin nikita-vanyasin force-pushed the feature/init-containers-copy-limits branch 2 times, most recently from bead917 to d1a617c Compare September 14, 2023 15:51
@nikita-vanyasin nikita-vanyasin force-pushed the feature/init-containers-copy-limits branch from de5994d to cd86b72 Compare September 18, 2023 11:30
@nikita-vanyasin nikita-vanyasin force-pushed the feature/init-containers-copy-limits branch from cd86b72 to 57650f6 Compare September 18, 2023 13:48
@jwierzbo jwierzbo merged commit 5c5fda4 into master Sep 19, 2023
@jwierzbo jwierzbo deleted the feature/init-containers-copy-limits branch September 19, 2023 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Arangodeployment can't start on GKE Autopilot since init container is OOMkilled
3 participants