Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Frequent errors from docker.io on external worker #274

Closed
flawedmatrix opened this issue Feb 8, 2016 · 3 comments
Closed

Frequent errors from docker.io on external worker #274

flawedmatrix opened this issue Feb 8, 2016 · 3 comments

Comments

@flawedmatrix
Copy link

We're seeing the following error about one out of every five builds on our external vSphere worker.

Get https://registry-1.docker.io/v2: net/http: request canceled while waiting for connection

Are there any steps to improve the reliability of connections to docker.io, or a feature to introduce additional retry logic?
@concourse-bot
Copy link
Collaborator

Hi there!

We use Pivotal Tracker to provide visibility into what our team is working on. A story for this issue has been automatically created.

The current status is as follows:

  • #113317551 Frequent errors from docker.io on external worker

This comment, as well as the labels on the issue, will be automatically updated as the status in Tracker changes.

@superbeeny
Copy link

+1 we're seeing this as well

cjcjameson pushed a commit to cjcjameson/concourse that referenced this issue Mar 2, 2016
@vito
bump cli
2bf494d 
[finishes #102114438]

Submodule src/github.com/codegangsta/cli 9b2bd2b..96f1fed:
  > Merge pull request concourse#281 from thiderman/patch-1
  > Merge pull request concourse#267 from tristanz/master
  > Merge pull request concourse#279 from nodirt/version
  > Merge pull request concourse#280 from gummiboll/fix-for-testcommandignoreflags
  > Merge pull request concourse#275 from langri-sha/master
  > Merge pull request concourse#274 from klueska/bash-completion-default-prog
  > Merge pull request concourse#264 from wardn/master
  > Merge pull request concourse#246 from cpuguy83/propigate_cmd_parent_to_app_setup
  > Merge pull request concourse#251 from MartyMacGyver/more-cleanups
  > Merge pull request concourse#253 from databus23/fix-isssue-252
  > Merge pull request concourse#245 from cpuguy83/get_parent_ctx
  > Merge pull request concourse#238 from polds/copyright
  > Merge pull request concourse#244 from wallclockbuilder/patch-1
  > Merge pull request concourse#241 from codegangsta/travis-update
  > Merge pull request concourse#232 from codegangsta/fix-231
  > Merge pull request concourse#233 from codegangsta/add-flag-docs
  > Merge pull request concourse#230 from codegangsta/do-not-swallow-after-error
  > Merge pull request concourse#227 from databus23/global-flags-from-all-parents
  > Merge pull request concourse#229 from saromanov/fix-missing-value
  > Add instructions for distributing bash completion
  > Set additional information on command's 'app'
  > Merge pull request concourse#218 from codegangsta/add-another-help-test
  > Rename TestShowAppHelp to be more verbose
  > Add test for inclusion of 'AUTHORS' in App help
  > do not print 'AUTHOR(S):' is there is no author
  > Merge pull request concourse#217 from codegangsta/readd-print-help-function
  > Merge pull request concourse#211 from netbun/patch-1

Signed-off-by: Corbin Halliwill <challiwill@pivotal.io>
@vito
Copy link
Member

vito commented Apr 18, 2016

This should be better now with image_resource + the Docker resource performing retry logic.

@vito vito closed this as completed Apr 18, 2016
cdutra pushed a commit that referenced this issue Aug 4, 2016
bump atc fly testflight ginkgo gomega
ebee7fc 
[finishes #127219723]

Submodule src/github.com/concourse/atc 1ee1c48..a3023dd:
  > ginkgo blur reoreded everything
  > return all public pipelines on GetAllPipelines endpoint
  > only check for basic auth on get token endpoint
  > do not default team name to main if not provided
  > do not default team name to main in api
Submodule src/github.com/concourse/fly cb98e4c..3269297:
  > make --team-name required
  > retrieves token if there is no auth method set
Submodule src/github.com/concourse/testflight fdd0c3f..abd306e:
  > do not default team name to 'main'
Submodule src/github.com/onsi/ginkgo e43390e..74c678d:
  > Make JUnit reporter include failure location in message. (#262)
  > remove 1.4 from travis.yml
  > Add gcflags option (#276)
  > Revert "Use the go1.5 build tag to handle vendor exceptions" (#274)
  > Merge pull request #272 from fsouza/fix-vendor
  > Add flaky test mitigation (#261)
  > Allow units and precision in benchmark (#266)
  > Add Solaris support (#264)
  > Merge pull request #259 from kwadrat/master
  > Merge branch 'apvail-spell-fix'
  > Fix go16 vendor
  > Merge pull request #250 from james-lawrence/master
  > Merge pull request #228 from jayunit100/RegexFileNameFiltering
  > Fix test flakiness
  > Merge pull request #235 from mboersma/fix-travis
  > fix compilation on older versions of go
  > fix issue where packages that reference vendored dependencies weren't compiling
  > Merge pull request #216 from sha1sum/master
  > Merge pull request #209 from luxas/build_on_arm64
  > Merge pull request #212 from cfmobile/master
  > Merge pull request #210 from cfmobile/master
Submodule src/github.com/onsi/gomega 2152b45..9ed8da1:
  > Merge pull request #166 from trayo/patch-2
  > Merge pull request #164 from wendorf/assert_typo
  > Merge remote-tracking branch 'origin/pr/163'
  > Merge pull request #160 from tinygrasshopper/fix_failing_close_ghttp
  > Merge pull request #150 from tinygrasshopper/build-fix
  > Merge pull request #159 from WesleyJeanette/patch-1
  > Merge pull request #157 from kwadrat/master
  > Merge pull request #141 from mariantalla/gomega-yaml-matcher
  > Reset tmpDir in gexec.CleanupBuildArtifacts
  > Update test description for match json tests.
  > Make the error message for expected JSON values having the wrong type accurate
  > Merge pull request #133 from tjarratt/be-identical-to-matcher
  > Merge pull request #132 from tjarratt/improve-match-json-error-message
  > Merge pull request #128 from tinygrasshopper/have-cap
  > drop 1.4 from travis
  > ghttp tests should now pass in 1.6
  > CloseClientConnections test uses http.Post instead of http.Get to avoid retries
  > add tip to .travis.yml
  > Merge pull request #125 from cfmobile/master
  > Merge pull request #122 from cfmobile/master
  > Merge pull request #119 from jim-slattery-rs/gitignore_idea
  > Merge pull request #118 from jim-slattery-rs/fix_up_succeed

Signed-off-by: Yucheng Tu <ytu@pivotal.io>
vito added a commit that referenced this issue Apr 11, 2018
@vito
bump fly yaml.v2
0700c38 
vmware-archive/fly#201

Submodule src/github.com/concourse/fly f9d92f5..2be951a:
  > Merge branch 'govau-unmarshalstrict'
  > Merge pull request #214 from alepee/patch-1
Submodule src/gopkg.in/yaml.v2 e4d366fc..5420a8b6:
  > Use underlying float precision when formatting floats (#353)
  > Fix typo in tab error message (#208).
  > Fix misspell of precede in ported code (#216)
  > Fix type on Marshal docs (#206).
  > Fixed typos in docstrings (#179).
  > Drop unnecessary explicit timestamp tags.
  > Fix broken test from last merge.
  > increment non-zero scanner error lines (#319)
  > Convert int to float when explicitly tagged.
  > Encode and decode arrays.
  > Fix edge case when decoding MinInt as -0b.
  > Remove mention of non-existent examples folder.
  > Fix curious assumption from the original C reader.
  > Ensure scanner has data before checking for blanks.
  > Drop invalid simple key assertion.
  > Improve map stabilization logic.
  > Fix unstable map key ordering (#195).
  > Merge pull request #336 from rogpeppe/025-go.mod
  > Merge pull request #335 from rogpeppe/024-merge-devel
  > Merge pull request #253 from heldtogether/patch-1
  > Merge pull request #308 from rogpeppe/016-revert-v2-PR273
  > Merge pull request #273 from rogpeppe/006-timestamps
  > Merge pull request #299 from rogpeppe/009-gofmt
  > Merge pull request #281 from houshengbo/fix-incorrect-line-number
  > Correct documentation for Marshal (#287)
  > Merge pull request #289 from rliebz/null-fix
  > Replace LICENSE text with actual license (#274)
  > Merge pull request #272 from rogpeppe/005-cleaner-tag-scan
  > Merge pull request #103 from andreychernih/bugfix/non-specific-tags
  > Merge pull request #271 from rogpeppe/004-embedded-example
  > Merge pull request #94 from mvo5/feature/embeded-structs-example
  > Merge pull request #264 from hiveminded/v2
  > Merge pull request #262 from wupeka/v2
  > Remove unreachable code to fix go vet (#249)
  > Fix dead URL for yaml specification (#240)
  > Tighten restrictions on float decoding (#171)
  > Fix decode test for Go 1.8 (#217)
  > Fix unmarshaler handling of empty strings.
  > new license in the README file (#189)
mhuangpivotal added a commit that referenced this issue May 30, 2018
@mhuangpivotal @nader-ziada
bump atc web
a13d5d8 
Submodule src/github.com/concourse/atc 27c1f7e..1898498:
  > Add /api/v1/resources endpoint
  > Merge pull request #274 from databus23/prometheus-job-label
Submodule src/github.com/concourse/web 90475f1..a185dc7:
  > Add resource error indicator in dashboard view

Signed-off-by: Nader Ziada <nziada@pivotal.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@vito @flawedmatrix @superbeeny @concourse-bot