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

(GH-33) Resolve HyperV deprecation warning for differencing vs linked clone #30

Merged
merged 1 commit into from
Jul 31, 2019

Conversation

FranklinYu
Copy link
Contributor

@FranklinYu FranklinYu commented Nov 12, 2018

Reference:
https://github.com/hashicorp/vagrant/blob/v2.1.2/website/source/docs/hyperv/configuration.html.md

Technically speaking, even

v.differencing_disk = true

requires 1.8.3, but I think nobody is using that old Vagrant anyway.

Closes #33

@@ -50,7 +50,11 @@ Vagrant.configure("2") do |config|
# The time in seconds to wait for the virtual machine to report an IP address
v.ip_address_timeout = 130
# Use differencing disk instead of cloning whole VHD
v.differencing_disk = true
Copy link
Contributor

Choose a reason for hiding this comment

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

When did we add the bit about differencing? Linked Clone is all I've used... hmm

Copy link
Contributor

Choose a reason for hiding this comment

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

Nevermind, this is HyperV - good to see they are making the naming consistent (as they should be).

@ferventcoder ferventcoder changed the title Fix deprecation warning Fix - HyperV deprecation warning for differencing vs linked clone Dec 6, 2018
@ferventcoder ferventcoder changed the title Fix - HyperV deprecation warning for differencing vs linked clone (GH-33) Resolve HyperV deprecation warning for differencing vs linked clone Dec 6, 2018
@FranklinYu
Copy link
Contributor Author

I amended the commit message to refer to the issue.

@FranklinYu
Copy link
Contributor Author

Hey is there any update? Or anything I can help?

Copy link
Member

@gep13 gep13 left a comment

Choose a reason for hiding this comment

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

LGTM!

@gep13 gep13 merged commit a9af978 into chocolatey-community:master Jul 31, 2019
@gep13
Copy link
Member

gep13 commented Jul 31, 2019

@FranklinYu your changes have been merged, thanks for your contribution 👍

@FranklinYu FranklinYu deleted the fix/linked-clone branch August 1, 2019 15:18
Copy link
Contributor

@ferventcoder ferventcoder left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants