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

Drop SLE Micro version refrences from the code #1801

Conversation

davidcassany
Copy link
Contributor

These are the changes required in this repository for the builds I prepared in here.

Note this changes image and charts URLs, so this is likely to be a breaking change from CI perspective.

Part of rancher/elemental#815

@davidcassany davidcassany requested a review from a team as a code owner July 14, 2023 10:04
@davidcassany davidcassany marked this pull request as draft July 14, 2023 10:04
#!BuildTag: rancher/elemental-builder-image/5.3:latest
#!BuildTag: rancher/elemental-builder-image/5.3:%VERSION%
#!BuildTag: rancher/elemental-builder-image/5.3:%VERSION%-%RELEASE%
#!BuildTag: rancher/elemental-builder-image:latest
Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm, this might fail while we build two versions (like 5.3 and 5.4) in parallel.

We should probably just drop the :latest tag ?!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done, we keep latest and also the slemicro version, but it resolves dynamically to 5.3 or 5.4

Signed-off-by: David Cassany <dcassany@suse.com>
Signed-off-by: David Cassany <dcassany@suse.com>
@davidcassany davidcassany force-pushed the drop_slemicro_version_references_from_code branch from 351b8a0 to 25817fa Compare July 27, 2023 10:46
Add again SLEMicro version within the repository to be sure we can still
make use of the 'latest' tag for each base distro version. Now it will
be dynamically computed from project configuration at the expense of
increasing _service file complexity in OBS.

Signed-off-by: David Cassany <dcassany@suse.com>
@davidcassany davidcassany force-pushed the drop_slemicro_version_references_from_code branch from bbb26e8 to a001b9b Compare July 31, 2023 17:10
@davidcassany davidcassany marked this pull request as ready for review July 31, 2023 20:52
Copy link
Member

@fgiudici fgiudici left a comment

Choose a reason for hiding this comment

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

lgtm

LABEL org.opencontainers.image.created="%BUILDTIME%"
LABEL org.opencontainers.image.vendor="SUSE LLC"
LABEL org.opensuse.reference="%%IMG_REPO%%/rancher/elemental-builder-image/5.3"
LABEL org.opensuse.reference=$IMAGE_REPO
LABEL org.openbuildservice.disturl="%DISTURL%"
LABEL com.suse.supportlevel="techpreview"
Copy link
Member

Choose a reason for hiding this comment

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

the supportlevel here is "techpreview": should we bump it to l2 (I don't remember if it really was l2 or something elese 😓 )?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not sure, this image is not customer facing, this is is just the ISO builder image, the one including elemental to actually call elemental build-iso. This is just part of internal tooling to build the vanilla ISO. This was left as is in purpose.

@kkaempf kkaempf self-requested a review August 1, 2023 10:33
@davidcassany davidcassany merged commit 6b3223c into rancher:main Aug 1, 2023
@davidcassany davidcassany deleted the drop_slemicro_version_references_from_code branch August 1, 2023 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants