[all] Update k/k dependency to v1.22.0 #1601

checked-out the branch from this PR
make image-manila-csi-plugin
for deployment I actually used the helm chart from [manila-csi-plugin] Bump sidecar versions for v1.22 release #1618 , I modified values.yaml accordingly (img repo and tag).

What was tested:

creating a volume
creating a snapshot
creating a volume from a snapshot
expanding a volume
cleaning up all created resources

$ kubectl get all | grep manila
pod/ss-openstack-manila-csi-controllerplugin-0     4/4     Running   0          4m44s
pod/ss-openstack-manila-csi-nodeplugin-wsfqn       2/2     Running   0          4m44s
service/ss-openstack-manila-csi-controllerplugin   ClusterIP   10.0.0.173   <none>        12345/TCP   4m44s
daemonset.apps/ss-openstack-manila-csi-nodeplugin   1         1         1       1            1           <none>          4m44s
statefulset.apps/ss-openstack-manila-csi-controllerplugin   1/1     4m45s

$  kubectl logs ss-openstack-manila-csi-controllerplugin-0 -c cephfs-nodeplugin
I0812 13:22:44.184954       1 driver.go:127] Driver: cephfs.manila.csi.openstack.org
I0812 13:22:44.185003       1 driver.go:128] Driver version: 0.9.0@8d384fbf

Provisioning CephFS PVC

$ kubectl create -f pvc.yaml
persistentvolumeclaim/new-cephfs-share-pvc created
$  kubectl get pvc
NAME                   STATUS   VOLUME                                     CAPACITY   ACCESS MODES   STORAGECLASS      AGE
new-cephfs-share-pvc   Bound    pvc-196023a4-7a92-45b6-b7c0-5b45434a9e85   1Gi        RWX            devstack-cephfs   9s
$ openstack share list
+--------------------------------------+------------------------------------------+------+-------------+-----------+-----------+-----------------+--------------------------------------+-------------------+
| ID                                   | Name                                     | Size | Share Proto | Status    | Is Public | Share Type Name | Host                                 | Availability Zone |
+--------------------------------------+------------------------------------------+------+-------------+-----------+-----------+-----------------+--------------------------------------+-------------------+
| 359132b5-5d58-4fb2-a429-a75da62b33ac | pvc-196023a4-7a92-45b6-b7c0-5b45434a9e85 |    1 | CEPHFS      | available | False     | cephfstype      | rvasek-devstack@cephfsnative1#cephfs | manila-zone-0     |
+--------------------------------------+------------------------------------------+------+-------------+-----------+-----------+-----------------+--------------------------------------+-------------------+

Creating a snapshot:

$  kubectl create -f snapshotcreate.yaml 
volumesnapshot.snapshot.storage.k8s.io/new-cephfs-share-snap created
$  kubectl get volumesnapshot
NAME                    READYTOUSE   SOURCEPVC              SOURCESNAPSHOTCONTENT   RESTORESIZE   SNAPSHOTCLASS     SNAPSHOTCONTENT                                    CREATIONTIME   AGE
new-cephfs-share-snap   true         new-cephfs-share-pvc                           1Gi           devstack-cephfs   snapcontent-1cbf826c-b1df-483e-b7db-9f2e21bd3ad1   51s            51s
$ openstack share snapshot list
+--------------------------------------+-----------------------------------------------+
| ID                                   | Name                                          |
+--------------------------------------+-----------------------------------------------+
| a4ff6908-6747-466e-ba32-c68ed796ca00 | snapshot-1cbf826c-b1df-483e-b7db-9f2e21bd3ad1 |
+--------------------------------------+-----------------------------------------------+

Creating a PVC from snapshot:

$ kubectl create -f snapshotrestore.yaml 
persistentvolumeclaim/new-cephfs-share-snap-restore created
$ kubectl get pvc
NAME                            STATUS   VOLUME                                     CAPACITY   ACCESS MODES   STORAGECLASS      AGE
new-cephfs-share-pvc            Bound    pvc-196023a4-7a92-45b6-b7c0-5b45434a9e85   1Gi        RWX            devstack-cephfs   8m46s
new-cephfs-share-snap-restore   Bound    pvc-72ede75a-1c65-416b-9065-d3139c63d9d2   1Gi        RWX            devstack-cephfs   2m25s
$ openstack share list
+--------------------------------------+------------------------------------------+------+-------------+-----------+-----------+-----------------+--------------------------------------+-------------------+
| ID                                   | Name                                     | Size | Share Proto | Status    | Is Public | Share Type Name | Host                                 | Availability Zone |
+--------------------------------------+------------------------------------------+------+-------------+-----------+-----------+-----------------+--------------------------------------+-------------------+
| 359132b5-5d58-4fb2-a429-a75da62b33ac | pvc-196023a4-7a92-45b6-b7c0-5b45434a9e85 |    1 | CEPHFS      | available | False     | cephfstype      | rvasek-devstack@cephfsnative1#cephfs | manila-zone-0     |
| 56c37f32-19a8-4f4b-9c3a-6a70d2e8fa49 | pvc-72ede75a-1c65-416b-9065-d3139c63d9d2 |    1 | CEPHFS      | available | False     | cephfstype      | rvasek-devstack@cephfsnative1#cephfs | manila-zone-0     |
+--------------------------------------+------------------------------------------+------+-------------+-----------+-----------+-----------------+--------------------------------------+-------------------+

Expanding PVC to 2GiB:

$ kubectl patch persistentvolumeclaim/new-cephfs-share-snap-restore -p '{"spec": {"resources": {"requests": {"storage": "2Gi"}}}}'
persistentvolumeclaim/new-cephfs-share-snap-restore patched
$ openstack share list
+--------------------------------------+------------------------------------------+------+-------------+-----------+-----------+-----------------+--------------------------------------+-------------------+
| ID                                   | Name                                     | Size | Share Proto | Status    | Is Public | Share Type Name | Host                                 | Availability Zone |
+--------------------------------------+------------------------------------------+------+-------------+-----------+-----------+-----------------+--------------------------------------+-------------------+
| 359132b5-5d58-4fb2-a429-a75da62b33ac | pvc-196023a4-7a92-45b6-b7c0-5b45434a9e85 |    1 | CEPHFS      | available | False     | cephfstype      | rvasek-devstack@cephfsnative1#cephfs | manila-zone-0     |
| 56c37f32-19a8-4f4b-9c3a-6a70d2e8fa49 | pvc-72ede75a-1c65-416b-9065-d3139c63d9d2 |    2 | CEPHFS      | available | False     | cephfstype      | rvasek-devstack@cephfsnative1#cephfs | manila-zone-0     |
+--------------------------------------+------------------------------------------+------+-------------+-----------+-----------+-----------------+--------------------------------------+-------------------+

Mounting a PVC into a Pod:

$  kubectl create -f pod.yaml 
pod/new-cephfs-share-snap-restore-pod created
$ kubectl get pod/new-cephfs-share-snap-restore-pod 
NAME                                READY   STATUS    RESTARTS   AGE
new-cephfs-share-snap-restore-pod   1/1     Running   0          9s

Clean-up was successful too.

jichenjc · 2021-08-13T01:22:55Z

@ramineni looks like pull-cloud-provider-openstack-check need to be handled ? it should not be affected by env

ramineni · 2021-08-13T05:55:56Z

@ramineni looks like pull-cloud-provider-openstack-check need to be handled ? it should not be affected by env

yes, Ill have a look at it. Thanks

theopenlab-ci · 2021-08-13T06:35:10Z

Build failed.

cloud-provider-openstack-acceptance-test-keystone-authentication-authorization : NODE_FAILURE in 0s

theopenlab-ci · 2021-08-13T06:39:24Z

Build succeeded.

cloud-provider-openstack-format : SUCCESS in 3m 56s

theopenlab-ci · 2021-08-13T06:40:23Z

Build succeeded.

cloud-provider-openstack-sanity-test-csi-cinder : SUCCESS in 5m 16s

theopenlab-ci · 2021-08-13T06:41:25Z

Build succeeded.

cloud-provider-openstack-unittest : SUCCESS in 7m 49s

theopenlab-ci · 2021-08-13T06:57:22Z

Build failed.

cloud-provider-openstack-test-occm : RETRY_LIMIT in 2m 14s

theopenlab-ci · 2021-08-13T06:57:43Z

Build failed.

cloud-provider-openstack-e2e-test-csi-cinder : RETRY_LIMIT in 2m 14s

theopenlab-ci · 2021-08-13T07:00:10Z

Build failed.

cloud-provider-openstack-acceptance-test-csi-manila : RETRY_LIMIT in 2m 25s

theopenlab-ci · 2021-08-13T07:03:43Z

Build failed.

cloud-provider-openstack-acceptance-test-e2e-conformance : RETRY_LIMIT in 2m 53s

theopenlab-ci · 2021-08-13T07:04:09Z

Build failed.

cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.21 : RETRY_LIMIT in 2m 48s

theopenlab-ci · 2021-08-13T07:05:29Z

Build failed.

cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.20 : RETRY_LIMIT in 2m 48s

theopenlab-ci · 2021-08-13T07:09:58Z

Build failed.

cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.19 : POST_FAILURE in 16m 17s

theopenlab-ci · 2021-08-13T07:17:21Z

Build failed.

cloud-provider-openstack-multinode-csi-migration-test : RETRY_LIMIT in 9m 27s

lingxiankong · 2021-08-16T23:50:00Z

Conformance test passed (in a v1.22.0 cluster), result:

Ran 340 of 6432 Specs in 6340.585 seconds
SUCCESS! -- 340 Passed | 0 Failed | 0 Pending | 6092 Skipped
PASS

openstack-cloud-controller-manager functional test also passed by running tests/e2e/cloudprovider/test-lb-service.sh

lingxiankong · 2021-08-16T23:50:35Z

/lgtm

jichenjc · 2021-08-17T00:56:31Z

/approve

looks good @lingxiankong I assume you will manual merge this anyway..

k8s-ci-robot · 2021-08-17T00:56:46Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jichenjc

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:

~~OWNERS~~ [jichenjc]

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

lingxiankong · 2021-08-17T01:28:01Z

Yeah, I will force merge it after receiving @ramineni's confirmation about cinder-csi test.

ramineni · 2021-08-17T04:39:08Z

Yeah, I will force merge it after receiving @ramineni's confirmation about cinder-csi test.

@lingxiankong Thanks. I did run acceptance tests of cinder csi, https://github.com/theopenlab/openlab-zuul-jobs/blob/master/playbooks/cloud-provider-openstack-acceptance-test-csi-cinder . It is successful.

Should be good to go.

lingxiankong · 2021-08-17T05:39:26Z

@lingxiankong Thanks. I did run acceptance tests of cinder csi, https://github.com/theopenlab/openlab-zuul-jobs/blob/master/playbooks/cloud-provider-openstack-acceptance-test-csi-cinder . It is successful.

Should be good to go.

Great, here we go!

k8s-ci-robot added release-note-none Denotes a PR that doesn't merit a release note. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jul 20, 2021

k8s-ci-robot requested review from adisky and chrigl July 20, 2021 11:02

k8s-ci-robot added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Jul 20, 2021

ramineni force-pushed the update-kk-1.22 branch from c65adfd to 89d2748 Compare August 9, 2021 06:04

ramineni changed the title ~~[WIP]Update k/k dependency to latest~~ [all] Update k/k dependency to v1.22.0 Aug 9, 2021

k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 9, 2021

gman0 mentioned this pull request Aug 12, 2021

[manila-csi-plugin] Bump sidecar versions for v1.22 release #1618

Merged

Update k/k dependency to v1.22.0

ce48c67

ramineni force-pushed the update-kk-1.22 branch from 8d384fb to ce48c67 Compare August 13, 2021 06:32

k8s-ci-robot assigned lingxiankong Aug 16, 2021

k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 16, 2021

k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 17, 2021

lingxiankong merged commit 1f88695 into kubernetes:master Aug 17, 2021

[all] Update k/k dependency to v1.22.0 #1601

[all] Update k/k dependency to v1.22.0 #1601

Conversation

ramineni commented Jul 20, 2021 • edited Loading

theopenlab-ci bot commented Jul 20, 2021

theopenlab-ci bot commented Jul 20, 2021

theopenlab-ci bot commented Jul 20, 2021

theopenlab-ci bot commented Jul 20, 2021

theopenlab-ci bot commented Jul 20, 2021

theopenlab-ci bot commented Jul 20, 2021

theopenlab-ci bot commented Jul 20, 2021

theopenlab-ci bot commented Jul 20, 2021

theopenlab-ci bot commented Jul 20, 2021

theopenlab-ci bot commented Jul 20, 2021

theopenlab-ci bot commented Jul 20, 2021

theopenlab-ci bot commented Aug 9, 2021

theopenlab-ci bot commented Aug 9, 2021

theopenlab-ci bot commented Aug 9, 2021

theopenlab-ci bot commented Aug 9, 2021

theopenlab-ci bot commented Aug 9, 2021

theopenlab-ci bot commented Aug 9, 2021

theopenlab-ci bot commented Aug 9, 2021

theopenlab-ci bot commented Aug 9, 2021

theopenlab-ci bot commented Aug 9, 2021

theopenlab-ci bot commented Aug 9, 2021

theopenlab-ci bot commented Aug 12, 2021

gman0 commented Aug 12, 2021

jichenjc commented Aug 13, 2021

ramineni commented Aug 13, 2021

theopenlab-ci bot commented Aug 13, 2021

theopenlab-ci bot commented Aug 13, 2021

theopenlab-ci bot commented Aug 13, 2021

theopenlab-ci bot commented Aug 13, 2021

theopenlab-ci bot commented Aug 13, 2021

theopenlab-ci bot commented Aug 13, 2021

theopenlab-ci bot commented Aug 13, 2021

theopenlab-ci bot commented Aug 13, 2021

theopenlab-ci bot commented Aug 13, 2021

theopenlab-ci bot commented Aug 13, 2021

theopenlab-ci bot commented Aug 13, 2021

theopenlab-ci bot commented Aug 13, 2021

lingxiankong commented Aug 16, 2021

lingxiankong commented Aug 16, 2021

jichenjc commented Aug 17, 2021

k8s-ci-robot commented Aug 17, 2021

lingxiankong commented Aug 17, 2021

ramineni commented Aug 17, 2021

lingxiankong commented Aug 17, 2021

ramineni commented Jul 20, 2021 •

edited

Loading