Skip to content
This repository was archived by the owner on Jun 3, 2025. It is now read-only.

add should fail on 40x#1326

Merged
tejal29 merged 4 commits intoGoogleContainerTools:masterfrom
greut:add-to-err-on-404
Jul 29, 2020
Merged

add should fail on 40x#1326
tejal29 merged 4 commits intoGoogleContainerTools:masterfrom
greut:add-to-err-on-404

Conversation

@greut
Copy link
Copy Markdown
Contributor

@greut greut commented Jun 20, 2020

Description

ADD command will fail when the HTTP response is not a successful one, e.g. 404

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

  • Includes unit tests
  • Adds integration tests if needed.

See the contribution guide for more details.

Reviewer Notes

  • The code flow looks good.
  • Unit tests and or integration tests added.

Release Notes

Describe any changes here so maintainer can include it in the release notes, or delete this block.

- `ADD` command to fail in case of an HTTP error, e.g. 404

@googlebot googlebot added the cla: yes CLA signed by all commit authors label Jun 20, 2020
greut added 4 commits June 20, 2020 09:49
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
@tejal29 tejal29 merged commit 92917ba into GoogleContainerTools:master Jul 29, 2020
@greut greut deleted the add-to-err-on-404 branch July 29, 2020 05:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes CLA signed by all commit authors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants