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 docs for the carvel plugin #4127

Merged
merged 3 commits into from
Jan 24, 2022
Merged

Conversation

antgamdia
Copy link
Contributor

Description of the change

This PR brings a new docs item covering how to use the carvel kapp-controller plugin in Kubeapps.

Benefits

The plugin will get finally documented :P

Possible drawbacks

N/A

Applicable issues

Additional information

In progress, I've sent the PR if someone is interested in having a look at the current progress.

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
@antgamdia antgamdia marked this pull request as ready for review January 20, 2022 12:11
@castelblanque
Copy link
Collaborator

Superb! Really nice and comprehensive guide to the Carvel in Kubeapps.
Just for being picky: in the package repository section we show a simple repository CR, shouldn't we mention which features Kubeapps is able to use? e.g. authentication for repos, etc.

@antgamdia
Copy link
Contributor Author

the package repository section we show a simple repository CR, shouldn't we mention which features Kubeapps is able to use? e.g. authentication for repos, etc.

Mmm, currently I haven't explored authenticated repos in kapp-controller. I'm aware there is a secretgen-controller, but you can manually create the repo adding a secretRef like:

  fetch:
    imgpkgBundle:
      image: k8slt/corp-com-pkg-repo:1.0.0
      secretRef:
        name: my-registry-creds

(reference: https://carvel.dev/kapp-controller/docs/latest/private-registry-auth/#manual-configuration-without-secretgen-controller)

However, since regardless of the cool features it has, Kubeapps will never show any Carvel repo in the UI as long as the repository API is not refactored. That's why I just try to keep it simple, we will have to re-word the whole section in the near future.

Anyway, happy to add more details if you think so.

@antgamdia antgamdia merged commit 96dacb6 into vmware-tanzu:master Jan 24, 2022
@antgamdia antgamdia deleted the carvelDocs branch January 24, 2022 12:44
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

Successfully merging this pull request may close these issues.

Document the current support for the Carvel plugin
2 participants