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
chore: Fix manifest generation in release and make quay.io the lead #5831
Conversation
Signed-off-by: jannfis <jann@mistrust.net>
Codecov Report
@@ Coverage Diff @@
## master #5831 +/- ##
=======================================
Coverage 41.01% 41.01%
=======================================
Files 146 146
Lines 19630 19630
=======================================
Hits 8052 8052
Misses 10464 10464
Partials 1114 1114 Continue to review full report at Codecov.
|
.github/workflows/release.yaml
Outdated
docker push ${IMAGE_NAMESPACE}/argocd:v${TARGET_VERSION} | ||
# Remove the following when Docker Hub is gone | ||
docker login --username "${DOCKER_USERNAME}" --password "${DOCKER_TOKEN}" | ||
docker tag ${IMAGE_NAMESPACE}/argocd:v${TARGET_VERSION} argoproj/argocd:v${TARGET_VERSION} | ||
docker push quay.io/${IMAGE_NAMESPACE}/argocd:v${TARGET_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.
I think this docker push line needs to be changed as well, in order to push to docker hub instead of quay.io
docker push argoproj/argocd:v${TARGET_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.
Good catch! Will fix it.
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.
Fixed.
Signed-off-by: jannfis <jann@mistrust.net>
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
…5831) * chore: Fix manifest generation in release and make quay.io the lead Signed-off-by: jannfis <jann@mistrust.net> * Fix docker push command Signed-off-by: jannfis <jann@mistrust.net>
…rgoproj#5831) * chore: Fix manifest generation in release and make quay.io the lead Signed-off-by: jannfis <jann@mistrust.net> * Fix docker push command Signed-off-by: jannfis <jann@mistrust.net>
Signed-off-by: jannfis jann@mistrust.net
Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist: