Merge pull request #73686 from ShivangGoswami/master

Cleanup: Audit log and error capitalization
pull/564/head
Kubernetes Prow Robot 2019-02-17 23:08:39 -08:00 committed by GitHub
commit cdbc9d8567
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 6 deletions

View File

@ -19,6 +19,7 @@ package main
import (
"bytes"
"encoding/json"
"errors"
goflag "flag"
"fmt"
"net/http"
@ -202,21 +203,21 @@ func getVersion(lastSeenBinaryVersion *string) error {
// Create the get request for the etcd version endpoint.
req, err := http.NewRequest("GET", etcdVersionScrapeURI, nil)
if err != nil {
return fmt.Errorf("Failed to create GET request for etcd version: %v", err)
return fmt.Errorf("failed to create GET request for etcd version: %v", err)
}
// Send the get request and receive a response.
client := &http.Client{}
resp, err := client.Do(req)
if err != nil {
return fmt.Errorf("Failed to receive GET response for etcd version: %v", err)
return fmt.Errorf("failed to receive GET response for etcd version: %v", err)
}
defer resp.Body.Close()
// Obtain EtcdVersion from the JSON response.
var version EtcdVersion
if err := json.NewDecoder(resp.Body).Decode(&version); err != nil {
return fmt.Errorf("Failed to decode etcd version JSON: %v", err)
return fmt.Errorf("failed to decode etcd version JSON: %v", err)
}
// Return without updating the version if it stayed the same since last time.
@ -228,7 +229,7 @@ func getVersion(lastSeenBinaryVersion *string) error {
if *lastSeenBinaryVersion != "" {
deleted := etcdVersion.Delete(prometheus.Labels{"binary_version": *lastSeenBinaryVersion})
if !deleted {
return fmt.Errorf("Failed to delete previous version's metric")
return errors.New("failed to delete previous version's metric")
}
}
@ -259,14 +260,14 @@ func getVersionPeriodically(stopCh <-chan struct{}) {
func scrapeMetrics() (map[string]*dto.MetricFamily, error) {
req, err := http.NewRequest("GET", etcdMetricsScrapeURI, nil)
if err != nil {
return nil, fmt.Errorf("Failed to create GET request for etcd metrics: %v", err)
return nil, fmt.Errorf("failed to create GET request for etcd metrics: %v", err)
}
// Send the get request and receive a response.
client := &http.Client{}
resp, err := client.Do(req)
if err != nil {
return nil, fmt.Errorf("Failed to receive GET response for etcd metrics: %v", err)
return nil, fmt.Errorf("failed to receive GET response for etcd metrics: %v", err)
}
defer resp.Body.Close()