Replace tag.NewKey calls with tag.MustNewKey where appropriate (#1146)

Author: semistrict

README.md

@@ -78,7 +78,7 @@ Package `tag` allows adding or modifying tags in the current context.
 
 [embedmd]:# (internal/readme/tags.go new)
 ```go
-ctx, err = tag.New(ctx,
+ctx, err := tag.New(ctx,
 	tag.Insert(osKey, "macOS-10.12.5"),
 	tag.Upsert(userIDKey, "cde36753ed"),
 )

examples/helloworld/main.go

@@ -54,10 +54,7 @@ func main() {
 
 	trace.ApplyConfig(trace.Config{DefaultSampler: trace.AlwaysSample()})
 
-	frontendKey, err = tag.NewKey("example.com/keys/frontend")
-	if err != nil {
-		log.Fatal(err)
-	}
+	frontendKey = tag.MustNewKey("example.com/keys/frontend")
 	videoSize = stats.Int64("example.com/measure/video_size", "size of processed videos", stats.UnitBytes)
 	view.SetReportingPeriod(2 * time.Second)
 

examples/quickstart/stats.go

@@ -56,7 +56,7 @@ var (
 
 // TagKeys for the stats quickstart.
 var (
-	keyMethod, _ = tag.NewKey("method")
+	keyMethod = tag.MustNewKey("method")
 )
 
 // Views for the stats quickstart.

internal/readme/README.md

@@ -1,6 +1,6 @@
 Use the following commands to regenerate the README.
 
 ```bash
-$ go get github.com/rakyll/embedmd
+$ GO11MODULE=off go get github.com/rakyll/embedmd
 $ embedmd -w ../../README.md
 ```

internal/readme/tags.go

@@ -24,17 +24,11 @@ import (
 func tagsExamples() {
 	ctx := context.Background()
 
-	osKey, err := tag.NewKey("example.com/keys/user-os")
-	if err != nil {
-		log.Fatal(err)
-	}
-	userIDKey, err := tag.NewKey("example.com/keys/user-id")
-	if err != nil {
-		log.Fatal(err)
-	}
+	osKey := tag.MustNewKey("example.com/keys/user-os")
+	userIDKey := tag.MustNewKey("example.com/keys/user-id")
 
 	// START new
-	ctx, err = tag.New(ctx,
+	ctx, err := tag.New(ctx,
 		tag.Insert(osKey, "macOS-10.12.5"),
 		tag.Upsert(userIDKey, "cde36753ed"),
 	)

plugin/ocgrpc/client_stats_handler_test.go

@@ -36,8 +36,8 @@ import (
 )
 
 func TestClientDefaultCollections(t *testing.T) {
-	k1, _ := tag.NewKey("k1")
-	k2, _ := tag.NewKey("k2")
+	k1 := tag.MustNewKey("k1")
+	k2 := tag.MustNewKey("k2")
 
 	type tagPair struct {
 		k tag.Key
@@ -340,7 +340,7 @@ func TestClientDefaultCollections(t *testing.T) {
 }
 
 func TestClientRecordExemplar(t *testing.T) {
-	key, _ := tag.NewKey("test_key")
+	key := tag.MustNewKey("test_key")
 	tagInfo := &stats.RPCTagInfo{FullMethodName: "/package.service/method"}
 	out := &stats.OutPayload{Length: 2000}
 	end := &stats.End{Error: nil}

plugin/ocgrpc/end_to_end_test.go

@@ -27,7 +27,7 @@ import (
 	"go.opencensus.io/tag"
 )
 
-var keyAccountId, _ = tag.NewKey("account_id")
+var keyAccountId = tag.MustNewKey("account_id")
 
 func TestEndToEnd_Single(t *testing.T) {
 	view.Register(ocgrpc.DefaultClientViews...)

plugin/ocgrpc/server_stats_handler_test.go

@@ -32,8 +32,8 @@ import (
 )
 
 func TestServerDefaultCollections(t *testing.T) {
-	k1, _ := tag.NewKey("k1")
-	k2, _ := tag.NewKey("k2")
+	k1 := tag.MustNewKey("k1")
+	k2 := tag.MustNewKey("k2")
 
 	type tagPair struct {
 		k tag.Key
@@ -338,7 +338,7 @@ func newDistributionData(countPerBucket []int64, count int64, min, max, mean, su
 }
 
 func TestServerRecordExemplar(t *testing.T) {
-	key, _ := tag.NewKey("test_key")
+	key := tag.MustNewKey("test_key")
 	tagInfo := &stats.RPCTagInfo{FullMethodName: "/package.service/method"}
 	out := &stats.OutPayload{Length: 2000}
 	end := &stats.End{Error: nil}

plugin/ocgrpc/stats_common.go

@@ -61,14 +61,14 @@ var (
 // Server tags are applied to the context used to process each RPC, as well as
 // the measures at the end of each RPC.
 var (
-	KeyServerMethod, _ = tag.NewKey("grpc_server_method")
-	KeyServerStatus, _ = tag.NewKey("grpc_server_status")
+	KeyServerMethod = tag.MustNewKey("grpc_server_method")
+	KeyServerStatus = tag.MustNewKey("grpc_server_status")
 )
 
 // Client tags are applied to measures at the end of each RPC.
 var (
-	KeyClientMethod, _ = tag.NewKey("grpc_client_method")
-	KeyClientStatus, _ = tag.NewKey("grpc_client_status")
+	KeyClientMethod = tag.MustNewKey("grpc_client_method")
+	KeyClientStatus = tag.MustNewKey("grpc_client_status")
 )
 
 var (

plugin/ochttp/stats.go

@@ -92,38 +92,38 @@ var (
 	// The value of this tag can be controlled by the HTTP client, so you need
 	// to watch out for potentially generating high-cardinality labels in your
 	// metrics backend if you use this tag in views.
-	Host, _ = tag.NewKey("http.host")
+	Host = tag.MustNewKey("http.host")
 
 	// StatusCode is the numeric HTTP response status code,
 	// or "error" if a transport error occurred and no status code was read.
-	StatusCode, _ = tag.NewKey("http.status")
+	StatusCode = tag.MustNewKey("http.status")
 
 	// Path is the URL path (not including query string) in the request.
 	//
 	// The value of this tag can be controlled by the HTTP client, so you need
 	// to watch out for potentially generating high-cardinality labels in your
 	// metrics backend if you use this tag in views.
-	Path, _ = tag.NewKey("http.path")
+	Path = tag.MustNewKey("http.path")
 
 	// Method is the HTTP method of the request, capitalized (GET, POST, etc.).
-	Method, _ = tag.NewKey("http.method")
+	Method = tag.MustNewKey("http.method")
 
 	// KeyServerRoute is a low cardinality string representing the logical
 	// handler of the request. This is usually the pattern registered on the a
 	// ServeMux (or similar string).
-	KeyServerRoute, _ = tag.NewKey("http_server_route")
+	KeyServerRoute = tag.MustNewKey("http_server_route")
 )
 
 // Client tag keys.
 var (
 	// KeyClientMethod is the HTTP method, capitalized (i.e. GET, POST, PUT, DELETE, etc.).
-	KeyClientMethod, _ = tag.NewKey("http_client_method")
+	KeyClientMethod = tag.MustNewKey("http_client_method")
 	// KeyClientPath is the URL path (not including query string).
-	KeyClientPath, _ = tag.NewKey("http_client_path")
+	KeyClientPath = tag.MustNewKey("http_client_path")
 	// KeyClientStatus is the HTTP status code as an integer (e.g. 200, 404, 500.), or "error" if no response status line was received.
-	KeyClientStatus, _ = tag.NewKey("http_client_status")
+	KeyClientStatus = tag.MustNewKey("http_client_status")
 	// KeyClientHost is the value of the request Host header.
-	KeyClientHost, _ = tag.NewKey("http_client_host")
+	KeyClientHost = tag.MustNewKey("http_client_host")
 )
 
 // Default distributions used by views in this package.