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

mds: fix assertion caused by system clock backwards #3970

Merged
1 commit merged into from Mar 20, 2015

Conversation

Projects
None yet
4 participants
@ukernel
Member

ukernel commented Mar 12, 2015

Fixes: #11053
Signed-off-by: Yan, Zheng zyan@redhat.com

mds: fix assertion caused by system clock backwards
Fixes: #11053
Signed-off-by: Yan, Zheng <zyan@redhat.com>
@loic-bot

This comment has been minimized.

loic-bot commented Mar 12, 2015

SUCCESS: the output of run-make-check.sh on centos-7 for 4d55ca8 is http://paste2.org/HVKNW56z

:octocat: Sent from GH.

@tchaikov tchaikov added this to the firefly milestone Mar 13, 2015

@ghost ghost assigned jcsp Mar 16, 2015

@jcsp

This comment has been minimized.

Contributor

jcsp commented Mar 16, 2015

I've merged the master version of this, we could let it pass some tests before merging backport, although given we're replacing an assertion with an if(), we won't notice the difference unless something was touching the assertion (nothing in our suite does).

@ghost

This comment has been minimized.

ghost commented Mar 16, 2015

@jcsp thanks for the update, I see #3922 is merged indeed. I'll check back in a few days and look for updates at http://tracker.ceph.com/issues/11053, assuming that's where you would report problems, if any.

@jcsp

This comment has been minimized.

Contributor

jcsp commented Mar 19, 2015

@dachary this has gone through a fs suite on master without introducing problems, so it's good to go

@jcsp jcsp assigned ghost and unassigned jcsp Mar 19, 2015

@ghost

This comment has been minimized.

ghost commented Mar 19, 2015

@jcsp cool, feel free to merge then !

ghost pushed a commit that referenced this pull request Mar 20, 2015

Merge pull request #3970 from ceph/firefly-11053
mds: fix assertion caused by system clock backwards

Reviewed-by: John Spray <john.spray@redhat.com>

@ghost ghost merged commit ea79a3b into firefly Mar 20, 2015

@ukernel ukernel deleted the firefly-11053 branch Mar 22, 2015

This issue was closed.

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