Skip to content

Close connection after building or pulling#1374

Merged
dnephin merged 1 commit intodocker:masterfrom
aanand:close-before-attaching
Apr 29, 2015
Merged

Close connection after building or pulling#1374
dnephin merged 1 commit intodocker:masterfrom
aanand:close-before-attaching

Conversation

@aanand
Copy link
Copy Markdown

@aanand aanand commented Apr 29, 2015

This ensures that the connection is not recycled, which can cause the Docker daemon to complain if we perform two streaming actions in a row.

Closes #1275.

@LK4D4
Copy link
Copy Markdown

LK4D4 commented Apr 29, 2015

@aanand Not sure what's going on, but you need to close it after each build :) Sorry if this is exactly what you done.

@aanand aanand force-pushed the close-before-attaching branch from 93debf5 to a5c1445 Compare April 29, 2015 17:32
@aanand aanand changed the title Close connection before attaching on 'up' and 'run' Close connection after building or pulling Apr 29, 2015
@aanand
Copy link
Copy Markdown
Author

aanand commented Apr 29, 2015

@LK4D4 Ah, good point. I've made it close after every build or pull.

@LK4D4
Copy link
Copy Markdown

LK4D4 commented Apr 29, 2015

@aanand Why pull? :) CloseNotifier used only in POST /build.

This ensures that the connection is not recycled, which can cause the
Docker daemon to complain if we've already performed another streaming
call such as doing a build.

Signed-off-by: Aanand Prasad <aanand.prasad@gmail.com>
@aanand aanand force-pushed the close-before-attaching branch from a5c1445 to 1a77fee Compare April 29, 2015 17:42
@aanand
Copy link
Copy Markdown
Author

aanand commented Apr 29, 2015

Oh, I assumed it applied to anything with streaming. Just build then.

@dnephin
Copy link
Copy Markdown

dnephin commented Apr 29, 2015

LGTM

dnephin added a commit that referenced this pull request Apr 29, 2015
Close connection after building or pulling
@dnephin dnephin merged commit 878d90f into docker:master Apr 29, 2015
@aanand aanand deleted the close-before-attaching branch May 21, 2015 17:38
infraAnchor pushed a commit to infraAnchor/compose that referenced this pull request Mar 6, 2026
…r#1374)

Bumps [github.com/prometheus/client_golang](https://github.com/prometheus/client_golang) from 1.20.5 to 1.21.0.
- [Release notes](https://github.com/prometheus/client_golang/releases)
- [Changelog](https://github.com/prometheus/client_golang/blob/main/CHANGELOG.md)
- [Commits](prometheus/client_golang@v1.20.5...v1.21.0)

---
updated-dependencies:
- dependency-name: github.com/prometheus/client_golang
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

APIError: 500 Server Error on first docker-compose up / build

4 participants