Use a low statement timeout in load balancer
What does this MR do and why?
Lowers statement timeout in the load balancer to 100ms to help with Improve application database load balancing log... (#472161)
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
Merge request reports
Activity
assigned to @stomlinson
mentioned in issue #472161
added pipelinetier-1 label
- A deleted user
added backend database databasereview pending labels
1 Warning The master pipeline status page reported failures in - rspec-ee migration pg14 2/2
- rspec unit pg14 14/44
- rspec migration pg14 26/26
- rspec migration pg14 23/26
- rspec migration pg14 20/26
- rspec migration pg14 17/26
- rspec migration pg14 16/26
- rspec migration pg14 13/26
- rspec migration pg14 12/26
- rspec migration pg14 5/26
- rspec migration pg14 3/26
- rspec migration pg14 2/26
- rspec-ee unit pg14 single-db-ci-connection 26/39
- rspec-ee unit pg14 single-db 32/39
- rspec integration pg14 single-db 1/20
- rspec unit pg14 single-db-ci-connection 35/44
- rspec unit pg14 single-db-ci-connection 11/44
- rspec unit pg14 single-db-ci-connection 1/44
- rspec unit pg14 single-db 14/44
- rspec unit pg14 single-db 6/44
- rspec unit pg14 single-redis 36/44
If these jobs fail in your merge request with the same errors, then they are not caused by your changes.
Please check for any on-going incidents in the incident issue tracker or in the#master-broken
Slack channel.1 Message CHANGELOG missing: If this merge request needs a changelog entry, add the
Changelog
trailer to the commit message you want to add to the changelog.If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.
Reviewer roulette
Category Reviewer Maintainer backend @shubhamkrai
(UTC+2, 7 hours ahead of author)
@eugielimpin
(UTC+8, 13 hours ahead of author)
database @bmarjanovic
(UTC+2, 7 hours ahead of author)
@mbobin
(UTC+3, 8 hours ahead of author)
Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.
If needed, you can retry the
danger-review
job that generated this comment.Generated by
DangerEdited by Ghost User- Resolved by Simon Tomlinson
@stomlinson
- please see the following guidance and update this merge request.1 Error Please add typebug typefeature, or typemaintenance label to this merge request.
added groupdatabase label
added Category:Database devopsdata stores sectioncore platform labels
- Resolved by Simon Tomlinson
- Resolved by Simon Tomlinson
added pipeline:run-all-rspec label
- A deleted user
added feature flag label
- Resolved by Simon Tomlinson
added 2554 commits
-
86909617...d85d4755 - 2553 commits from branch
master
- 0bdb394b - Low statement timeout for lb health checks
-
86909617...d85d4755 - 2553 commits from branch
- Resolved by Simon Tomlinson
@mattkasa can you take a first look here? I think this is feature complete but I'm curious if you can think of any more tests or anything else we can do to make this safer.