vmware-archive / kubeless

Kubernetes Native Serverless Framework
https://kubeless.io
Apache License 2.0
6.86k stars 754 forks source link

Custom Cluster Domains #832

Open clatko opened 6 years ago

clatko commented 6 years ago

Is this a BUG REPORT or FEATURE REQUEST?: FEATURE REQUEST

What happened: Cluster Domain hardcoded into NATS/Kafka.

What you expected to happen: To be able to use my custom cluster domain.

How to reproduce it (as minimally and precisely as possible): Set up kafka trigger on a non-cluster.local cluster. Watch it fail.

Anything else we need to know?:

Environment:

andresmgot commented 6 years ago

Hi @clatko, can you give us more details about the error? How is the domain you are trying to use? The logs of the Kafka/NATS controller should help. I assume the culprit is this piece:

https://github.com/kubeless/kubeless/blob/master/pkg/utils/event_sender.go#L55

The same should happen for CronJobs since they use a similar address.

cc/ @murali-reddy

murali-reddy commented 6 years ago

Agree with @andresmgot. Issue is related to use of cluster.local when using Kubernetes service backing the function pods.

@clatko Just to be sure, you are referring to the custom domain name that can be specified with --cluster-domain?

clatko commented 6 years ago

That is correct. I have a PR to add this as an argument to triggers basically finished. Under a tight deadline so will finish when i can. In the meantime, i just forked and swapped out the hard coded cluster name and rebuilt the kafka trigger controller.

andresmgot commented 6 years ago

thanks @clatko, a PR will be very welcomed

alexander-alvarez commented 6 years ago

:+1: on this. I ran into this issue, but couldn't track it down and then went on vacation. Glad someone was able to find it

clatko commented 6 years ago

i have it finished, just trying to run tests and running into issues on osx. hope to have it done before weekend.

clatko commented 6 years ago

PR here - https://github.com/kubeless/kubeless/pull/857

mgolovatiy-atconsulting-ru commented 4 years ago

https://github.com/kubeless/kafka-trigger/issues/22 The same issue for kafka-trigger