Skip to content
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

luminous: tools: Add clear-data-digest command to objectstore tool. #29366

Merged
merged 1 commit into from
Oct 1, 2019

Conversation

dzafman
Copy link
Contributor

@dzafman dzafman commented Jul 26, 2019

https://tracker.ceph.com/issues/37748


There may be a situation where data digest in object info is
inconsistent with that computed from object data, then deep-scrub
will fail even though all three repicas have the same object data.

Fixes: https://tracker.ceph.com/issues/37935

Signed-off-by: Li Yichao liyichao.good@gmail.com
(cherry picked from commit da5832b)

Adjustments for luminous:
Add ObjectStore::Sequencer
Use begin() instead of cbegin()
Eliminate use of open_collection()

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

There may be a situation where data digest in object info is
inconsistent with that computed from object data, then deep-scrub
will fail even though all three repicas have the same object data.

Fixes: https://tracker.ceph.com/issues/37935

Signed-off-by: Li Yichao <liyichao.good@gmail.com>
(cherry picked from commit da5832b)

Adjustments for luminous:
	Add ObjectStore::Sequencer
	Use begin() instead of cbegin()
	Eliminate use of open_collection()
Copy link
Contributor

@smithfarm smithfarm left a comment

Choose a reason for hiding this comment

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

Thanks for including descriptions of the manual adjustments you made, but the commit message is missing this text:

Conflicts:
    qa/standalone/special/ceph_objectstore_tool.py
    src/tools/ceph_objectstore_tool.cc

(of course, you would only include the files that actually had cherry-pick conflicts)

@dzafman
Copy link
Contributor Author

dzafman commented Aug 19, 2019

@smithfarm There weren't any cherry-pick conflicts, so I having "Conflicts:" doesn't apply.

@yuriw
Copy link
Contributor

yuriw commented Sep 30, 2019

@yuriw yuriw merged commit ebada6e into ceph:luminous Oct 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants