Merge pull request #5351 from mikebrow/resolver-debug-improvement

estesp · web-flow · commit ce5d42fa2ea5 · 2021-04-14T14:54:58.000-04:00
adds some debug log for each failed host
diff --git a/remotes/docker/resolver.go b/remotes/docker/resolver.go
@@ -286,12 +286,14 @@ func (r *dockerResolver) Resolve(ctx context.Context, ref string) (string, ocisp
 				if lastErr == nil {
 					lastErr = err
 				}
+				log.G(ctx).WithError(err).Info("trying next host")
 				continue // try another host
 			}
 			resp.Body.Close() // don't care about body contents.
 
 			if resp.StatusCode > 299 {
 				if resp.StatusCode == http.StatusNotFound {
+					log.G(ctx).Info("trying next host - response was http.StatusNotFound")
 					continue
 				}
 				return "", ocispec.Descriptor{}, errors.Errorf("unexpected status code %v: %v", u, resp.Status)

Original file line number	Diff line number	Diff line change
`@@ -286,12 +286,14 @@ func (r *dockerResolver) Resolve(ctx context.Context, ref string) (string, ocisp`
`286`	`286`	`if lastErr == nil {`
`287`	`287`	`lastErr = err`
`288`	`288`	`}`
	`289`	`+ log.G(ctx).WithError(err).Info("trying next host")`
`289`	`290`	`continue // try another host`
`290`	`291`	`}`
`291`	`292`	`resp.Body.Close() // don't care about body contents.`
`292`	`293`
`293`	`294`	`if resp.StatusCode > 299 {`
`294`	`295`	`if resp.StatusCode == http.StatusNotFound {`
	`296`	`+ log.G(ctx).Info("trying next host - response was http.StatusNotFound")`
`295`	`297`	`continue`
`296`	`298`	`}`
`297`	`299`	`return "", ocispec.Descriptor{}, errors.Errorf("unexpected status code %v: %v", u, resp.Status)`