Skip to content
This repository has been archived by the owner on Jul 15, 2024. It is now read-only.

Issue #170 is not solved when using server, only when using name for destination #282

Merged
merged 4 commits into from
Jul 13, 2021

Conversation

jgwest
Copy link
Member

@jgwest jgwest commented Jul 2, 2021

This is the same fix as I made on Argo CD here but since we support older Argo CDs in ApplicationSet controller, it needs to be made here as well.

  • In addition to verifying using the name field (as before), we now also verify using the server field
  • Unit test to verify the logic is correct

Fixes #220

@jgwest jgwest requested a review from wtam2018 July 2, 2021 14:25
@jgwest jgwest marked this pull request as ready for review July 2, 2021 14:25
Copy link
Collaborator

@wtam2018 wtam2018 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
thanks @jgwest

@jgwest jgwest merged commit be44c32 into argoproj:master Jul 13, 2021
@jgwest jgwest deleted the validate-destination-220 branch July 13, 2021 05:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue #170 is not solved when using server, only when using name for destination
2 participants