[exporters/otlptracehttp] Do not log errors for 2XX response

CHANGELOG.md

@@ -8,6 +8,10 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 
 ## [Unreleased]
 
+### Fixed
+
+- Do not log errors for 202 Accepted response for otlptracehttp exporter. (#3706)
+
 ### Added
 
 - Semantic conventions of the `event` type are now generated. (#3697)

exporters/otlp/otlptrace/otlptracehttp/client.go

@@ -165,7 +165,7 @@ func (d *client) UploadTraces(ctx context.Context, protoSpans []*tracepb.Resourc
  }
 
  switch resp.StatusCode {
- case http.StatusOK:
+ case http.StatusOK, http.StatusAccepted:
  // Success, do not retry.
  // Read the partial success message, if any.
  var respData bytes.Buffer

exporters/otlp/otlptrace/otlptracehttp/client_test.go

@@ -67,6 +67,12 @@ func TestEndToEnd(t *testing.T) {
  otlptracehttp.WithCompression(otlptracehttp.GzipCompression),
  },
  },
+ {
+ name: "no errors on 202 Accepted",
+ mcCfg: mockCollectorConfig{
+ InjectHTTPStatus: []int{202},
+ },
+ },
  {
  name: "retry",
  opts: []otlptracehttp.Option{

exporters/otlp/otlptrace/otlptracehttp/mock_collector_test.go

@@ -103,9 +103,12 @@ func (c *mockCollector) serveTraces(w http.ResponseWriter, r *http.Request) {
  return
  }
  h := c.getInjectResponseHeader()
- if injectedStatus := c.getInjectHTTPStatus(); injectedStatus != 0 {
+ injectedStatus := c.getInjectHTTPStatus()
+ if injectedStatus != 0 {
  writeReply(w, rawResponse, injectedStatus, c.injectContentType, h)
- return
+ if injectedStatus != http.StatusAccepted {
+ return
+ }
  }
  rawRequest, err := readRequest(r)
  if err != nil {
@@ -118,7 +121,9 @@ func (c *mockCollector) serveTraces(w http.ResponseWriter, r *http.Request) {
  w.WriteHeader(http.StatusBadRequest)
  return
  }
- writeReply(w, rawResponse, 0, c.injectContentType, h)
+ if injectedStatus == 0 {
+ writeReply(w, rawResponse, 0, c.injectContentType, h)
+ }
  c.spanLock.Lock()
  defer c.spanLock.Unlock()
  c.spansStorage.AddSpans(request)