Skip to content

Commit 991df24

Browse files
committed
Correct the conformance tests.
1 parent e4fe22d commit 991df24

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

conformance/tests/inferencepool_resolvedrefs_condition.go

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,10 @@ import (
3636
)
3737

3838
func init() {
39-
ConformanceTests = append(ConformanceTests, InferencePoolParentStatus)
39+
ConformanceTests = append(ConformanceTests, InferencePoolResolvedRefsCondition)
4040
}
4141

42-
var InferencePoolParentStatus = suite.ConformanceTest{
42+
var InferencePoolResolvedRefsCondition = suite.ConformanceTest{
4343
ShortName: "InferencePoolResolvedRefsCondition",
4444
Description: "Verify that an InferencePool correctly updates its parent-specific status (e.g., Accepted condition) when referenced by HTTPRoutes attached to shared Gateways, and clears parent statuses when no longer referenced.",
4545
Manifests: []string{"tests/inferencepool_resolvedrefs_condition.yaml"},
@@ -98,16 +98,16 @@ var InferencePoolParentStatus = suite.ConformanceTest{
9898
t,
9999
s.RoundTripper,
100100
s.TimeoutConfig,
101-
gwPrimaryAddr,
101+
gwSecondaryAddr,
102102
gwhttp.ExpectedResponse{
103103
Request: gwhttp.Request{
104104
Host: hostnameSecondaryGw,
105105
Path: pathSecondaryGw,
106106
},
107107
Response: gwhttp.Response{
108-
StatusCodes: []int{http.StatusNotFound},
108+
StatusCodes: []int{http.StatusOK},
109109
},
110-
Backend: resources.PrimaryModelServerDeploymentName,
110+
Backend: resources.PrimaryModelServerDeploymentName, // Primary because in this test, both primary and secondary httpRoute is backnedRef the primary InferecePool.
111111
Namespace: resources.AppBackendNamespace,
112112
},
113113
)
@@ -139,7 +139,7 @@ var InferencePoolParentStatus = suite.ConformanceTest{
139139
Response: gwhttp.Response{
140140
StatusCodes: []int{http.StatusOK},
141141
},
142-
Backend: resources.PrimaryModelServerDeploymentName,
142+
Backend: resources.PrimaryModelServerDeploymentName, // Primary because in this test, both primary and secondary httpRoute is backnedRef the primary InferecePool.
143143
Namespace: resources.AppBackendNamespace,
144144
},
145145
)
@@ -178,7 +178,7 @@ var InferencePoolParentStatus = suite.ConformanceTest{
178178
t,
179179
s.RoundTripper,
180180
s.TimeoutConfig,
181-
gwPrimaryAddr,
181+
gwSecondaryAddr,
182182
gwhttp.ExpectedResponse{
183183
Request: gwhttp.Request{
184184
Host: hostnameSecondaryGw,

0 commit comments

Comments
 (0)