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 sidecars #236

Merged
merged 1 commit into from
Dec 17, 2019
Merged

Update sidecars #236

merged 1 commit into from
Dec 17, 2019

Conversation

timoreimann
Copy link
Contributor

@timoreimann timoreimann commented Dec 11, 2019

  • Update csi-provisioner from v1.1.0 to v1.4.0.
  • Update csi-attacher from v1.1.1 to v2.0.0.
  • provisioner: return content source field in CreatVolumeResponse to accommodate to behavioral change (see release nodes).
  • attacher: add RBAC rules for PATCH operations (see release nodes).
  • attacher: return OK response when droplet is found missing during ControllerUnpublishVolume operation to accommodate to behavior change (see release nodes).

Note that we did not update external-snapshotter to v1.2.0 since it requires a disruptive update to the alpha CRDs. Suggestion is to move to v2.0.0 straight on Kubernetes 1.17 where snapshotting graduated to beta.

All changes were extensively tested using the upstream e2e tests. (Integration into our CSI repo coming, promised.)

@timoreimann timoreimann force-pushed the update-sidecars branch 2 times, most recently from 1fb57ae to 68f266d Compare December 11, 2019 13:56
Copy link
Contributor

@adamwg adamwg left a comment

Choose a reason for hiding this comment

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

lgtm.

- Update csi-provisioner from v1.1.0 to v1.4.0.
- Update csi-attacher from v1.1.1 to v2.0.0.
- provisioner: return content source field in CreatVolumeResponse to
  accommodate to behavioral change (see release nodes [1]).
- attacher: add RBAC rules for PATCH operations (see release nodes [2]).
- attacher: return OK response when droplet is found missing during
  ControllerUnpublishVolume operation to accommodate to behavior change
  (see release nodes [2]).

Note that we did not update external-snapshotter to v1.2.0 since it
requires a disruptive update to the alpha CRDs. Suggestion is to move to
v2.0.0 straight on Kubernetes 1.17 where snapshotting graduated to beta.

[1]: https://github.com/kubernetes-csi/external-provisioner/blob/v1.4.0/CHANGELOG-1.4.md
[2]: https://github.com/kubernetes-csi/external-attacher/blob/v2.0.0/CHANGELOG-2.0.md
@timoreimann timoreimann merged commit 06b7443 into master Dec 17, 2019
@timoreimann timoreimann deleted the update-sidecars branch December 17, 2019 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants