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

rgw: LCWorker's worktime is not the same as config rgw_lifecycle_work_time. #11963

Merged
merged 1 commit into from Apr 4, 2017

Conversation

Projects
None yet
4 participants
@zhangsw
Contributor

zhangsw commented Nov 14, 2016

The judgement in should_work is not correctly. The worktime should between start time and end time.

Signed-off-by: Zhang Shaowen zhangshaowen@cmss.chinamobile.com

@ghost

This comment has been minimized.

ghost commented Nov 16, 2016

jenkins test this please (eio now ignored in master)

@ghost

This comment has been minimized.

ghost commented Nov 17, 2016

jenkins test this please (fast-forward failure)

@ghost

This comment has been minimized.

ghost commented Nov 17, 2016

jenkins test this please (asok error now fixed in master)

@ghost

This comment has been minimized.

ghost commented Nov 18, 2016

jenkins test this please (eio now ignored in master)

@ghost

This comment has been minimized.

ghost commented Nov 18, 2016

jenkins test this please (unittest_journal)

@yehudasa

This comment has been minimized.

Member

yehudasa commented Nov 29, 2016

@zhangsw you open a ticket for this one, and add it in the commit message? @dang we need to make sure that this is covered by teuthology

rgw: Fix a bug that LCWorker's worktime is not the same as config
rgw_lifecycle_work_time.

Fixes: http://tracker.ceph.com/issues/18087

Signed-off-by: Zhang Shaowen <zhangshaowen@cmss.chinamobile.com>
@zhangsw

This comment has been minimized.

Contributor

zhangsw commented Nov 30, 2016

@yehudasa I have created a tracker and added that in commit message.
http://tracker.ceph.com/issues/18087

@zhangsw

This comment has been minimized.

Contributor

zhangsw commented Feb 9, 2017

@dang please help review this,thanks~

@zhangsw

This comment has been minimized.

Contributor

zhangsw commented Mar 29, 2017

jenkins test this please

@dang

dang approved these changes Mar 31, 2017

LGTM

@cbodley

This comment has been minimized.

@cbodley cbodley merged commit 1e8b763 into ceph:master Apr 4, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment