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

When ui_dir is set in windows, ui does not work. Separate UI download/extraction seems to not be required for current version of consul on Windows. #339

Closed
jvogt opened this issue Aug 5, 2016 · 4 comments · Fixed by #360

Comments

@jvogt
Copy link

jvogt commented Aug 5, 2016

The current version of consul only needs ui = true for the ui to work (at least on Windows). No separate download required. But if ui_dir is set in the consul.json config file, visiting the ui results in a 404. Even when it is pointing to a linked location where the ui is actually separately extracted to (as this cookbook's default behavior attempts to do)

I couldn't figure out how to override this option in a way that does not print the ui_dir line to the consul config.

Tried various ways of setting node['consul']['config']['ui_dir'] to false, nil, '', etc.

Not sure of best way to correct or I would submit a PR myself.

@sonnysideup
Copy link

Setting ui = true using Consul v0.6.4 on Linux (debian 8.5) results in the following behavior:

  1. The webui package is automatically downloaded and installed under /opt/consul-webui.
  2. The webui installation is symlinked to /var/lib/consul/ui.
  3. Consul server the UI component correctly.

As of Consul v0.6.4 (possibly earlier):

-ui - Enables the built-in web UI server and the required HTTP routes. This eliminates the need to maintain the Consul web UI files separately from the binary.

Ideally, setting ui = true should just add the option to the main config file. Another attribute could trigger whether or not the consumer wants to install the webui as a separate package, and maybe that's the default behavior for now in order to maintain backwards compatibility.

@jeffbyrnes
Copy link
Contributor

#315 resolves this, but a version of this cookbook with those changes hasn’t been released yet.

@pdf
Copy link
Contributor

pdf commented Oct 19, 2016

FYI, #315 needs #360 to work as expected.

@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.

5 participants