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

Cherry-pick fixes from v1.0.1 to release-0.4 #150

Merged
merged 10 commits into from
Apr 26, 2019

Conversation

jcodybaker
Copy link
Collaborator

@jcodybaker jcodybaker commented Apr 26, 2019

This cherry-picks the following PRs from v1.0.1. Once merged, a separate PR will be created for the v0.4.1 release via make bump-version. Lastly, a final PR will be created to update the master changelog to reflect what's in v0.4.1.

  • Cherry-pick: Add tagging support for Volumes via the new --do-tag flag
    [GH-130]
  • Cherry-pick: Fix support for volume snapshots by setting snapshot id on volume creation
    [GH-129]
  • Cherry-pick: Goreportcard fixes (typos, exported variables, etc..)
    [GH-121]
  • Cherry-pick: Rename the cluster role bindings for the node-driver-registrar to be
    consistent with the other role bindings.
    [GH-118]
  • Cherry-pick: Remove the --token flag for the csi-do-node driver. Drivers running on
    the node don't need the token anymore.
    [GH-118]
  • Cherry-pick: Don't check the volume limits on the worker nodes (worker nodes are not able
    to talk to DigitalOcean API)
    [GH-142]
  • Cherry-pick: Update godo (DigitalOcean API package) version to v1.13.0
    [GH-143]
  • Cherry-pick: Fix race in snapshot integration test.
    [GH-146]
  • Cherry-pick: Add tagging support for Volume snapshots via the new --do-tag flag
    [GH-145]

jcodybaker and others added 10 commits April 25, 2019 18:53
* tag DO volumes on create/attach
* tag snapshots at creation
Fix support for volume snapshots by setting snapshot id on volume creation.
This test failed more often than not in my local testing.
Previously we only waited for the pod to be running
which didn't guarantee the file had been created, nor
that it had been sync'ed to the block device.
- sembump (github.com/jessfraz/junk/sembump) is unsupported by
  the author and won't build on mac/windows. Make NEW_VERSION explicit.
@jcodybaker
Copy link
Collaborator Author

Verified this passes both the DO CSI integration tests and Kubernetes Conformance E2E tests.

Copy link
Member

@nanzhong nanzhong left a comment

Choose a reason for hiding this comment

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

lgtm

@jcodybaker jcodybaker merged commit e5a9941 into release-0.4 Apr 26, 2019
@jcodybaker jcodybaker deleted the release-0.4-cp-1-0-1 branch April 26, 2019 13:52
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.

3 participants