sous-chefs / consul

Development repository for the consul cookbook
https://supermarket.chef.io/cookbooks/consul
Apache License 2.0
192 stars 244 forks source link

Consul command not available on Windows #451

Closed emachnic closed 5 years ago

emachnic commented 7 years ago

Just a quick fix but currently, the Windows installation does not add the Consul directory to the path and trying to run consul commands from Powershell do not work. I was able to get around this by using windows_path from the Windows cookbook like so:

windows_path "C:/Program Files/consul/#{node['consul']['version']}" do
  action :add
end

It might be cool to go ahead and do this in the cookbook already so that the consumers of the cookbook don't have to.

Cheers, Evan

gdavison commented 7 years ago

A shortcut might be a better option, in the same way that the *nix versions use a symlink. I haven't tested that this works, though

gdavison commented 7 years ago

@emachnic, a Windows symlink will do it. I have code on https://github.com/paybyphone/consul-cookbook/tree/windows_executable_symlink that creates it.

It's waiting on #452 to fix Windows issues with cookbook version 3.0

gdavison commented 6 years ago

@emachnic This is available on the master branch, but not yet in a numbered release

damacus commented 5 years ago

Closing due to inactivity.

If this is still an issue please reopen or open another issue. Alternatively drop by the #sous-chefs channel on the Chef Community Slack and we'll be happy to help!

Thanks, Sous-Chefs

lock[bot] commented 4 years ago

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.