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 description for containers #381

Open
xiaojinghu93 opened this issue Nov 11, 2021 · 2 comments
Open

Add description for containers #381

xiaojinghu93 opened this issue Nov 11, 2021 · 2 comments

Comments

@xiaojinghu93
Copy link

xiaojinghu93 commented Nov 11, 2021

Hi, all,

We've deployed sregistry in our clusters, of which the software and hardware environments are both quite complex. We'd like to add descriptions to containers to help users/developers grasp infos like the architecture, the version, the compiler and so on.

Currently we have to differentiate the containers by name(with long suffix).

Sregistry's already provided a separate page for each container. It would be nice if the description can be displayed and edited on that page.

Please feel free to close this issue if you are not interested in it. Or if you have better alternatives(better than long suffix), please share it with me.

Thanks in advance!

@vsoch
Copy link
Member

vsoch commented Nov 11, 2021

Yep, can definitely add this! And likely we can store with the container metadata so the database doesn’t need a migration. And you’d be ok with editing the interface to add the description?

mine more suggestion I’ll make in terms of labels is to use org.opencontainers.image.description - it’s currently used by GitHub packages containers for the same goal.

I should have some time to work on this soon.

@vsoch
Copy link
Member

vsoch commented Nov 11, 2021

okay, please review #382 !

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 a pull request may close this issue.

2 participants