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

artifacthub.io metadata #30

Closed
viceice opened this issue Jan 18, 2021 · 17 comments
Closed

artifacthub.io metadata #30

viceice opened this issue Jan 18, 2021 · 17 comments
Labels
enhancement New feature or request

Comments

@viceice
Copy link
Contributor

viceice commented Jan 18, 2021

Move the chart repo on artifact.io to drone org.

I've invited @gtaylor to the newly created org. He can remove me after accepting.

After this some repo metadata should be added. see https://artifacthub.io/docs/topics/repositories/#helm-charts-repositories

I'm happy to send a pr.

@jimsheldon
Copy link
Contributor

Hello @viceice, thank you for suggesting this.

What advantage(s) will artifact.io give the community over the current https://charts.drone.io?

@jimsheldon jimsheldon added the enhancement New feature or request label May 2, 2022
@viceice viceice changed the title artifact.io metadata artifacthub.io metadata May 3, 2022
@viceice
Copy link
Contributor Author

viceice commented May 3, 2022

You would have a nice web page for viewing possible config options for the chart.

checkout https://artifacthub.io/packages/helm/community-charts/drone

So it's not moving, instead that's a search index and this chart should be registred there to have more visibillity. Will update issue description.

@jimsheldon
Copy link
Contributor

Got it, I agree this would be a nice improvement for drone helm users.

@gtaylor is not involved with this project these days, could you invite me instead?

Also, feel free to submit a PR.

Thanks!

@viceice
Copy link
Contributor Author

viceice commented May 10, 2022

@jimsheldon Invited you using your github username.

image

@viceice
Copy link
Contributor Author

viceice commented May 10, 2022

@jimsheldon
Copy link
Contributor

Thanks for working on this.

I merged the PR, but I think there is more to be done based on https://artifacthub.io/docs/topics/repositories/#helm-charts-repositories

There is an extra metadata file that you can add at the repository URL’s path named artifacthub-repo.yml, which can be used to setup features like Verified Publisher or Ownership claim. Please note that the artifacthub-repo.yml metadata file must be located at the same level of the chart repository index.yaml file, and it must be served from the chart repository HTTP server as well.

This might mean artifacthub-repo.yml just needs to move under the docs directory, I will research more when I have time.

@viceice
Copy link
Contributor Author

viceice commented May 12, 2022

@viceice
Copy link
Contributor Author

viceice commented May 24, 2022

After we get the verified publisher, we can ask for official status.

https://artifacthub.io/docs/topics/repositories/#official-status

which looks like this: https://artifacthub.io/packages/helm/renovate/renovate

@jimsheldon
Copy link
Contributor

I am very confused, the file is now in place https://charts.drone.io/artifacthub-repo.yml

However, when I try to claim ownership of the drone charts, it won't let me select them. I am not sure when I will have time to pick this up again, but I will keep investigating.

@jimsheldon
Copy link
Contributor

On second thought, perhaps this means the repo is claimed successfully? Does that mean #67 can be merged?

@viceice
Copy link
Contributor Author

viceice commented May 24, 2022

yes, i already claimed it

@viceice
Copy link
Contributor Author

viceice commented May 24, 2022

image
🎉

Now getting official status?

@jimsheldon
Copy link
Contributor

Making progress! Looks like each chart needs a README, I will work on that, then submit for official status.

@jimsheldon
Copy link
Contributor

Adding README in PR #74, which does not address #68, I will submit another PR for all charts for that.

@jimsheldon
Copy link
Contributor

PR for official status artifacthub/hub#2022

@jimsheldon
Copy link
Contributor

The repo is now official!

Thanks for you help and patience @viceice, this is definitely going to improve the user experience with these charts.

I'll be working on the other related issues as I find the time.

I will close this out now.

@viceice
Copy link
Contributor Author

viceice commented Jun 3, 2022

thank you 🤗

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants