Skip to content
This repository has been archived by the owner on Oct 21, 2024. It is now read-only.

Fix FS Events forwarding to xhyve VM #150

Merged

Conversation

fgrehm
Copy link
Contributor

@fgrehm fgrehm commented Mar 2, 2016

Just noticed that today and I tracked down the root cause to a blanky SSHKeyPath on the VM's state that results in wrong arguments being passed on to fsevents_to_vm:

$ docker-machine create remove --driver xhyve
....
Docker is up and running!
To see how to connect your Docker Client to the Docker Engine running on
this virtual machine, run: docker-machine env remove

$ docker-machine inspect remove | grep -i sshkey
        "SSHKeyPath": "",

This might be something to be fixed on the driver's side but I guess it does the trick for now.

Just noticed that today and I tracked down the root cause to a blanky
`SSHKeyPath` on the VM's state that results in wrong arguments being
passed on to `fsevents_to_vm`:

```
$ docker-machine create remove --driver xhyve
....
Docker is up and running!
To see how to connect your Docker Client to the Docker Engine running on
this virtual machine, run: docker-machine env remove

$ docker-machine inspect remove | grep -i sshkey
        "SSHKeyPath": "",
```

This might be something to be fixed on the driver's side but I guess it
does the trick for now.
@codekitchen
Copy link
Owner

Oh good catch thanks. Have you filed an issue with the xhyve driver? It does seem like something they should fix.

@fgrehm
Copy link
Contributor Author

fgrehm commented Mar 2, 2016

Have you filed an issue with the xhyve driver? It does seem like something they should fix.

Not yet, I haven't had a chance to dig into it as I was unable to build it locally neither I know how exactly drivers should behave in regard to that. But I'll try to look into it tomorrow.

Either way, unless we want to implement custom logic based on the driver version, I guess we might want to keep this little hack around while the behavior gets dealt with upstream and a new release is out 😅

WDYT?

@codekitchen
Copy link
Owner

Yeah for sure, I just want to make sure they're aware of it

@codekitchen
Copy link
Owner

Doh, I just released dinghy 4.3.0 and forgot to merge this first 😵

Merging and releasing 4.3.1. Thanks again!

codekitchen added a commit that referenced this pull request Mar 3, 2016
@codekitchen codekitchen merged commit 91e9989 into codekitchen:master Mar 3, 2016
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.

2 participants