Merge pull request #11 from rsvoboda/more.context.logging

moutonjeremy · web-flow · commit 2212d4c8dae6 · 2021-03-10T10:29:31.000+01:00
Provide more details in error message when GH retuns unexpected HTTP status code
diff --git a/metrics/get_billable_from_github.go b/metrics/get_billable_from_github.go
@@ -56,7 +56,7 @@ func GetBillableFromGithub() {
 					log.Fatal(err)
 				}
 				if resp.StatusCode != 200 {
-					log.Fatalf("the status code returned by the server is different from 200: %d", resp.StatusCode)
+					log.Fatalf("the status code returned by the server for workflows in repo %s is different from 200: %d", repo, resp.StatusCode)
 				}
 				err = json.NewDecoder(resp.Body).Decode(&bill)
 				if err != nil {
diff --git a/metrics/get_jobs_from_github.go b/metrics/get_jobs_from_github.go
@@ -53,7 +53,7 @@ func GetJobsFromGithub() {
 				log.Fatal(err)
 			}
 			if resp.StatusCode != 200 {
-				log.Fatalf("the status code returned by the server is different from 200: %d", resp.StatusCode)
+				log.Fatalf("the status code returned by the server for runs in repo %s is different from 200: %d", repo, resp.StatusCode)
 			}
 			err = json.NewDecoder(resp.Body).Decode(&p)
 			if err != nil {
diff --git a/metrics/get_runners_from_github.go b/metrics/get_runners_from_github.go
@@ -48,7 +48,7 @@ func GetRunnersFromGithub() {
 				log.Fatal(err)
 			}
 			if resp.StatusCode != 200 {
-				log.Fatalf("the status code returned by the server is different from 200: %d", resp.StatusCode)
+				log.Fatalf("the status code returned by the server for runners in repo %s is different from 200: %d", repo, resp.StatusCode)
 			}
 			err = json.NewDecoder(resp.Body).Decode(&p)
 			if err != nil {
diff --git a/metrics/get_runners_organization_from_github.go b/metrics/get_runners_organization_from_github.go
@@ -41,7 +41,7 @@ func GetRunnersOrganizationFromGithub() {
 				log.Fatal(err)
 			}
 			if resp.StatusCode != 200 {
-				log.Fatalf("the status code returned by the server is different from 200: %d", resp.StatusCode)
+				log.Fatalf("the status code returned by the server for runners in organization %s is different from 200: %d", orga, resp.StatusCode)
 			}
 			err = json.NewDecoder(resp.Body).Decode(&p)
 			if err != nil {
diff --git a/metrics/worflows_cache.go b/metrics/worflows_cache.go
@@ -37,7 +37,7 @@ func WorkflowsCache() {
 				log.Fatal(err)
 			}
 			if resp.StatusCode != 200 {
-				log.Fatalf("the status code returned by the server is different from 200: %d", resp.StatusCode)
+				log.Fatalf("the status code returned by the server for workflows in repo %s is different from 200: %d", repo, resp.StatusCode)
 			}
 			err = json.NewDecoder(resp.Body).Decode(&p)
 			if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ func GetBillableFromGithub() {`
`56`	`56`	`log.Fatal(err)`
`57`	`57`	`}`
`58`	`58`	`if resp.StatusCode != 200 {`
`59`		`- log.Fatalf("the status code returned by the server is different from 200: %d", resp.StatusCode)`
	`59`	`+ log.Fatalf("the status code returned by the server for workflows in repo %s is different from 200: %d", repo, resp.StatusCode)`
`60`	`60`	`}`
`61`	`61`	`err = json.NewDecoder(resp.Body).Decode(&bill)`
`62`	`62`	`if err != nil {`
Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ func GetJobsFromGithub() {`
`53`	`53`	`log.Fatal(err)`
`54`	`54`	`}`
`55`	`55`	`if resp.StatusCode != 200 {`
`56`		`- log.Fatalf("the status code returned by the server is different from 200: %d", resp.StatusCode)`
	`56`	`+ log.Fatalf("the status code returned by the server for runs in repo %s is different from 200: %d", repo, resp.StatusCode)`
`57`	`57`	`}`
`58`	`58`	`err = json.NewDecoder(resp.Body).Decode(&p)`
`59`	`59`	`if err != nil {`
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ func GetRunnersFromGithub() {`
`48`	`48`	`log.Fatal(err)`
`49`	`49`	`}`
`50`	`50`	`if resp.StatusCode != 200 {`
`51`		`- log.Fatalf("the status code returned by the server is different from 200: %d", resp.StatusCode)`
	`51`	`+ log.Fatalf("the status code returned by the server for runners in repo %s is different from 200: %d", repo, resp.StatusCode)`
`52`	`52`	`}`
`53`	`53`	`err = json.NewDecoder(resp.Body).Decode(&p)`
`54`	`54`	`if err != nil {`
Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ func GetRunnersOrganizationFromGithub() {`
`41`	`41`	`log.Fatal(err)`
`42`	`42`	`}`
`43`	`43`	`if resp.StatusCode != 200 {`
`44`		`- log.Fatalf("the status code returned by the server is different from 200: %d", resp.StatusCode)`
	`44`	`+ log.Fatalf("the status code returned by the server for runners in organization %s is different from 200: %d", orga, resp.StatusCode)`
`45`	`45`	`}`
`46`	`46`	`err = json.NewDecoder(resp.Body).Decode(&p)`
`47`	`47`	`if err != nil {`
Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ func WorkflowsCache() {`
`37`	`37`	`log.Fatal(err)`
`38`	`38`	`}`
`39`	`39`	`if resp.StatusCode != 200 {`
`40`		`- log.Fatalf("the status code returned by the server is different from 200: %d", resp.StatusCode)`
	`40`	`+ log.Fatalf("the status code returned by the server for workflows in repo %s is different from 200: %d", repo, resp.StatusCode)`
`41`	`41`	`}`
`42`	`42`	`err = json.NewDecoder(resp.Body).Decode(&p)`
`43`	`43`	`if err != nil {`