jimmykodes commented on code in PR #52953:
URL: https://github.com/apache/airflow/pull/52953#discussion_r3149755727


##########
chart/templates/api-server/api-server-ingress.yaml:
##########
@@ -76,7 +76,7 @@ spec:
             pathType: {{ .pathType }}
             backend:
               service:
-                name: {{ .serviceName }}
+                name: {{ $fullname }}-api-server

Review Comment:
   I know I'm 7 months late, and honestly not really interested in doing the 
work to open (or check for) a GH issue for this, but this breaks the ability to 
route traffic to a separate backend service, which I've been doing for a while. 
Maybe that wasn't intended behavior, but I've been using preceding paths to run 
more than just airflow on a single domain for a couple years at this point, and 
now I have to figure out how to patch this myself.
   (just trying to surface some side effects here, not trying to be whiny, 
y'all do good work)



-- 
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]

Reply via email to