Update SpanKind for root span

Requires review, thanks.

Sounds strange, but looks good :) Implementation is ok, left a comment in the MR.

Updated MR, please review

Now is okay, merged.

MR for config settings form
https://git.drupalcode.org/project/opentelemetry/-/merge_requests/41

Im not agree with this development, only external calls should be marked as span client. In your first reference check comment "//a guzzle middleware to wrap http calls in a span, and inject trace headers" and check official documentations about span kinds https://opentelemetry.io/docs/concepts/signals/traces/#client this development is not correct in my opinion.

Thanks @tarikflz for sharing. In that case we can switch from spankind "client" to default spankind "internal" for "RequestTraceEventSubscriber" span
I will prepare MR and will ask for review.

gaurav_manerkar → changed the visibility of the branch 3422535-update-spankind-for-settings-form to hidden.

gaurav_manerkar → changed the visibility of the branch 3422535-update-spankind-for-settings-form to active.

@tarikflz please review
MR - https://git.drupalcode.org/project/opentelemetry/-/merge_requests/42

Its okey for me @gaurav_manerkar, If we are on the same page about usage of span kinds.

Looks good for me, even better than the first version! :) Thanks, merged.

Automatically closed - issue fixed for 2 weeks with no activity.