Skip to content
This repository was archived by the owner on Oct 19, 2024. It is now read-only.

Feat: Automated release to GitHub and GHCR#193

Merged
alexmt merged 2 commits into
argoproj-labs:masterfrom
mylesagray:master
Mar 12, 2021
Merged

Feat: Automated release to GitHub and GHCR#193
alexmt merged 2 commits into
argoproj-labs:masterfrom
mylesagray:master

Conversation

@mylesagray
Copy link
Copy Markdown
Contributor

@mylesagray mylesagray commented Mar 6, 2021

As per discussion in #190 - this adds support for automatic releases to DockerHub and GHCR.

This workflow can only be invoked via the "Run workflow" in the GitHub Actions UI.

It will publish images with tags reflecting what is in Makefile - the output of "v" + VERSION file and will tag whatever image that is with the latest tag as well.

I have noted on DockerHub that the image tags are of format v1.0.0 and this PR reflects that - would we rather a different tagging scheme or is this okay? /cc @alexmt for 👀

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 6, 2021

Codecov Report

Merging #193 (bd787b9) into master (fb56f31) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #193   +/-   ##
=======================================
  Coverage   51.53%   51.53%           
=======================================
  Files          34       34           
  Lines        1690     1690           
=======================================
  Hits          871      871           
  Misses        647      647           
  Partials      172      172           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fb56f31...bd787b9. Read the comment docs.

Comment thread .github/workflows/release.yaml Outdated
push: true
tags: |
argoproj-labs/argocd-notifications:${{ env.IMAGE_TAG }}
argoproj-labs/argocd-notifications:latest
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ops, I just realized that I incorrectly asked to avoid pushing the latest tag as part of CI workflow #190 . Sorry

Typically latest tag has latest code. @mylesagray , do you mind to move ghcr.io/argoproj-labs/argocd-notifications:latest and argoproj-labs/argocd-notifications:latest tags to https://github.com/argoproj-labs/argocd-notifications/blob/master/.github/workflows/ci.yaml#L69 ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@alexmt I've made those changes to the PR - So I can understand the desired outcome:

  • CI builds on every push to master and always tags latest (no discrete commit based hash tags desired here?)
  • Release builds are manual only - and will derive their tag appending v to the output of $(cat ./VERSION)

Is that what you're after?

Thoughts on adding a tag for commit hash to the CI images too in addition to latest tag so commits can be easier tested?

Copy link
Copy Markdown
Collaborator

@alexmt alexmt Mar 12, 2021

Choose a reason for hiding this comment

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

Is that what you're after?

Yes, thank you for making the change!

Thoughts on adding a tag for commit hash to the CI images too in addition to latest tag so commits can be easier tested?

I think it is a good idea but only for ghcr.io/argoproj-labs/argocd-notifications . We are trying to have only officially released images in dockerhub.com/argoproj-labs and dockerhub.com/argoproj .

Copy link
Copy Markdown
Collaborator

@alexmt alexmt left a comment

Choose a reason for hiding this comment

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

PR is still "draft" looks ready for review, is not it? Can you "move" it to ready for review state?

@mylesagray mylesagray marked this pull request as ready for review March 12, 2021 07:45
@mylesagray
Copy link
Copy Markdown
Contributor Author

Made ready for review

Copy link
Copy Markdown
Collaborator

@alexmt alexmt left a comment

Choose a reason for hiding this comment

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

LGTM!

@alexmt alexmt merged commit 1c2dafe into argoproj-labs:master Mar 12, 2021
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