[stable/prometheus-operator] Update README.md for wrong example prometheus.prometheusSpec.retentionSize
#17725
Conversation
Hi @Eji4h. Thanks for your PR. I'm waiting for a helm member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Fix wrong example value of `prometheus.prometheusSpec.retentionSize` to correct. Signed-off-by: Yosapol Jitrak <yosapoljittarak@hotmail.com>
/retest |
@Eji4h: Cannot trigger testing until a trusted user reviews the PR and leaves an In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/ok-to-test |
@Eji4h: Cannot trigger testing until a trusted user reviews the PR and leaves an In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/ok-to-test |
You'll need to bump the chart version for any change to be valid |
I update document to correct and don't change anything. Why I need to bump the chart version? |
The readme is packed into the chart when it is built. If you do not change the version, a new chart version cannot be built and pushed to the chart repository. You must bump the chart version according to semver rules (For this change, changing the minor version is ok) whenever you make any change to any file. This is enforced by the CI process and I will not be able to merge the changes until you do it. |
Signed-off-by: Yosapol Jitrak <yosapoljittarak@hotmail.com>
/retest |
/ok-to-test |
/retest |
@@ -10,7 +10,7 @@ name: prometheus-operator | |||
sources: | |||
- https://github.com/coreos/prometheus-operator | |||
- https://coreos.com/operators/prometheus | |||
version: 6.16.0 | |||
version: 6.17.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this is a bugfix, only the patch version should be changed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you. Sorry for mistake.
Signed-off-by: Yosapol Jitrak <yosapoljittarak@hotmail.com>
/ok-to-test |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Eji4h, vsliouniaev The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Fix wrong example value of
prometheus.prometheusSpec.retentionSize
to correct.Signed-off-by: Yosapol Jitrak yosapoljittarak@hotmail.com
What this PR does / why we need it:
README in example is wrong units of retentionSize. It can't be start prometheus-prometheus-operator-prometheus workload.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
[stable/chart]
)