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

ci/test: bump various versions #3162

Merged
merged 1 commit into from
Aug 28, 2023

Conversation

zhangzujian
Copy link
Member

@zhangzujian zhangzujian commented Aug 25, 2023

What type of this PR

Examples of user facing changes:

  • Features
  • Bug fixes
  • Docs
  • Tests

Which issue(s) this PR fixes:

Fixes #(issue-number)

WHAT

🤖 Generated by Copilot at 0318217

Updated some dependencies and tools versions in the Makefile and the GitHub workflow files. Added some comments and constants to suppress gosec warnings and improve code quality in the pkg/ovs and pkg/util packages.

🤖 Generated by Copilot at 0318217

Sing, O Muse, of the valiant pull request
That updated the tools of the code warriors
Who battle with bugs and vulnerabilities
In the realm of ovs and helm and gosec.

HOW

🤖 Generated by Copilot at 0318217

  • Update gosec, helm, and cilium versions in workflow files and Makefile (link, link, link, link)
  • Add // #nosec comments to suppress gosec warnings for using unsafe package in pkg/ovs files (link, link, link, link, link)
  • Add // #nosec comment and const keyword to vpcExternalNet variable in pkg/util/net.go file (link)
  • Update kwok version in Makefile (link)

@zhangzujian zhangzujian added test automation tests ci labels Aug 25, 2023
@zhangzujian zhangzujian marked this pull request as ready for review August 25, 2023 06:00
@zhangzujian zhangzujian merged commit 58efd1d into kubeovn:master Aug 28, 2023
60 checks passed
@zhangzujian zhangzujian deleted the ci-bump-versions branch August 28, 2023 01:36
bobz965 pushed a commit that referenced this pull request Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci test automation tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants