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

ci: deploy multus in thick-plugin mode #2628

Merged
merged 1 commit into from Apr 10, 2023

Conversation

zhangzujian
Copy link
Member

@zhangzujian zhangzujian commented Apr 9, 2023

What type of this PR

  • CI

Which issue(s) this PR fixes:

With the multus 4.0 release, we introduce a new client/server-style plugin deployment. This new deployment is called 'thick plugin', in contrast to deployments in previous versions, which is now called a 'thin plugin'. The new thick plugin consists of two binaries, multus-daemon and multus-shim CNI plugin. The 'multus-daemon' will be deployed to all nodes as a local agent and supports additional features, such as metrics, which were not available with the 'thin plugin' deployment before. Due to these additional features, the 'thick plugin' comes with the trade-off of consuming more resources than the 'thin plugin'.

By deploying multus-cni in thick-plugin mode, we can collect arguments passed to and results returned from CNI plugins for debugging purposes.

WHAT

馃 Generated by Copilot at 7880d60

Update Makefile and Helm chart to support Multus CNI thick plugin mode. This mode bundles multiple CNI binaries into the Multus binary, simplifying the network attachment for pods.

馃 Generated by Copilot at 7880d60

Multus supports thick
Bundling all CNI plugins
Simpler for spring pods

HOW

馃 Generated by Copilot at 7880d60

  • Update the Multus image and YAML file to use the thick plugin mode (link)

@zhangzujian zhangzujian added the ci label Apr 9, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Apr 9, 2023

  • The commit message should provide more context about the changes made in this patch.

@zhangzujian zhangzujian marked this pull request as ready for review April 9, 2023 09:24
@zhangzujian zhangzujian merged commit 3e67e89 into kubeovn:master Apr 10, 2023
53 checks passed
@zhangzujian zhangzujian deleted the ci-multus branch April 10, 2023 02:09
zhangzujian added a commit that referenced this pull request Apr 10, 2023
zhangzujian added a commit that referenced this pull request Apr 10, 2023
zhangzujian added a commit that referenced this pull request Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants