Remove http.status_code attribute

Author: Muriel Picone Farías

instrumentation/grape/lib/opentelemetry/instrumentation/grape/event_handler.rb

@@ -26,11 +26,7 @@ def endpoint_run_finish(_name, _id, payload)
             token = payload.delete(:__opentelemetry_ctx_token)
             return unless span && token
 
-            if payload[:exception_object]
-              handle_payload_exception(span, payload[:exception_object])
-            else
-              span.set_attribute(OpenTelemetry::SemanticConventions::Trace::HTTP_STATUS_CODE, payload[:endpoint].status)
-            end
+            handle_payload_exception(span, payload[:exception_object]) if payload[:exception_object]
 
             span.finish
             OpenTelemetry::Context.detach(token)
@@ -111,9 +107,6 @@ def run_attributes(payload)
           def handle_payload_exception(span, exception)
             span.record_exception(exception)
             span.status = OpenTelemetry::Trace::Status.error("Unhandled exception of type: #{exception.class}")
-            return unless exception.respond_to?('status') && exception.status
-
-            span.set_attribute(OpenTelemetry::SemanticConventions::Trace::HTTP_STATUS_CODE, exception.status)
           end
 
           def request_method(endpoint)

instrumentation/grape/test/opentelemetry/instrumentation/grape_test.rb

@@ -55,7 +55,6 @@ class BasicAPI < Grape::API
         _(span.attributes['http.route']).must_equal '/hello'
         _(span.attributes['grape.route.method']).must_equal 'GET'
         _(span.attributes['http.method']).must_equal 'GET'
-        _(span.attributes['http.status_code']).must_equal 200
       end
 
       it 'produces a child endpoint_render span with the expected attributes' do
@@ -258,7 +257,6 @@ class ValidationErrorAPI < Grape::API
           _(span.events.first.name).must_equal 'exception'
           _(span.events.first.attributes['exception.type']).must_equal expected_error_type
           _(span.events.first.attributes['exception.message']).must_equal 'name is missing'
-          _(span.attributes['http.status_code']).must_equal(400)
         end
       end
 
@@ -308,12 +306,6 @@ class RaisedErrorAPI < Grape::API
           _(span.status.code).wont_equal OpenTelemetry::Trace::Status::ERROR
         end
       end
-
-      it 'does not add the http.status_code attribute to the endpoint_run span' do
-        span = run_spans.first
-
-        _(span.attributes).wont_include 'http.status_code'
-      end
     end
 
     describe 'when an API endpoint raises an error in the filters' do
@@ -422,12 +414,6 @@ class ErrorResponseAPI < Grape::API
           _(span.status.code).wont_equal OpenTelemetry::Trace::Status::ERROR
         end
       end
-
-      it 'sets endpoint_run span status code to expected value' do
-        span = run_spans.first
-
-        _(span.attributes['http.status_code']).must_equal 404
-      end
     end
 
     describe 'when an API endpoint receives a request and some events are ignored in the configs' do