[Fix] charts large number of field structure errors exist

openkruise / charts

OpenKruise Helm Charts.

Apache License 2.0

9 stars 24 forks source link

[Fix] charts large number of field structure errors exist #64

Closed ls-2018 closed 11 months ago

ls-2018 commented 11 months ago

I replaced yaml in kruise version 1.4 and fixed yaml in charts. A large number of inconsistencies were found.

Checklist:

[x] I have bumped the chart version according to versioning
[x] I have updated the chart changelog with all the changes that come with this pull request according to changelog.
[x] Any new values are backwards compatible and/or have sensible default.
[x] I have signed off all my commits as required by DCO.

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

kruise-bot commented 11 months ago

Welcome @ls-2018! It looks like this is your first PR to openkruise/charts 🎉

ls-2018 commented 11 months ago

/assign @FillZpp @furykerry @zmberg

kruise-bot commented 11 months 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 ask for approval from fillzpp 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

ls-2018 commented 10 months ago

will fix in version 1.5

https://github.com/openkruise/charts/pull/65

https://github.com/openkruise/charts/pull/65/files#diff-247d5405f65c859826cea0f1b2c67d1fddcb7ece3e0c6c3617e71872de6aacd5