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: applicationset reduce redundant reconciles (#12457) (cherry-pick #12480) #13029

Merged
merged 1 commit into from
Mar 28, 2023

Conversation

gcp-cherry-pick-bot[bot]
Copy link

Cherry-picked fix: applicationset reduce redundant reconciles (#12457) (#12480)

  • fix: applicationset reduce redundant reconciles

Signed-off-by: rumstead 37445536+rumstead@users.noreply.github.com

  • fix: applicationset reduce redundant reconciles

Signed-off-by: rumstead 37445536+rumstead@users.noreply.github.com

  • adding tests

Signed-off-by: rumstead 37445536+rumstead@users.noreply.github.com

  • every line counts

Signed-off-by: rumstead 37445536+rumstead@users.noreply.github.com

  • deep copy applications from event object

Signed-off-by: rumstead 37445536+rumstead@users.noreply.github.com

  • update from code review

Signed-off-by: rumstead 37445536+rumstead@users.noreply.github.com

  • check progressive sync fields

Signed-off-by: rumstead 37445536+rumstead@users.noreply.github.com

  • check progressive sync fields

Signed-off-by: rumstead 37445536+rumstead@users.noreply.github.com

  • selective checks for progressive syncs

Signed-off-by: rumstead 37445536+rumstead@users.noreply.github.com

  • selective checks for progressive syncs

Signed-off-by: rumstead 37445536+rumstead@users.noreply.github.com

  • pural

Signed-off-by: rumstead 37445536+rumstead@users.noreply.github.com


Signed-off-by: rumstead 37445536+rumstead@users.noreply.github.com

* fix: applicationset reduce redundant reconciles

Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>

* fix: applicationset reduce redundant reconciles

Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>

* adding tests

Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>

* every line counts

Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>

* deep copy applications from event object

Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>

* update from code review

Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>

* check progressive sync fields

Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>

* check progressive sync fields

Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>

* selective checks for progressive syncs

Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>

* selective checks for progressive syncs

Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>

* pural

Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>

---------

Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
@crenshaw-dev crenshaw-dev merged commit 5f1fc31 into release-2.7 Mar 28, 2023
@crenshaw-dev crenshaw-dev deleted the cherry-pick-59a4e1-release-2.7 branch March 28, 2023 16:06
schakrad pushed a commit to schakrad/argo-cd that referenced this pull request Jul 24, 2023
…oproj#12480) (argoproj#13029)

* fix: applicationset reduce redundant reconciles

* fix: applicationset reduce redundant reconciles

* adding tests

* every line counts

* deep copy applications from event object

* update from code review

* check progressive sync fields

* check progressive sync fields

* selective checks for progressive syncs

* selective checks for progressive syncs

* pural

---------

Signed-off-by: rumstead <37445536+rumstead@users.noreply.github.com>
Co-authored-by: rumstead <37445536+rumstead@users.noreply.github.com>
Signed-off-by: schakrad <58915923+schakrad@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.

2 participants