Issue 623854: Support unit annotations in ts_mon metrics

go/src/infra/monitoring/proxy/main.go

Index: go/src/infra/monitoring/proxy/main.go
diff --git a/go/src/infra/monitoring/proxy/main.go b/go/src/infra/monitoring/proxy/main.go
index 038d75bd667d38341bbffa1311930db7554f076a..f77eb7747aafa9e3d47d688242428b5259bb95fa 100644
--- a/go/src/infra/monitoring/proxy/main.go
+++ b/go/src/infra/monitoring/proxy/main.go
@@ -25,6 +25,7 @@ import (
 	"github.com/luci/luci-go/common/tsmon/distribution"
 	"github.com/luci/luci-go/common/tsmon/field"
 	"github.com/luci/luci-go/common/tsmon/metric"
+	"github.com/luci/luci-go/common/tsmon/types"
 	"golang.org/x/net/context"
 	"google.golang.org/cloud/pubsub"
 )
@@ -41,9 +42,11 @@ const (
 var (
 	sentCount = metric.NewCounter("mon_proxy/endpoint/sent",
 		"Count of messages proxied to the endpoint",
+		types.MetricMetadata{},

[Sergey Berezin, 2016/07/06 22:01:24]

I'd split the go changes into a separate CL. The reason is, luci-go is pinned in
infra.git, and you first need to land the Go CL, and only then modify the go
code in infra.git, together with the pin roll (in top-level DEPS file).

IIUC, the Python change is independent from the Go change, so they can land
separately.

[ddoman, 2016/07/07 05:06:39]

Done.

 		field.String("result"))
 	sentDuration = metric.NewCumulativeDistribution("mon_proxy/endpoint/duration",
 		"Time taken to send messages to the endpoint, in milliseconds",
+		types.MetricMetadata{Units: types.Milliseconds},
 		distribution.DefaultBucketer)
 )