Skip to content

Specify what presence of an image means - #1260

Open
mbuechse wants to merge 1 commit into
mainfrom
issue/1259a
Open

Specify what presence of an image means#1260
mbuechse wants to merge 1 commit into
mainfrom
issue/1259a

Conversation

@mbuechse

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Matthias Büchse <matthias.buechse@alasca.cloud>
@mbuechse

Copy link
Copy Markdown
Contributor Author

Note: This has to be debated in SIG Std/Cert, particularly the case of visibility=community.
I presume @berendt will want this, but I'm not sure this finds consensus.

@toothstone

Copy link
Copy Markdown
Contributor

@mbuechse Could you please elaborate/confirm that this change only aligns the canonical texts with the existing compliance checks. Since I see no diff in Tests I assume so, but I think it would be good to have it explicitly spelled out.

@berendt

berendt commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Visibility of community < public. Fine for me the add this. I don't like the public visibility and prefer community over public.

@mbuechse

Copy link
Copy Markdown
Contributor Author

@toothstone I can confirm that:

c.add_function('images', lambda c: [img for img in c.conn.list_images(show_all=True) if img.visibility in ('public', 'community')])

However, in my mind, it's more important that the standard makes sense. This is v2, which is still a draft, so we can and should discuss in the SIG what makes most sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants