-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Refactored vmware tools scripts #638
Conversation
I don't see |
vmware-iso|vmware-vmx) | ||
apt-get install -y open-vm-tools; | ||
mkdir /mnt/hgfs; | ||
echo "platform specific vmware.sh executed"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like this is indented unnecessarily
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed. Fixed it.
Now that I've got 2.2.9 out the door I can look at testing this across all the providers. |
Hi @cheeseplus, did you had the chance to test this one up? |
Let's get this in and see how we do. |
Are there any specific host requirements to use this? I tried to build a CentOS 7.2 template incorporating this pull request but the guest is unable to access shared folders, eventually timing out with:
|
Doesn't look like this helps for CentOS platforms as there is no HGFS distributed with open-vm-tools so that makes 99% of the reason why we'd use it moot :( See: #667 (comment) |
Indeed, AFAIK HGFS will only be available when the CentOS team upgrades to OVT 10. I don't use the shared folders feature that much since it is not something I would have in production (i'm a rsync guy myself :) but I imagine that I may be the exception in this case. Nonetheless, what do you think about a variable to control this behavior? So we can use the variable file to select between OVT and VMT boxes? |
Update Centos 7 to use OVT and refactored the scripts structure to make it easier to include new distributions. #600 #621