Skip to content
This repository has been archived by the owner on May 6, 2022. It is now read-only.

Update catalog charts README configuration #1823

Merged
merged 1 commit into from
Mar 12, 2018

Conversation

lilic
Copy link
Contributor

@lilic lilic commented Mar 9, 2018

This seemed to be out of sync with the values.yaml file. I just more or less copied over the same description that was in that file.

cc @pmorie

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Mar 9, 2018
Copy link
Contributor

@carolynvs carolynvs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM once the copy/paste error is fixed.

| `apiserver.audit.activated` | If true, enables the use of audit features via this chart. | `false` |
| `apiserver.audit.logPath` | If specified, audit log goes to specified path. | `"/tmp/service-catalog-apiserver-audit.log"` |
| `apiserver.serviceAccount` | Service account. | `service-catalog-apiserver` |
| `apiserver.serveOpenAPISpec` | If specified, audit log goes to specified path. | `"/tmp/service-catalog-apiserver-audit.log"` |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be "if true, makes the API server serve the OpenAPI schema"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@carolynvs Done, sorry about that! Please have another look, thanks!

@lilic lilic force-pushed the lili/fix-config-docs branch from 854ad20 to 7416557 Compare March 11, 2018 22:37
Copy link
Contributor

@carolynvs carolynvs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pmorie pmorie added the LGTM2 label Mar 12, 2018
@pmorie pmorie merged commit 55505be into kubernetes-retired:master Mar 12, 2018
@lilic lilic deleted the lili/fix-config-docs branch March 12, 2018 16:23
jeremyrickard pushed a commit to jeremyrickard/service-catalog that referenced this pull request Mar 13, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. LGTM1 LGTM2 size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants