Prometheus not accessible via cilium ingress

[adippl]

Prometheus not accessible via cilium ingress

[squeed]

I wonder if the pathType: ImplementationSpecific is causing issues. What if you change it to Prefix?

[squeed]

Aha, I bet https://github.com/cilium/cilium/issues/30071is the same issue

[youngnick]

Yes, after checking, because ImplementationSpecific paths are implemented in Cilium's Envoys as Regex matches, in the absence of special characters to make the regex match any prefix, they behave like an Exact match.

We already have users using ImplementationSpecific as a regex match, so I don't think we can change this now.

For kube-prometheus-stack, it appears as though the pathType is configurable as a Helm value (see https://github.com/prometheus-community/helm-charts/blob/a2a395ad078a330f9afaf3962f7cc47e2cfaf8fc/charts/prometheus/values.yaml#L355), so I'd recommend giving that a go and/or logging an issue with them if the default is still ImplementationSpecific.

[adippl]

I've configured pathType: Prefix in helm values and got prometheus working.

I've also tried wildcard in the path path: /.* without setting pathType and this also worked.

Is there any documentation default behavior of pathType: ImplementationSpecific on docs.cilium.io?
It would be nice to warn users about this behavior.

[lmb]

@youngnick can we close this as a duplicate?

[ti-mo]

Let's not close this out yet, I've added requires-doc-change because this seems a little surprising from user POV. Of course it makes sense if you're familiar with regex, but this might be good to call out in the docs.

[youngnick]

Agreed, I think we should make this clearer in the docs.

[github-actions]

This issue has been automatically marked as stale because it has not
had recent activity. It will be closed if no further activity occurs.

[github-actions]

This issue has not seen any activity since it was marked stale.
Closing.