Skip to content

Commit 2e4466f

Browse files
committed
skip deleted aigatewayroutes for extproc secret updates
Signed-off-by: siddharth1036 <[email protected]>
1 parent a2ba2c9 commit 2e4466f

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

internal/controller/gateway_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -296,7 +296,7 @@ func TestGatewayController_reconcileFilterConfigSecret_SkipsDeletedRoutes(t *tes
296296
kube := fake2.NewClientset()
297297
ctrl.SetLogger(zap.New(zap.UseFlagOptions(&zap.Options{Development: true, Level: zapcore.DebugLevel})))
298298
c := NewGatewayController(fakeClient, kube, ctrl.Log,
299-
"docker.io/envoyproxy/ai-gateway-extproc:latest", false, nil, true)
299+
"docker.io/envoyproxy/ai-gateway-extproc:latest", false, nil)
300300

301301
const gwNamespace = "ns"
302302
now := metav1.Now()
@@ -319,7 +319,7 @@ func TestGatewayController_reconcileFilterConfigSecret_SkipsDeletedRoutes(t *tes
319319
{
320320
Headers: []gwapiv1.HTTPHeaderMatch{
321321
{
322-
Name: internalapi.ModelNameHeaderKeyDefault,
322+
Name: aigv1a1.AIModelHeaderKey,
323323
Value: "mymodel",
324324
},
325325
},
@@ -345,7 +345,7 @@ func TestGatewayController_reconcileFilterConfigSecret_SkipsDeletedRoutes(t *tes
345345
{
346346
Headers: []gwapiv1.HTTPHeaderMatch{
347347
{
348-
Name: internalapi.ModelNameHeaderKeyDefault,
348+
Name: aigv1a1.AIModelHeaderKey,
349349
Value: "deletedmodel",
350350
},
351351
},
@@ -380,7 +380,7 @@ func TestGatewayController_reconcileFilterConfigSecret_SkipsDeletedRoutes(t *tes
380380
configName := FilterConfigSecretPerGatewayName("gw", gwNamespace)
381381

382382
// Reconcile filter config secret.
383-
err := c.reconcileFilterConfigSecret(t.Context(), configName, someNamespace, routes, nil, "foouuid")
383+
err := c.reconcileFilterConfigSecret(t.Context(), configName, someNamespace, routes, "foouuid")
384384
require.NoError(t, err)
385385

386386
// Verify the secret was created and only contains data from the active route.

0 commit comments

Comments
 (0)