-
Notifications
You must be signed in to change notification settings - Fork 279
Remove Argo CD integration, and update Argo CD dependencies #502
Remove Argo CD integration, and update Argo CD dependencies #502
Conversation
Signed-off-by: Jonathan West <jonwest@redhat.com>
6f2c313
to
e594609
Compare
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!
nit: Do we want to update the go version to 1.17.6
in github/workflows/image.yaml?
@@ -34,19 +43,6 @@ The `manifests/install.yaml` file contains the Kubernetes manifests required to | |||
- RoleBinding to bind the ServiceAccount and Role | |||
|
|||
|
|||
### B) Install ApplicationSet and Argo CD together |
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.
C
below should be changed to B
.
sigs.k8s.io/yaml v1.3.0 | ||
) | ||
|
||
require ( |
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.
Just for my own knowledge: is this just a pinning mechanism for transient dependencies, in order to get (relatively) deterministic builds?
hack/generate-manifests.sh
Outdated
@@ -42,11 +42,5 @@ cd ${SRCROOT}/manifests/namespace-install && ${KUSTOMIZE} build . >> ${TEMPFILE} | |||
mv ${TEMPFILE} ${SRCROOT}/manifests/install.yaml | |||
cd ${SRCROOT} && chmod 644 manifests/install.yaml | |||
|
|||
# Verify that the 'install-with-argo-cd.yaml' is targetting the expected Argo CD version | |||
# Verify that the GitHub actions is targetting the expected Argo CD version |
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.
# Verify that the GitHub actions is targetting the expected Argo CD version | |
# Verify that the GitHub actions are targeting the expected Argo CD version |
Signed-off-by: William Tam <wtam@redhat.com>
Signed-off-by: William Tam <wtam@redhat.com>
Signed-off-by: William Tam <wtam@redhat.com>
update go-version to 1.17.6 in `publish` job
Signed-off-by: William Tam <wtam@redhat.com>
Signed-off-by: William Tam <email.wtam@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 @jgwest !
This PR: