Skip to content
This repository has been archived by the owner on Sep 26, 2021. It is now read-only.

Fix issue #4424 - Pre-create check: "Hyper-V PowerShell Module is not available" #4426

Merged
merged 2 commits into from
Mar 20, 2018
Merged

Fix issue #4424 - Pre-create check: "Hyper-V PowerShell Module is not available" #4426

merged 2 commits into from
Mar 20, 2018

Conversation

beppler
Copy link
Contributor

@beppler beppler commented Mar 19, 2018

Pull request #4404 adds the ability to run Hyper-V driver when VMware PowerCLI is installed, but it make the pre-create check for Hyper-V module fail.

This corrects the test as indicated as best solution on #4424.

@GordonTheTurtle
Copy link

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "4424-powershell-module-not-available" git@github.com:beppler/machine.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842357010440
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Carlos Alberto Costa Beppler added 2 commits March 19, 2018 15:13
Signed-off-by: Carlos Alberto Costa Beppler <carlos.beppler@mps.com.br>
Signed-off-by: Carlos Alberto Costa Beppler <carlos.beppler@mps.com.br>
@dgageot
Copy link
Member

dgageot commented Mar 20, 2018

LGTM

@dgageot dgageot merged commit e501d1b into docker:master Mar 20, 2018
@gbraad
Copy link
Contributor

gbraad commented Mar 21, 2018

Can this be tagged as a new (minor) release? As this fixes a breaking change...

Ref: minishift/minishift#2159 (comment)

@krzysztofantczak
Copy link

Quick hack for people in hurry. Editing docker-machine binary file and replacing (ie. with vim or other editor - kind of - safe for binary files) :%s/Hyper-V/hyper-v/g works.

@gbraad
Copy link
Contributor

gbraad commented Mar 21, 2018

I'd rather not choose to go for local patches, as this will result in issues later on.
At the moment we pull a revision ... waiting for a tag

@beppler beppler deleted the 4424-powershell-module-not-available branch March 21, 2018 11:58
@mtudor
Copy link

mtudor commented Apr 6, 2018

Although hacky, @krzysztofantczak suggestion did work for me (on Windows, C:\Program Files\Docker\Docker\resources\bin\docker-machine.exe - make a backup of the file first) if anyone is in a bind and needs a quick fix; at your own risk and bearing in mind the warning from @gbraad.

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.

6 participants