vagrant-managed-servers icon indicating copy to clipboard operation
vagrant-managed-servers copied to clipboard

Wrong exit code when server is not reachable

Open JMLX42 opened this issue 7 years ago • 3 comments

Expected behavior

$ vagrant provision dev
==> dev: The managed server is not reachable. Check if the `config.managed.server` is correct.
$ echo $?
1

Actual behavior

$ vagrant provision dev
==> dev: The managed server is not reachable. Check if the `config.managed.server` is correct.
$ echo $?
0

JMLX42 avatar Jun 27 '18 07:06 JMLX42

This is not good :(

pvalena avatar Nov 18 '19 11:11 pvalena

Hey @promethe42 , @pvalena ,

sorry for the delay in response. I'm currently not finding time to properly support this as needed, but I'm happy to accept pull requests for this.

Only issue is that before making any functional changes:

  1. the plugin should be lifted to be compatible with latest vagrant version
  2. the.travis.yml is currently failing and needs to be fixed

I will try to find some time during the next days. If you want to support, I'm happy about PRs as well.

Cheers, Torben

tknerr avatar Nov 20 '19 10:11 tknerr

the plugin should be lifted to be compatible with latest vagrant version

How's that? We're experimenting with it, and it works fine, currently. Do you have some specifics?

I'll try to check travis-ci failures later on, but I'm no expert.

pvalena avatar Nov 20 '19 16:11 pvalena