Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Do not read conscience persistence file if a hub is running #1778

Merged
merged 2 commits into from
May 23, 2019

Conversation

fvennetier
Copy link
Member

SUMMARY

If a hub is running, reinjecting the old conscience state can have nasty effects, like respawning service entries that have been explicitly flushed when the local conscience service was down.

Also remove an sqliterepo parameter that is no more used.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME
  • Consicence
  • sqliterepo
SDS VERSION
openio 4.4.3.dev14

@fvennetier fvennetier changed the title Do not read consicience persistence file if a hub is running Do not read conscience persistence file if a hub is running May 16, 2019
@murlock murlock self-requested a review May 16, 2019 16:58
@fvennetier
Copy link
Member Author

This needs a little more work to either:

  • adapt the failing test to retry a few seconds later;
  • make the conscience refuse to serve requests until it has fetched a list of services from the other conscience instances.

If a hub is running, reinjecting the old state can have nasty effects
like respawning service entries that have been explicitly flushed when
one of the conscience services was down.
@codecov-io
Copy link

Codecov Report

Merging #1778 into 4.4.x will increase coverage by 0.04%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           4.4.x    #1778      +/-   ##
=========================================
+ Coverage   79.9%   79.94%   +0.04%     
=========================================
  Files        371      371              
  Lines      68836    68842       +6     
  Branches    6961     6963       +2     
=========================================
+ Hits       54996    55026      +30     
+ Misses     13736    13712      -24     
  Partials     104      104
Impacted Files Coverage Δ
sqliterepo/election.c 89.97% <ø> (ø) ⬇️
cluster/module/server.c 79.98% <100%> (+0.25%) ⬆️
oio/blob/indexer.py 67.43% <0%> (-1.51%) ⬇️
proxy/metacd_http.c 89.93% <0%> (-1.1%) ⬇️
meta1v2/meta1_gridd_dispatcher.c 68.35% <0%> (-0.91%) ⬇️
sqliterepo/gridd_client_pool.c 81.58% <0%> (-0.87%) ⬇️
metautils/lib/gridd_client.c 75.5% <0%> (-0.49%) ⬇️
proxy/cs_actions.c 89.38% <0%> (-0.36%) ⬇️
server/transport_gridd.c 77.92% <0%> (-0.2%) ⬇️
sqliterepo/replication_dispatcher.c 64.2% <0%> (-0.15%) ⬇️
... and 28 more

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 3efd4c2...5ade0c2. Read the comment docs.

@fvennetier fvennetier merged commit 5b849fa into open-io:4.4.x May 23, 2019
@fvennetier fvennetier deleted the review-options branch May 23, 2019 12:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants