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

update version of CNI plugins #3733

Merged
merged 13 commits into from
Jul 21, 2024
Merged

update version of CNI plugins #3733

merged 13 commits into from
Jul 21, 2024

Conversation

weizhoublue
Copy link
Collaborator

  1. automatically update plugins version
  2. change image tag to commit hash
  3. add sriov cni
  4. update version
# https://github.com/containernetworking/plugins
export CNI_VERSION=${CNI_VERSION:-"v1.5.1"}
# https://github.com/k8snetworkplumbingwg/ovs-cni
export OVS_VERSION=${OVS_VERSION:-"v0.34.1"}
# https://github.com/k8snetworkplumbingwg/rdma-cni
export RDMA_VERSION=${RDMA_VERSION:-"v1.2.0"}
# https://github.com/k8snetworkplumbingwg/sriov-cni
export SRIOV_VERSION=${SRIOV_VERSION:-"v2.8.0"}
# https://github.com/k8snetworkplumbingwg/ib-sriov-cni
export IB_SRIOV_VERSION=${IB_SRIOV_VERSION:-"v1.1.1"}
# https://github.com/Mellanox/ipoib-cni
export IPOIB_VERSION=${IPOIB_VERSION:-"v1.2.0"}

Signed-off-by: weizhou.lan@daocloud.io <weizhou.lan@daocloud.io>
Signed-off-by: weizhou.lan@daocloud.io <weizhou.lan@daocloud.io>
@weizhoublue weizhoublue added the release/feature-new release note for new feature label Jul 19, 2024
@weizhoublue weizhoublue changed the title Pr/welan/cniplugins update version of CNI plugins Jul 19, 2024
Signed-off-by: weizhou.lan@daocloud.io <weizhou.lan@daocloud.io>
Signed-off-by: weizhou.lan@daocloud.io <weizhou.lan@daocloud.io>
Signed-off-by: weizhou.lan@daocloud.io <weizhou.lan@daocloud.io>
Copy link
Collaborator

@cyclinder cyclinder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

所以这个流程大概是 每天都会跑?只要有版本更新就提个 PR 构建镜像?镜像tag 如何变更?

@@ -91,6 +91,8 @@ spec:
value: {{ .Values.plugins.installOvsCNI | quote }}
- name: INSTALL_RDMA_PLUGIN
value: {{ .Values.plugins.installRdmaCNI | quote }}
- name: INSTALL_SRIOV_PLUGIN
value: {{ .Values.plugins.installSriovCNI | quote }}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sriov-cni 也注册进去了? 目前它是通过 sriov-operator 安装的

Copy link
Collaborator Author

@weizhoublue weizhoublue Jul 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

那边 被我砍了,少了个镜像

spidernet-io/sriov-network-operator@f7244ad

@weizhoublue
Copy link
Collaborator Author

weizhoublue commented Jul 19, 2024

所以这个流程大概是 每天都会跑?只要有版本更新就提个 PR 构建镜像?镜像tag 如何变更

像 spdierpool 的 base 镜像一样,跟着 dockerfile 的 commit hash 走,可追踪

只要 pr 批入,就会有 新 cni 镜像, chart 那边手动 更新

Signed-off-by: weizhou.lan@daocloud.io <weizhou.lan@daocloud.io>
Signed-off-by: weizhou.lan@daocloud.io <weizhou.lan@daocloud.io>
Signed-off-by: weizhou.lan@daocloud.io <weizhou.lan@daocloud.io>
Signed-off-by: weizhou.lan@daocloud.io <weizhou.lan@daocloud.io>
Signed-off-by: weizhou.lan@daocloud.io <weizhou.lan@daocloud.io>
Signed-off-by: weizhou.lan@daocloud.io <weizhou.lan@daocloud.io>
@weizhoublue weizhoublue temporarily deployed to release-base-images July 19, 2024 08:20 — with GitHub Actions Inactive
@weizhoublue
Copy link
Collaborator Author

Signed-off-by: weizhou.lan@daocloud.io <weizhou.lan@daocloud.io>
Signed-off-by: weizhou.lan@daocloud.io <weizhou.lan@daocloud.io>
@weizhoublue
Copy link
Collaborator Author

the failed CI is about kubevirt, not relevant to this PR

@weizhoublue weizhoublue merged commit 7c14015 into main Jul 21, 2024
48 of 54 checks passed
@weizhoublue weizhoublue deleted the pr/welan/cniplugins branch July 21, 2024 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release/feature-new release note for new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants