search

ruromero / cloudevents-player

Cloudevents Player tool
Apache License 2.0
40 stars 15 forks source link

fix: baseUri of broker ingress starts by broker name #67

Closed dmartinol closed 1 year ago

dmartinol commented 1 year ago

When BROKER_NAME is provided instead of BROKER_URI, the baseUri generated is: http://broker-ingress.knative-eventing.svc.cluster.local/dmartino-kafka/kafka-broker

But the .status.address.url of the broker kafka-broker is instead: http://kafka-broker-ingress.knative-eventing.svc.cluster.local/dmartino-kafka/kafka-broker

The first part has to match the BROKER_NAME variable

ruromero commented 1 year ago

The problem is not the broker_name, is the type of the broker that uses a different ingress. For kafka it's the kafka-broker-ingress. For that case it's better to use the BROKER_URI