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

csi: allow volume detach to take a node ID prefix #9041

Merged
merged 1 commit into from
Oct 7, 2020

Conversation

tgross
Copy link
Member

@tgross tgross commented Oct 7, 2020

Fixes #8963

Fixes a bug where the nomad volume detach command would not accept a node ID
prefix instead of a full node ID. The volume ID is already prefix matched
server-side.

@vercel
Copy link

vercel bot commented Oct 7, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/hashicorp/nomad/bxzceg2n7
✅ Preview: https://nomad-git-b-csi-volume-detach-node-prefix.hashicorp.vercel.app

@tgross tgross added this to the 0.13 milestone Oct 7, 2020
@tgross tgross requested review from shoenig and drewbailey October 7, 2020 13:56
@tgross tgross marked this pull request as ready for review October 7, 2020 13:56
@tgross tgross force-pushed the b-csi-volume-detach-node-prefix branch from ee69d76 to 330d36a Compare October 7, 2020 13:57
@tgross tgross force-pushed the b-csi-volume-detach-node-prefix branch from 330d36a to b42b838 Compare October 7, 2020 13:57
Fixes a bug where the `nomad volume detach` command would not accept a node ID
prefix instead of a full node ID. The volume ID is already prefix matched
server-side.
@tgross
Copy link
Member Author

tgross commented Oct 7, 2020

Fixed the merge conflict with the changelog. Will merge this once CI gives me the ✅ again.

@tgross tgross merged commit c04e7f6 into master Oct 7, 2020
@tgross tgross deleted the b-csi-volume-detach-node-prefix branch October 7, 2020 15:15
fredrikhgrelland pushed a commit to fredrikhgrelland/nomad that referenced this pull request Oct 22, 2020
Fixes a bug where the `nomad volume detach` command would not accept a node ID
prefix instead of a full node ID. The volume ID is already prefix matched
server-side.
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 15, 2022
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.

Cannot use short node-id for detaching nomad volumes
2 participants