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

Users need a public reference for mccp chart values.yaml #2213

Open
kingdonb opened this issue Jan 11, 2023 · 1 comment
Open

Users need a public reference for mccp chart values.yaml #2213

kingdonb opened this issue Jan 11, 2023 · 1 comment
Labels
team/pesto team/tangerine type/documentation Improvements or additions to documentation

Comments

@kingdonb
Copy link
Contributor

kingdonb commented Jan 11, 2023

The mccp chart isn't published anywhere but within the Helm Repository, as far as I can tell. I noticed this when I was working out weaveworks/weave-gitops#3263 and I was looking for the configuration options that weren't described all the way there. I was looking for options related to oidc and further explanation of the staticClients section that's presented in the Dex guide (the top-line subject of that issue linked above.)

I'm thinking that values.yaml is the usual reference point for users trying to understand how to configure the chart. Yes, users can helm fetch and untar the chart, and inspect it there, but I think that's not what type of experience we'd expect to show on enterprise software. Not sure how to fix this without making the repo public, but there needs to be discoverability of the options in values.yaml and it should ideally be not too far away from the chart repo itself.

Maybe would it make sense to make the chart itself public? Since you still can't use the chart without an entitlement... it might seem like something that shouldn't be public, but I don't think that loses anything (and if it isn't public then both users and potential users can't easily learn about the options available unless they have the option of going through support channels.)

@foot foot added type/documentation Improvements or additions to documentation team/pesto labels Feb 3, 2023
@Callisto13
Copy link
Contributor

@foot

There are 4 charts in charts/, do we want a reference for all of them or just the mccp?

For any chart we want publicly rendered in the docs there needs to be as much documentation in comments as possible (skipping the really obvious fields), so some work needs to be done there too.

We can store the chart somewhere public, or move a copy into the docs on release, I can make either work so that's not a problem. The weirdness is around docs versioning. The OSS chart reference is generated automatically when there is a new release, so the published chart is pinned to that version. WGEE is not related to that release process or that versioning sequence. Does WGEE's own release process have an automated docs update bit? Or is that manual? Do we want the EE chart(s) versioned at all or just have the latest always?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team/pesto team/tangerine type/documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

4 participants