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

hammer: OSD coredumps with leveldb compact on mount = true #7645

Merged
merged 1 commit into from Feb 24, 2016
Merged

hammer: OSD coredumps with leveldb compact on mount = true #7645

merged 1 commit into from Feb 24, 2016

Conversation

ghost
Copy link

@ghost ghost commented Feb 15, 2016

Initialize the performance counter(l_rocksdb_compact)
before use it in compact().

Signed-off-by: Xiaoxi Chen <xiaoxi.chen@intel.com>
(cherry picked from commit 932d59d)
@ghost ghost self-assigned this Feb 15, 2016
@ghost ghost added this to the hammer milestone Feb 15, 2016
@ghost ghost added bug-fix core labels Feb 15, 2016
ghost pushed a commit that referenced this pull request Feb 15, 2016
…on mount = true

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Feb 16, 2016
…on mount = true

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Feb 22, 2016
…on mount = true

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link
Author

ghost commented Feb 24, 2016

@liewegas does this backport look good to merge ? It passed a run of the hammer rados suite ( see http://tracker.ceph.com/issues/14692#note-9 for details ). The upgrade/hammer suite also passes ( http://tracker.ceph.com/issues/14692#note-11 ) after a few tries to get rid of environmental noise.

@ghost ghost assigned liewegas and unassigned ghost Feb 24, 2016
liewegas added a commit that referenced this pull request Feb 24, 2016
hammer: OSD coredumps with leveldb compact on mount = true
@liewegas liewegas merged commit 293e8c7 into ceph:hammer Feb 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants