From 244a4a9cdb6f6f9f5b84ca3e6a4508462cee63c6 Mon Sep 17 00:00:00 2001 From: "Matt T. Proud" Date: Tue, 14 May 2013 12:39:08 +0200 Subject: [PATCH] Update to go1.1. This commit updates the documentation, Makefiles, formatting, and code semantics to support the 1.1. runtime, which includes ... 1. ``make advice``, 2. ``make format``, and 3. ``go fix`` on various targets. --- .travis.yml | 4 ++-- Makefile.INCLUDE | 2 +- README.md | 6 +++--- model/fingerprinting.go | 6 +++--- retrieval/format/discriminator.go | 2 -- rules/load.go | 1 - rules/testdata.go | 2 +- storage/metric/tiered.go | 2 -- 8 files changed, 10 insertions(+), 15 deletions(-) diff --git a/.travis.yml b/.travis.yml index b1c6ffc8f..6a2b5e3bf 100644 --- a/.travis.yml +++ b/.travis.yml @@ -1,8 +1,8 @@ language: go before_script: -- gvm install go1.0.3 || true -- gvm use go1.0.3 || true +- gvm install go1.1 || true +- gvm use go1.1 || true script: - make diff --git a/Makefile.INCLUDE b/Makefile.INCLUDE index cb83b97a0..58062e8b0 100644 --- a/Makefile.INCLUDE +++ b/Makefile.INCLUDE @@ -25,7 +25,7 @@ else export THIRD_PARTY_BUILD_OUTPUT := endif -GO_VERSION := 1.0.3 +GO_VERSION := 1.1 LEVELDB_VERSION := 1.9.0 PROTOCOL_BUFFERS_VERSION := 2.5.0 SNAPPY_VERSION := 1.1.0 diff --git a/README.md b/README.md index 557726c86..6537ca3a5 100644 --- a/README.md +++ b/README.md @@ -18,7 +18,7 @@ action if some condition is observed to be true. If you read below in the _Getting Started_ section, the build infrastructure will take care of the following things for you in most cases: - 1. Go 1.0.3. + 1. Go 1.1. 2. GVM: [https://github.com/moovweb/gvm](https://github.com/moovweb/gvm) is highly recommended as well. 3. LevelDB: [https://code.google.com/p/leveldb/](https://code.google.com/p/leveldb/). 4. Protocol Buffers Compiler: [http://code.google.com/p/protobuf/](http://code.google.com/p/protobuf/). @@ -39,8 +39,8 @@ For basic help how to get started: ### Working with GVM Starting out, the following workflow is advised: - $ gvm install go1.0.3 - $ gvm use go1.0.3 + $ gvm install go1.1 + $ gvm use go1.1 $ gvm pkgset create prometheus $ gvm pkgset use prometheus diff --git a/model/fingerprinting.go b/model/fingerprinting.go index c18e5b875..123e1292a 100644 --- a/model/fingerprinting.go +++ b/model/fingerprinting.go @@ -105,9 +105,9 @@ func NewFingerprintFromMetric(metric Metric) (f Fingerprint) { return fingerprint{ firstCharacterOfFirstLabelName: firstCharacterOfFirstLabelName, - hash: binary.LittleEndian.Uint64(summer.Sum(nil)), - labelMatterLength: uint(labelMatterLength % 10), - lastCharacterOfLastLabelValue: lastCharacterOfLastLabelValue, + hash: binary.LittleEndian.Uint64(summer.Sum(nil)), + labelMatterLength: uint(labelMatterLength % 10), + lastCharacterOfLastLabelValue: lastCharacterOfLastLabelValue, } } diff --git a/retrieval/format/discriminator.go b/retrieval/format/discriminator.go index eabafc0af..5c7f8f8cf 100644 --- a/retrieval/format/discriminator.go +++ b/retrieval/format/discriminator.go @@ -71,6 +71,4 @@ func (r *registry) ProcessorForRequestHeader(header http.Header) (processor Proc err = fmt.Errorf("Unrecognized API version %s", prometheusApiVersion) return } - - return } diff --git a/rules/load.go b/rules/load.go index 690071c75..7bd988e2f 100644 --- a/rules/load.go +++ b/rules/load.go @@ -92,7 +92,6 @@ func LoadFromReader(rulesReader io.Reader, singleExpr bool) (interface{}, error) } else { return lexer.parsedRules, nil } - panic("") } func LoadRulesFromReader(rulesReader io.Reader) ([]Rule, error) { diff --git a/rules/testdata.go b/rules/testdata.go index 6a4f4780b..c1c43719b 100644 --- a/rules/testdata.go +++ b/rules/testdata.go @@ -143,7 +143,7 @@ var testMatrix = ast.Matrix{ { Metric: model.Metric{ model.MetricNameLabel: "x", - "y": "testvalue", + "y": "testvalue", }, Values: getTestValueStream(0, 100, 10), }, diff --git a/storage/metric/tiered.go b/storage/metric/tiered.go index 640344221..55b96591f 100644 --- a/storage/metric/tiered.go +++ b/storage/metric/tiered.go @@ -49,8 +49,6 @@ func (c chunk) TruncateBefore(t time.Time) chunk { default: return c[index-1:] } - - panic("unreachable") } // TieredStorage both persists samples and generates materialized views for