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

Make resources in agent and operator helm chart configurable #10296

Merged
merged 1 commit into from Feb 25, 2020
Merged

Make resources in agent and operator helm chart configurable #10296

merged 1 commit into from Feb 25, 2020

Conversation

maxbischoff
Copy link
Contributor

@maxbischoff maxbischoff commented Feb 21, 2020

Please ensure your pull request adheres to the following guidelines:

  • For first time contributors, read Submitting a pull request
  • All code is covered by unit and/or runtime tests where feasible.
  • All commits contain a well written commit description including a title,
    description and a Fixes: #XXX line if the commit addresses a particular
    GitHub issue.
  • All commits are signed off. See the section Developer’s Certificate of Origin
  • Provide a title or release-note blurb suitable for the release notes.
  • Thanks for contributing!

Enables users of the helm chart to provide resource requests and limits to the containers of the agent and operator deployment/daemonset. Also provides default requests for the agent's initContainer.


This change is Reviewable

Signed-off-by: Maximilian Bischoff <maximilian.bischoff@inovex.de>
@maxbischoff maxbischoff requested a review from a team February 21, 2020 18:50
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.8.0 Feb 21, 2020
resources: {}

# Specifies the resources for the clean-cilium-state init container
initResources:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not particularly fond of this naming but also could not come up with anything better. If you have a suggestion I would happily change it.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.007%) to 45.537% when pulling d7fd53a on maxbischoff:add-resources-to-containers into a1d93e0 on cilium:master.

@joestringer
Copy link
Member

test-me-please

@maintainer-s-little-helper
Copy link

Release note label not set, please set the appropriate release note.

@aanm aanm added the release-note/misc This PR makes changes that have no direct user impact. label Feb 24, 2020
@aanm aanm added release-note/minor This PR changes functionality that users may find relevant to operating Cilium. pending-review and removed release-note/misc This PR makes changes that have no direct user impact. pending-review labels Feb 24, 2020
@aanm aanm requested review from tgraf and tklauser February 25, 2020 08:33
@aanm
Copy link
Member

aanm commented Feb 25, 2020

test-me-please

@joestringer joestringer merged commit e075f8c into cilium:master Feb 25, 2020
1.8.0 automation moved this from In progress to Merged Feb 25, 2020
@maxbischoff maxbischoff deleted the add-resources-to-containers branch February 26, 2020 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/minor This PR changes functionality that users may find relevant to operating Cilium.
Projects
No open projects
1.8.0
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

5 participants