WIP: doc/dev: add explanation of how to use deduplication
Signed-off-by: Myoungwon Oh [email protected]
Contribution Guidelines
-
To sign and title your commits, please refer to Submitting Patches to Ceph.
-
If you are submitting a fix for a stable branch (e.g. "pacific"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.
Checklist
- Tracker (select at least one)
- [ ] References tracker ticket
- [ ] Very recent bug; references commit where it was introduced
- [ ] New feature (ticket optional)
- [ ] Doc update (no ticket needed)
- [ ] Code cleanup (no ticket needed)
- Component impact
- [ ] Affects Dashboard, opened tracker ticket
- [ ] Affects Orchestrator, opened tracker ticket
- [ ] No impact that needs to be tracked
- Documentation (select at least one)
- [ ] Updates relevant documentation
- [ ] No doc update is appropriate
- Tests (select at least one)
- [ ] Includes unit test(s)
- [ ] Includes integration test(s)
- [ ] Includes bug reproducer
- [ ] No tests
Show available Jenkins commands
-
jenkins retest this please -
jenkins test classic perf -
jenkins test crimson perf -
jenkins test signed -
jenkins test make check -
jenkins test make check arm64 -
jenkins test submodules -
jenkins test dashboard -
jenkins test dashboard cephadm -
jenkins test api -
jenkins test docs -
jenkins render docs -
jenkins test ceph-volume all -
jenkins test ceph-volume tox -
jenkins test windows
Please amend the PR and commit descriptions with the preferred format beginning with the full relative directory, in this case something like:
doc/dev: Add explanation of how to use deduplication
Add a warning at the top that this feature is highly experimental and is subject to change or removal.
warning
Very good point.
@athanatos Can you take a look?
@myoungwon: The "pull request checklist" wasn't filled out, which was preventing this PR from being mergeable.
I've filled out the checklist, and will now merge and backport this PR.
https://github.com/ceph/ceph/pull/48567 - Quincy backport https://github.com/ceph/ceph/pull/48568 - Pacific backport
@zdover23 This probably shouldn't be backported.
@zdover23 This probably shouldn't be backported.
@athanatos: I'll create a couple of new PRs that take this feature out, since you're the architect of this thing and the authority on it. I acted here (wrongly, as it turns out) after consulting an email to [email protected] on 16 Apr 2020 with the subject line "CDS Pacific highlights - RADOS", which suggested to me (again, wrongly, which is totally on me) that this feature would be new in Pacific.
In short: sorry about that.
@athanatos: Is deduplication expected to land in Reef?
https://github.com/ceph/ceph/pull/48570 - Quincy backport reversion
https://github.com/ceph/ceph/pull/48571 - Pacific backport reversion
@zdover23 No worries, thanks!