Skip to content

Conversation

rinsuki
Copy link
Contributor

@rinsuki rinsuki commented Apr 8, 2025

this is basically a re-apply of fd6924b, since it is accidentally removed by d1bb05d

Fix #971

see more details in original pull request, #941

this is basically a re-apply of fd6924b, since it is accidentally removed by d1bb05d
@rinsuki rinsuki marked this pull request as ready for review April 8, 2025 07:48
@rinsuki rinsuki requested a review from a team as a code owner April 8, 2025 07:48
@rinsuki rinsuki requested review from yannikmesserli and removed request for a team April 8, 2025 07:48
@yannikmesserli
Copy link
Contributor

Oh no, did I? I am really sorry! I'll review your PR asap, so you'll get this back quickly.

Copy link
Contributor

@yannikmesserli yannikmesserli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good! Tested with baseUrl "/" and works 👌 Again, sorry about that!

@yannikmesserli yannikmesserli merged commit 12c937d into cilium:master Apr 8, 2025
7 of 8 checks passed
@vazkarvishal
Copy link

vazkarvishal commented May 29, 2025

Hey folks, looks like this has been pushed but there is no helm/app release which enables this functionality. Is this intentional? or can we have a minor/bug fix version rolled out?

We are trying to add a base path /hubble to our ingress for this UI as we have a shared ingress for all internal cluster tools, but unable to proxy the request using ingress rules, as the route validation logic seems to be happening on the client side (on the users browser), before the route can be proxied from /hubble to / before sending it to the hubble UI. Is there any way we can handle this?

@nguu0123
Copy link

having the same problem with cilium helm chart version 1.17.5

@rinsuki
Copy link
Contributor Author

rinsuki commented Jun 20, 2025

@vazkarvishal @nguu0123 just for workaround, you can use image from the CI, quay.io/cilium/hubble-ui-ci:12c937d8cc8cd47ac66b9a7a10d7d6b1b1a86f0e@sha256:873632cda74618532270abf519a4f91c2c3734edf3eafad20af0a7e81c3fe61c (taken from https://github.com/cilium/hubble-ui/actions/runs/14329917577/job/40163362696 , and I think helm chart have a tag override settings)

UPDATE: in Helm chart, you can override it by hubble.ui.frontend.image.*

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

Successfully merging this pull request may close these issues.

HubbleUI still does not load on a Prefix
4 participants