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

Add Ubuntu 24.04 as mandatory image, drop mandatory for 22.04 #682

Merged
merged 5 commits into from
Aug 23, 2024

Conversation

martinmo
Copy link
Member

@martinmo martinmo commented Aug 2, 2024

No description provided.

@martinmo martinmo self-assigned this Aug 2, 2024
@martinmo martinmo force-pushed the feat/add-ubuntu-24.04-mandatory-image branch from 29ac496 to b13ec7c Compare August 2, 2024 13:47
@martinmo martinmo added SCS is standardized SCS is standardized SCS-VP10 Related to tender lot SCS-VP10 IaaS Issues or pull requests relevant for Team1: IaaS labels Aug 2, 2024
name_scheme: "ubuntu-capi-image v[0-9]\\.[0-9]+(\\.[0-9]+)?"
source: https://swift.services.a.regiocloud.tech/swift/v1/AUTH_b182637428444b9aa302bb8d5a5a418c/openstack-k8s-capi-images/ubuntu-2204-kube
status: recommended
- name: "Ubuntu 20.04"
Copy link
Member

Choose a reason for hiding this comment

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

While touching this list, should we remove this?

Copy link
Member Author

Choose a reason for hiding this comment

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

@berendt I tend to to agree. However: the status quo of the standard and check is as follows: without a status, the conformance check will not complain at all if this image is missing. However, it will complain if it is available but from the wrong source.

What we probably want, and what we discussed internally: there needs to be a way phase out old OS images, i.e., new status values in the YAML. @mbuechse proposed discouraged and forbidden (the counterparts for recommended and mandatory). It needs to be discussed, whether such an image should be removed completely, or visibility changed to community or something.

Copy link
Member Author

Choose a reason for hiding this comment

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

To be discussed as part #684

Copy link
Contributor

Choose a reason for hiding this comment

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

Two questions here:

  1. does it hurt to specify the preferred source url for a distro that is EOL? Maybe not.
  2. shall we discourage CSPs from providing these EOL distros? Maybe yes.

So yeah, in the end, maybe keep them in the file, but go ahead with #684 and mark them "discouraged".

- name: "Debian 10"
source:
- https://cloud.debian.org/images/cloud/buster/
- https://cdimage.debian.org/cdimage/cloud/buster/
Copy link
Member

Choose a reason for hiding this comment

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

Dito.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it can be problematic if an environment has Debian 10, but NOT 11 or 12, as seems to be the case with Syseleven, see #696. Then users have a hard time choosing a current version. They can always upload one, I guess, but that is an additional barrier.

Copy link
Member

Choose a reason for hiding this comment

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

I would expect Debian 12 to be mandatory at this point, same for 11 and 10 recommended.

The whole point of requiring a minimal set of public images is to ensure that customers do not have to upload their own stock images and remember to update them once per month ...
This is certainly a good answer to folks that want specific images. But mainstream images are better provided by the cloud operator, as this is more efficient for everyone.

Copy link
Contributor

Choose a reason for hiding this comment

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

Then we shall do that for v5 of SCS-compatible IaaS

Copy link
Contributor

@mbuechse mbuechse left a comment

Choose a reason for hiding this comment

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

Make the latest Ubuntu LTS mandatory. Make Debian 12 recommended. Find a better filename. Keep v1 at the front, maybe add v5 to the back, as a reference to the certificate scope version?

@mbuechse mbuechse marked this pull request as ready for review August 21, 2024 15:32
@mbuechse mbuechse requested a review from garloff August 21, 2024 15:32
@mbuechse mbuechse force-pushed the feat/add-ubuntu-24.04-mandatory-image branch from 9449ad8 to 7d7fdd0 Compare August 23, 2024 06:25
Signed-off-by: Matthias Büchse <[email protected]>
@mbuechse mbuechse assigned mbuechse and unassigned martinmo Aug 23, 2024
Copy link
Contributor

@mbuechse mbuechse left a comment

Choose a reason for hiding this comment

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

as per SIG Std/Cert of 2024-08-22

@mbuechse mbuechse merged commit 20131c8 into main Aug 23, 2024
3 checks passed
@mbuechse mbuechse deleted the feat/add-ubuntu-24.04-mandatory-image branch August 23, 2024 06:37
OgarOgarovic pushed a commit that referenced this pull request Aug 27, 2024
as per SIG Std/Cert of 2024-08-22

Signed-off-by: Martin Morgenstern <[email protected]>
Signed-off-by: Matthias Büchse <[email protected]>
Co-authored-by: Matthias Büchse <[email protected]>
Signed-off-by: Ivan Vnučko <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IaaS Issues or pull requests relevant for Team1: IaaS SCS is standardized SCS is standardized SCS-VP10 Related to tender lot SCS-VP10
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants