diff options
author | Corey Stewart <cbstewart9393@gmail.com> | 2016-12-06 14:22:26 -0500 |
---|---|---|
committer | Joe Handzik <joseph.t.handzik@hpe.com> | 2017-01-08 10:23:58 -0600 |
commit | a8c94d48e66ade7e18de532c27f7adfc3f38e4ee (patch) | |
tree | 5548aacc4d52f8bc4e6320ec54ed5efe5aec3eaf /collector/zfs_linux.go | |
parent | c95bfa705edf1ed9e0b49e71a5c94e2c877bed10 (diff) | |
download | prometheus_node_collector-a8c94d48e66ade7e18de532c27f7adfc3f38e4ee.tar.bz2 prometheus_node_collector-a8c94d48e66ade7e18de532c27f7adfc3f38e4ee.tar.xz prometheus_node_collector-a8c94d48e66ade7e18de532c27f7adfc3f38e4ee.zip |
Style changes and cleanup
This patch makes stylistic changes to error strings, unexports method names by lower casing them, removes unused dataSetMetric, and adds copyright/licence information.
Signed-Off-By: Corey Stewart <stewa169@purdue.edu>
Diffstat (limited to 'collector/zfs_linux.go')
-rw-r--r-- | collector/zfs_linux.go | 26 |
1 files changed, 14 insertions, 12 deletions
diff --git a/collector/zfs_linux.go b/collector/zfs_linux.go index 0a1c10e..17cfcae 100644 --- a/collector/zfs_linux.go +++ b/collector/zfs_linux.go | |||
@@ -1,3 +1,16 @@ | |||
1 | // Copyright 2016 The Prometheus Authors | ||
2 | // Licensed under the Apache License, Version 2.0 (the "License"); | ||
3 | // you may not use this file except in compliance with the License. | ||
4 | // You may obtain a copy of the License at | ||
5 | // | ||
6 | // http://www.apache.org/licenses/LICENSE-2.0 | ||
7 | // | ||
8 | // Unless required by applicable law or agreed to in writing, software | ||
9 | // distributed under the License is distributed on an "AS IS" BASIS, | ||
10 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
11 | // See the License for the specific language governing permissions and | ||
12 | // limitations under the License. | ||
13 | |||
1 | package collector | 14 | package collector |
2 | 15 | ||
3 | import ( | 16 | import ( |
@@ -17,14 +30,6 @@ const ( | |||
17 | zfsArcstatsProcpath = "spl/kstat/zfs/arcstats" | 30 | zfsArcstatsProcpath = "spl/kstat/zfs/arcstats" |
18 | ) | 31 | ) |
19 | 32 | ||
20 | func (c *zfsCollector) zfsAvailable() (err error) { | ||
21 | file, err := c.openArcstatsFile() | ||
22 | if err != nil { | ||
23 | file.Close() | ||
24 | } | ||
25 | return err | ||
26 | } | ||
27 | |||
28 | func (c *zfsCollector) openArcstatsFile() (file *os.File, err error) { | 33 | func (c *zfsCollector) openArcstatsFile() (file *os.File, err error) { |
29 | file, err = os.Open(procFilePath(zfsArcstatsProcpath)) | 34 | file, err = os.Open(procFilePath(zfsArcstatsProcpath)) |
30 | if err != nil { | 35 | if err != nil { |
@@ -35,7 +40,6 @@ func (c *zfsCollector) openArcstatsFile() (file *os.File, err error) { | |||
35 | } | 40 | } |
36 | 41 | ||
37 | func (c *zfsCollector) updateArcstats(ch chan<- prometheus.Metric) (err error) { | 42 | func (c *zfsCollector) updateArcstats(ch chan<- prometheus.Metric) (err error) { |
38 | |||
39 | file, err := c.openArcstatsFile() | 43 | file, err := c.openArcstatsFile() |
40 | if err != nil { | 44 | if err != nil { |
41 | return err | 45 | return err |
@@ -43,13 +47,11 @@ func (c *zfsCollector) updateArcstats(ch chan<- prometheus.Metric) (err error) { | |||
43 | defer file.Close() | 47 | defer file.Close() |
44 | 48 | ||
45 | return c.parseArcstatsProcfsFile(file, func(s zfsSysctl, v zfsMetricValue) { | 49 | return c.parseArcstatsProcfsFile(file, func(s zfsSysctl, v zfsMetricValue) { |
46 | ch <- c.ConstSysctlMetric(arc, s, v) | 50 | ch <- c.constSysctlMetric(arc, s, v) |
47 | }) | 51 | }) |
48 | |||
49 | } | 52 | } |
50 | 53 | ||
51 | func (c *zfsCollector) parseArcstatsProcfsFile(reader io.Reader, handler func(zfsSysctl, zfsMetricValue)) (err error) { | 54 | func (c *zfsCollector) parseArcstatsProcfsFile(reader io.Reader, handler func(zfsSysctl, zfsMetricValue)) (err error) { |
52 | |||
53 | scanner := bufio.NewScanner(reader) | 55 | scanner := bufio.NewScanner(reader) |
54 | 56 | ||
55 | parseLine := false | 57 | parseLine := false |