Skip to content
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

No container tags corresponding to version releases #69

Closed
spantaleev opened this issue Feb 25, 2023 · 6 comments · Fixed by #70
Closed

No container tags corresponding to version releases #69

spantaleev opened this issue Feb 25, 2023 · 6 comments · Fixed by #70

Comments

@spantaleev
Copy link

It looks like there's no v1.8 (or 1.8) container image tag published to the ghcr.io/matrix-org/rageshake container registry.

It's just master, etc., which is a moving target and dangerous to use.

@richvdh
Copy link
Member

richvdh commented Feb 25, 2023

Looks like https://github.com/matrix-org/rageshake/blob/master/.github/workflows/docker.yaml needs updating to run on tag events

@t3chguy
Copy link
Member

t3chguy commented Feb 27, 2023

@richvdh why tag over release ooi?

@richvdh
Copy link
Member

richvdh commented Feb 27, 2023

If you get GHA notifications for releases then yes, that would work well too

@t3chguy
Copy link
Member

t3chguy commented Feb 27, 2023

Its what we use in the web app team's automations, means you can draft releases without them getting deployed until they're published - https://github.com/matrix-org/matrix-js-sdk/blob/develop/.github/workflows/release.yml

@michaelkaye
Copy link
Contributor

I think it's not even as complicated as that:

Warning: v1.8 is not a valid semver. More info: https://semver.org/
Warning: v1.8 is not a valid semver. More info: https://semver.org/
Warning: v1.8 is not a valid semver. More info: https://semver.org/

I think it's actually fine; we just need to be "more semver" rather than just versioned.

0f14dc8 is the commit that (would) trigger the build of the tag
https://github.com/matrix-org/rageshake/actions/runs/3913755336/jobs/6690023916 is the run that ran, but it didn't label anything because semver.

@michaelkaye
Copy link
Contributor

https://github.com/michaelkaye/rageshake - yeah, i tried to release a v2.0.0 of rageshakes on this fork and the right thing happened with the docker build. I think we could avoid fighting the system and say we'll move to semver versions. Which looks like it's actually the reason behind #51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants