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.

Sign up for GitHub

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

Jump to bottom

Fix docker-machine rm command #17

Closed
shouze opened this issue Apr 19, 2016 · 2 comments · Fixed by #18
Closed

Fix docker-machine rm command #17

shouze opened this issue Apr 19, 2016 · 2 comments · Fixed by #18
Assignees
@QuentinPerez
Labels
bug

Comments

@shouze
Copy link

shouze commented Apr 19, 2016

Given I want to remove a docker machine
When I execute the command

$ docker-machine rm test-docker-machine                                                                                                                                    About to remove test-docker-machine
Are you sure? (y/n): y
Error removing host "test-docker-machine": The method is not allowed for the requested URL.

Then I execute docker ls
And the output looks like

test-docker-machine            -        scaleway       Error                                                     Unknown   "95bdefdc-21c7-4487-a09c-b0e1adae59fb" not found

Note: the scaleway machine if effectively removed, running a consecutive docker-machine rm -f solve the problem on the docker side.

Environment:

  • scaleway/scaleway/docker-machine-driver-scaleway: stable 1.0.1, HEAD
  • docker-machine version 0.7.0, build a650a40
@moul moul added the bug label Apr 20, 2016
@moul
Copy link
Contributor

moul commented Apr 20, 2016

Hi @shouze, thank you for your feedback,

@QuentinPerez was already working on this bug, it should be fixed by today

@QuentinPerez QuentinPerez mentioned this issue Apr 20, 2016
Merged
@QuentinPerez
Copy link
Contributor

QuentinPerez commented Apr 20, 2016 •
edited
Loading

Hi @shouze,

The PR 18 seems fix this bug 😉

@moul moul closed this as completed in #18 Apr 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@QuentinPerez QuentinPerez

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

rm: fix UUID ip QuentinPerez/docker-machine-driver-scaleway
3 participants
@shouze @moul @QuentinPerez