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

Apply agent resources to init containers #690

Merged
merged 1 commit into from
Mar 8, 2023

Conversation

dmitryax
Copy link
Contributor

@dmitryax dmitryax commented Mar 8, 2023

Azure Defender throws warnings expecting all containers to have resource limits specified, even on init containers. This change applies the same resources to init containers as allocated to the collector agent. The pod effective limits are not affected because init container limits don't adds up, highest value is applied instead.

@dmitryax dmitryax requested review from a team as code owners March 8, 2023 07:39
@dmitryax dmitryax force-pushed the set-resources-on0init-containers branch from 428c055 to a6400d0 Compare March 8, 2023 07:39
Azure Defender throws warnings expecting all containers to have resource limits specified, even on init containers. This change applies the same resources to init containers as allocated to the collector agent. The pod effective limits are not affected because init container limits don't adds up, highest value is applied instead.
@dmitryax dmitryax force-pushed the set-resources-on0init-containers branch from a6400d0 to 1c459d2 Compare March 8, 2023 07:40
@dmitryax dmitryax merged commit 8ffd2b9 into signalfx:main Mar 8, 2023
@dmitryax dmitryax deleted the set-resources-on0init-containers branch April 10, 2023 21:32
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.

3 participants