[SEMANTIC CONVENTIONS] Upgrade to semconv version 1.23.1#2428
[SEMANTIC CONVENTIONS] Upgrade to semconv version 1.23.1#2428marcalff merged 4 commits intoopen-telemetry:mainfrom
Conversation
lalitb
left a comment
There was a problem hiding this comment.
Thanks. Difficult to identify the difference due to the change in the order of definitions. Hopefully, the generate script should be good to rely on. Else, let me know if something needs to be reviewed specifically.
Yes, relying on the generated code too. I think the build-tools can be improved to may be sort all symbols first, to generate them in a more predictable and stable order, to limit spurious diffs ... I will raise that with build-tools. I use a smoke test to check the diff sanity, looked ok: it will list a lot of semconv with +1 and -1 lines for the same name, indicating the declaration moved but was unchanged. |
Agree, a more predictable order would help. |
Fixes #2419
Changes
Upgrade to:
For significant contributions please make sure you have completed the following items:
CHANGELOG.mdupdated for non-trivial changes