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

CLI: check peer count before actually rebalancing meta1 #1972

Merged

Conversation

fvennetier
Copy link
Member

SUMMARY

During a rebalance operation, check each reference prefix is managed by the right number of meta1 services.

Also implement a new 'openio directory check' command, checking each reference prefix is managed by the right number of meta1 services. This complements 'openio-admin directory check' (which does not check this number).

Jira: OS-470

ISSUE TYPE
  • Feature Pull Request
  • Bugfix Pull Request
COMPONENT NAME
  • CLI
SDS VERSION
openio 5.3.0.dev3
ADDITIONAL INFORMATION

read_timeout=parsed_args.meta0_timeout)
self.log.info("Moved %s", moved)
else:
self.log.warn("Nothing done due to errors")
Copy link
Member

Choose a reason for hiding this comment

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

We should also add self.success = False.

Also implement a new 'openio directory check' command, checking each
reference prefix is managed by the right number of meta1 services. This
complements 'openio-admin directory check' (which does not check this
number).

Jira: OS-470
@fvennetier fvennetier force-pushed the check-peer-count-before-rebalance branch from 5781188 to c1e7b0a Compare January 16, 2020 13:32
@codecov-io
Copy link

codecov-io commented Jan 16, 2020

Codecov Report

Merging #1972 into 5.x will decrease coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             5.x    #1972      +/-   ##
=========================================
- Coverage   78.7%   78.65%   -0.04%     
=========================================
  Files        377      377              
  Lines      72279    72279              
  Branches    7025     7022       -3     
=========================================
- Hits       56877    56841      -36     
- Misses     15294    15327      +33     
- Partials     108      111       +3
Impacted Files Coverage Δ
tests/functional/cli/admin/test_service_decache.py 66.67% <0%> (-16.66%) ⬇️
oio/directory/admin.py 47.52% <0%> (-7.09%) ⬇️
tests/functional/event/filters/test_filters.py 94.52% <0%> (-3.79%) ⬇️
oio/common/wsgi.py 43.21% <0%> (-3.7%) ⬇️
tests/functional/audit/test_audit_storage.py 80.49% <0%> (-3.25%) ⬇️
metautils/lib/rrd.c 74.36% <0%> (-2.56%) ⬇️
tests/functional/api/test_rdir.py 96.05% <0%> (-2.51%) ⬇️
rdir/routes.c.rl 93.19% <0%> (-2.27%) ⬇️
metautils/lib/utils_hashstr.c 81.49% <0%> (-1.85%) ⬇️
meta2v2/meta2_filters_check.c 77.62% <0%> (-1.49%) ⬇️
... and 26 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 493b14e...c1e7b0a. Read the comment docs.

@fvennetier fvennetier merged commit b7493c3 into open-io:5.x Jan 17, 2020
@fvennetier fvennetier deleted the check-peer-count-before-rebalance branch January 17, 2020 08:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants