Skip to content
This repository was archived by the owner on Nov 9, 2022. It is now read-only.
This repository is currently being migrated. It's locked while the migration is in progress.

Bump csi-provisioner to v2.1.1#317

Merged
croomes merged 1 commit intomasterfrom
bump-csi-provisioner
Apr 15, 2021
Merged

Bump csi-provisioner to v2.1.1#317
croomes merged 1 commit intomasterfrom
bump-csi-provisioner

Conversation

@croomes
Copy link
Contributor

@croomes croomes commented Apr 13, 2021

Bumps the default version of the CSI provisioner, which we fork: https://github.com/storageos/external-provisioner/tree/v2.1.1-patched

v2.1.1 requires k8s 1.17+.

@croomes croomes requested review from avestuk, darkowlzz and mhmxs April 13, 2021 18:06
@@ -9,7 +9,7 @@ const (
CSIv1ClusterDriverRegistrarContainerImage = "quay.io/k8scsi/csi-cluster-driver-registrar:v1.0.1"
CSIv1NodeDriverRegistrarContainerImage = "quay.io/k8scsi/csi-node-driver-registrar:v1.2.0"
CSIv1ExternalProvisionerContainerImageV1 = "storageos/csi-provisioner:v1.4.0"
Copy link
Contributor

@darkowlzz darkowlzz Apr 14, 2021

Choose a reason for hiding this comment

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

We can remove this constant completely. Not being used anywhere anymore.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@darkowlzz
Copy link
Contributor

v2.1.2 was release 2 days ago with a minor logging change kubernetes-csi/external-provisioner@v2.1.1...v2.1.2 .

@croomes croomes force-pushed the bump-csi-provisioner branch from 50fa0cb to 52cd5c6 Compare April 15, 2021 20:14
@croomes
Copy link
Contributor Author

croomes commented Apr 15, 2021

v2.1.2 was release 2 days ago with a minor logging change kubernetes-csi/external-provisioner@v2.1.1...v2.1.2 .

I think it's ok to leave this for the next update.

@croomes croomes merged commit de3bcb5 into master Apr 15, 2021
@croomes croomes deleted the bump-csi-provisioner branch April 15, 2021 20:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants