-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
feat(kustomize): support envsubst in commonAnnotations and commonLabels #11345
feat(kustomize): support envsubst in commonAnnotations and commonLabels #11345
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #11345 +/- ##
==========================================
+ Coverage 47.98% 48.00% +0.01%
==========================================
Files 246 246
Lines 42225 42234 +9
==========================================
+ Hits 20263 20275 +12
+ Misses 19939 19937 -2
+ Partials 2023 2022 -1
... and 4 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
7282890
to
4915d47
Compare
af03c96
to
5589e91
Compare
Oren from Komodor, Would love to see this added ❤️ |
5ec1fe5
to
77b4c4b
Compare
bbf9274
to
906231d
Compare
906231d
to
8567523
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your patience @AndrewChubatiuk! Would you mind documenting the new field here? https://github.com/argoproj/argo-cd/blob/master/docs/operator-manual/application.yaml
Signed-off-by: Andrii Chubatiuk <andrew.chubatiuk@motional.com>
Signed-off-by: Andrii Chubatiuk <andrew.chubatiuk@motional.com>
b13e5ea
to
a5a3b14
Compare
Added this new field to application.yaml file |
…vsubst Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
…vsubst Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
…ls (argoproj#11345) * support envsubst in kustomize commonAnnotations and commonLabels Signed-off-by: Andrii Chubatiuk <andrew.chubatiuk@motional.com> * added description for new parameter Signed-off-by: Andrii Chubatiuk <andrew.chubatiuk@motional.com> --------- Signed-off-by: Andrii Chubatiuk <andrew.chubatiuk@motional.com> Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> Co-authored-by: Andrii Chubatiuk <andrew.chubatiuk@motional.com> Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Replacement for #11333 PR