-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Unable to push to internal registry using Docker 1.8 due to "Error parsing HTTP response: invalid character 'M' looking for beginning of value: "Method not allowed\n"" #4415
Comments
This comment might be related: |
Also might be related to this: #4378 if Docker registry use Route. |
The issue is that our copy of the registry doesn't support PATCH when uploading layers, but Docker 1.8 has moved to using that HTTP verb. We'll have to upgrade our registry, which we can't do until we refactor pruning (#3333). |
Our copy of the registry's handler := http.Handler(handlers.MethodHandler{
"POST": http.HandlerFunc(luh.StartLayerUpload),
"GET": http.HandlerFunc(luh.GetUploadStatus),
"HEAD": http.HandlerFunc(luh.GetUploadStatus),
// TODO(stevvooe): Must implement patch support.
// "PATCH": http.HandlerFunc(luh.PutLayerChunk),
"PUT": http.HandlerFunc(luh.PutLayerUploadComplete),
"DELETE": http.HandlerFunc(luh.CancelLayerUpload),
}) whereas upstream master has this: handler := http.Handler(handlers.MethodHandler{
"POST": http.HandlerFunc(buh.StartBlobUpload),
"GET": http.HandlerFunc(buh.GetUploadStatus),
"HEAD": http.HandlerFunc(buh.GetUploadStatus),
"PATCH": http.HandlerFunc(buh.PatchBlobData),
"PUT": http.HandlerFunc(buh.PutBlobUploadComplete),
"DELETE": http.HandlerFunc(buh.CancelBlobUpload),
}) |
FYI, it should be possible to bump our docker/distribution godep to the 2.0.1 tag. It's not a straightforward patch, but we should be able to get it in w/o too much difficulty. |
I am still hitting this error with latest origin build on F21/docker 1.8.
|
Make sure you are building images and running with --latest-images=true. The fix is in the registry image, which you might still be pulling an old version of from dockerhub |
I added
|
Sorry, that was a typo... the registry image is the one containing the fix |
You have to specify --latest-images when invoking oadm registry too. On Sunday, September 6, 2015, Jordan Liggitt [email protected]
|
I am still getting same error.
|
@nak3 remove all docker-registry images from your system and build again |
After clean up all of my local images and download newest images, it worked. At this time, I downloaded this ID's registry image.
|
Env:
NOTE: I tested on Fedora21 with docker 1.8. But I could reproduce CentOS7 with docker 1.8
Issue:
Reproduce steps:
oc new-app https://github.com/nak3/helloworld-v3.git -l app=hello
oc new-app https://github.com/openshift/sti-python.git --context-dir=3.3/test/standalone-test-app -l app=python --strategy=source
, but same result.The text was updated successfully, but these errors were encountered: