-
Notifications
You must be signed in to change notification settings - Fork 308
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: gw transient errors crash #3397
Conversation
b051e38
to
91a4872
Compare
9b03a06
to
7584aa8
Compare
4bc381b
to
4371b87
Compare
8c1a026
to
e49ab2d
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #3397 +/- ##
==========================================
- Coverage 68.47% 68.31% -0.16%
==========================================
Files 330 330
Lines 53042 53114 +72
==========================================
- Hits 36319 36284 -35
- Misses 14358 14433 +75
- Partials 2365 2397 +32
☔ View full report in Codecov by Sentry. |
3d9c4e3
to
0f12e3f
Compare
0f12e3f
to
7e757b5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
waiting to review changes after rebasing lint fixes introduced in master
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
waiting to review changes after rebasing lint fixes introduced in master
Co-authored-by: Leonidas Vrachnis <leo.al.vra@gmail.com>
Co-authored-by: Aris Tzoumas <atzoumas@rudderstack.com>
Description
PR modifies maintenance loop in jobsdb to make handling of transient errors configurable.
Notion Ticket
https://www.notion.so/rudderstacks/Gateway-shall-not-panic-on-transient-DNS-errors-ff53bcb47f6045b1a409aecea927841c?pvs=4
Security