Go Report Card

Preparing report...

Report for github.com/kubernetes/heapster

A+    Excellent!    Found 98 issues across 153 files

Tweet
Last refresh:

Refresh now

gofmt100%

Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command

No problems detected. Good job!

go_vet100%

go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.

No problems detected. Good job!

gocyclo88%

Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.

    • heapster/metrics/sinks/stackdriver/stackdriver.go
    • Line 665: warning: cyclomatic complexity 35 of function (*StackdriverSink).TranslateMetric() is high (> 15) (gocyclo)
    • Line 311: warning: cyclomatic complexity 27 of function CreateStackdriverSink() is high (> 15) (gocyclo)
    • Line 126: warning: cyclomatic complexity 18 of function (*StackdriverSink).ExportData() is high (> 15) (gocyclo)

golint38%

Golint is a linter for Go source code.

    • heapster/common/kafka/glogadapter.go
    • Line 1: warning: package comment should be of the form "Package kafka ..." (golint)
    • Line 25: warning: exported type GologAdapterLogger should have comment or be unexported (golint)
    • Line 28: warning: exported method GologAdapterLogger.Print should have comment or be unexported (golint)
    • Line 32: warning: exported method GologAdapterLogger.Printf should have comment or be unexported (golint)
    • Line 36: warning: exported method GologAdapterLogger.Println should have comment or be unexported (golint)
    • heapster/common/kafka/kafka.go
    • Line 43: warning: exported const TimeSeriesTopic should have comment (or a comment on this block) or be unexported (golint)
    • Line 47: warning: exported type KafkaClient should have comment or be unexported (golint)
    • Line 184: warning: exported function NewKafkaClient should have comment or be unexported (golint)
    • heapster/common/elasticsearch/esVersionManager.go
    • Line 1: warning: package comment should be of the form "Package elasticsearch ..." (golint)
    • Line 26: warning: exported type UnsupportedVersion should have comment or be unexported (golint)
    • Line 41: warning: exported function NewMockClient should have comment or be unexported (golint)
    • Line 162: warning: don't use underscores in Go names; func bulkAfterCB_v2 should be bulkAfterCBV2 (golint)
    • heapster/metrics/processors/namespace_based_enricher.go
    • Line 32: warning: exported type NamespaceBasedEnricher should have comment or be unexported (golint)
    • Line 37: warning: exported method NamespaceBasedEnricher.Name should have comment or be unexported (golint)
    • Line 37: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 41: warning: exported method NamespaceBasedEnricher.Process should have comment or be unexported (golint)
    • Line 41: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 49: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 80: warning: exported function NewNamespaceBasedEnricher should have comment or be unexported (golint)
    • heapster/metrics/processors/pod_based_enricher.go
    • Line 29: warning: exported type PodBasedEnricher should have comment or be unexported (golint)
    • Line 34: warning: exported method PodBasedEnricher.Name should have comment or be unexported (golint)
    • Line 34: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 38: warning: exported method PodBasedEnricher.Process should have comment or be unexported (golint)
    • Line 38: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 68: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 79: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 132: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 238: warning: exported function NewPodBasedEnricher should have comment or be unexported (golint)
    • heapster/metrics/sources/summary/summary.go
    • Line 22: warning: should not use dot imports (golint)
    • Line 48: warning: comment on exported const VolumeResourcePrefix should be of the form "VolumeResourcePrefix ..." (golint)
    • Line 55: warning: exported type NodeInfo should have comment or be unexported (golint)
    • Line 69: warning: exported function NewSummaryMetricsSource should have comment or be unexported (golint)
    • Line 76: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 80: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 84: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 108: warning: exported const RootFsKey should have comment (or a comment on this block) or be unexported (golint)
    • Line 120: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 140: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 148: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 175: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 216: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 244: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 254: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 263: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 271: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 280: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 293: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 306: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 318: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 332: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 355: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 370: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 384: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 403: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 418: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 437: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 463: warning: exported function NewSummaryProvider should have comment or be unexported (golint)
    • heapster/metrics/sinks/hawkular/types.go
    • Line 25: warning: exported type Filter should have comment or be unexported (golint)
    • Line 26: warning: exported type FilterType should have comment or be unexported (golint)
    • Line 29: warning: comment on exported const Label should be of the form "Label ..." (golint)
    • Line 31: warning: comment on exported const Name should be of the form "Name ..." (golint)
    • Line 37: warning: exported method FilterType.From should have comment or be unexported (golint)
    • heapster/common/gce/gce.go
    • Line 35: warning: exported function EnsureOnGCE should have comment or be unexported (golint)
    • Line 45: warning: exported function GetProjectId should have comment or be unexported (golint)
    • heapster/metrics/sources/manager.go
    • Line 21: warning: should not use dot imports (golint)
    • Line 28: warning: exported const DefaultMetricsScrapeTimeout should have comment (or a comment on this block) or be unexported (golint)
    • Line 62: warning: exported function NewSourceManager should have comment or be unexported (golint)
    • Line 74: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 78: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • heapster/version/version.go
    • Line 19: warning: comment on exported var HeapsterVersion should be of the form "HeapsterVersion ..." (golint)
    • Line 26: warning: comment on exported var GitCommit should be of the form "GitCommit ..." (golint)
    • Line 29: warning: exported function VersionInfo should have comment or be unexported (golint)
    • heapster/events/util/dummies.go
    • Line 24: warning: exported type DummySink should have comment or be unexported (golint)
    • Line 32: warning: exported method DummySink.Name should have comment or be unexported (golint)
    • Line 32: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 35: warning: exported method DummySink.ExportEvents should have comment or be unexported (golint)
    • Line 35: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 43: warning: exported method DummySink.Stop should have comment or be unexported (golint)
    • Line 43: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 51: warning: exported method DummySink.IsStopped should have comment or be unexported (golint)
    • Line 51: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 57: warning: exported method DummySink.GetExportCount should have comment or be unexported (golint)
    • Line 57: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 63: warning: exported function NewDummySink should have comment or be unexported (golint)
    • Line 72: warning: exported type DummyEventSource should have comment or be unexported (golint)
    • Line 76: warning: exported method DummyEventSource.GetNewEvents should have comment or be unexported (golint)
    • Line 76: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 80: warning: exported function NewDummySource should have comment or be unexported (golint)
    • heapster/common/influxdb/dummy_influxdb.go
    • Line 24: warning: exported type PointSavedToInfluxdb should have comment or be unexported (golint)
    • Line 28: warning: exported type FakeInfluxDBClient should have comment or be unexported (golint)
    • Line 32: warning: exported function NewFakeInfluxDBClient should have comment or be unexported (golint)
    • Line 43: warning: exported method FakeInfluxDBClient.Query should have comment or be unexported (golint)
    • Line 52: warning: exported method FakeInfluxDBClient.Ping should have comment or be unexported (golint)
    • Line 56: warning: exported var Client should have comment or be unexported (golint)
    • Line 58: warning: exported var Config should have comment or be unexported (golint)
    • heapster/metrics/storage/nodemetrics/reststorage.go
    • Line 37: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
    • Line 40: warning: exported type MetricStorage should have comment or be unexported (golint)
    • Line 51: warning: exported function NewStorage should have comment or be unexported (golint)
    • Line 59: warning: comment on exported method MetricStorage.New should be of the form "New ..." (golint)
    • Line 64: warning: comment on exported method MetricStorage.Kind should be of the form "Kind ..." (golint)
    • Line 69: warning: comment on exported method MetricStorage.NewList should be of the form "NewList ..." (golint)
    • Line 74: warning: comment on exported method MetricStorage.List should be of the form "List ..." (golint)
    • Line 101: warning: comment on exported method MetricStorage.Get should be of the form "Get ..." (golint)
    • heapster/metrics/sources/factory.go
    • Line 26: warning: exported type SourceFactory should have comment or be unexported (golint)
    • Line 29: warning: exported method SourceFactory.Build should have comment or be unexported (golint)
    • Line 29: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 42: warning: exported method SourceFactory.BuildAll should have comment or be unexported (golint)
    • Line 42: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 49: warning: exported function NewSourceFactory should have comment or be unexported (golint)
    • heapster/common/influxdb/influxdb.go
    • Line 29: warning: exported type InfluxdbClient should have comment or be unexported (golint)
    • Line 35: warning: exported type InfluxdbConfig should have comment or be unexported (golint)
    • Line 49: warning: exported function NewClient should have comment or be unexported (golint)
    • Line 76: warning: exported function BuildConfig should have comment or be unexported (golint)
    • heapster/metrics/core/labels.go
    • Line 20: warning: exported var LabelMetricSetType should have comment or be unexported (golint)
    • Line 75: warning: comment on exported var LabelCustomMetricName should be of the form "LabelCustomMetricName ..." (golint)
    • Line 110: warning: exported type LabelDescriptor should have comment or be unexported (golint)
    • Line 169: warning: exported function CommonLabels should have comment or be unexported (golint)
    • Line 175: warning: exported function ContainerLabels should have comment or be unexported (golint)
    • Line 181: warning: exported function PodLabels should have comment or be unexported (golint)
    • Line 187: warning: exported function MetricLabels should have comment or be unexported (golint)
    • Line 194: warning: exported function SupportedLabels should have comment or be unexported (golint)
    • Line 200: warning: exported function GcmLabels should have comment or be unexported (golint)
    • Line 207: warning: exported function GcmNodeAutoscalingLabels should have comment or be unexported (golint)
    • heapster/metrics/sinks/statsd/formatter.go
    • Line 24: warning: exported type Formatter should have comment or be unexported (golint)
    • Line 28: warning: exported type LabelStyle should have comment or be unexported (golint)
    • Line 29: warning: exported type CustomizeLabel should have comment or be unexported (golint)
    • Line 31: warning: exported type LabelCustomizer should have comment or be unexported (golint)
    • Line 36: warning: exported method LabelCustomizer.Customize should have comment or be unexported (golint)
    • Line 43: warning: exported function NewFormatter should have comment or be unexported (golint)
    • Line 54: warning: exported function DefaultLabelStyle should have comment or be unexported (golint)
    • Line 58: warning: exported function SnakeToLowerCamel should have comment or be unexported (golint)
    • Line 68: warning: exported function SnakeToUpperCamel should have comment or be unexported (golint)
    • heapster/metrics/sinks/riemann/driver.go
    • Line 27: warning: comment on exported type RiemannSink should be of the form "RiemannSink ..." (with optional leading article) (golint)
    • Line 34: warning: comment on exported function CreateRiemannSink should be of the form "CreateRiemannSink ..." (golint)
    • Line 48: warning: comment on exported method RiemannSink.Name should be of the form "Name ..." (golint)
    • Line 53: warning: exported method RiemannSink.Stop should have comment or be unexported (golint)
    • heapster/metrics/manager/manager.go
    • Line 27: warning: exported const DefaultScrapeOffset should have comment (or a comment on this block) or be unexported (golint)
    • Line 48: warning: exported type Manager should have comment or be unexported (golint)
    • Line 64: warning: exported function NewManager should have comment or be unexported (golint)
    • heapster/common/honeycomb/dummy_honeycomb.go
    • Line 17: warning: exported type BatchPointsSavedToHoneycomb should have comment or be unexported (golint)
    • Line 21: warning: exported type FakeHoneycombClient should have comment or be unexported (golint)
    • Line 25: warning: exported function NewFakeHoneycombClient should have comment or be unexported (golint)
    • Line 29: warning: exported method FakeHoneycombClient.SendBatch should have comment or be unexported (golint)
    • Line 36: warning: exported var Config should have comment or be unexported (golint)
    • heapster/common/honeycomb/honeycomb.go
    • Line 41: warning: exported function BuildConfig should have comment or be unexported (golint)
    • Line 69: warning: exported type Client should have comment or be unexported (golint)
    • Line 73: warning: exported type HoneycombClient should have comment or be unexported (golint)
    • Line 78: warning: exported function NewClient should have comment or be unexported (golint)
    • Line 86: warning: exported type BatchPoint should have comment or be unexported (golint)
    • Line 91: warning: exported type Batch should have comment or be unexported (golint)
    • heapster/metrics/sinks/graphite/driver.go
    • Line 32: warning: exported const DefaultHost should have comment (or a comment on this block) or be unexported (golint)
    • Line 122: warning: exported type Sink should have comment or be unexported (golint)
    • Line 127: warning: exported function NewGraphiteSink should have comment or be unexported (golint)
    • Line 154: warning: exported method Sink.Name should have comment or be unexported (golint)
    • Line 158: warning: exported method Sink.ExportData should have comment or be unexported (golint)
    • Line 201: warning: exported method Sink.Stop should have comment or be unexported (golint)
    • heapster/metrics/sources/kubelet/kubelet.go
    • Line 24: warning: should not use dot imports (golint)
    • Line 74: warning: exported function NewKubeletMetricsSource should have comment or be unexported (golint)
    • Line 85: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 89: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 93: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 104: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 121: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 236: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 261: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 275: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 313: warning: exported function GetNodeHostnameAndIP should have comment or be unexported (golint)
    • Line 339: warning: exported function NewKubeletProvider should have comment or be unexported (golint)
    • heapster/metrics/options/options.go
    • Line 26: warning: exported type HeapsterRunOptions should have comment or be unexported (golint)
    • Line 57: warning: exported function NewHeapsterRunOptions should have comment or be unexported (golint)
    • Line 66: warning: exported method HeapsterRunOptions.AddFlags should have comment or be unexported (golint)
    • heapster/metrics/sinks/manager.go
    • Line 27: warning: exported const DefaultSinkStopTimeout should have comment (or a comment on this block) or be unexported (golint)
    • Line 74: warning: exported function NewDataSinkManager should have comment or be unexported (golint)
    • Line 106: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 126: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 130: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • heapster/metrics/sinks/influxdb/influxdb_historical.go
    • Line 153: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 159: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 189: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 229: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 396: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 546: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • heapster/metrics/util/dummies.go
    • Line 24: warning: exported type DummySink should have comment or be unexported (golint)
    • Line 32: warning: exported method DummySink.Name should have comment or be unexported (golint)
    • Line 32: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 35: warning: exported method DummySink.ExportData should have comment or be unexported (golint)
    • Line 35: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 43: warning: exported method DummySink.Stop should have comment or be unexported (golint)
    • Line 43: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 51: warning: exported method DummySink.IsStopped should have comment or be unexported (golint)
    • Line 51: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 57: warning: exported method DummySink.GetExportCount should have comment or be unexported (golint)
    • Line 57: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 63: warning: exported function NewDummySink should have comment or be unexported (golint)
    • Line 72: warning: exported type DummyMetricsSource should have comment or be unexported (golint)
    • Line 77: warning: exported method DummyMetricsSource.Name should have comment or be unexported (golint)
    • Line 77: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 81: warning: exported method DummyMetricsSource.ScrapeMetrics should have comment or be unexported (golint)
    • Line 81: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 100: warning: exported function NewDummyMetricsSource should have comment or be unexported (golint)
    • Line 107: warning: exported type DummyMetricsSourceProvider should have comment or be unexported (golint)
    • Line 111: warning: exported method DummyMetricsSourceProvider.GetMetricsSources should have comment or be unexported (golint)
    • Line 111: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 115: warning: exported function NewDummyMetricsSourceProvider should have comment or be unexported (golint)
    • Line 121: warning: exported type DummyDataProcessor should have comment or be unexported (golint)
    • Line 125: warning: exported method DummyDataProcessor.Name should have comment or be unexported (golint)
    • Line 125: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 129: warning: exported method DummyDataProcessor.Process should have comment or be unexported (golint)
    • Line 129: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 134: warning: exported function NewDummyDataProcessor should have comment or be unexported (golint)
    • heapster/metrics/cmd/heapster-apiserver/app/server.go
    • Line 31: warning: exported type HeapsterAPIServer should have comment or be unexported (golint)
    • Line 38: warning: comment on exported method HeapsterAPIServer.RunServer should be of the form "RunServer ..." (golint)
    • Line 44: warning: exported function NewHeapsterApiServer should have comment or be unexported (golint)
    • heapster/events/sinks/riemann/driver.go
    • Line 30: warning: comment on exported type RiemannSink should be of the form "RiemannSink ..." (with optional leading article) (golint)
    • Line 37: warning: comment on exported function CreateRiemannSink should be of the form "CreateRiemannSink ..." (golint)
    • Line 51: warning: comment on exported method RiemannSink.Name should be of the form "Name ..." (golint)
    • Line 56: warning: exported method RiemannSink.Stop should have comment or be unexported (golint)
    • Line 115: warning: exported method RiemannSink.ExportEvents should have comment or be unexported (golint)
    • heapster/metrics/sinks/factory.go
    • Line 41: warning: exported type SinkFactory should have comment or be unexported (golint)
    • Line 44: warning: exported method SinkFactory.Build should have comment or be unexported (golint)
    • Line 44: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 83: warning: exported method SinkFactory.BuildAll should have comment or be unexported (golint)
    • Line 83: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 127: warning: exported function NewSinkFactory should have comment or be unexported (golint)
    • heapster/metrics/processors/rate_calculator.go
    • Line 23: warning: exported type RateCalculator should have comment or be unexported (golint)
    • Line 28: warning: exported method RateCalculator.Name should have comment or be unexported (golint)
    • Line 28: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 32: warning: exported method RateCalculator.Process should have comment or be unexported (golint)
    • Line 32: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 131: warning: exported function NewRateCalculator should have comment or be unexported (golint)
    • heapster/metrics/api/v1/api.go
    • Line 28: warning: exported type Api should have comment or be unexported (golint)
    • Line 41: warning: comment on exported function NewApi should be of the form "NewApi ..." (golint)
    • Line 203: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • heapster/metrics/sinks/stackdriver/stackdriver.go
    • Line 47: warning: exported type StackdriverSink should have comment or be unexported (golint)
    • Line 99: warning: exported method StackdriverSink.Name should have comment or be unexported (golint)
    • Line 103: warning: exported method StackdriverSink.Stop should have comment or be unexported (golint)
    • Line 126: warning: exported method StackdriverSink.ExportData should have comment or be unexported (golint)
    • Line 311: warning: exported function CreateStackdriverSink should have comment or be unexported (golint)
    • Line 334: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 489: warning: exported method StackdriverSink.LegacyTranslateLabeledMetric should have comment or be unexported (golint)
    • Line 537: warning: exported method StackdriverSink.LegacyTranslateMetric should have comment or be unexported (golint)
    • Line 610: warning: exported method StackdriverSink.TranslateLabeledMetric should have comment or be unexported (golint)
    • Line 665: warning: exported method StackdriverSink.TranslateMetric should have comment or be unexported (golint)
    • heapster/metrics/api/v1/historical_handlers.go
    • Line 771: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 784: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • heapster/common/librato/dummy_librato.go
    • Line 17: warning: exported type MeasurementsSavedToLibrato should have comment or be unexported (golint)
    • Line 21: warning: exported type FakeLibratoClient should have comment or be unexported (golint)
    • Line 25: warning: exported function NewFakeLibratoClient should have comment or be unexported (golint)
    • Line 36: warning: exported var FakeClient should have comment or be unexported (golint)
    • Line 38: warning: exported var Config should have comment or be unexported (golint)
    • heapster/metrics/storage/podmetrics/reststorage.go
    • Line 37: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
    • Line 40: warning: exported type MetricStorage should have comment or be unexported (golint)
    • Line 51: warning: exported function NewStorage should have comment or be unexported (golint)
    • Line 59: warning: comment on exported method MetricStorage.New should be of the form "New ..." (golint)
    • Line 64: warning: comment on exported method MetricStorage.Kind should be of the form "Kind ..." (golint)
    • Line 69: warning: comment on exported method MetricStorage.NewList should be of the form "NewList ..." (golint)
    • Line 74: warning: comment on exported method MetricStorage.List should be of the form "List ..." (golint)
    • Line 99: warning: comment on exported method MetricStorage.Get should be of the form "Get ..." (golint)
    • heapster/events/manager/manager.go
    • Line 35: warning: comment on exported var LatestScrapeTime should be of the form "LatestScrapeTime ..." (golint)
    • Line 43: warning: exported type Manager should have comment or be unexported (golint)
    • Line 55: warning: exported function NewManager should have comment or be unexported (golint)
    • heapster/metrics/core/metrics.go
    • Line 26: warning: exported const CustomMetricPrefix should have comment (or a comment on this block) or be unexported (golint)
    • Line 29: warning: comment on exported var StandardMetrics should be of the form "StandardMetrics ..." (golint)
    • Line 46: warning: comment on exported var AdditionalMetrics should be of the form "AdditionalMetrics ..." (golint)
    • Line 55: warning: comment on exported var RateMetrics should be of the form "RateMetrics ..." (golint)
    • Line 67: warning: exported var RateMetricsMapping should have comment or be unexported (golint)
    • Line 78: warning: exported var LabeledMetrics should have comment or be unexported (golint)
    • Line 93: warning: exported var NodeAutoscalingMetrics should have comment or be unexported (golint)
    • Line 108: warning: exported var CpuMetrics should have comment or be unexported (golint)
    • Line 119: warning: exported var FilesystemMetrics should have comment or be unexported (golint)
    • Line 126: warning: exported var MemoryMetrics should have comment or be unexported (golint)
    • Line 142: warning: exported var NetworkMetrics should have comment or be unexported (golint)
    • Line 153: warning: comment on exported var ResourceRequestMetrics should be of the form "ResourceRequestMetrics ..." (golint)
    • Line 162: warning: exported type MetricFamily should have comment or be unexported (golint)
    • Line 165: warning: exported const MetricFamilyCpu should have comment (or a comment on this block) or be unexported (golint)
    • Line 172: warning: exported var MetricFamilies should have comment or be unexported (golint)
    • Line 179: warning: exported function MetricFamilyForName should have comment or be unexported (golint)
    • Line 190: warning: exported var AllMetrics should have comment or be unexported (golint)
    • Line 193: warning: comment on exported var MetricUptime should be of the form "MetricUptime ..." (golint)
    • Line 213: warning: exported var MetricRestartCount should have comment or be unexported (golint)
    • Line 223: warning: exported var MetricCpuLoad should have comment or be unexported (golint)
    • Line 242: warning: exported var MetricCpuUsage should have comment or be unexported (golint)
    • Line 261: warning: exported var MetricEphemeralStorageUsage should have comment or be unexported (golint)
    • Line 270: warning: exported var MetricMemoryUsage should have comment or be unexported (golint)
    • Line 289: warning: exported var MetricMemoryCache should have comment or be unexported (golint)
    • Line 308: warning: exported var MetricMemoryRSS should have comment or be unexported (golint)
    • Line 327: warning: exported var MetricMemoryWorkingSet should have comment or be unexported (golint)
    • Line 346: warning: exported var MetricMemoryPageFaults should have comment or be unexported (golint)
    • Line 365: warning: exported var MetricMemoryMajorPageFaults should have comment or be unexported (golint)
    • Line 384: warning: exported var MetricNetworkRx should have comment or be unexported (golint)
    • Line 408: warning: exported var MetricNetworkRxErrors should have comment or be unexported (golint)
    • Line 432: warning: exported var MetricNetworkTx should have comment or be unexported (golint)
    • Line 456: warning: exported var MetricNetworkTxErrors should have comment or be unexported (golint)
    • Line 480: warning: comment on exported var MetricCpuRequest should be of the form "MetricCpuRequest ..." (golint)
    • Line 491: warning: exported var MetricCpuLimit should have comment or be unexported (golint)
    • Line 501: warning: exported var MetricMemoryRequest should have comment or be unexported (golint)
    • Line 511: warning: exported var MetricMemoryLimit should have comment or be unexported (golint)
    • Line 521: warning: exported var MetricEphemeralStorageRequest should have comment or be unexported (golint)
    • Line 531: warning: exported var MetricEphemeralStorageLimit should have comment or be unexported (golint)
    • Line 541: warning: comment on exported var MetricCpuUsageRate should be of the form "MetricCpuUsageRate ..." (golint)
    • Line 552: warning: exported var MetricMemoryPageFaultsRate should have comment or be unexported (golint)
    • Line 562: warning: exported var MetricMemoryMajorPageFaultsRate should have comment or be unexported (golint)
    • Line 572: warning: exported var MetricNetworkRxRate should have comment or be unexported (golint)
    • Line 582: warning: exported var MetricNetworkRxErrorsRate should have comment or be unexported (golint)
    • Line 592: warning: exported var MetricNetworkTxRate should have comment or be unexported (golint)
    • Line 602: warning: exported var MetricNetworkTxErrorsRate should have comment or be unexported (golint)
    • Line 612: warning: exported var MetricNodeCpuCapacity should have comment or be unexported (golint)
    • Line 622: warning: exported var MetricNodeMemoryCapacity should have comment or be unexported (golint)
    • Line 632: warning: exported var MetricNodeEphemeralStorageCapacity should have comment or be unexported (golint)
    • Line 642: warning: exported var MetricNodeCpuAllocatable should have comment or be unexported (golint)
    • Line 652: warning: exported var MetricNodeMemoryAllocatable should have comment or be unexported (golint)
    • Line 662: warning: exported var MetricNodeEphemeralStorageAllocatable should have comment or be unexported (golint)
    • Line 672: warning: exported var MetricNodeCpuUtilization should have comment or be unexported (golint)
    • Line 682: warning: exported var MetricNodeMemoryUtilization should have comment or be unexported (golint)
    • Line 692: warning: exported var MetricNodeEphemeralStorageUtilization should have comment or be unexported (golint)
    • Line 702: warning: exported var MetricNodeCpuReservation should have comment or be unexported (golint)
    • Line 712: warning: exported var MetricNodeMemoryReservation should have comment or be unexported (golint)
    • Line 722: warning: exported var MetricNodeEphemeralStorageReservation should have comment or be unexported (golint)
    • Line 734: warning: exported var MetricFilesystemUsage should have comment or be unexported (golint)
    • Line 765: warning: exported var MetricFilesystemLimit should have comment or be unexported (golint)
    • Line 796: warning: exported var MetricFilesystemAvailable should have comment or be unexported (golint)
    • Line 827: warning: exported var MetricFilesystemInodes should have comment or be unexported (golint)
    • Line 860: warning: exported var MetricFilesystemInodesFree should have comment or be unexported (golint)
    • Line 893: warning: exported var MetricAcceleratorMemoryTotal should have comment or be unexported (golint)
    • Line 930: warning: exported var MetricAcceleratorMemoryUsed should have comment or be unexported (golint)
    • Line 967: warning: exported var MetricAcceleratorDutyCycle should have comment or be unexported (golint)
    • Line 1004: warning: exported var MetricDiskIORead should have comment or be unexported (golint)
    • Line 1045: warning: exported var MetricDiskIOWrite should have comment or be unexported (golint)
    • Line 1086: warning: exported var MetricDiskIOReadRate should have comment or be unexported (golint)
    • Line 1097: warning: exported var MetricDiskIOWriteRate should have comment or be unexported (golint)
    • Line 1108: warning: exported function IsNodeAutoscalingMetric should have comment or be unexported (golint)
    • Line 1117: warning: exported type MetricDescriptor should have comment or be unexported (golint)
    • heapster/metrics/processors/namespace_aggregator.go
    • Line 22: warning: exported type NamespaceAggregator should have comment or be unexported (golint)
    • Line 26: warning: exported method NamespaceAggregator.Name should have comment or be unexported (golint)
    • Line 26: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 30: warning: exported method NamespaceAggregator.Process should have comment or be unexported (golint)
    • Line 30: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • heapster/common/librato/librato.go
    • Line 28: warning: exported type Measurement should have comment or be unexported (golint)
    • Line 40: warning: exported type Client should have comment or be unexported (golint)
    • Line 44: warning: exported type LibratoClient should have comment or be unexported (golint)
    • Line 72: warning: exported type LibratoConfig should have comment or be unexported (golint)
    • Line 80: warning: exported function NewClient should have comment or be unexported (golint)
    • Line 96: warning: exported function BuildConfig should have comment or be unexported (golint)
    • heapster/events/core/types.go
    • Line 23: warning: exported type EventBatch should have comment or be unexported (golint)
    • Line 30: warning: comment on exported type EventSource should be of the form "EventSource ..." (with optional leading article) (golint)
    • Line 37: warning: exported type EventSink should have comment or be unexported (golint)
    • heapster/metrics/core/historical_types.go
    • Line 1: warning: package comment should be of the form "Package core ..." (golint)
    • Line 21: warning: exported type AggregationType should have comment or be unexported (golint)
    • Line 24: warning: exported var AggregationTypeAverage should have comment or be unexported (golint)
    • heapster/metrics/core/types.go
    • Line 21: warning: exported type MetricType should have comment or be unexported (golint)
    • Line 24: warning: exported const MetricCumulative should have comment (or a comment on this block) or be unexported (golint)
    • Line 29: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 41: warning: exported type ValueType should have comment or be unexported (golint)
    • Line 44: warning: exported const ValueInt64 should have comment (or a comment on this block) or be unexported (golint)
    • Line 48: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 58: warning: exported type UnitsType should have comment or be unexported (golint)
    • Line 61: warning: comment on exported const UnitsCount should be of the form "UnitsCount ..." (golint)
    • Line 63: warning: comment on exported const UnitsBytes should be of the form "UnitsBytes ..." (golint)
    • Line 65: warning: comment on exported const UnitsMilliseconds should be of the form "UnitsMilliseconds ..." (golint)
    • Line 67: warning: comment on exported const UnitsNanoseconds should be of the form "UnitsNanoseconds ..." (golint)
    • Line 69: warning: comment on exported const UnitsMillicores should be of the form "UnitsMillicores ..." (golint)
    • Line 73: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 87: warning: exported type MetricValue should have comment or be unexported (golint)
    • Line 94: warning: exported method MetricValue.GetValue should have comment or be unexported (golint)
    • Line 94: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 104: warning: exported type LabeledMetric should have comment or be unexported (golint)
    • Line 110: warning: exported method LabeledMetric.GetValue should have comment or be unexported (golint)
    • Line 110: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 120: warning: exported type MetricSet should have comment or be unexported (golint)
    • Line 134: warning: exported type DataBatch should have comment or be unexported (golint)
    • Line 140: warning: comment on exported type MetricsSource should be of the form "MetricsSource ..." (with optional leading article) (golint)
    • Line 146: warning: comment on exported type MetricsSourceProvider should be of the form "MetricsSourceProvider ..." (with optional leading article) (golint)
    • Line 151: warning: exported type DataSink should have comment or be unexported (golint)
    • Line 161: warning: exported type DataProcessor should have comment or be unexported (golint)
    • heapster/metrics/sources/kubelet/kubelet_client.go
    • Line 37: warning: exported type Host should have comment or be unexported (golint)
    • Line 47: warning: exported type KubeletClient should have comment or be unexported (golint)
    • Line 52: warning: exported type ErrNotFound should have comment or be unexported (golint)
    • Line 60: warning: exported function IsNotFoundError should have comment or be unexported (golint)
    • Line 72: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 101: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 133: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 136: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 141: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 151: warning: comment on exported method KubeletClient.GetAllRawContainers should be of the form "GetAllRawContainers ..." (golint)
    • Line 152: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 158: warning: exported method KubeletClient.GetSummary should have comment or be unexported (golint)
    • Line 158: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 174: warning: exported method KubeletClient.GetPort should have comment or be unexported (golint)
    • Line 174: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 178: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 216: warning: exported function NewKubeletClient should have comment or be unexported (golint)
    • heapster/metrics/api/v1/types/model_types.go
    • Line 21: warning: exported type MetricPoint should have comment or be unexported (golint)
    • Line 30: warning: exported type MetricResult should have comment or be unexported (golint)
    • Line 35: warning: exported type MetricResultList should have comment or be unexported (golint)
    • Line 39: warning: exported type Stats should have comment or be unexported (golint)
    • Line 45: warning: exported type ExternalStatBundle should have comment or be unexported (golint)
    • Line 51: warning: exported type StatsResponse should have comment or be unexported (golint)
    • heapster/events/sinks/log/log_sink.go
    • Line 25: warning: exported type LogSink should have comment or be unexported (golint)
    • Line 28: warning: exported method LogSink.Name should have comment or be unexported (golint)
    • Line 28: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 32: warning: exported method LogSink.Stop should have comment or be unexported (golint)
    • Line 32: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 45: warning: exported method LogSink.ExportEvents should have comment or be unexported (golint)
    • Line 45: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 49: warning: exported function CreateLogSink should have comment or be unexported (golint)
    • heapster/common/riemann/riemann.go
    • Line 28: warning: comment on exported type RiemannConfig should be of the form "RiemannConfig ..." (with optional leading article) (golint)
    • Line 37: warning: comment on exported type RiemannSink should be of the form "RiemannSink ..." (with optional leading article) (golint)
    • Line 44: warning: comment on exported function CreateRiemannSink should be of the form "CreateRiemannSink ..." (golint)
    • Line 101: warning: comment on exported function GetRiemannClient should be of the form "GetRiemannClient ..." (golint)
    • Line 114: warning: comment on exported function SendData should be of the form "SendData ..." (golint)
    • Line 127: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • heapster/metrics/processors/cluster_aggregator.go
    • Line 19: warning: exported type ClusterAggregator should have comment or be unexported (golint)
    • Line 23: warning: exported method ClusterAggregator.Name should have comment or be unexported (golint)
    • Line 23: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 27: warning: exported method ClusterAggregator.Process should have comment or be unexported (golint)
    • Line 27: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • heapster/events/sources/kubernetes/kubernetes_source.go
    • Line 34: warning: comment on exported const LocalEventsBufferSize should be of the form "LocalEventsBufferSize ..." (golint)
    • Line 69: warning: comment on exported type KubernetesEventSource should be of the form "KubernetesEventSource ..." (with optional leading article) (golint)
    • Line 79: warning: exported method KubernetesEventSource.GetNewEvents should have comment or be unexported (golint)
    • Line 79: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 105: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 175: warning: exported function NewKubernetesSource should have comment or be unexported (golint)
    • heapster/common/elasticsearch/elasticsearch.go
    • Line 1: warning: package comment should be of the form "Package elasticsearch ..." (golint)
    • Line 32: warning: exported const ESIndex should have comment (or a comment on this block) or be unexported (golint)
    • Line 36: warning: exported type ElasticSearchService should have comment or be unexported (golint)
    • Line 42: warning: exported method ElasticSearchService.Index should have comment or be unexported (golint)
    • Line 45: warning: exported method ElasticSearchService.IndexAlias should have comment or be unexported (golint)
    • Line 49: warning: exported method ElasticSearchService.FlushData should have comment or be unexported (golint)
    • Line 53: warning: comment on exported method ElasticSearchService.SaveData should be of the form "SaveData ..." (golint)
    • Line 124: warning: comment on exported function CreateElasticSearchService should be of the form "CreateElasticSearchService ..." (golint)
    • heapster/metrics/sinks/metric/metric_sink.go
    • Line 24: warning: comment on exported type MetricSink should be of the form "MetricSink ..." (with optional leading article) (golint)
    • Line 73: warning: exported method MetricSink.Name should have comment or be unexported (golint)
    • Line 73: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 77: warning: exported method MetricSink.Stop should have comment or be unexported (golint)
    • Line 77: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 81: warning: exported method MetricSink.ExportData should have comment or be unexported (golint)
    • Line 81: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 92: warning: exported method MetricSink.GetLatestDataBatch should have comment or be unexported (golint)
    • Line 92: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 102: warning: exported method MetricSink.GetShortStore should have comment or be unexported (golint)
    • Line 102: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 113: warning: exported method MetricSink.GetMetric should have comment or be unexported (golint)
    • Line 113: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 173: warning: exported method MetricSink.GetLabeledMetric should have comment or be unexported (golint)
    • Line 173: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 216: warning: exported method MetricSink.GetMetricNames should have comment or be unexported (golint)
    • Line 216: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 235: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 253: warning: comment on exported method MetricSink.GetMetricSetKeys should be of the form "GetMetricSetKeys ..." (golint)
    • Line 256: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 262: warning: exported method MetricSink.GetNodes should have comment or be unexported (golint)
    • Line 262: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 268: warning: exported method MetricSink.GetPods should have comment or be unexported (golint)
    • Line 268: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 276: warning: exported method MetricSink.GetNamespaces should have comment or be unexported (golint)
    • Line 276: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 284: warning: exported method MetricSink.GetPodsFromNamespace should have comment or be unexported (golint)
    • Line 284: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 295: warning: exported method MetricSink.GetContainersForPodFromNamespace should have comment or be unexported (golint)
    • Line 295: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 307: warning: exported method MetricSink.GetSystemContainersFromNode should have comment or be unexported (golint)
    • Line 307: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 338: warning: exported function NewMetricSink should have comment or be unexported (golint)
    • heapster/metrics/core/ms_keys.go
    • Line 26: warning: exported function PodContainerKey should have comment or be unexported (golint)
    • Line 30: warning: exported function PodKey should have comment or be unexported (golint)
    • Line 34: warning: exported function NamespaceKey should have comment or be unexported (golint)
    • Line 38: warning: exported function NodeKey should have comment or be unexported (golint)
    • Line 42: warning: exported function NodeContainerKey should have comment or be unexported (golint)
    • Line 46: warning: exported function ClusterKey should have comment or be unexported (golint)
    • heapster/integration/heapster_api_test.go
    • Line 92: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 101: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • heapster/metrics/processors/pod_aggregator.go
    • Line 25: warning: exported var LabelsToPopulate should have comment or be unexported (golint)
    • Line 34: warning: exported type PodAggregator should have comment or be unexported (golint)
    • Line 38: warning: exported method PodAggregator.Name should have comment or be unexported (golint)
    • Line 38: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 42: warning: exported method PodAggregator.Process should have comment or be unexported (golint)
    • Line 42: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 107: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 122: warning: exported function NewPodAggregator should have comment or be unexported (golint)
    • heapster/events/sources/factory.go
    • Line 27: warning: exported type SourceFactory should have comment or be unexported (golint)
    • Line 30: warning: exported method SourceFactory.Build should have comment or be unexported (golint)
    • Line 30: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 40: warning: exported method SourceFactory.BuildAll should have comment or be unexported (golint)
    • Line 40: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 56: warning: exported function NewSourceFactory should have comment or be unexported (golint)
    • heapster/events/sinks/manager.go
    • Line 27: warning: exported const DefaultSinkExportEventsTimeout should have comment (or a comment on this block) or be unexported (golint)
    • Line 64: warning: exported function NewEventSinkManager should have comment or be unexported (golint)
    • Line 96: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 116: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 120: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • heapster/common/kubernetes/configs.go
    • Line 30: warning: exported const APIVersion should have comment (or a comment on this block) or be unexported (golint)
    • Line 60: warning: exported function GetKubeClientConfig should have comment or be unexported (golint)
    • heapster/metrics/sinks/elasticsearch/driver.go
    • Line 38: warning: exported type EsFamilyPoints should have comment or be unexported (golint)
    • Line 42: warning: exported type EsSinkPointGeneral should have comment or be unexported (golint)
    • Line 48: warning: exported type EsSinkPointFamily should have comment or be unexported (golint)
    • Line 142: warning: exported function EsPointValue should have comment or be unexported (golint)
    • Line 156: warning: exported function NewElasticSearchSink should have comment or be unexported (golint)
    • heapster/common/flags/flags.go
    • Line 25: warning: exported type Uri should have comment or be unexported (golint)
    • Line 38: warning: exported method Uri.Set should have comment or be unexported (golint)
    • Line 55: warning: exported type Uris should have comment or be unexported (golint)
    • Line 70: warning: exported method Uris.Set should have comment or be unexported (golint)
    • Line 79: warning: exported method Uris.Type should have comment or be unexported (golint)
    • heapster/events/sinks/factory.go
    • Line 33: warning: exported type SinkFactory should have comment or be unexported (golint)
    • Line 36: warning: exported method SinkFactory.Build should have comment or be unexported (golint)
    • Line 36: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 57: warning: exported method SinkFactory.BuildAll should have comment or be unexported (golint)
    • Line 57: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 70: warning: exported function NewSinkFactory should have comment or be unexported (golint)
    • heapster/integration/framework.go
    • Line 366: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 370: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 379: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 388: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 402: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 416: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 422: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 436: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 441: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 453: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 459: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 481: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 485: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 490: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 494: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 498: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 510: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 514: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 518: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 543: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • heapster/metrics/processors/node_aggregator.go
    • Line 22: warning: comment on exported type NodeAggregator should be of the form "NodeAggregator ..." (with optional leading article) (golint)
    • Line 27: warning: exported method NodeAggregator.Name should have comment or be unexported (golint)
    • Line 27: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 31: warning: exported method NodeAggregator.Process should have comment or be unexported (golint)
    • Line 31: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • heapster/metrics/processors/node_autoscaling_enricher.go
    • Line 30: warning: exported type NodeAutoscalingEnricher should have comment or be unexported (golint)
    • Line 36: warning: exported method NodeAutoscalingEnricher.Name should have comment or be unexported (golint)
    • Line 36: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 40: warning: exported method NodeAutoscalingEnricher.Process should have comment or be unexported (golint)
    • Line 40: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 104: warning: exported function NewNodeAutoscalingEnricher should have comment or be unexported (golint)
    • heapster/metrics/sinks/log/log_sink.go
    • Line 26: warning: exported type LogSink should have comment or be unexported (golint)
    • Line 29: warning: exported method LogSink.Name should have comment or be unexported (golint)
    • Line 29: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 33: warning: exported method LogSink.Stop should have comment or be unexported (golint)
    • Line 33: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 80: warning: exported method LogSink.ExportData should have comment or be unexported (golint)
    • Line 80: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 84: warning: exported function NewLogSink should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign96%

IneffAssign detects ineffectual assignments in Go code.

misspell99%

Misspell Finds commonly misspelled English words