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
jewel: tools: Add export-remove ceph-objectstore-tool command option #21193
Conversation
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.
lgtm
Signed-off-by: David Zafman <dzafman@redhat.com> (cherry picked from commit 49ca1ff) Conflicts: qa/standalone/special/ceph_objectstore_tool.py - file does not exist in jewel; backported the changes manually to src/test/ceph_objectstore_tool.py
Add new export-remove to combine the 2 operations Fixes: http://tracker.ceph.com/issues/21272 Signed-off-by: David Zafman <dzafman@redhat.com> (cherry picked from commit 3bb20f6) Conflicts: qa/standalone/special/ceph_objectstore_tool.py - file does not exist in jewel; backported changes manually to src/test/ceph_objectstore_tool.py qa/tasks/divergent_priors2.py - jewel uses pgid 1.0 qa/tasks/reg11184.py - jewel uses pgid 1.0 and lacks some of the code that is being changed src/tools/ceph_objectstore_tool.cc - jewel has slightly different list of subcommands
Signed-off-by: David Zafman <dzafman@redhat.com> (cherry picked from commit 5083287)
Fixes: http://tracker.ceph.com/issues/22086 Signed-off-by: David Zafman <dzafman@redhat.com> (cherry picked from commit a8b8d54) Conflicts: qa/standalone/special/ceph_objectstore_tool.py src/tools/ceph_objectstore_tool.cc src/tools/ceph_objectstore_tool.h
8eeac27
to
e1c0a2b
Compare
@dzafman Please take another look - I cherry-picked the |
|
src/tools/ceph_objectstore_tool.cc
Outdated
formatter->flush(cout); | ||
|
||
formatter->open_object_section("pg_missing_t"); | ||
ms.missing.dump(formatter); |
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.
@dzafman The compilation failure happens here - metadata_section
does not have a member missing
. If you immediately see how to fix, please advise?
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.
44cb6d3 introduced code to persist the missing set. That commit is too complicated to backport to jewel, so remove code that refers to the "missing" component of "metadata_section". Signed-off-by: Nathan Cutler <ncutler@suse.com>
merge conflict with #20882 and still doesn't pass make check |
|
||
# Remove the same pg that was exported | ||
cmd = ((prefix + "--op remove --pgid 1.0"). | ||
cmd = ((prefix + "--op export-remove --pgid 2.0 --file {file}"). |
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.
The pgid is 1.0 in Jewel
Jewel is EOL |
http://tracker.ceph.com/issues/21373
http://tracker.ceph.com/issues/22390