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

Carvel package support in Kubeapps #3221

Closed
absoludity opened this issue Jul 29, 2021 · 2 comments · Fixed by #3816
Closed

Carvel package support in Kubeapps #3221

absoludity opened this issue Jul 29, 2021 · 2 comments · Fixed by #3816
Assignees
Labels
component/apis-server Issue related to kubeapps api-server component/ui Issue related to kubeapps UI kind/feature An issue that reports a feature (approved) to be implemented

Comments

@absoludity
Copy link
Contributor

Description:

We're currently working on plugin-based support for different packaging systems, moving across our existing Helm support as well as adding fluxv2 support.

We included Carvel support in the original POC, but due to it changing constantly at the time, as well as a higher priority for flux support, we paused the carvel work to focus on flux and porting the existing helm support to a plugin.

We'd love to come back and add a Carvel/kapp-controller plugin once other priorities are completed (especially as it's now stable).

@absoludity absoludity added component/apis-server Issue related to kubeapps api-server component/ui Issue related to kubeapps UI kind/feature An issue that reports a feature (approved) to be implemented size/XL labels Jul 29, 2021
@ppbaena ppbaena added this to the 2021-Q4 milestone Aug 10, 2021
@absoludity absoludity self-assigned this Nov 8, 2021
@antgamdia
Copy link
Contributor

antgamdia commented Nov 10, 2021

Just adding a list to track the remaining operations:

Additionally, we should be fixing some minor UI issues and features we encounter (e.g., icons, filtering by package plugin, etc...) [will update this comment to track them]

Currently working on: implementing tests (nov 22th)

Tests done so far:

  • 'GetPackageRepositories'
  • 'DeleteInstalledPackage'
  • 'UpdateInstalledPackage'
  • 'CreateInstalledPackage'
  • 'GetInstalledPackageDetail'
  • 'GetInstalledPackageSummaries'
  • 'GetAvailablePackageDetail'
  • 'GetAvailablePackageVersions'
  • 'GetAvailablePackageSummaries'

Old videos

createCarvelUI.mp4
carvelInstalled.mp4

@antgamdia
Copy link
Contributor

antgamdia commented Nov 19, 2021

I've created a bunch of PRs aimed at having initial Carvel support. However, the final result is far from being seamless and smooth.

carvelDemo.mp4

For instance, just in this demo I'd remark a couple of issues:

  • The UI redirects when not every resource is created. Currently, we are throwing errors when sth is not found. Not the best UX
  • The loading times for the catalog are way too long... I'd foresee we're gonna need a cache (even if I tried using goroutines... it seems it's not enough)
    • Especially if you add the Bitnami repo as Carvel packages...
  • The demo package gets installed, but I' haven't been able to get to work another pkg but this one. I mean, not having a default values.yaml file hinders quicks 1-click deployments.
  • There is plenty of to-do and questions in the code. But I think we can start merging soon and start to have quick iterations for improving it.
  • I haven't checked the support for multiple namespaces yet.
    • Also, the "all namespaces" functionality remains unimplemented.
  • I haven't checked the support for multiple clusters yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/apis-server Issue related to kubeapps api-server component/ui Issue related to kubeapps UI kind/feature An issue that reports a feature (approved) to be implemented
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants