What's the right way to have _OTHER values? They're creating an error.

@JamesNK Have u tried removing the underscore from the Id? Note fields which are enums are not restricted to those known values but in fact other values can be sent hence reducing the need for _other.

I need to investigate more for feedback on changing/removing some results and using error.type more.

Updated. I used most feedback. I notably I think aspnetcore.identity.result is still required.

Note: Don't merge this until .NET 10 preview 7 ships.

I fixed the generated markdown files by copying output from build. Is it possible to run the tooling on Windows to generate markdown? Or is WSL recommended?

@JamesNK

I fixed the generated markdown files by copying output from build. Is it possible to run the tooling on Windows to generate markdown? Or is WSL recommended?

wsl is recommended. It's possible to install weaver https://github.com/open-telemetry/weaver/releases (or use docker image - otel/weaver) and run something like these two commands

weaver registry update-markdown --registry .\model --param registry_base_url=/docs/registry/ --target markdown --templates .\templates --future .\docs

and

weaver registry generate --registry .\model --templates .\templates markdown .\docs\registry\

which would be the same as make table-generation attribute-registry-generation

@open-telemetry/semconv-dotnet-approver @open-telemetry/dotnet-contrib-approvers @open-telemetry/dotnet-approvers PTAL

All changes have been applied. Please take another look.

LGTM!

could we please get an approval from @open-telemetry/semconv-dotnet-approver ?