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: fix RadosGW hang during multi-chunk upload of AWSv4. #14770

Merged
merged 1 commit into from May 1, 2017

Conversation

Projects
None yet
3 participants
@rzarzynski
Contributor

rzarzynski commented Apr 25, 2017

Fixes: http://tracker.ceph.com/issues/19754
Signed-off-by: Radoslaw Zarzynski rzarzynski@mirantis.com


Still needs regression testing.

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>
@mattbenjamin

This comment has been minimized.

Contributor

mattbenjamin commented Apr 25, 2017

@oritwas, do we need this downstream?

@oritwas

This comment has been minimized.

Contributor

oritwas commented Apr 25, 2017

@mattbenjamin , we need to backport it to jewel so yes

@mattbenjamin

This comment has been minimized.

Contributor

mattbenjamin commented Apr 25, 2017

@oritwas is it in teuthology now? I'll add this to Marcus' bz as soon as this is merged (we still have the civetweb issue for Jewel/downstream)

@oritwas

This comment has been minimized.

Contributor

oritwas commented Apr 25, 2017

@mattbenjamin , not yet waiting for the gitbuilders to complete (for wip-orit-testing) and than I will schedule it

@rzarzynski

This comment has been minimized.

Contributor

rzarzynski commented Apr 25, 2017

Hopefully the problem is restricted to Kraken only. Most likely Jewel is unaffected as it has the check. However, I haven't verified that in testing.

@oritwas

This comment has been minimized.

Contributor

oritwas commented Apr 25, 2017

@rzarzynski , that is good news (one less backport)

@mattbenjamin

This comment has been minimized.

Contributor

mattbenjamin commented Apr 25, 2017

@oritwas , @rzarzynski thanks, that's great news (in particular, if it's on v10.2.7)

@mattbenjamin

This comment has been minimized.

Contributor

mattbenjamin commented Apr 27, 2017

@oritwas update?

@mattbenjamin mattbenjamin merged commit 1c9a48c into ceph:master May 1, 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
@rzarzynski

This comment has been minimized.

Contributor

rzarzynski commented May 1, 2017

Tested using the Javier's UploadObjectSingleOperation.java.

Some useful commands:

$ ls
aws-java-sdk-1.11.123  aws-java-sdk.zip  cmds.txt  my-file-name-1111  UploadObjectSingleOperation.class  UploadObjectSingleOperation.java
$ cat cmds.txt
javac -cp aws-java-sdk-1.11.123/lib/aws-java-sdk-1.11.123.jar UploadObjectSingleOperation.java
java -cp '.:aws-java-sdk-1.11.123/lib/*:aws-java-sdk-1.11.123/third-party/lib/*' UploadObjectSingleOperation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment