-
Notifications
You must be signed in to change notification settings - Fork 41
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
Release leader for life lock in case pod is preempted. #157
Conversation
Signed-off-by: Jagpreet Singh Tamber <jagpreetstamber@gmail.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #157 +/- ##
==========================================
+ Coverage 81.03% 87.44% +6.41%
==========================================
Files 17 17
Lines 485 494 +9
==========================================
+ Hits 393 432 +39
+ Misses 82 49 -33
- Partials 10 13 +3 ☔ View full report in Codecov by Sentry. |
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.
Thanks @jagpreetstamber for the PR! I had one nit that I left as a comment, but it's more of a suggestion - feel free to make the change or leave as is.
Signed-off-by: Jagpreet Singh Tamber <jagpreetstamber@gmail.com>
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.
/lgtm
Signed-off-by: Jagpreet Singh Tamber <jagpreetstamber@gmail.com>
New changes are detected. LGTM label has been removed. |
Description of the change:
Fix for issue #156
Motivation for the change:
Fix for issue #156