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-521] Placeholder pods are not cleaned when the job is deleted #232
Conversation
…is deleted Filled ownreference field for the placeholder pods
Codecov Report
@@ Coverage Diff @@
## master #232 +/- ##
==========================================
- Coverage 59.75% 59.43% -0.33%
==========================================
Files 35 35
Lines 3133 3229 +96
==========================================
+ Hits 1872 1919 +47
- Misses 1180 1228 +48
- Partials 81 82 +1
Continue to review full report at Codecov.
|
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.
Can we add some UT in general_test.go
? that needs to cover the case where the pod has ownerReference and hasn't.
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.
Couple of small points more about style than code content.
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
…ed (#232) Add the owner reference to the placeholder pods in order to do proper cleanup. The owner reference is the same value as the first pod of the app's owner reference, if the first pod doesn't have an owner reference set, it is set to the pod's object.
…ed (apache#232) Add the owner reference to the placeholder pods in order to do proper cleanup. The owner reference is the same value as the first pod of the app's owner reference, if the first pod doesn't have an owner reference set, it is set to the pod's object.
Filled ownreference field for the placeholder pods