provectus / kafka-ui

Open-Source Web UI for Apache Kafka Management
Apache License 2.0
9.69k stars 1.17k forks source link

ALB ingress with wildcard not support pathType: Prefix #3253

Closed Dgadavin closed 1 year ago

Dgadavin commented 1 year ago

Good day. I faced with problem when ALB ingress controller failed to build model because wildcard /* in path not supported with pathType: Prefix.

Could you please add a value to override pathType in values.yaml?

Set up Chart version: 0.4.2

Steps to Reproduce

Create ingress with such values

    ingress:
      enabled: true
      annotations:
        kubernetes.io/ingress.class: alb
        alb.ingress.kubernetes.io/scheme: internal
        alb.ingress.kubernetes.io/target-type: ip
        alb.ingress.kubernetes.io/load-balancer-attributes: idle_timeout.timeout_seconds=900
        external-dns.alpha.kubernetes.io/hostname: "kafka-ui.example.com"
        alb.ingress.kubernetes.io/certificate-arn: <CERT-ARN>
        alb.ingress.kubernetes.io/listen-ports: '[{"HTTP": 80}, {"HTTPS":443}]'
        alb.ingress.kubernetes.io/actions.ssl-redirect: '{"Type": "redirect",
          "RedirectConfig": { "Protocol": "HTTPS", "Port": "443", "StatusCode":
          "HTTP_301"}}'
        alb.ingress.kubernetes.io/group.name: prod-int

      path: "/*"
      host: "kafka-ui.example.com"

And you will have such error:

Failed build model due to ingress: dev/kafka-ui: prefix path shouldn't contain wildcards: /*

Expected behavior Ingress work and model build without errors. For this pathType should be ImplementationSpecific

github-actions[bot] commented 1 year ago

Hello there Dgadavin! πŸ‘‹

Thank you and congratulations πŸŽ‰ for opening your very first issue in this project! πŸ’–

In case you want to claim this issue, please comment down below! We will try to get back to you as soon as we can. πŸ‘€

Haarolean commented 1 year ago

Hey, thanks for reaching out.

We'll take a look!

Haarolean commented 1 year ago

@Narekmat please

Dgadavin commented 1 year ago

Thanks a lot @Haarolean and @Narekmat. Superfast.