[release/1.6] retry request on writer reset#7461
[release/1.6] retry request on writer reset#7461samuelkarp merged 1 commit intocontainerd:release/1.6from
Conversation
when a put request is retried due to the response from registry, the body of the request should be seekable. A dynamic pipe is added to the body so that the content of the body can be read again. Currently a maximum of 5 resets are allowed, above which will fail the request. A new error ErrReset is introduced which informs that a reset has occured and request needs to be retried. also added tests for Copy() and push() to test the new functionality Signed-off-by: Akhil Mohan <[email protected]> (cherry picked from commit 8f4c23b) Signed-off-by: Akhil Mohan <[email protected]>
|
Hi @akhilerm. Thanks for your PR. I'm waiting for a containerd member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
/ok-to-test |
|
did you encounter any merge issues or was it a clean cherry pick? nvm I checked :-) clean cherry.. added comment to PR text.. Cheers. |
| testcase.source.size, | ||
| testcase.source.digest) | ||
|
|
||
| // if an error is expected then further comparisons are not required |
There was a problem hiding this comment.
@mikebrow There was one merge conflict while cherry-picking here in the tests. Other than that it was clean.
7 participants
Cherry-pick (clean) #6995
when a put request is retried due to the response from registry, the body of the request should be seekable. A dynamic pipe is added to the body so that the content of the body can be read again. Currently a maximum of 5 resets are allowed, above which will fail the request. A new error ErrReset is introduced which informs that a reset has occured and request needs to be retried.
also added tests for Copy() and push() to test the new functionality
Signed-off-by: Akhil Mohan [email protected]
(cherry picked from commit 8f4c23b)
Signed-off-by: Akhil Mohan [email protected]