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

Implement 'openio-admin container repair' command #1743

Merged
merged 4 commits into from
Apr 19, 2019

Conversation

AymericDu
Copy link
Member

SUMMARY
  • Allow to rebuild meta2 databases when the quorum isn't reached
  • Implement 'openio-admin container repair' command
ISSUE TYPE
  • Feature Pull Request
  • Bugfix Pull Request
COMPONENT NAME
  • Admin CLI
  • Python API
SDS VERSION
openio 5.0.0.0a1.dev51
ADDITIONAL INFORMATION
$ openio-admin container repair toto titi
START last_report=2019-04-18T17:49:44 0.00s containers=0 0.00/s errors=0 0.00% start_time=2019-04-18T17:49:44 0.00s total_containers=0 0.00/s total_errors=0 0.00% progress=0/2 0.00%
DONE last_report=2019-04-18T17:49:44 0.19s containers=2 10.28/s errors=0 0.00% start_time=2019-04-18T17:49:44 0.19s total_containers=2 10.28/s total_errors=0 0.00% progress=2/2 100.00%
+-----------------------+--------+
| Object                | Status |
+-----------------------+--------+
| OPENIO|myaccount|toto | OK     |
| OPENIO|myaccount|titi | OK     |
+-----------------------+--------+

@AymericDu AymericDu force-pushed the admin-cli-container-repair branch 2 times, most recently from 73f6047 to b9bab29 Compare April 19, 2019 08:24
from oio.cli.admin.common import ContainerCommandMixin, ObjectCommandMixin


class ItemRepaiCommand(lister.Lister):
Copy link
Contributor

Choose a reason for hiding this comment

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

ItemRepairCommand

@AymericDu AymericDu force-pushed the admin-cli-container-repair branch from b9bab29 to 6cb3035 Compare April 19, 2019 09:56
@codecov-io
Copy link

codecov-io commented Apr 19, 2019

Codecov Report

Merging #1743 into master will decrease coverage by 0.04%.
The diff coverage is 46.43%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1743      +/-   ##
==========================================
- Coverage   79.52%   79.49%   -0.03%     
==========================================
  Files         375      375              
  Lines       69444    69471      +27     
  Branches     6920     6920              
==========================================
- Hits        55221    55219       -2     
- Misses      14118    14147      +29     
  Partials      105      105
Impacted Files Coverage Δ
oio/directory/meta2.py 69.54% <46.43%> (-8.63%) ⬇️
oio/ecd/app.py 60.12% <0%> (-5.95%) ⬇️
oio/account/client.py 81.61% <0%> (-5.74%) ⬇️
oio/crawler/storage_tierer.py 57.8% <0%> (-2.75%) ⬇️
oio/conscience/client.py 80.73% <0%> (-2.4%) ⬇️
sqliterepo/replication.c 84.29% <0%> (-1.14%) ⬇️
meta1v2/meta1_prefixes.c 75.78% <0%> (-1.03%) ⬇️
meta2v2/meta2_bean_utils.c 59% <0%> (-1%) ⬇️
meta1v2/meta1_gridd_dispatcher.c 68.35% <0%> (-0.91%) ⬇️
resolver/hc_resolver.c 92.55% <0%> (-0.31%) ⬇️
... and 18 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 3b03c96...2eb604a. Read the comment docs.

@AymericDu AymericDu force-pushed the admin-cli-container-repair branch from 6cb3035 to 2eb604a Compare April 19, 2019 14:30
@AymericDu AymericDu merged commit 8965a16 into open-io:master Apr 19, 2019
@AymericDu AymericDu deleted the admin-cli-container-repair branch April 19, 2019 16:16
# 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.

4 participants