diff --git a/.chloggen/remove_misleading_jaegerremotesampling_endpoint.yaml b/.chloggen/remove_misleading_jaegerremotesampling_endpoint.yaml new file mode 100755 index 000000000000..629f91c8a3e5 --- /dev/null +++ b/.chloggen/remove_misleading_jaegerremotesampling_endpoint.yaml @@ -0,0 +1,16 @@ +# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix' +change_type: breaking + +# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver) +component: extension/jaegerremotesampling + +# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`). +note: Remove misleading http endpoint from jaegerremotesampling extension. + +# One or more tracking issues related to the change +issues: [18058] + +# (Optional) One or more lines of additional information to render under the primary note. +# These lines will be padded with 2 spaces and then inserted directly into the document. +# Use pipe (|) for multiline entries. +subtext: diff --git a/extension/jaegerremotesampling/internal/http.go b/extension/jaegerremotesampling/internal/http.go index 0a8d614b1c4a..8d7c23d2a14c 100644 --- a/extension/jaegerremotesampling/internal/http.go +++ b/extension/jaegerremotesampling/internal/http.go @@ -58,9 +58,6 @@ func NewHTTP(telemetry component.TelemetrySettings, settings confighttp.HTTPServ } srv.mux = http.NewServeMux() - // the legacy endpoint - srv.mux.Handle("/", http.HandlerFunc(srv.samplingStrategyHandler)) - // the new endpoint -- not strictly necessary, as the previous one would match it // already, but good to have it explicit here srv.mux.Handle("/sampling", http.HandlerFunc(srv.samplingStrategyHandler)) diff --git a/extension/jaegerremotesampling/internal/http_test.go b/extension/jaegerremotesampling/internal/http_test.go index c574c7943481..c4325aac9938 100644 --- a/extension/jaegerremotesampling/internal/http_test.go +++ b/extension/jaegerremotesampling/internal/http_test.go @@ -56,10 +56,6 @@ func TestEndpointsAreWired(t *testing.T) { desc string endpoint string }{ - { - desc: "legacy", - endpoint: "/", - }, { desc: "new", endpoint: "/sampling",