-
Notifications
You must be signed in to change notification settings - Fork 763
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
upgrade argo-cd version to 2.12.1 #1513
upgrade argo-cd version to 2.12.1 #1513
Conversation
Signed-off-by: Ishita Sequeira <ishiseq29@gmail.com>
Signed-off-by: Ishita Sequeira <ishiseq29@gmail.com>
Signed-off-by: Ishita Sequeira <ishiseq29@gmail.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.
Update controller-gen version to 0.14.0
Signed-off-by: Ishita Sequeira <ishiseq29@gmail.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
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.
Dockerfile also needs an update, rest all looks good. Thanks.
argocd-operator/build/util/Dockerfile
Lines 1 to 2 in 6a5b081
# Argo CD v2.11.6 | |
FROM quay.io/argoproj/argocd@sha256:7f82e7eec131866c3cbd91e5035381d62bd9318269cd3d16e6e2d42ede402292 as argocd |
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.
Are the YAML files in https://github.com/argoproj-labs/argocd-operator/tree/master/bundle/manifests updated to take the upstream ArgoCD v2.12.1 ?
Oh, yes. We also need to update the Argo CD Application, Project & ApplicationSet CRD in https://github.com/argoproj-labs/argocd-operator/tree/master/config/crd/bases. Just copy these files from upstream Argo CD source tag for new version. |
Signed-off-by: Ishita Sequeira <ishiseq29@gmail.com>
Signed-off-by: Ishita Sequeira <ishiseq29@gmail.com>
Signed-off-by: Ishita Sequeira <ishiseq29@gmail.com>
Signed-off-by: Ishita Sequeira <ishiseq29@gmail.com>
Signed-off-by: Ishita Sequeira <ishiseq29@gmail.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
Interesting. I see some latest changes in Application and Appset CRD in upstream https://github.com/argoproj/argo-cd/tree/v2.12.1/manifests/crds which are not present in our copy of CRDs here: https://github.com/argoproj-labs/argocd-operator/tree/master/config/crd/bases . |
Signed-off-by: Siddhesh Ghadi <sghadi1203@gmail.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.. Thanks
go 1.21 | ||
go 1.21.0 | ||
|
||
toolchain go1.21.9 |
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 we can remove this.
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.
Not sure how to do it. go mod tidy
is adding it again even if I remove it and set to just go 1.21
.
Is this ready for a new release so we can get this pushed to https://operatorhub.io/operator/argocd-operator? |
Hi @Invincibear, it will still take a few weeks to get a release out. In the meantime, you can use the |
@svghadi Thanks, that worked! |
What type of PR is this?
/kind chore
What does this PR do / why we need it:
upgrade argo-cd version to 2.12.1