Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

Hook WaitForCommunicator instead of Boot #22

Closed
wants to merge 1 commit into from
Closed

Hook WaitForCommunicator instead of Boot #22

wants to merge 1 commit into from

Conversation

nrwiersma
Copy link

If you are hooking Boot, this will fail on first boot trying to send the plugin with communicator. Instead hook WaitForCommunicator, then we know we send the plugin.

@OndraM
Copy link

OndraM commented Jun 12, 2015

+1, hooking to the Boot event causes this (Vagrant 1.7.2) during vagrant up.

...
==> default: Forwarding ports...
    default: 22 => 2222 (adapter 1)
==> default: Running 'pre-boot' VM customizations...
==> default: Booting VM...
Vagrant timed out while attempting to connect via SSH. This usually
means that the VM booted, but there are issues with the SSH configuration                                                                                        
or network connectivity issues. Please try to `vagrant reload` or                                                                                                
`vagrant up` again

And with this fix it is working properly.

@fgrehm
Copy link
Owner

fgrehm commented Jul 24, 2015

Sorry for the long silence but as per GH-24 I'm looking for help to maintain this plugin and for now I'm holding back from bringing in new changes =/

If you are interested on maintaining the project and keep things moving please shoot me an email or drop a message on GH-24 so I can arrange things.

Cheers 🍻

@alpha01 alpha01 mentioned this pull request Apr 27, 2016
@alpha01 alpha01 mentioned this pull request May 11, 2016
@alpha01
Copy link
Collaborator

alpha01 commented Jun 22, 2016

Update has been applied to vagrant-notify 0.5.0.

@alpha01 alpha01 closed this Jun 22, 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.

None yet

4 participants