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

Standardize image names and source URLs #326

Closed
5 tasks done
mbuechse opened this issue Aug 7, 2023 · 3 comments · Fixed by #314
Closed
5 tasks done

Standardize image names and source URLs #326

mbuechse opened this issue Aug 7, 2023 · 3 comments · Fixed by #314
Assignees
Labels
IaaS Issues or pull requests relevant for Team1: IaaS SCS is standardized SCS is standardized SCS-VP10 Related to tender lot SCS-VP10
Milestone

Comments

@mbuechse
Copy link
Contributor

mbuechse commented Aug 7, 2023

We want users to be able to upload their own images to Glance, so we mandate that this must be possible (with some quota).

We want to standardize image names and source URLs with the intention that the CSP can provide any subset of these images at will; AND if an image with a name from that list is provided, it has to be from the given source URL. The list should be given by a yaml file, whose schema could be a subset of the one for the image-manager yaml files.

The only mandatory image should be the kubernetes-capi images. These could be listed in the yaml file as well and marked mandatory.

Work on this issue can be done inside #314 -- at least the applicable parts from that PR should be reused.

Definition of done:

@mbuechse mbuechse added IaaS Issues or pull requests relevant for Team1: IaaS SCS is standardized SCS is standardized SCS-VP10 Related to tender lot SCS-VP10 labels Aug 7, 2023
@mbuechse mbuechse added this to the R5 (v6.0.0) milestone Aug 7, 2023
@mbuechse
Copy link
Contributor Author

Documentation about quotas in Glance: https://docs.openstack.org/glance/latest/admin/quotas.html
Team IaaS opined about minima:

maybe start with at least 200 GB (and 40 images)

https://input.scs.community/2023-scs-team-iaas?both#Standardize-image-names-and-source-URLs-standards326

@mbuechse
Copy link
Contributor Author

The matter of image_source seems to be kind of tricky.

So we can't really specify the image_source, but we can specify a prefix, or if we want to get fancy, we can use a regular expression. It would be important though that the update script in the image manager preserve the source URL to be used for the properties regardless of mirroring.

@mbuechse
Copy link
Contributor Author

mbuechse commented Aug 22, 2023

The next problem then is whether to use (for the example of Ubuntu 20.04 LTS)

Or we could allow both, either by (again) using a regular expression, or by providing a list of acceptable sources.

Needless to say, the original image manager uses the daily build, whereas the image crawler prefers release builds.

Oh, also, there is a difference between

that I totally missed (even in the previous comment)!! Which one should we prescribe?

@mbuechse mbuechse unassigned fkr and berendt Aug 25, 2023
mbuechse added a commit that referenced this issue Sep 13, 2023
* Split off standard images (updated to remove CentOS 8) from image metadata standard, rough first step

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Apply suggestions from code review

Many thanks to @fkr for spotting the blunders!

Co-authored-by: Felix Kronlage-Dammers <fkr@hazardous.org>
Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Incorporated remarks by @osfrickler; also improved formatting

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Improved formatting by using tables

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Split test script, corrected AlmaLinux spelling (without space)

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Adjusted/amended Conformance Tests sections according to scs-0001-v1.

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Incorporated discussion of 2023-07-05 session of Team IaaS; mainly to remove AlmaLinux

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Added section on Operational tooling, adapted from flavors standard

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Also remove Alma from yaml file

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Change standard in place, for the change doesn't render existing implementations unconformant

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Prescribe image_source prefix

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Allow image specs with regex (name_scheme), allow multiple source URL prefixes

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Provide the actual standard for the standard images

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Pacify markdownlint

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Fix name scheme of Ubuntu CAPI image

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

* Update example in scs-0104

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>

---------

Signed-off-by: Matthias Büchse <matthias.buechse@cloudandheat.com>
Co-authored-by: Felix Kronlage-Dammers <fkr@hazardous.org>
@anjastrunk anjastrunk mentioned this issue Apr 15, 2024
59 tasks
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
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants