PR feedback

Author: jack-berg

content/en/docs/collector/deployment/agent.md

@@ -129,7 +129,7 @@ Cons:
 [instrumentation]: /docs/languages/
 [otlp]: /docs/specs/otel/protocol/
 [collector]: /docs/collector/
-[instrument-java-metrics]: /docs/languages/java/instrumentation/#metrics
+[instrument-java-metrics]: /docs/languages/java/api-components/#meterprovider
 [otlp-exporter]: /docs/specs/otel/protocol/exporter/
 [java-otlp-example]:
   https://github.com/open-telemetry/opentelemetry-java-docs/tree/main/otlp

content/en/docs/concepts/instrumentation/libraries.md

@@ -265,7 +265,7 @@ After you create a span, pass new trace context to the application code
 (callback or handler), by making the span active; if possible, do this
 explicitly. The following Java example shows how to add trace context and
 activate a span. See the
-[Context extraction in Java](/docs/languages/java/instrumentation/#context-propagation),
+[Context extraction in Java](/docs/languages/java/api-components/#contextpropagators),
 for more examples.
 
 ```java
@@ -289,9 +289,7 @@ try (Scope unused = span.makeCurrent()) {
 ```
 
 In the case of a messaging system, you might receive more than one message at
-once. Received messages become
-[links](/docs/languages/java/instrumentation/#create-spans-with-links) on the
-span you create. Refer to
+once. Received messages become links on the span you create. Refer to
 [messaging conventions](/docs/specs/semconv/messaging/messaging-spans/) for
 details.
 

content/en/docs/languages/go/getting-started.md

@@ -377,22 +377,25 @@ Modify `rolldice.go` to include custom instrumentation using OpenTelemetry API:
 package main
 
 import (
+	"fmt"
+	"io"
+	"log"
+	"math/rand"
+	"net/http"
+	"strconv"
+
 	"go.opentelemetry.io/contrib/bridges/otelslog"
 	"go.opentelemetry.io/otel"
+	"go.opentelemetry.io/otel/attribute"
 	"go.opentelemetry.io/otel/metric"
-	"go.opentelemetry.io/otel/sdk/instrumentation"
 )
 
-const name = "rolldice"
+const name = "go.opentelemetry.io/otel/example/dice"
 
 var (
 	tracer = otel.Tracer(name)
 	meter  = otel.Meter(name)
-	logger = otelslog.NewLogger(
-		otelslog.WithInstrumentationScope(instrumentation.Scope{
-			Name: name,
-		}),
-	)
+	logger = otelslog.NewLogger(name)
 	rollCnt metric.Int64Counter
 )
 
@@ -405,6 +408,30 @@ func init() {
 		panic(err)
 	}
 }
+
+func rolldice(w http.ResponseWriter, r *http.Request) {
+	ctx, span := tracer.Start(r.Context(), "roll")
+	defer span.End()
+
+	roll := 1 + rand.Intn(6)
+
+	var msg string
+	if player := r.PathValue("player"); player != "" {
+		msg = fmt.Sprintf("%s is rolling the dice", player)
+	} else {
+		msg = "Anonymous player is rolling the dice"
+	}
+	logger.InfoContext(ctx, msg, "result", roll)
+
+	rollValueAttr := attribute.Int("roll.value", roll)
+	span.SetAttributes(rollValueAttr)
+	rollCnt.Add(ctx, 1, metric.WithAttributes(rollValueAttr))
+
+	resp := strconv.Itoa(roll) + "\n"
+	if _, err := io.WriteString(w, resp); err != nil {
+		log.Printf("Write failed: %v\n", err)
+	}
+}
 ```
 <!-- prettier-ignore-end -->