Cleanup: Audit log and error capitalization

pull/564/head
ShivangGoswami 2019-02-04 10:59:34 +05:30
parent 0bd35d1b68
commit 2866b1a3b8
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()