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

docs: Update zookeeper-operator helm configuration docs #423

Merged
merged 1 commit into from
Dec 16, 2021

Conversation

cbuto
Copy link
Contributor

@cbuto cbuto commented Dec 13, 2021

Change log description

Update zookeeper-operator Helm configuration documentation.

Purpose of the change

Fixes #421

What the code does

Updates the charts/zookeeper-operator/README.md to reflect all the Helm configuration parameters for the zookeeper-operator chart.

How to verify it

(Steps to verify that the changes are effective)

@cbuto cbuto force-pushed the issue-421-helm-docs branch from e1efa7c to 97e478b Compare December 13, 2021 17:23
@codecov
Copy link

codecov bot commented Dec 13, 2021

Codecov Report

Merging #423 (e1efa7c) into master (75d75e7) will not change coverage.
The diff coverage is n/a.

❗ Current head e1efa7c differs from pull request most recent head 97e478b. Consider uploading reports for the commit 97e478b to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master     #423   +/-   ##
=======================================
  Coverage   84.11%   84.11%           
=======================================
  Files          12       12           
  Lines        1643     1643           
=======================================
  Hits         1382     1382           
  Misses        177      177           
  Partials       84       84           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 75d75e7...97e478b. Read the comment docs.

@anishakj
Copy link
Contributor

@cbuto Thank you for the contribution

Copy link
Contributor

@anishakj anishakj left a comment

Choose a reason for hiding this comment

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

LGTM

@anishakj anishakj merged commit bfc3277 into pravega:master Dec 16, 2021
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.

[docs] Update zookeeper-operator Helm chart configuration documentation (README)
2 participants