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.

Sign up for GitHub

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

Jump to bottom

os/bluestore: fix improper blob usage while handling deferred big write. #34754

Merged
merged 1 commit into from May 1, 2020
Merged

os/bluestore: fix improper blob usage while handling deferred big write. #34754

merged 1 commit into from May 1, 2020

Conversation

ifed01
Copy link
Contributor

@ifed01 ifed01 commented Apr 25, 2020 •
edited

The root cause is tail extent iterator invalidation when rewriting head
one.

Fixes: https://tracker.ceph.com/issues/45195
Signed-off-by: Igor Fedotov ifedotov@suse.com

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug
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 backend
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@ifed01
os/bluestore: fix improper blob usage while handling deferred big write.
f47ccfa 
The root cause is tail extent iterator invalidation when rewriting head
one.

Fixes: https://tracker.ceph.com/issues/45195
Signed-off-by: Igor Fedotov <ifedotov@suse.com>
badone
badone approved these changes Apr 27, 2020
View reviewed changes
Copy link
Contributor

@badone badone left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks Igor!

@badone badone mentioned this pull request Apr 27, 2020
Merged
3 tasks
aclamk
aclamk approved these changes Apr 27, 2020
View reviewed changes
Copy link
Contributor

@aclamk aclamk left a comment

Choose a reason for hiding this comment

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

Looks good.

@tchaikov
Copy link
Contributor

tchaikov commented May 1, 2020

@tchaikov tchaikov merged commit c21200b into ceph:master May 1, 2020
@ifed01 ifed01 deleted the wip-ifed-fix-45195 branch May 1, 2020 09:07
@disaster123
Copy link

Was this backported to nautilus?

@tchaikov
Copy link
Contributor

Was this backported to nautilus?

see https://tracker.ceph.com/issues/45195#note-17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@badone badone badone approved these changes

@aclamk aclamk aclamk approved these changes

@jdurgin jdurgin Awaiting requested review from jdurgin

@neha-ojha neha-ojha Awaiting requested review from neha-ojha

Assignees
No one assigned
Labels
bluestore bug-fix needs-qa wip-kefu-testing
Projects
None yet
Milestone
No milestone
5 participants
@ifed01 @tchaikov @disaster123 @badone @aclamk