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

Mount under root owned folder fails in 1.3.0 #65

Closed
jlebon opened this issue Dec 21, 2016 · 1 comment · Fixed by #66
Closed

Mount under root owned folder fails in 1.3.0 #65

jlebon opened this issue Dec 21, 2016 · 1 comment · Fixed by #66

Comments

@jlebon
Copy link
Collaborator

jlebon commented Dec 21, 2016

After upgrading to 1.3.0, I get stuck at "Checking Mount.." when starting a halted machine. Full output:

$ rpm -q vagrant ruby vagrant-sshfs
vagrant-1.8.1-5.fc24.noarch
ruby-2.3.1-56.fc24.x86_64
vagrant-sshfs-1.3.0-1.fc24.noarch
$ vagrant up
Bringing machine 'vmcheck' up with 'libvirt' provider...
==> vmcheck: Starting domain.
==> vmcheck: Waiting for domain to get an IP address...
==> vmcheck: Waiting for SSH to become available...
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/usr/share/gems/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:84:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
==> vmcheck: Creating shared folders metadata...
==> vmcheck: Mounting SSHFS shared folder...
==> vmcheck: Mounting folder via SSHFS: /code/github.com/projectatomic/rpm-ostree/vagrant-dev => /var/roothome/sync
==> vmcheck: Checking Mount..
==> vmcheck: Checking Mount..
==> vmcheck: Checking Mount..
==> vmcheck: Checking Mount..
==> vmcheck: Checking Mount..
==> vmcheck: Checking Mount..
==> vmcheck: Checking Mount..
Mounting SSHFS shared folder via slave SSHFS mount failed. Please
look at the below STDERR output from the processes that were run.

SSH command:

Warning: Permanently added '192.168.121.201' (ECDSA) to the list of known hosts.


SFTP command:

This is the Vagrantfile that I'm using: https://github.com/projectatomic/rpm-ostree/blob/master/Vagrantfile (though note the fuse-sshfs workaround mentioned in: https://github.com/projectatomic/rpm-ostree/tree/master/vagrant).

Reverting back to 1.2.1 works.
@jlebon jlebon changed the title Regression in 1.3.0 Regression in 1.3.0 -- stuck at "Checking Mount.." Dec 21, 2016
@dustymabe
Copy link
Owner

This has to do with the fact that you are trying to mount in a directory owned by root. The checks that I added recently run some commands without sudo and those checks are failing now.

I'll put in a PR soon to resolve this.

dustymabe added a commit that referenced this issue Dec 22, 2016
@dustymabe
add sudo to readlink and test -e commands
d0fc26a 
In b3eae35 and b3eae35 we added some attempt to make resolve the
path of symbolic links and also a check to make sure a file path
exists before checking it. The commands we used for that don't run
privileged by default, which causes failures if you try to mount
under a directory owned by root.

Fixes #65
@dustymabe dustymabe changed the title Regression in 1.3.0 -- stuck at "Checking Mount.." Mount under root owned folder fails in 1.3.0 Dec 22, 2016
@dustymabe dustymabe mentioned this issue Dec 22, 2016
Merged
dustymabe added a commit that referenced this issue Dec 22, 2016
@dustymabe
add sudo to readlink and test -e commands
895264b 
In b3eae35 and a4abb8a we added some attempt to make resolve the
path of symbolic links and also a check to make sure a file path
exists before checking it. The commands we used for that don't run
privileged by default, which causes failures if you try to mount
under a directory owned by root.

Fixes #65
dustymabe added a commit that referenced this issue Dec 22, 2016
@dustymabe
add sudo to readlink and test -e commands
ba12edf 
In b3eae35 and a4abb8a we added some attempt to resolve the
path of symbolic links and also a check to make sure a file path
exists before checking it. The commands we used for that don't run
privileged by default, which causes failures if you try to mount
under a directory owned by root.

Fixes #65
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add sudo to readlink and test -e commands dustymabe/vagrant-sshfs
2 participants
@dustymabe @jlebon