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

Helm Deployment - UI support kubernetes ingress #14227

Open
AlexChenSkyBlue opened this issue Oct 11, 2021 · 7 comments
Open

Helm Deployment - UI support kubernetes ingress #14227

AlexChenSkyBlue opened this issue Oct 11, 2021 · 7 comments
Assignees
Labels
stale The PR/Issue does not have recent activities and will be closed automatically type-feature This issue is a feature request

Comments

@AlexChenSkyBlue
Copy link

Is your feature request related to a problem? Please describe.
when deploy helm, current UI does not have way to set href as base_url for Ingress Setting. Please help add this to UI code
Describe the solution you'd like
A clear and concise description of what you want to happen.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Urgency
Explain why the feature is important

Additional context
Add any other context or screenshots about the feature request here.

@AlexChenSkyBlue AlexChenSkyBlue added the type-feature This issue is a feature request label Oct 11, 2021
@yuzhu
Copy link
Contributor

yuzhu commented Oct 11, 2021

@ZhuTopher any thoughts on this?

@ZhuTopher
Copy link
Contributor

@AlexChenSkyBlue Could you provide some more details? I'm not clear on what exactly you're requesting.

  1. What UI are you referring to? Alluxio pod UI? Kubernetes dashboard UI? Something else?

  2. What is this "Ingress Setting"? Are you talking about a Kubernetes Ingress or something else?

  3. What are you trying to modify in the UI? Can you link or describe specifically what the href/base_url you're referencing is?

Thanks for your help!

@yuzhu
Copy link
Contributor

yuzhu commented Oct 12, 2021

@AlexChenSkyBlue do you mind joining our community slack so that we can discuss this issue interactively? https://alluxio.io/slack

@AlexChenSkyBlue
Copy link
Author

@AlexChenSkyBlue Could you provide some more details? I'm not clear on what exactly you're requesting.

  1. What UI are you referring to? Alluxio pod UI? Kubernetes dashboard UI? Something else?
    I am referring to alluxio ui that listen on 19999.
  2. What is this "Ingress Setting"? Are you talking about a Kubernetes Ingress or something else?
    yes. I am referring to Kubernetes Ingress. In order to have multiple endpoint behind ingress proxy. We need to have base path similar what kibana did. https://www.elastic.co/guide/en/kibana/current/settings.html#server-basePath
  3. What are you trying to modify in the UI? Can you link or describe specifically what the href/base_url you're referencing is?
    see above
    Thanks for your help!

@ZhuTopher
Copy link
Contributor

Thanks for you feedback, I'm looking into this! I'll post an update when I have one.

@ZhuTopher
Copy link
Contributor

ZhuTopher commented Oct 22, 2021

Issue requirements update:

  1. We need to add an Alluxio configuration property (eg: alluxio.master.web.basepath) to specify the base path for web resources (CSS, .js, etc.) so that our UI elements can properly fetch their resources when behind a proxy (or in this case, a k8s ingress). This property would be similar to the reporter's linked Kibana basePath setting.

  2. We need to add the k8s Ingress definition to the Helm chart.

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The PR/Issue does not have recent activities and will be closed automatically label Jan 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale The PR/Issue does not have recent activities and will be closed automatically type-feature This issue is a feature request
Projects
None yet
Development

No branches or pull requests

3 participants