Skip to content

Commit 99b5e74

Browse files
authored
ignore target group health check timeout for NLB (#1727)
1 parent cb5ba72 commit 99b5e74

File tree

3 files changed

+0
-14
lines changed

3 files changed

+0
-14
lines changed

pkg/service/model_build_target_group.go

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -94,10 +94,6 @@ func (t *defaultModelBuildTask) buildTargetGroupHealthCheckConfig(ctx context.Co
9494
if err != nil {
9595
return nil, err
9696
}
97-
timeoutSeconds, err := t.buildTargetGroupHealthCheckTimeoutSeconds(ctx)
98-
if err != nil {
99-
return nil, err
100-
}
10197
healthyThresholdCount, err := t.buildTargetGroupHealthCheckHealthyThresholdCount(ctx)
10298
if err != nil {
10399
return nil, err
@@ -111,7 +107,6 @@ func (t *defaultModelBuildTask) buildTargetGroupHealthCheckConfig(ctx context.Co
111107
Protocol: &healthCheckProtocol,
112108
Path: healthCheckPathPtr,
113109
IntervalSeconds: &intervalSeconds,
114-
TimeoutSeconds: &timeoutSeconds,
115110
HealthyThresholdCount: &healthyThresholdCount,
116111
UnhealthyThresholdCount: &unhealthyThresholdCount,
117112
}, nil

pkg/service/model_build_target_group_test.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,6 @@ func Test_defaultModelBuilderTask_buildTargetHealthCheck(t *testing.T) {
179179
Port: &trafficPort,
180180
Protocol: (*elbv2.Protocol)(aws.String(string(elbv2.ProtocolTCP))),
181181
IntervalSeconds: aws.Int64(10),
182-
TimeoutSeconds: aws.Int64(10),
183182
HealthyThresholdCount: aws.Int64(3),
184183
UnhealthyThresholdCount: aws.Int64(3),
185184
},
@@ -205,7 +204,6 @@ func Test_defaultModelBuilderTask_buildTargetHealthCheck(t *testing.T) {
205204
Protocol: (*elbv2.Protocol)(aws.String("HTTP")),
206205
Path: aws.String("/healthz"),
207206
IntervalSeconds: aws.Int64(10),
208-
TimeoutSeconds: aws.Int64(30),
209207
HealthyThresholdCount: aws.Int64(2),
210208
UnhealthyThresholdCount: aws.Int64(2),
211209
},
@@ -225,7 +223,6 @@ func Test_defaultModelBuilderTask_buildTargetHealthCheck(t *testing.T) {
225223
Protocol: (*elbv2.Protocol)(aws.String("HTTP")),
226224
Path: aws.String("/"),
227225
IntervalSeconds: aws.Int64(10),
228-
TimeoutSeconds: aws.Int64(10),
229226
HealthyThresholdCount: aws.Int64(3),
230227
UnhealthyThresholdCount: aws.Int64(3),
231228
},

pkg/service/model_builder_test.go

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,6 @@ func Test_defaultModelBuilderTask_Build(t *testing.T) {
165165
"port":"traffic-port",
166166
"protocol":"TCP",
167167
"intervalSeconds":10,
168-
"timeoutSeconds":10,
169168
"healthyThresholdCount":3,
170169
"unhealthyThresholdCount":3
171170
},
@@ -324,7 +323,6 @@ func Test_defaultModelBuilderTask_Build(t *testing.T) {
324323
"port":"traffic-port",
325324
"protocol":"TCP",
326325
"intervalSeconds":10,
327-
"timeoutSeconds":10,
328326
"healthyThresholdCount":3,
329327
"unhealthyThresholdCount":3
330328
},
@@ -523,7 +521,6 @@ func Test_defaultModelBuilderTask_Build(t *testing.T) {
523521
"protocol":"HTTP",
524522
"path":"/healthz",
525523
"intervalSeconds":10,
526-
"timeoutSeconds":30,
527524
"healthyThresholdCount":2,
528525
"unhealthyThresholdCount":2
529526
},
@@ -546,7 +543,6 @@ func Test_defaultModelBuilderTask_Build(t *testing.T) {
546543
"protocol":"HTTP",
547544
"path":"/healthz",
548545
"intervalSeconds":10,
549-
"timeoutSeconds":30,
550546
"healthyThresholdCount":2,
551547
"unhealthyThresholdCount":2
552548
},
@@ -852,7 +848,6 @@ func Test_defaultModelBuilderTask_Build(t *testing.T) {
852848
"protocol":"HTTP",
853849
"path":"/healthz",
854850
"intervalSeconds":10,
855-
"timeoutSeconds":30,
856851
"healthyThresholdCount":2,
857852
"unhealthyThresholdCount":2
858853
},
@@ -875,7 +870,6 @@ func Test_defaultModelBuilderTask_Build(t *testing.T) {
875870
"protocol":"HTTP",
876871
"path":"/healthz",
877872
"intervalSeconds":10,
878-
"timeoutSeconds":30,
879873
"healthyThresholdCount":2,
880874
"unhealthyThresholdCount":2
881875
},

0 commit comments

Comments
 (0)