Allow graphql.field.parent and code.namespace to be nil

[elenatanasoiu]

Description of the bug

Source of the problem: https://github.com/open-telemetry/opentelemetry-ruby-contrib/blob/main/instrumentation/graphql/lib/opentelemetry/instrumentation/graphql/tracers/graphql_tracer.rb#L88

After turning on the enable_platform_field option to report execute_field and execute_field_lazy spans in our APM, we can now see some tests raise this error:

invalid span attribute value type NilClass for key 'graphql.field.parent' on span 'graphql.execute_field'

There are cases where we will add nil to graphql.field.parent, for example when the spans are parentless.

OpenTelemetry will validate the span attributes, will ignore invalid attributes and optionally call handle_error when doing so.

Our test helper currently raises an exception when handling such an error, which causes test failures.

We've had to specifically ignore the invalidgraphql.field.parent attribute message. We also do a similar thing for the code.namespace field:

next if message&.start_with?("invalid span attribute value type NilClass for key 'code.namespace'")
next if message&.start_with?("invalid span attribute value type NilClass for key 'graphql.field.parent'")

Share details about your runtime

N/A

[elenatanasoiu]

Will be fixed by #867