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.

Sign up for GitHub

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

Jump to bottom

helm: adding resources override for the operator manager controller #83

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

helm: adding resources override for the operator manager controller #83

wants to merge 6 commits into from

Conversation

AmitBenAmi
Copy link
Contributor

@AmitBenAmi AmitBenAmi commented Oct 27, 2023 •
edited

Description

  • adding configuration overrides for the operator pod's resources
  • using the common-library
  • adding override options for tolerations and affinity to the Operator Deployment

Type of change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • New feature / enhancement (non-breaking change which adds functionality)
  • Security fix
  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • Add changelog entry following the contributing guide
  • Documentation has been updated
  • This change requires changes in testing:
    • unit tests
    • E2E tests

@AmitBenAmi AmitBenAmi force-pushed the feat/helm/allow-resource-override branch from 9647755 to f5933c5 Compare October 27, 2023 00:31
@AmitBenAmi AmitBenAmi force-pushed the feat/helm/allow-resource-override branch 2 times, most recently from d06efe3 to 4100eeb Compare November 16, 2023 01:18
@AmitBenAmi
feat(helm): adding configuration overrides for the operator pod's res…
fe3df57 
…ources

Signed-off-by: Amit Ben Ami <benami.amit1@gmail.com>
@AmitBenAmi
chore(helm): bumping version
158d67f 
Signed-off-by: Amit Ben Ami <benami.amit1@gmail.com>
@AmitBenAmi AmitBenAmi force-pushed the feat/helm/allow-resource-override branch from 4100eeb to 158d67f Compare February 26, 2024 22:19
@AmitBenAmi AmitBenAmi force-pushed the feat/helm/allow-resource-override branch 4 times, most recently from 15ffb7f to c6f152b Compare February 26, 2024 23:22
@AmitBenAmi
Copy link
Contributor Author

@juanjjaramillo any chance you can review this PR?

@AmitBenAmi AmitBenAmi force-pushed the feat/helm/allow-resource-override branch from c6f152b to 15daf87 Compare February 26, 2024 23:28
@AmitBenAmi
chore(helm): fixing indentation formatting
13cb0c6 
Signed-off-by: Amit Ben Ami <benami.amit1@gmail.com>
@AmitBenAmi
chore: adding CHANGELOG.md entry
6bdafe6 
Signed-off-by: Amit Ben Ami <benami.amit1@gmail.com>
@AmitBenAmi AmitBenAmi force-pushed the feat/helm/allow-resource-override branch from 15daf87 to 4fec97d Compare February 26, 2024 23:29
@AmitBenAmi
fix: fixing broken links from within Pull Requests
358113b 
Signed-off-by: Amit Ben Ami <benami.amit1@gmail.com>
@AmitBenAmi AmitBenAmi force-pushed the feat/helm/allow-resource-override branch 3 times, most recently from 707c190 to 358113b Compare February 26, 2024 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@AmitBenAmi