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: correctly compare application destinations with inferred cluster URL #4937
Conversation
… URL Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #4937 +/- ##
==========================================
- Coverage 41.44% 41.34% -0.11%
==========================================
Files 128 128
Lines 17666 17695 +29
==========================================
- Hits 7322 7316 -6
- Misses 9292 9330 +38
+ Partials 1052 1049 -3
Continue to review full report at Codecov.
|
@@ -2398,6 +2398,17 @@ func (source *ApplicationSource) ExplicitType() (*ApplicationSourceType, error) | |||
|
|||
// Equals compares two instances of ApplicationDestination and return true if instances are equal. | |||
func (dest ApplicationDestination) Equals(other ApplicationDestination) bool { | |||
// ignore destination cluster name and isServerInferred fields during comparison |
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.
Do you mean to say that "ignore destination cluster URL" because we are setting dest.Server = ""
?
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
… URL (#4937) Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
… URL (#4937) Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com> Signed-off-by: Remington Breeze <remington@breeze.software>
Fixes #4926
Argo CD incorrectly performs full app reconciliation if app destination contains cluster name instead of cluster URL
Signed-off-by: Alexander Matyushentsev AMatyushentsev@gmail.com