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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

add workflows for release chart #2672

Merged
merged 1 commit into from Apr 19, 2023
Merged

add workflows for release chart #2672

merged 1 commit into from Apr 19, 2023

Conversation

hongzhen-ma
Copy link
Collaborator

@hongzhen-ma hongzhen-ma commented Apr 18, 2023

What type of this PR

Which issue(s) this PR fixes:

Fixes #2354

WHAT

馃 Generated by Copilot at 9710ad9

This pull request adds a GitHub workflow to automatically release Helm charts for the kube-ovn project. It also disables the chart signing feature in the cr.yaml file.

馃 Generated by Copilot at 9710ad9

chart-releaser-action
Publishes Helm charts to site
No signing in spring

HOW

馃 Generated by Copilot at 9710ad9

  • Add a GitHub workflow to release Helm charts (link)
  • Disable chart signing for chart-releaser-action (link)

@github-actions
Copy link
Contributor

  • The commit message should provide more context about the changes made in the patch.
  • It is recommended to add a description of what this workflow does and how it works.
  • The permissions field is empty, which might cause issues with the workflow. It is recommended to remove it or add appropriate permissions.
  • The runs-on field can be optimized for better performance based on the requirements of the workflow.
  • The fetch-depth field can be increased to improve the speed of the checkout process.
  • The Configure Git step might fail if the $GITHUB_ACTOR variable is not set. It is recommended to add a check for this variable before running the command.
  • The Install Helm step might take a long time to complete, which can affect the overall performance of the workflow. It is recommended to cache the Helm installation to improve the speed of subsequent runs.
  • The chart-releaser-action version used in the workflow can be updated to the latest version for bug fixes and new features.
  • The sign field in cr.yaml is set to false, which might cause security issues. It is recommended to set it to true if possible.

@hongzhen-ma hongzhen-ma merged commit a9993da into master Apr 19, 2023
53 checks passed
@hongzhen-ma hongzhen-ma deleted the chart-ci branch April 19, 2023 05:26
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.

Release the Helm Chart on Github Pages
2 participants