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

Fix deployment missing If condition for tolerations #630

Merged

Conversation

ammartins
Copy link
Contributor

If you use ArgoCD to deploy this helm chart and leave toleration empty, the helm chart keeps getting reported as not sync

What this PR does / why we need it:

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

  • fixes #

Special notes for your reviewer:

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • PR is based off the current tip of the main branch.
  • Changes are documented under the "Unreleased" header in CHANGELOG.md
  • New or modified sections of values.yaml are documented in the README.md
  • Commits follow the Kong commit message guidelines

@ammartins ammartins requested a review from a team as a code owner July 6, 2022 12:30
@CLAassistant
Copy link

CLAassistant commented Jul 6, 2022

CLA assistant check
All committers have signed the CLA.

charts/kong/CHANGELOG.md Show resolved Hide resolved
@rainest
Copy link
Contributor

rainest commented Jul 6, 2022

@ammartins it looks like your local git installation isn't configured with your email. You'll need to set that to match your GitHub email, edit commits to use your new author info, and then sign the CLA.

@ammartins
Copy link
Contributor Author

@ammartins it looks like your local git installation isn't configured with your email. You'll need to set that to match your GitHub email, edit commits to use your new author info, and then sign the CLA.

Will do.

If you use ArgoCD to deploy this helm chart and leave toleration empty, helm chart keeps getting repor as not sync
If you use ArgoCD to deploy this helm chart and leave toleration empty, helm chart keeps getting repor as not sync
@ammartins ammartins force-pushed the fix-deployment-missing-condition-tolerations branch from a427e63 to 92fe31f Compare July 6, 2022 18:27
Co-authored-by: Travis Raines <571832+rainest@users.noreply.github.com>
@ammartins ammartins force-pushed the fix-deployment-missing-condition-tolerations branch from b53d8f8 to ddda098 Compare July 6, 2022 18:31
@ammartins ammartins requested a review from rainest July 6, 2022 18:32
@rainest rainest enabled auto-merge (squash) July 6, 2022 19:27
@rainest rainest merged commit 21bc132 into Kong:main Jul 6, 2022
sgrzemski pushed a commit to sgrzemski/charts that referenced this pull request Jul 11, 2022
Co-authored-by: Travis Raines <571832+rainest@users.noreply.github.com>
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.

None yet

3 participants