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

Fully Deprecate internal chartmuseum deployment #220

Open
michaeljguarino opened this issue May 3, 2022 · 2 comments
Open

Fully Deprecate internal chartmuseum deployment #220

michaeljguarino opened this issue May 3, 2022 · 2 comments

Comments

@michaeljguarino
Copy link
Member

Use Case

The plural api currently proxies and partially implements the chartmuseum protocol to handle chart storage, with some operations like storage and retrieval done with an internal instance of chartmuseum. It's not clear this is even necessary and we could simplify+save a bit of space by eliminating this chartmuseum layer and just uploading/retrieving from s3 directly.

Ideas of Implementation

Use ExAws' s3 uploader in the upload chart endpoint (instead of proxying to chartmuseum get) and proxy against a signed url for the chart GET endpoint

Additional Info

We should probably test this during a weekend and be prepared to rollback (version the server appropriately when deploying)


Message from the maintainers:

Excited about this feature? Give it a 👍. We factor engagement into prioritization.

@davidspek
Copy link
Contributor

With the release process now implemented it should be easier to test this and roll back if needed.

@davidspek
Copy link
Contributor

Rather than using chartmuseum it might also be an option to use OCI instead going forward so the registry can be used for both containers and charts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

2 participants