openkruise / charts

OpenKruise Helm Charts.
Apache License 2.0
9 stars 29 forks source link

Fix: kruise-rollout chart structure does not follow best practice #43

Open StevenLeiZhang opened 1 year ago

StevenLeiZhang commented 1 year ago

Signed-off-by: StevenLeiZhang zhangleiic@163.com

Note on DCO:

If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.

Fix: #41 Checklist:

Changes are automatically published when merged to master. They are not published on branches.

kruise-bot commented 1 year ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign fillzpp for approval by writing /assign @fillzpp in a comment. For more information see:The Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/openkruise/charts/blob/master/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment