-
Notifications
You must be signed in to change notification settings - Fork 118
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
[YUNIKORN-2630] Release context lock early for config changes #842
Conversation
Release the lock of the context in the shim when processing is done. When the config changes are sent to the core the k8shim should not be locked. The context changes have been finalised at that point. The core handles its own locking and serialises config changes that come in from the k8shim.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #842 +/- ##
==========================================
- Coverage 67.33% 67.26% -0.07%
==========================================
Files 70 70
Lines 7598 7604 +6
==========================================
- Hits 5116 5115 -1
- Misses 2271 2276 +5
- Partials 211 213 +2 ☔ 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.
+1
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.
@wilfred-s nice patch.
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.
+1 LGTM
Release the lock of the context in the shim when processing is done. When the config changes are sent to the core the k8shim should not be locked. The context changes have been finalised at that point. The core handles its own locking and serialises config changes that come in from the k8shim. review: remove call through api to get config. Closes: #842 Signed-off-by: Chia-Ping Tsai <chia7712@gmail.com>
What is this PR for?
Release the lock of the context in the shim when processing is done. When the config changes are sent to the core the k8shim should not be locked. The context changes have been finalised at that point.
The core handles its own locking and serialises config changes that come in from the k8shim.
What type of PR is it?
What is the Jira issue?
How should this be tested?
Current unit and e2e tests cover the changed paths