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

Fix startup mount error on Parallels 14 guest tools #1122

Merged
merged 1 commit into from
Oct 10, 2018
Merged

Fix startup mount error on Parallels 14 guest tools #1122

merged 1 commit into from
Oct 10, 2018

Conversation

max-arnold
Copy link
Contributor

Description

This is a workaround for the following error (happens on first startup with the latest Parallels Tools):

Failed to mount folders in Linux guest. This is usually because
the "prl_fs" file system is not available. Please verify that
Parallels Tools are properly installed in the guest and
can work properly. If so, the VM reboot can solve a problem.
The command attempted was:

mount -t prl_fs -o uid=`id -u vagrant`,gid=`getent group vagrant | cut -d: -f3` media_DockerData /media/DockerData
mount -t prl_fs -o uid=`id -u vagrant`,gid=`id -g vagrant` media_DockerData /media/DockerData

Issues Resolved

#1115

@sttts
Copy link

sttts commented Oct 9, 2018

/lgtm

Copy link
Contributor

@legal90 legal90 left a comment

Choose a reason for hiding this comment

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

👍 LGTM.
It's the same what we have in vagrant-parallels: https://github.com/Parallels/vagrant-parallels/blob/master/lib/vagrant-parallels/guest_cap/linux/mount_parallels_shared_folder.rb#L98-L109

But due to the Parallels/vagrant-parallels#325 we need to have it applied on the box build time.

Copy link
Contributor

@cheeseplus cheeseplus left a comment

Choose a reason for hiding this comment

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

Thanks a bunch, will pass this through the builders and see how it fares!

@cheeseplus cheeseplus merged commit 2794986 into chef:master Oct 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants