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
chore: rename 'progressive rollouts' to 'progressive syncs' #12265
Conversation
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Codecov ReportBase: 47.45% // Head: 47.45% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #12265 +/- ##
=======================================
Coverage 47.45% 47.45%
=======================================
Files 246 246
Lines 41883 41883
=======================================
Hits 19877 19877
Misses 20008 20008
Partials 1998 1998
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 at Codecov. |
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.
LGTM!
LGTM. |
1. Pass `--enable-progressive-syncs` to the ApplicationSet controller args. | ||
1. Set `ARGOCD_APPLICATIONSET_CONTROLLER_ENABLE_PROGRESSIVE_SYNCS=true` in the ApplicationSet controller environment variables. | ||
1. Set `applicationsetcontroller.enable.progressive.syncs: true` in the Argo CD ConfigMap. |
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.
1. Pass `--enable-progressive-syncs` to the ApplicationSet controller args. | |
1. Set `ARGOCD_APPLICATIONSET_CONTROLLER_ENABLE_PROGRESSIVE_SYNCS=true` in the ApplicationSet controller environment variables. | |
1. Set `applicationsetcontroller.enable.progressive.syncs: true` in the Argo CD ConfigMap. | |
1. Pass `--enable-progressive-syncs` to the ApplicationSet controller args. | |
2. Set `ARGOCD_APPLICATIONSET_CONTROLLER_ENABLE_PROGRESSIVE_SYNCS=true` in the ApplicationSet controller environment variables. | |
3. Set `applicationsetcontroller.enable.progressive.syncs: true` in the Argo CD ConfigMap. |
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.
I think you need to change this as well.
cmd/argocd-applicationset-controller/commands/applicationset_controller.go
Outdated
Show resolved
Hide resolved
…ontroller.go Co-authored-by: Panagiotis Georgiadis <pgeorgia@redhat.com> Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
…syncs Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
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.
LGTM
* chore: rename 'progressive rollouts' to 'progressive syncs' Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> * rename docs file Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> * Update cmd/argocd-applicationset-controller/commands/applicationset_controller.go Co-authored-by: Panagiotis Georgiadis <pgeorgia@redhat.com> Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> --------- Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> Co-authored-by: Panagiotis Georgiadis <pgeorgia@redhat.com>
Cherry-picked onto release-2.6 for 2.6.0. |
…#12265) * chore: rename 'progressive rollouts' to 'progressive syncs' Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> * rename docs file Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> * Update cmd/argocd-applicationset-controller/commands/applicationset_controller.go Co-authored-by: Panagiotis Georgiadis <pgeorgia@redhat.com> Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> --------- Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com> Co-authored-by: Panagiotis Georgiadis <pgeorgia@redhat.com> Signed-off-by: schakrad <chakradari.sindhu@gmail.com>
Thanks to @christianh814 for the idea. I think this is clearer and avoids clash with Argo Rollouts. The rename is also endorsed by @wmgroot (who wrote the feature) and @zachaller (who is an Approver on Rollouts).