carina icon indicating copy to clipboard operation
carina copied to clipboard

Compatible with kubernetes 1.25

Open duanhongyi opened this issue 1 year ago • 18 comments

Carina v0.11.0 is not compatible with the latest kubernetes 1.25. I upgraded the dependent project and it is running well in my kubernetes 1.25 cluster.

duanhongyi avatar Sep 04 '22 01:09 duanhongyi

/lgtm

zhangkai8048 avatar Sep 05 '22 02:09 zhangkai8048

Carina bot is dead. Wait for him to start

antmoveh avatar Sep 05 '22 02:09 antmoveh

@duanhongyi

  1. 这个golang版本为何要升级到1.19?在1.17.5不可以吗?
  2. 1.25废弃了psp, 这样升级完还是无法部署的吧,在部署yaml中我们使用了psp https://github.com/carina-io/carina/blob/main/deploy/kubernetes/csi-controller-psp.yaml

antmoveh avatar Sep 05 '22 02:09 antmoveh

@duanhongyi Does it work in k8s 1.18~1.24, if you upgraded the dependent project

fanhaouu avatar Sep 05 '22 02:09 fanhaouu

@duanhongyi

  1. 这个golang版本为何要升级到1.19?在1.17.5不可以吗?
  2. 1.25废弃了psp, 这样升级完还是无法部署的吧,在部署yaml中我们使用了psp https://github.com/carina-io/carina/blob/main/deploy/kubernetes/csi-controller-psp.yaml

1、kubernetes 1.25的go.mod声明使用的是1.19,我当时在1.17.5下编译报错,或许最终能解决但是感觉费时费力。 2、这个在我的k3s 1.25集群中没有影响,至于正常的kube我这边没有这个环境,无法测试。

duanhongyi avatar Sep 05 '22 07:09 duanhongyi

@duanhongyi Does it work in k8s 1.18~1.24, if you upgraded the dependent project

1.24我已经测试过了,我稍后会在1.18上做一下测试。

duanhongyi avatar Sep 05 '22 07:09 duanhongyi

image

duanhongyi avatar Sep 05 '22 07:09 duanhongyi

image

你是用Helm部署的 还是使用那个部署脚本部署的。 目前helm里没有psp的内容。

antmoveh avatar Sep 05 '22 08:09 antmoveh

Helm chart没有部署psp,也是能正常使用 @fanhaouu 我们是不是可以考虑去掉psp的yaml呢

antmoveh avatar Sep 05 '22 08:09 antmoveh

Helm chart没有部署psp,也是能正常使用 @fanhaouu 我们是不是可以考虑去掉psp的yaml呢 1.25+记得有内置的准入控制策略;其次,低版本的k8s没有psp是不是没法正常运行

fanhaouu avatar Sep 05 '22 09:09 fanhaouu

@antmoveh 是helm

duanhongyi avatar Sep 05 '22 11:09 duanhongyi

@duanhongyi @fanhaouu

① 这个为经过测试在 <k8s 1.18版本中,由于csidriver驱动版本原因无法部署成功,也就是k8s>=1.18将会定义为他支持的版本范围并加入到readme.md中 ② 在k8s1.18版本中不创建psp将导致carina无法创建存储卷.

  Warning  ProvisioningFailed    4s (x5 over 19s)   carina.storage.io_csi-carina-controller-69f6fc6ddb-2spdn_5f7c9421-0e5d-462c-a2ff-3adaa1bab4f8  failed to provision volume with StorageClass "csi-carina-lvm": rpc error: code = Internal desc = an empty namespace may not be set when a resource name is provided

③ helm中需要将psp 内容添加, @zhangkai8048 ④ 我们将制作一个表格,来表示k8s版本和支持他的carina版本 ⑤ 该pr暂时挂这,例会讨论一下关于如何支持k8s>=1.25的问题,可能需要划分不同的分支

antmoveh avatar Sep 06 '22 03:09 antmoveh

/do-not-merge hold

antmoveh avatar Sep 06 '22 03:09 antmoveh

收到 @antmoveh

duanhongyi avatar Sep 06 '22 05:09 duanhongyi

@duanhongyi main分支支持范围为v1.18-1.24 这是一个主流版本范围长期支持

对于kubernetes >1.25版本 将会专门为他建立一个分支,来合并这块代码。 可以稍后将PR提到release-k8s-1.25 关于release-k8s-1.25处于实验状态,新的版本功能更新会落后于main分支

antmoveh avatar Sep 08 '22 02:09 antmoveh

@duanhongyi @fanhaouu

① 这个为经过测试在 <k8s 1.18版本中,由于csidriver驱动版本原因无法部署成功,也就是k8s>=1.18将会定义为他支持的版本范围并加入到readme.md中 ② 在k8s1.18版本中不创建psp将导致carina无法创建存储卷.

  Warning  ProvisioningFailed    4s (x5 over 19s)   carina.storage.io_csi-carina-controller-69f6fc6ddb-2spdn_5f7c9421-0e5d-462c-a2ff-3adaa1bab4f8  failed to provision volume with StorageClass "csi-carina-lvm": rpc error: code = Internal desc = an empty namespace may not be set when a resource name is provided

③ helm中需要将psp 内容添加, @zhangkai8048 ④ 我们将制作一个表格,来表示k8s版本和支持他的carina版本 ⑤ 该pr暂时挂这,例会讨论一下关于如何支持k8s>=1.25的问题,可能需要划分不同的分支

你这个错误是哪个组件的错误,我测试1.18.0没有问题?

duanhongyi avatar Sep 08 '22 07:09 duanhongyi

image

duanhongyi avatar Sep 08 '22 07:09 duanhongyi

#141

duanhongyi avatar Sep 13 '22 06:09 duanhongyi

k8s > 1.25的支持 现在应该需要了。

antmoveh avatar Dec 19 '22 02:12 antmoveh

Any update on 1.25 compatibility timeline ?

Used quite the same PR our side, would be nice to have it upstream.

redref avatar May 02 '23 11:05 redref

要支持k8s > 1.25, 该分支即将合并进main分支

antmoveh avatar May 04 '23 02:05 antmoveh

/lgtm /approve

antmoveh avatar May 05 '23 05:05 antmoveh

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: antmoveh, duanhongyi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

carina-ci-bot avatar May 05 '23 05:05 carina-ci-bot

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: antmoveh, duanhongyi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

carina-ci-bot avatar May 05 '23 05:05 carina-ci-bot

Unless mistaken, this change is not included in 0.12.0. Would it be possible to do a new release for this ? Thanks.

redref avatar May 09 '23 07:05 redref