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: mon: MonitorDBStore: get_next_key() only if prefix matches #5361
Conversation
@jecluis could this backport be the cause of these failures : http://pulpito.ceph.com/loic-2015-08-15_22:14:11-upgrade:hammer-hammer-backports---basic-multi/ ? (see more at http://tracker.ceph.com/issues/12701#note-10) |
@dachary I don't think so, but feel free to test without it. Besides, the stack trace in the tracker issue feels a lot more related with memory issues rather than the store itself, but I may be wrong. |
@jecluis thanks for taking a look. Running the upgrade suite again to figure out if it's a permanent error or something else. http://pulpito.ceph.com/loic-2015-08-19_12:26:39-upgrade:hammer-hammer-backports---basic-vps/ |
@jecluis running the hammer upgrade suite after removing this commit works. It's easy to reproduce, it fails all upgrade jobs. |
Also cherry-pick #5834 to make it work |
patch to be cherry-picked is 70d3108 |
We introduced a significant bug with 2cc7aee, when we fixed issue #11786. Although that patch would fix the problem described in #11786, we managed to not increment the iterator upon returning the current key. This would have the iterator iterating over the same key, forever and ever. Signed-off-by: Joao Eduardo Luis <joao@suse.de> (cherry picked from commit 70d3108)
Patch has been cherry-picked. Okay to test. |
@jecluis thanks ! |
… prefix matches Reviewed-by: Loic Dachary <ldachary@redhat.com>
… prefix matches Reviewed-by: Loic Dachary <ldachary@redhat.com>
… prefix matches Reviewed-by: Loic Dachary <ldachary@redhat.com>
… prefix matches Reviewed-by: Loic Dachary <ldachary@redhat.com>
… prefix matches Reviewed-by: Loic Dachary <ldachary@redhat.com>
… prefix matches Reviewed-by: Loic Dachary <ldachary@redhat.com>
mon: MonitorDBStore: get_next_key() only if prefix matches Reviewed-by: Loic Dachary <ldachary@redhat.com>
mon: MonitorDBStore: get_next_key() only if prefix matches Reviewed-by: Loic Dachary <ldachary@redhat.com>
mon: MonitorDBStore: get_next_key() only if prefix matches Reviewed-by: Loic Dachary <ldachary@redhat.com>
mon: MonitorDBStore: get_next_key() only if prefix matches Reviewed-by: Loic Dachary <ldachary@redhat.com>
mon: MonitorDBStore: get_next_key() only if prefix matches Reviewed-by: Loic Dachary <ldachary@redhat.com>
http://tracker.ceph.com/issues/12483