voxpupuli / crafty

CRAFTY - Containerized Resources And Funky Tools (in) YAML
GNU Affero General Public License v3.0
13 stars 6 forks source link

improve container tags #21

Closed hadmut closed 8 months ago

hadmut commented 9 months ago

Hi, just a proposal:

I saw that there are tags ["7.13.0","main-8","latest","main-7","v1.0.0-7","v1.0.0-8"]

but none of them is really explained in the README.md. I found some description in https://github.com/voxpupuli/crafty/blob/main/docs/how-to-release.md

but it is rather confusing, because is says "don't use main-7, it can break any time", while "v1.0.0-7" is difficult to use, because it can change it's version number any time. Furthermore, there's no way to tell from the tags, that the containers are actually using 7.14.0 and 8.3.0 in build_versions.json.

My proposal is to follow current best practices and

regards

tuxmea commented 8 months ago

We already dicussed several options on how to ensure that changes within the container (not a new puppetxxx version) can be differentiated. Consider that one adds a new ENV var to configure something inside the container. e.g. enable/disable environment delete API endpoint.

Therefor we decided to have a normal version describing the functional basis and append the puppet major version which is used inside the container.

Yes, I agree that the option to have the full puppet version is desired.

@rwaffen @bastelfreak we should have another discussion on this.

Regarding main and latest:

At the moment we build from every commit to main brach. The tag main indicates that thinkgs might not have been fully tested.

How about the following:

rwaffen commented 8 months ago

to find out atm what is in a tag one can see this like this: https://github.com/voxpupuli/container-puppetserver/blob/v1.1.1/build_versions.json

we will work to make this more visible in the next releases,

rwaffen commented 8 months ago

The container tag latest is moving constantly. it is automatically added on each build. and the last container which is build keeps it until the next build happens.

tuxmea commented 8 months ago

Proposal:

<puppet_version>-<container version>

e.g.

7.13.0-1.1.2 or 8.2.3-1.0.1

rwaffen commented 8 months ago

i will move this issue to crafty and convert it into a discussion