-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add semantic conventions for specification v1.16.0 #6713
Conversation
00c41f3
to
a06ece5
Compare
Signed-off-by: Benedikt Bongartz <bongartz@klimlive.de>
a06ece5
to
5f3d04f
Compare
Codecov ReportBase: 89.64% // Head: 91.05% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #6713 +/- ##
==========================================
+ Coverage 89.64% 91.05% +1.41%
==========================================
Files 246 237 -9
Lines 14302 14306 +4
==========================================
+ Hits 12821 13027 +206
+ Misses 1231 1027 -204
- Partials 250 252 +2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
any updates here @open-telemetry/collector-approvers ? 😃 |
Is something else needed? :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like you had to change the invocation used to generate these constants, but have not updated the gensemconv
make target. That needs to be done as well. If changes are required in the generator then they should be made first and the dependency updated.
$ make gensemconv SPECPATH=../opentelemetry-specification SPECTAG=v1.16.0
Generating semantic convention constants from specification version v1.16.0 at ../opentelemetry-specification
semconvgen -o semconv/v1.16.0 -t semconv/template.j2 -s v1.16.0 -i ../opentelemetry-specification/semantic_conventions/resource -p conventionType=resource -f generated_resource.go
semconvgen -o semconv/v1.16.0 -t semconv/template.j2 -s v1.16.0 -i ../opentelemetry-specification/semantic_conventions/trace -p conventionType=trace -f generated_trace.go
panic: unable to render template: exit status 1
Traceback (most recent call last):
File "/usr/local/bin/gen-semconv", line 8, in <module>
sys.exit(main())
File "/usr/local/lib/python3.8/site-packages/opentelemetry/semconv/main.py", line 67, in main
semconv = parse_semconv(args, parser)
File "/usr/local/lib/python3.8/site-packages/opentelemetry/semconv/main.py", line 38, in parse_semconv
semconv.finish()
File "/usr/local/lib/python3.8/site-packages/opentelemetry/semconv/model/semantic_convention.py", line 317, in finish
fixpoint_ref = self.resolve_ref(semconv)
File "/usr/local/lib/python3.8/site-packages/opentelemetry/semconv/model/semantic_convention.py", line 458, in resolve_ref
raise ValidationError.from_yaml_pos(
opentelemetry.semconv.model.exceptions.ValidationError: Semantic Convention trace-exception reference `exception.type` but it cannot be found! - @2:5
goroutine 1 [running]:
main.main()
/Users/mirabant/go/pkg/mod/go.opentelemetry.io/build-tools/semconvgen@v0.3.0/generator.go:61 +0x44e
make: *** [gensemconv] Error 2
Signed-off-by: Benedikt Bongartz <bongartz@klimlive.de>
Thanks for the feedback @Aneurysm9. I have adjusted the |
Once the generator change is landed this will need to update |
@Aneurysm9 I updated the internal semconvgen to the latest version. |
I just made a |
Signed-off-by: Benedikt Bongartz <bongartz@klimlive.de>
a3d994d
to
7ad7296
Compare
done 👍 |
Description:
Add semantic conventions for specification v1.16.0.
Link to tracking Issue:
Belongs to #6696 (comment)
Closes #6714
Additional:
Convention type
trace
trace leads to the following output:According to the sem-conv changelog, there should have been no breaking change. The last one was in version v0.12.0 and was resolved here.Can someone help/show me what to look for?update
Thanks for the hint from @Aneurysm9. He noticed that exception was moved from tracing[1][2]. Now I could now generate trace and resources as follows: