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 support for JSON index to chartmuseum #738

Closed
MeurillonGuillaume opened this issue Dec 7, 2023 · 1 comment
Closed

Add support for JSON index to chartmuseum #738

MeurillonGuillaume opened this issue Dec 7, 2023 · 1 comment
Assignees

Comments

@MeurillonGuillaume
Copy link
Contributor

MeurillonGuillaume commented Dec 7, 2023

Recently a PR was merged into Helm (helm/helm#12245) where support was added to store repository indexes in JSON format. This is especially helpful in situations where the repository is very large. The problem that was solved here is that parsing the YAML-index is slow, and it could be sped up if the index was parsed as JSON instead.

I have personally noticed our chartmuseum becoming slower over time as it grew, and looked into chartmuseum compatibility with this fix. Some digging in Helm learned me that one of our issues is in fact parsing the index.yaml returned by chartmuseum.

I checked, and chartmuseum has no support for what was added in the mentionned PR yet. I felt like this was a nice option to add for people with large repositories. I'm opening this issue for suggestions, and I'm also submitting a PR with the changes needed to generate a JSON index.yaml.

@scbizu
Copy link
Contributor

scbizu commented Dec 29, 2023

Great , I think it will make big sense for large tendency storage . I will look it soon .

@scbizu scbizu self-assigned this Dec 29, 2023
@scbizu scbizu closed this as completed Feb 18, 2024
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

No branches or pull requests

2 participants