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

kraken: rgw: fix RadosGW hang during multi-chunk upload of AWSv4. #14939

Merged
merged 1 commit into from Jul 6, 2017

Conversation

Projects
None yet
3 participants
@rzarzynski
Contributor

rzarzynski commented May 3, 2017

http://tracker.ceph.com/issues/19837


Fixes: http://tracker.ceph.com/issues/19754
Signed-off-by: Radoslaw Zarzynski rzarzynski@mirantis.com
(cherry picked from commit 72c1e2e)


This is the Kraken backport of PR #14770.
Corresponding backport ticket: http://tracker.ceph.com/issues/19837.

CC: @mattbenjamin, @oritwas.

@mattbenjamin

This comment has been minimized.

Show comment
Hide comment
@mattbenjamin

mattbenjamin May 3, 2017

Contributor

@rzarzynski it is identical to that on master :)

Contributor

mattbenjamin commented May 3, 2017

@rzarzynski it is identical to that on master :)

@mattbenjamin

lgtm

@smithfarm smithfarm added this to the kraken milestone May 3, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Jun 20, 2017

Contributor

Jenkins re-test this please

Contributor

smithfarm commented Jun 20, 2017

Jenkins re-test this please

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Jun 30, 2017

Contributor

Jenkins re-test this please

Contributor

smithfarm commented Jun 30, 2017

Jenkins re-test this please

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Jul 3, 2017

Contributor

@rzarzynski The make check failure has hopefully been fixed by #16069 - could you please rebase to pick it up?

Contributor

smithfarm commented Jul 3, 2017

@rzarzynski The make check failure has hopefully been fixed by #16069 - could you please rebase to pick it up?

@smithfarm smithfarm changed the title from kraken: rgw: fix RadosGW hang during multi-chunk upload of AWSv4. to [DNM] kraken: rgw: fix RadosGW hang during multi-chunk upload of AWSv4. Jul 3, 2017

rgw: fix RadosGW hang during multi-chunk upload of AWSv4.
Fixes: http://tracker.ceph.com/issues/19754
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit 72c1e2e)
@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Jul 3, 2017

Contributor

Jenkins re-test this please

Contributor

smithfarm commented Jul 3, 2017

Jenkins re-test this please

@rzarzynski

This comment has been minimized.

Show comment
Hide comment
@rzarzynski

rzarzynski Jul 3, 2017

Contributor

@smithfarm: rebased.

Contributor

rzarzynski commented Jul 3, 2017

@smithfarm: rebased.

@smithfarm smithfarm changed the title from [DNM] kraken: rgw: fix RadosGW hang during multi-chunk upload of AWSv4. to kraken: rgw: fix RadosGW hang during multi-chunk upload of AWSv4. Jul 3, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Jul 3, 2017

Contributor

Nevermind, rebase not necessary (Jenkins merges into latest base branch before starting the test).

Jenkins re-test this please

Contributor

smithfarm commented Jul 3, 2017

Nevermind, rebase not necessary (Jenkins merges into latest base branch before starting the test).

Jenkins re-test this please

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Jul 3, 2017

Contributor

Jenkins re-test this please (eight tests were started, but none completed - timeout)

Contributor

smithfarm commented Jul 3, 2017

Jenkins re-test this please (eight tests were started, but none completed - timeout)

@smithfarm smithfarm merged commit 7133d4d into ceph:kraken Jul 6, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Jul 7, 2017

Contributor

This passed an rgw suite at http://tracker.ceph.com/issues/19009#note-60

Contributor

smithfarm commented Jul 7, 2017

This passed an rgw suite at http://tracker.ceph.com/issues/19009#note-60

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