-
Notifications
You must be signed in to change notification settings - Fork 129
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
Why does Docker have builds and not GitHub tags are pushed? #248
Comments
The github actions are not set up to create tags. Previously, tagging (and thus releasing) was a manual process which triggered builds on Docker Hub. Because of the new automation with Github actions, tags are no longer needed. That saying, suggestions how to extend our actions to create tags again are welcome. |
Okay, thanks |
We could consider to create tags for each version again but some questions come to my mind:
|
Trigger the same tag once more ?
This one is for me a bad option, avoid moving git tags.
Using git tags is quite easy and will do the trick And when a release is published, then it is built: https://github.com/sudo-bot/docker-phpmyadmin-snapshots/blob/main/.github/workflows/publish.yml#L8 |
I don't quite understand this issue. Roundcubemail uses git-tags only for releases, if I am not mistaken. And these releases are picked up by Can someone explain what's missing? |
For example this update has no git tag, it should have one. Each new released version send to the Docker hub should have a tag, and I noticed that there is no more |
@williamdes I'm still a little confused, sorry. Do you speak about git-tags in the commit-log of this repository, or container-tags on the built images? Or maybe container-labels on the images? Because the container-images do all have tags: https://hub.docker.com/r/roundcube/roundcubemail/tags I also don't understand your previous comments. How is "Trigger the same tag once more" different from moving a git-tag to a different commit? And the links you posted in that comment create tagged releases – but we don't do releases in this repository? |
To make it more concrete:
Which git-tag do you think that commit should have?
What do you mean by "released version"? A release from roundcube/roundcubemail? Which tag do you think the latest built container image should have? |
This is how I understand this discussion. It raises two questions
Both not easy to answer because very much a matter of taste and philosophy. So far the tags we published in the Docker repo represent the Roundcube release versions. They can be replaced/updated with new builds from this repository that inherit either changes in the base images or in the Dockerfiles. This way consumers can subscribe to a Roundcube release and still get the latest updates from the PHP base images. This pattern is hard to map 1:1 with Git tags and we did a bad job regarding this in the past. Changes to the Dockerfiles can be applied to tags by constantly shifting tags to new commits but some may consider this bad practice. Tags in Docker and tags in Git may not necessarily have the same meaning. Changes to the base images cannot be tracked in Git unless we store the sha sum of the base images in the Dockerfiles (-> update with renovate). With #281 the exact contents of a Docker image cannot be represented in Git anyway. So let's first try to agree to an answer for question one. Maybe it's time to re-think or extend the Docker tagging strategy. |
I think the image-tags should definitely reference the Roundcubemail version. But maybe we can additionally include more details, by using container labels, or additional tags. But to discuss that I still would like to know an actual use case for those details. Because without one we don't have any criteria (besides personal preference) for a decision, I think. |
@thomascube I noticed no new git tags are pushed but that the image gets updated.
What is the new process?
The text was updated successfully, but these errors were encountered: