Skip to content

Commit

Permalink
Merge pull request #275 from Akrog/default-route-timeouts
Browse files Browse the repository at this point in the history
Set HAProxy timeout to 60 seconds for storage

This patch sets the HAProxy timeout to 60 seconds for storage services (Cinder, Glance, and Manila).
This way it's in sync with the Apache default timeouts and matches what we have in OSP17.
This will be fixed in the operators themselves to have these same defaults, and this patch is compatible with that change.
When the operators code is merged and is run by the CI jobs the defaults in the operators will be ignored in favor of the defaults defined in this patch, so we'll want to revert this patch then.
These are the PRs that once their code is run on the CI we can revert this PR:

openstack-k8s-operators/cinder-operator#396
openstack-k8s-operators/glance-operator#550
openstack-k8s-operators/manila-operator#282
openstack-k8s-operators/openstack-operator#830

Jira: https://issues.redhat.com/browse/OSPRH-7393
Jira: https://issues.redhat.com/browse/OSPRH-7415

Reviewed-by: Andrew Bays <[email protected]>
  • Loading branch information
2 parents fbbf959 + feb0c92 commit 39f6b87
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions lib/control-plane/openstackcontrolplane.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ spec:
serviceUser: ceilometer
cinder:
apiOverride:
route: {}
route: {"haproxy.router.openshift.io/timeout": "60s"}
template:
customServiceConfig: |
# Debug logs by default, jobs can override as needed.
Expand Down Expand Up @@ -63,7 +63,7 @@ spec:
glance:
apiOverrides:
default:
route: {}
route: {"haproxy.router.openshift.io/timeout": "60s"}
template:
databaseInstance: openstack
glanceAPIs:
Expand Down Expand Up @@ -115,7 +115,7 @@ spec:
secret: osp-secret
manila:
apiOverride:
route: {}
route: {"haproxy.router.openshift.io/timeout": "60s"}
enabled: false
template:
manilaAPI:
Expand Down

0 comments on commit 39f6b87

Please sign in to comment.