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

rgw: rest-bench common/WorkQueue.cc: 54: FAILED assert(_threads.empty()) #5383

Merged
2 commits merged into from Sep 7, 2015

Conversation

Projects
None yet
3 participants
@ghost
Copy link

commented Jul 28, 2015

hjwsm1989 added some commits Jul 16, 2015

rest_bench: drain the work queue to fix a crash
  Fixes: #3896
  Signed-off-by: huangjun <hjwsm1989@gmail.com>

(cherry picked from commit f3d34d8)
rest_bench: bucketname is not mandatory as we have a default name
  Signed-off-by: huangjun <hjwsm1989@gmail.com>

(cherry picked from commit bc56a87)

@ghost ghost self-assigned this Jul 28, 2015

@ghost ghost added this to the hammer milestone Jul 28, 2015

@ghost ghost added bug fix rgw labels Jul 28, 2015

ghost pushed a commit that referenced this pull request Jul 29, 2015

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5383: rest-bench common/WorkQueue.cc: 54: FAILED …
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5383: rest-bench common/WorkQueue.cc: 54: FAILED …
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5383: rest-bench common/WorkQueue.cc: 54: FAILED …
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 31, 2015

Merge pull request #5383: rest-bench common/WorkQueue.cc: 54: FAILED …
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 31, 2015

Merge pull request #5383: rest-bench common/WorkQueue.cc: 54: FAILED …
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 31, 2015

Merge pull request #5383: rest-bench common/WorkQueue.cc: 54: FAILED …
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Sep 1, 2015

Merge pull request #5383: rest-bench common/WorkQueue.cc: 54: FAILED …
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Sep 2, 2015

Merge pull request #5383: rest-bench common/WorkQueue.cc: 54: FAILED …
…assert(_threads.empty())

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

This comment has been minimized.

Copy link
Author

commented Sep 2, 2015

@tchaikov does this backport look good to merge ? It passed a run of the hammer rgw suite ( see http://pulpito.ceph.com/loic-2015-09-01_17:25:32-rgw-hammer-backports---basic-multi/ ).

@ghost

This comment has been minimized.

Copy link
Author

commented Sep 6, 2015

@tchaikov ping ?

@ghost ghost assigned tchaikov and unassigned ghost Sep 6, 2015

@tchaikov tchaikov assigned ghost and unassigned tchaikov Sep 7, 2015

@tchaikov

This comment has been minimized.

Copy link
Contributor

commented Sep 7, 2015

👍

ghost pushed a commit that referenced this pull request Sep 7, 2015

Loic Dachary
Merge pull request #5383 from dachary/wip-12504-hammer
rest-bench common/WorkQueue.cc: 54: FAILED assert(_threads.empty())

Reviewed-by: Kefu Chai <kchai@redhat.com>

@ghost ghost merged commit c243f7a into ceph:hammer Sep 7, 2015

@ghost ghost changed the title rest-bench common/WorkQueue.cc: 54: FAILED assert(_threads.empty()) rgw: rest-bench common/WorkQueue.cc: 54: FAILED assert(_threads.empty()) Sep 13, 2015

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.