Skip to content

Commit e714b6c

Browse files
committed
increase clinic merge API endpoint timeout (part 2)
The changes from 5698df6 weren't effective. There are multiple places where timeouts can be configured. Hopefully this will catch them all. This is an internal-only endpoint, so making people wait up to a minute should be fine. BACK-4216
1 parent 5698df6 commit e714b6c

File tree

4 files changed

+33
-1
lines changed

4 files changed

+33
-1
lines changed

charts/tidepool/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
apiVersion: v2
22
description: A Helm chart for Tidepool
33
name: tidepool
4-
version: 0.24.0
4+
version: 0.24.1
55
maintainers:
66
- name: Todd Kazakov
77
email: todd@tidepool.org

charts/tidepool/charts/clinic/templates/4-routetable.yaml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,17 @@ metadata:
99
namespace: {{ .Release.Namespace }}
1010
spec:
1111
routes:
12+
- matchers:
13+
- methods:
14+
- POST
15+
regex: /v1/clinics/[^/]+/merge
16+
routeAction:
17+
single:
18+
upstream:
19+
name: clinic
20+
namespace: {{ .Release.Namespace }}
21+
options:
22+
timeout: 1m
1223
- matchers:
1324
- methods:
1425
- GET

charts/tidepool/charts/clinic/templates/7-serviceprofile.yaml

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -286,6 +286,21 @@ spec:
286286
min: 400
287287
max: 599
288288
isFailure: true
289+
- condition:
290+
method: POST
291+
pathRegex: /v1/clinics/[^/]*/merge
292+
name: POST /v1/clinics/{clinicId}/merge
293+
timeout: 1m
294+
responseClasses:
295+
- condition:
296+
status:
297+
min: 200
298+
max: 399
299+
- condition:
300+
status:
301+
min: 400
302+
max: 599
303+
isFailure: true
289304
- condition:
290305
method: POST
291306
pathRegex: /v1/clinics/[^/]*/merge

charts/tidepool/charts/glooingress/templates/2-http-internal-virtual-service.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,5 +44,11 @@ spec:
4444
namespace: {{ .Release.Namespace }}
4545
app: tidepool
4646

47+
- matchers:
48+
- regex: '/v1/clinics/[^/]*/merge'
49+
methods:
50+
- POST
51+
options:
52+
timeout: '60s'
4753
{{- end }}
4854
{{- end }}

0 commit comments

Comments
 (0)