Should the demo set exception message as span's status description

[cijothomas]

Bug Report

Which version of the demo you are using? : main branch

Symptom

https://github.com/open-telemetry/opentelemetry-demo/blob/main/src/cartservice/src/services/CartService.cs#L75 Is storing Exception message into status description. As per the SetStatus spec, the description should be documented and predictable. I am wondering if this is an intentional thing or a leftover from some previous state?

open-telemetry/opentelemetry-dotnet#5025 OTel .NET's instrumentation library removed setting exception message to status.description before its 1st stable release....

What is the expected behavior?

Not sure. This is more like a question - should we set exception message into status description.

[puckpuck]

I believe we are doing this properly. The specification says that Instrumentation Libraries should set this to be documented and predictable. Since this is not an instrumentation library, what we are doing would be the expected behavior to see the error message as returned to our code if an error occurs.

When the status is set to Error by Instrumentation Libraries, the Description SHOULD be documented and predictable.

[puckpuck]

Closing this issue since we are setting this in the application code and not as part of an instrumentation library. Feel free to re-open if you feel otherwise and we can further discuss.