-
Notifications
You must be signed in to change notification settings - Fork 949
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 cacheHits query & remove liveness probe of juicefs worker #2212
Conversation
Signed-off-by: zwwhdls <zww@hdls.me>
Codecov Report
@@ Coverage Diff @@
## master #2212 +/- ##
==========================================
- Coverage 51.36% 51.35% -0.02%
==========================================
Files 331 331
Lines 24293 24302 +9
==========================================
+ Hits 12479 12480 +1
- Misses 10421 10425 +4
- Partials 1393 1397 +4
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
pods, err := j.GetRunningPodsOfDaemonset(stsName, j.namespace) | ||
if err != nil || len(pods) == 0 { | ||
return | ||
var containerName string |
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.
Would you add some unit tests?
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.
done
Signed-off-by: zwwhdls <zww@hdls.me>
…cloudnative#2212) * fix cacheHits query & remove liveness probe of juicefs worker Signed-off-by: zwwhdls <zww@hdls.me> * add unittest Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: yn <yanghaihai690@163.com>
…cloudnative#2212) * fix cacheHits query & remove liveness probe of juicefs worker Signed-off-by: zwwhdls <zww@hdls.me> * add unittest Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me>
…cloudnative#2212) * fix cacheHits query & remove liveness probe of juicefs worker Signed-off-by: zwwhdls <zww@hdls.me> * add unittest Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: cheyang <cheyang@163.com>
* fix privileged configuration (#2137) Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: cheyang <cheyang@163.com> * delete juicefs dataload privileged (#2153) Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: cheyang <cheyang@163.com> * add juicefs diagnose script and fix alluxio diagnose script (#2156) * add juicefs diagnose script and fix alluxio diagnose script Signed-off-by: zwwhdls <zww@hdls.me> * rename diagnose script Signed-off-by: zwwhdls <zww@hdls.me> * fix doc Signed-off-by: zwwhdls <zww@hdls.me> * fix doc Signed-off-by: zwwhdls <zww@hdls.me> * fix doc Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: cheyang <cheyang@163.com> * fix runtime replicas is 0 (#2160) Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: cheyang <cheyang@163.com> * fix ro accessMode (#2183) Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: cheyang <cheyang@163.com> * fix juicefs cache capacity (#2186) Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: cheyang <cheyang@163.com> * fix cacheHits query & remove liveness probe of juicefs worker (#2212) * fix cacheHits query & remove liveness probe of juicefs worker Signed-off-by: zwwhdls <zww@hdls.me> * add unittest Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: cheyang <cheyang@163.com> * make worker manage policy parallel (#2237) Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: cheyang <cheyang@163.com> * add juicefs networkmode (#2343) Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: cheyang <cheyang@163.com> * fix juicefs yaml marshal & fix volume mount (#2346) * fix juicefs yaml marshal & fix volume mount Signed-off-by: zwwhdls <zww@hdls.me> * fix yaml marshal in dataload Signed-off-by: zwwhdls <zww@hdls.me> * udpate changelog Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: cheyang <cheyang@163.com> * add updateStrategy for juicefs fuse (#2375) Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: cheyang <cheyang@163.com> Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: cheyang <cheyang@163.com> Co-authored-by: Weiwei <zww@hdls.me>
…cloudnative#2212) * fix cacheHits query & remove liveness probe of juicefs worker Signed-off-by: zwwhdls <zww@hdls.me> * add unittest Signed-off-by: zwwhdls <zww@hdls.me> Signed-off-by: zwwhdls <zww@hdls.me>
Signed-off-by: zwwhdls zww@hdls.me
Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews