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

consul ui ingress resource only works if set to / #764

Closed
vtuson opened this issue Aug 27, 2018 · 9 comments
Closed

consul ui ingress resource only works if set to / #764

vtuson opened this issue Aug 27, 2018 · 9 comments
Labels
stale 15 days without activity

Comments

@vtuson
Copy link

vtuson commented Aug 27, 2018

I tried many many combinations of annotations but basically consul-ui always redirects to to /ui/ so the only way to get the ingress to work (that I have found) is to math path / (wont work for example if you make it /consul/)

This is due to a long standing issue with consul, seems that was fixed but broke again with the new UI - hashicorp/consul#1930

Also it might be possible to work around this added a custom config snippet (https://github.com/kubernetes/ingress-nginx/blob/master/docs/user-guide/nginx-configuration/annotations.md#configuration-snippet) but I dont know enough about nginx to suggest a solution

@juan131
Copy link
Contributor

juan131 commented Aug 27, 2018

Hi @vtuson

Thanks for opening this issue! It seems there's no way to configure the UI path on the application itself, therefore, it's sth we could fix by adding some kind of env. variable to the container to use a custom path for the UI.

As you mentioned, a possible workaround would be using a configuration snippet so we can modify the way NGINX handle the proxy redirection.

However, this configuration snippet will be including on the Ingress annotations (which are different depending on the Ingress solution you're using). Hence, my doubts about this approach. We could add some annotations for NGINX BUT they will be ignored by other Ingress Controllers...

@vtuson
Copy link
Author

vtuson commented Aug 27, 2018

you are right, that they wouldnt be generic, not sure if it is worth adding nginx as it is the standard upstream provided or not? but might be worth documenting it in the readme

@juan131
Copy link
Contributor

juan131 commented Aug 28, 2018

What do you think @sameersbn @prydonius ?

@prydonius
Copy link
Contributor

Perhaps the simplest thing for now is to document that Consul UI can only be exposed on the root in an Ingress. FYI, we have the same issue in Kubeapps, the Dashboard currently cannot be mounted in a subpath due to the way the React app loads things and uses routes.

@juan131
Copy link
Contributor

juan131 commented Aug 29, 2018

Are you ok with just documenting it @vtuson ? We could also modify the NOTES.txt so we show a warning in case we detect someone is using a different path on the Ingress Rule.

@stale
Copy link

stale bot commented Nov 24, 2018

This Issue has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thanks for the feedback.

@stale stale bot added the stale 15 days without activity label Nov 24, 2018
@stale
Copy link

stale bot commented Nov 30, 2018

Due to the lack of activity in the last 5 days since it was marked as "stale", we proceed to close this Issue. Do not hesitate to reopen it later if necessary.

@stale stale bot closed this as completed Nov 30, 2018
@mumubin
Copy link

mumubin commented Oct 9, 2019

Is there a solution?

@juan131
Copy link
Contributor

juan131 commented Oct 9, 2019

Hi @mumubin

We didn't document anything so far. There are other users requesting similar features for other charts. If you take a look to the issue below, you'll find out there's a way to play with Ingress annotations so you add extra configuration with redirections:

helm/charts#17196

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

No branches or pull requests

4 participants