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.

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

Windows Consul service does not start up #273

Closed
gdavison opened this issue Feb 5, 2016 · 7 comments · Fixed by #274 or #275
Closed

Windows Consul service does not start up #273

gdavison opened this issue Feb 5, 2016 · 7 comments · Fixed by #274 or #275

Comments

@gdavison
Copy link
Contributor

gdavison commented Feb 5, 2016

The consul service expects its data directory to exist. This was recently removed, in commit aac210a, causing consul to fail on startup

@johnbellone johnbellone added the bug label Feb 5, 2016
@Ginja
Copy link
Contributor

Ginja commented Feb 5, 2016

This was removed in response to this:

#259 (comment)

I haven't seen the behaviour you've described, but I'm testing it now with Test Kitchen.

@Ginja
Copy link
Contributor

Ginja commented Feb 5, 2016

Just verified that the data directory is being created. What does your run list look like? Could you send me the output of your failed run?

@Ginja
Copy link
Contributor

Ginja commented Feb 5, 2016

Ah, I see why it worked for me now. In our wrapper cookbook I'm setting:

node.default['consul']['config']['data_dir'] = join_path data_prefix, 'data'

I think we should revert one of my changes in #259. I'm submitting another PR shortly. I'll add that in there.

@Ginja Ginja mentioned this issue Feb 5, 2016
@gdavison
Copy link
Contributor Author

gdavison commented Feb 6, 2016

I've got at PR almost ready, with tests.

On Feb 5, 2016, at 15:28, Riley Shott notifications@github.com wrote:

Ah, I see why it worked for me now. In our wrapper cookbook I'm setting:

node.default['consul']['config']['data_dir'] = join_path data_prefix, 'data'

I think we should revert one of my changes in #259. I'm submitting another PR shortly. I'll add that in there.


Reply to this email directly or view it on GitHub.

@Ginja
Copy link
Contributor

Ginja commented Feb 6, 2016

@gdavison I have it included in #274. Please let me know if that works for you.

@gdavison
Copy link
Contributor Author

gdavison commented Feb 6, 2016

Ah, didn't see that you have a PR, too.

@lock
Copy link

lock bot commented Apr 25, 2020

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.

@lock lock bot locked as resolved and limited conversation to collaborators Apr 25, 2020
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 a pull request may close this issue.

3 participants