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

Update the interface used for syncing chart data from the DB to an assetsvc API that can be used from other namespaces and clusters #2394

Closed
danlishka opened this issue Feb 11, 2021 · 3 comments
Labels
component/packages Issue related to kubeapps packaging formats to be distributed (Helm chart and Carvel packages) kind/enhancement An issue that reports an enhancement for an implemented feature

Comments

@danlishka
Copy link

Update the interface used for syncing chart data from the DB to an assetsvc API that can be used from other namespaces and clusters
Currently the asset sync job requires access to the Kubeapps DB, meaning it can only be securely run in the Kubeapps namespace, see #1647
This same issue stops us from being able to set app repositories on other clusters (#1982)
While there we should update the DB actions to use transactions (or a framework where this comes for free) since many jobs can be posting data to the asset svc.

@danlishka danlishka added this to the Q1-2 milestone Feb 11, 2021
@absoludity
Copy link
Contributor

Thanks - I thought we already had this as an issue, but I see it's only implied from those linked (and described in the multi-cluster doc).

@absoludity
Copy link
Contributor

Bringing this into the discussion for next iteration again as it's going to soon be a blocker: If Kubeapps is installed on a specific cluster (such as a shared-services cluster in TKG), then users will only be able to use the app repositories configured with Kubeapps on install. They will be unable to add app repositories and hence unable to use private app repositories etc.

@danlishka danlishka reopened this Mar 2, 2021
@ppbaena ppbaena added component/assetsvc component/packages Issue related to kubeapps packaging formats to be distributed (Helm chart and Carvel packages) kind/enhancement An issue that reports an enhancement for an implemented feature and removed size/L labels May 7, 2021
@ppbaena ppbaena removed this from the 2021-Q1 milestone Jul 9, 2021
@ppbaena ppbaena added this to the App repository refactor milestone Sep 24, 2021
@ppbaena
Copy link
Collaborator

ppbaena commented Mar 9, 2022

To be closed as the new approach for package repositories API is in progress in #3496

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/packages Issue related to kubeapps packaging formats to be distributed (Helm chart and Carvel packages) kind/enhancement An issue that reports an enhancement for an implemented feature
Projects
None yet
Development

No branches or pull requests

3 participants