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

Install ui broken ? #19

Closed
KrisBuytaert opened this issue Sep 8, 2014 · 2 comments
Closed

Install ui broken ? #19

KrisBuytaert opened this issue Sep 8, 2014 · 2 comments

Comments

@KrisBuytaert
Copy link

Is it just me or should

https://github.com/solarkennedy/puppet-consul/blob/master/manifests/install.pp#L54
Read something different.
I'd expect this to be the "if gui" then install this package as one might not want to have the gui on every node.

One will probably set the ui_diir when one wants to install the gui so

  • if $consul::ui_package_name {
    
  • if $consul::ui_dir {

Should do the trick

@solarkennedy
Copy link
Contributor

I agree, installing via a zip seems to be different than installing the ui via a package. PR me? cc @croomes

@solarkennedy
Copy link
Contributor

This has been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants