Skip to content
This repository was archived by the owner on Dec 17, 2025. It is now read-only.

Conversation

@crenshaw-dev
Copy link
Member

@crenshaw-dev crenshaw-dev commented Jun 27, 2024

Should fix the following:

argoproj/argo-cd#13585
argoproj/argo-cd#15176
argoproj/argo-cd#16640
argoproj/argo-cd#17358
argoproj/argo-cd#17548
argoproj/argo-cd#17811
argoproj/argo-cd#18778

I've tested most of them manually. I don't think unit tests would be valuable. We can test old fields but not new ones.

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Copy link
Contributor

@leoluz leoluz left a comment

Choose a reason for hiding this comment

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

LGTM
Tks!

@crenshaw-dev crenshaw-dev merged commit fa0e8d6 into argoproj:master Jun 28, 2024
@crenshaw-dev crenshaw-dev deleted the update-static-scheme branch June 28, 2024 15:55
crenshaw-dev added a commit to crenshaw-dev/gitops-engine that referenced this pull request Jul 1, 2024
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
crenshaw-dev added a commit to crenshaw-dev/gitops-engine that referenced this pull request Jul 1, 2024
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

fix cherry-pick and test

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
crenshaw-dev added a commit to crenshaw-dev/gitops-engine that referenced this pull request Jul 1, 2024
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

fix cherry-pick and test

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
crenshaw-dev added a commit that referenced this pull request Jul 2, 2024
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
crenshaw-dev added a commit that referenced this pull request Jul 2, 2024
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
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.

2 participants