Don't set Span.Status for 4xx http status codes for SERVER spans (#1998)

* Don't set Span.Status for 4xx http status codes for SERVER spans

Author: iNikem

CHANGELOG.md

@@ -27,6 +27,9 @@ release.
 - BREAKING: Remove db.cassandra.keyspace and db.hbase.namespace, and clarify db.name
   ([#1973](https://github.com/open-telemetry/opentelemetry-specification/pull/1973))
 
+- Don't set Span.Status for 4xx http status codes for SERVER spans.
+  ([#1998](https://github.com/open-telemetry/opentelemetry-specification/pull/1998))
+
 ### Compatibility
 
 ### OpenTelemetry Protocol

specification/trace/semantic_conventions/http.md

@@ -46,8 +46,11 @@ default span name.
 the response body; or 3xx codes with max redirects exceeded), in which case status
 MUST be set to `Error`.
 
-For HTTP status codes in the 4xx and 5xx ranges, as well as any other code the client
-failed to interpret, status MUST be set to `Error`.
+For HTTP status codes in the 4xx range span status MUST be left unset in case of `SpanKind.SERVER`
+and MUST be set to `Error` in case of `SpanKind.CLIENT`.
+
+For HTTP status codes in the 5xx range, as well as any other code the client
+failed to interpret, span status MUST be set to `Error`.
 
 Don't set the span status description if the reason can be inferred from `http.status_code`.