docker: add comment about "connection reset by peer" error

pull/3621/head
Frank Schroeder 7 years ago
parent 164ec3ec39
commit b4d8c26194
No known key found for this signature in database
GPG Key ID: 4D65C6EAEC87DECD

@ -162,6 +162,12 @@ func (c *DockerClient) StartExec(containerID, execID string) (*circbuf.Buffer, e
uri := fmt.Sprintf("/exec/%s/start", execID) uri := fmt.Sprintf("/exec/%s/start", execID)
b, code, err := c.call("POST", uri, data) b, code, err := c.call("POST", uri, data)
switch { switch {
// todo(fs): https://github.com/hashicorp/consul/pull/3621
// todo(fs): for some reason the docker agent closes the connection during the
// todo(fs): io.Copy call in c.call which causes a "connection reset by peer" error
// todo(fs): even though both body and status code have been received. My current is
// todo(fs): that the docker agent closes this prematurely but I don't understand why.
// todo(fs): the code below ignores this error.
case err != nil && !strings.Contains(err.Error(), "connection reset by peer"): case err != nil && !strings.Contains(err.Error(), "connection reset by peer"):
return nil, fmt.Errorf("start exec failed for container %s: %s", containerID, err) return nil, fmt.Errorf("start exec failed for container %s: %s", containerID, err)
case code == 200: case code == 200:

Loading…
Cancel
Save