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
Merged

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

2 commits merged into from Sep 7, 2015

Conversation

ghost
Copy link

@ghost ghost commented Jul 28, 2015

  Fixes: #3896
  Signed-off-by: huangjun <hjwsm1989@gmail.com>

(cherry picked from commit f3d34d8)
  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
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 30, 2015
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 30, 2015
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 31, 2015
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 31, 2015
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 31, 2015
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Sep 1, 2015
…assert(_threads.empty())

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Sep 2, 2015
…assert(_threads.empty())

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

ghost 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
Copy link
Author

ghost 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
Copy link
Contributor

tchaikov commented Sep 7, 2015

👍

ghost pushed a commit that referenced this pull request Sep 7, 2015
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 pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants