Skip to content

Commit 705a512

Browse files
committed
increase clinic merge API endpoint timeout (part 3)
The changes from e714b6c were effective, but not long enough. This is an internal-only endpoint, so making people wait a few minutes should be fine. Additionally, it's possible that other services (the ELB for example) might have shorter timeouts, so if things time out before hitting this limit, that's a hint that it's something else timing out. BACK-4216 SIR-807
1 parent e714b6c commit 705a512

4 files changed

Lines changed: 3 additions & 15 deletions

File tree

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.1
4+
version: 0.24.2
55
maintainers:
66
- name: Todd Kazakov
77
email: todd@tidepool.org

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ spec:
1919
name: clinic
2020
namespace: {{ .Release.Namespace }}
2121
options:
22-
timeout: 1m
22+
timeout: 15m
2323
- matchers:
2424
- methods:
2525
- GET

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

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -290,7 +290,7 @@ spec:
290290
method: POST
291291
pathRegex: /v1/clinics/[^/]*/merge
292292
name: POST /v1/clinics/{clinicId}/merge
293-
timeout: 1m
293+
timeout: 15m
294294
responseClasses:
295295
- condition:
296296
status:
@@ -301,11 +301,6 @@ spec:
301301
min: 400
302302
max: 599
303303
isFailure: true
304-
- condition:
305-
method: POST
306-
pathRegex: /v1/clinics/[^/]*/merge
307-
name: POST /v1/clinics/{clinicId}/merge
308-
timeout: 1m
309304
- condition:
310305
method: POST
311306
pathRegex: /v1/clinics/[^/]*/migrate

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

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -43,12 +43,5 @@ spec:
4343
labels:
4444
namespace: {{ .Release.Namespace }}
4545
app: tidepool
46-
47-
- matchers:
48-
- regex: '/v1/clinics/[^/]*/merge'
49-
methods:
50-
- POST
51-
options:
52-
timeout: '60s'
5346
{{- end }}
5447
{{- end }}

0 commit comments

Comments
 (0)