Skip to content

Conversation

@amuraru
Copy link

@amuraru amuraru commented Jul 21, 2020

Force refresh of /data/conf/zoo.cfg content on zk pod restart

Fixes pravega#222

@amuraru
Copy link
Author

amuraru commented Jul 21, 2020

@alungu @azun @dobrerazvan @alexander-falca please review
I started a PR internally here to expedite the fix and plan to do a similar PR upstream

Copy link
Member

@alexander-falca alexander-falca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@codecov
Copy link

codecov bot commented Jul 21, 2020

Codecov Report

Merging #3 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master       #3   +/-   ##
=======================================
  Coverage   82.82%   82.82%           
=======================================
  Files          11       11           
  Lines        1223     1223           
=======================================
  Hits         1013     1013           
  Misses        142      142           
  Partials       68       68           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c10c018...b4a8a1c. Read the comment docs.

@amuraru amuraru force-pushed the issue-222 branch 2 times, most recently from 1088117 to df94953 Compare July 22, 2020 09:28
…ng restarts

Force refresh of /data/conf/zoo.cfg content
@alungu alungu merged commit 21d288e into master Jul 22, 2020
@alungu alungu deleted the issue-222 branch July 22, 2020 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

zoo.cfg updated parameters are not picked up during rolling restarts

6 participants