morhidi commented on a change in pull request #67:
URL:
https://github.com/apache/flink-kubernetes-operator/pull/67#discussion_r828788151
##########
File path: helm/flink-operator/templates/ingress.yaml
##########
@@ -17,6 +17,23 @@
################################################################################
---
{{- if .Values.ingress.create }}
+{{- if .Values.watchNamespaces }}
Review comment:
@wangyang0918 So it seems my assumption about the one LB address per
Ingress is not correct:
> With the NGINX Ingress controller you can also have multiple ingress
objects for multiple environments or namespaces with the same network load
balancer; with the ALB, each ingress object requires a new load balancer.
https://aws.amazon.com/blogs/opensource/network-load-balancer-nginx-ingress-controller-eks/
It seems we can create one ingress per deployment, and it get's merged by
nginx. I'll test this and see if it works. Having a single ingress per
deplyment would solve the ownership issue as well.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
