Update comment with correct naming

bjee19 · bjee19 · commit 7400fe0167a5 · 2025-12-10T17:04:30.000-08:00
diff --git a/internal/controller/nginx/config/servers.go b/internal/controller/nginx/config/servers.go
@@ -298,18 +298,18 @@ External location calls inference NJS module. The module gets the AI endpoint to
 then redirects to the internal inference location that proxies to the backend.
 
 location /coffee {
-	set $epp_internal_path /_ngf-internal-rule0-route0-inference;
-	js_content epp.getEndpoint; // gets endpoint and redirects to /_ngf-internal-rule0-route0-inference
+    set $epp_internal_path /_ngf-internal-proxy-pass-rule0-route0-backend0-inference;
+    js_content epp.getEndpoint; // gets endpoint and redirects to /_ngf-internal-proxy-pass-rule0-route0-backend0-inference
 }
-location /_ngf-internal-rule0-route0-inference {
+location /_ngf-internal-proxy-pass-rule0-route0-backend0-inference {
 	internal;
 	proxy_pass http://$inference-backend;
 }
 ---------------
 Inference extension with multiple inference backends.
 
 location /coffee {
-	rewrite $inference_backend_group_inference_conformance_app_backend__httproute_weighted_two_pools_rule0 last;
+	rewrite $inference_backend_group_routeNS__routeName_rule0 last;
 	proxy_http_version 1.1;
 }
 
@@ -318,7 +318,7 @@ location /_ngf-internal-proxy-pass-rule0-route0-backend0-inference {
 	proxy_pass http://$inference-backend0;
 }
 
-location /_ngf-internal-inference-conformance-app-backend_primary-inference-pool-pool-svc_3000-backend-0 {
+location /_ngf-internal-upstreamNamePrimary-routeNS-routeName-rule0 {
 	set $epp_internal_path /_ngf-internal-proxy-pass-rule0-route0-backend0-inference;
 	js_content epp.getEndpoint; // gets endpoint and redirects to /_ngf-internal-proxy-pass-rule0-route0-backend0-inference
 }
@@ -328,14 +328,14 @@ location /_ngf-internal-proxy-pass-rule0-route0-backend1-inference {
 	proxy_pass http://$inference-backend1;
 }
 
-location /_ngf-internal-inference-conformance-app-backend_primary-inference-pool-pool-svc_3000-backend-1 {
+location /_ngf-internal-upstreamNameSecondary-routeNS-routeName-rule0 {
 	set $epp_internal_path /_ngf-internal-proxy-pass-rule0-route0-backend1-inference;
 	js_content epp.getEndpoint; // gets endpoint and redirects to /_ngf-internal-proxy-pass-rule0-route0-backend1-inference
 }
 
-split_clients $request_id $inference_backend_group_inference_conformance_app_backend__httproute_weighted_two_pools_rule0 {
-    70.00% /_ngf-internal-inference-conformance-app-backend_primary-inference-pool-pool-svc_3000-backend-0;
-    30.00% /_ngf-internal-inference-conformance-app-backend_secondary-inference-pool-pool-svc_3000-backend-1;
+split_clients $request_id $inference_backend_group_routeNS__routeName_rule0 {
+    70.00% /_ngf-internal-upstreamNamePrimary-routeNS-routeName-rule0;
+    30.00% /_ngf-internal-upstreamNameSecondary-routeNS-routeName-rule0;
 }
 ---------------
 Inference extension with HTTP matching conditions.
@@ -353,7 +353,7 @@ it was the final location that had -inference in the name.
 location /coffee {
 	js_content httpmatches.match; // chooses backend and redirects to appropriate internal inference location
 }
-location /_ngf-internal-test_coffee_80-backend-0 {
+location /_ngf-internal-upstreamNamePrimary-routeNS-routeName-rule0 {
 	internal;
 
 	set $epp_internal_path /_ngf-internal-proxy-pass-rule0-route0-backend0-inference;
@@ -378,7 +378,7 @@ location /coffee {
 location /_ngf-internal-split-clients-rule0-route0-inference  {
 	internal;
 
-	rewrite $inference_backend_group_inference_conformance_app_backend__httproute_weighted_two_pools_rule0 last;
+	rewrite $inference_backend_group_routeNS__routeName_rule0 last;
 	proxy_http_version 1.1;
 }
 
@@ -387,7 +387,7 @@ location /_ngf-internal-proxy-pass-rule0-route0-backend0-inference {
 	proxy_pass http://$inference-backend0;
 }
 
-location /_ngf-internal-inference-conformance-app-backend_primary-inference-pool-pool-svc_3000-backend-0 {
+location /_ngf-internal-upstreamNamePrimary-routeNS-routeName-rule0 {
 	set $epp_internal_path /_ngf-internal-proxy-pass-rule0-route0-backend0-inference;
 	js_content epp.getEndpoint; // gets endpoint and redirects to /_ngf-internal-proxy-pass-rule0-route0-backend0-inference
 }
@@ -397,14 +397,14 @@ location /_ngf-internal-proxy-pass-rule0-route0-backend1-inference {
 	proxy_pass http://$inference-backend1;
 }
 
-location /_ngf-internal-inference-conformance-app-backend_primary-inference-pool-pool-svc_3000-backend-1 {
+location /_ngf-internal-upstreamNameSecondary-routeNS-routeName-rule0 {
 	set $epp_internal_path /_ngf-internal-proxy-pass-rule0-route0-backend1-inference;
 	js_content epp.getEndpoint; // gets endpoint and redirects to /_ngf-internal-proxy-pass-rule0-route0-backend1-inference
 }
 
-split_clients $request_id $inference_backend_group_inference_conformance_app_backend__httproute_weighted_two_pools_rule0 {
-    70.00% /_ngf-internal-inference-conformance-app-backend_primary-inference-pool-pool-svc_3000-backend-0;
-    30.00% /_ngf-internal-inference-conformance-app-backend_secondary-inference-pool-pool-svc_3000-backend-1;
+split_clients $request_id $inference_backend_group_routeNS__routeName_rule0 {
+    70.00% /_ngf-internal-upstreamNamePrimary-routeNS-routeName-rule0;
+    30.00% /_ngf-internal-upstreamNameSecondary-routeNS-routeName-rule0;
 }
 */
 

Original file line number	Diff line number	Diff line change
`@@ -298,18 +298,18 @@ External location calls inference NJS module. The module gets the AI endpoint to`
`298`	`298`	`then redirects to the internal inference location that proxies to the backend.`
`299`	`299`
`300`	`300`	`location /coffee {`
`301`		`- set $epp_internal_path /_ngf-internal-rule0-route0-inference;`
`302`		`- js_content epp.getEndpoint; // gets endpoint and redirects to /_ngf-internal-rule0-route0-inference`
	`301`	`+ set $epp_internal_path /_ngf-internal-proxy-pass-rule0-route0-backend0-inference;`
	`302`	`+ js_content epp.getEndpoint; // gets endpoint and redirects to /_ngf-internal-proxy-pass-rule0-route0-backend0-inference`
`303`	`303`	`}`
`304`		`-location /_ngf-internal-rule0-route0-inference {`
	`304`	`+location /_ngf-internal-proxy-pass-rule0-route0-backend0-inference {`
`305`	`305`	`internal;`
`306`	`306`	`proxy_pass http://$inference-backend;`
`307`	`307`	`}`
`308`	`308`	`---------------`
`309`	`309`	`Inference extension with multiple inference backends.`
`310`	`310`
`311`	`311`	`location /coffee {`
`312`		`- rewrite $inference_backend_group_inference_conformance_app_backend__httproute_weighted_two_pools_rule0 last;`
	`312`	`+ rewrite $inference_backend_group_routeNS__routeName_rule0 last;`
`313`	`313`	`proxy_http_version 1.1;`
`314`	`314`	`}`
`315`	`315`
`@@ -318,7 +318,7 @@ location /_ngf-internal-proxy-pass-rule0-route0-backend0-inference {`
`318`	`318`	`proxy_pass http://$inference-backend0;`
`319`	`319`	`}`
`320`	`320`
`321`		`-location /_ngf-internal-inference-conformance-app-backend_primary-inference-pool-pool-svc_3000-backend-0 {`
	`321`	`+location /_ngf-internal-upstreamNamePrimary-routeNS-routeName-rule0 {`
`322`	`322`	`set $epp_internal_path /_ngf-internal-proxy-pass-rule0-route0-backend0-inference;`
`323`	`323`	`js_content epp.getEndpoint; // gets endpoint and redirects to /_ngf-internal-proxy-pass-rule0-route0-backend0-inference`
`324`	`324`	`}`
`@@ -328,14 +328,14 @@ location /_ngf-internal-proxy-pass-rule0-route0-backend1-inference {`
`328`	`328`	`proxy_pass http://$inference-backend1;`
`329`	`329`	`}`
`330`	`330`
`331`		`-location /_ngf-internal-inference-conformance-app-backend_primary-inference-pool-pool-svc_3000-backend-1 {`
	`331`	`+location /_ngf-internal-upstreamNameSecondary-routeNS-routeName-rule0 {`
`332`	`332`	`set $epp_internal_path /_ngf-internal-proxy-pass-rule0-route0-backend1-inference;`
`333`	`333`	`js_content epp.getEndpoint; // gets endpoint and redirects to /_ngf-internal-proxy-pass-rule0-route0-backend1-inference`
`334`	`334`	`}`
`335`	`335`
`336`		`-split_clients $request_id $inference_backend_group_inference_conformance_app_backend__httproute_weighted_two_pools_rule0 {`
`337`		`- 70.00% /_ngf-internal-inference-conformance-app-backend_primary-inference-pool-pool-svc_3000-backend-0;`
`338`		`- 30.00% /_ngf-internal-inference-conformance-app-backend_secondary-inference-pool-pool-svc_3000-backend-1;`
	`336`	`+split_clients $request_id $inference_backend_group_routeNS__routeName_rule0 {`
	`337`	`+ 70.00% /_ngf-internal-upstreamNamePrimary-routeNS-routeName-rule0;`
	`338`	`+ 30.00% /_ngf-internal-upstreamNameSecondary-routeNS-routeName-rule0;`
`339`	`339`	`}`
`340`	`340`	`---------------`
`341`	`341`	`Inference extension with HTTP matching conditions.`
`@@ -353,7 +353,7 @@ it was the final location that had -inference in the name.`
`353`	`353`	`location /coffee {`
`354`	`354`	`js_content httpmatches.match; // chooses backend and redirects to appropriate internal inference location`
`355`	`355`	`}`
`356`		`-location /_ngf-internal-test_coffee_80-backend-0 {`
	`356`	`+location /_ngf-internal-upstreamNamePrimary-routeNS-routeName-rule0 {`
`357`	`357`	`internal;`
`358`	`358`
`359`	`359`	`set $epp_internal_path /_ngf-internal-proxy-pass-rule0-route0-backend0-inference;`
`@@ -378,7 +378,7 @@ location /coffee {`
`378`	`378`	`location /_ngf-internal-split-clients-rule0-route0-inference {`
`379`	`379`	`internal;`
`380`	`380`
`381`		`- rewrite $inference_backend_group_inference_conformance_app_backend__httproute_weighted_two_pools_rule0 last;`
	`381`	`+ rewrite $inference_backend_group_routeNS__routeName_rule0 last;`
`382`	`382`	`proxy_http_version 1.1;`
`383`	`383`	`}`
`384`	`384`
`@@ -387,7 +387,7 @@ location /_ngf-internal-proxy-pass-rule0-route0-backend0-inference {`
`387`	`387`	`proxy_pass http://$inference-backend0;`
`388`	`388`	`}`
`389`	`389`
`390`		`-location /_ngf-internal-inference-conformance-app-backend_primary-inference-pool-pool-svc_3000-backend-0 {`
	`390`	`+location /_ngf-internal-upstreamNamePrimary-routeNS-routeName-rule0 {`
`391`	`391`	`set $epp_internal_path /_ngf-internal-proxy-pass-rule0-route0-backend0-inference;`
`392`	`392`	`js_content epp.getEndpoint; // gets endpoint and redirects to /_ngf-internal-proxy-pass-rule0-route0-backend0-inference`
`393`	`393`	`}`
`@@ -397,14 +397,14 @@ location /_ngf-internal-proxy-pass-rule0-route0-backend1-inference {`
`397`	`397`	`proxy_pass http://$inference-backend1;`
`398`	`398`	`}`
`399`	`399`
`400`		`-location /_ngf-internal-inference-conformance-app-backend_primary-inference-pool-pool-svc_3000-backend-1 {`
	`400`	`+location /_ngf-internal-upstreamNameSecondary-routeNS-routeName-rule0 {`
`401`	`401`	`set $epp_internal_path /_ngf-internal-proxy-pass-rule0-route0-backend1-inference;`
`402`	`402`	`js_content epp.getEndpoint; // gets endpoint and redirects to /_ngf-internal-proxy-pass-rule0-route0-backend1-inference`
`403`	`403`	`}`
`404`	`404`
`405`		`-split_clients $request_id $inference_backend_group_inference_conformance_app_backend__httproute_weighted_two_pools_rule0 {`
`406`		`- 70.00% /_ngf-internal-inference-conformance-app-backend_primary-inference-pool-pool-svc_3000-backend-0;`
`407`		`- 30.00% /_ngf-internal-inference-conformance-app-backend_secondary-inference-pool-pool-svc_3000-backend-1;`
	`405`	`+split_clients $request_id $inference_backend_group_routeNS__routeName_rule0 {`
	`406`	`+ 70.00% /_ngf-internal-upstreamNamePrimary-routeNS-routeName-rule0;`
	`407`	`+ 30.00% /_ngf-internal-upstreamNameSecondary-routeNS-routeName-rule0;`
`408`	`408`	`}`
`409`	`409`	`*/`
`410`	`410`