-
Notifications
You must be signed in to change notification settings - Fork 282
feat: introduce new service.criticality attribute (#2986) #3088
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
base: main
Are you sure you want to change the base?
feat: introduce new service.criticality attribute (#2986) #3088
Conversation
I need help with guidance on how to provide prototype for a newly introduced semantic convention attribute. Please, feel free drop your idea. Thanks in advance! |
|
Note: The build is failing - My approval is for the proposed change, but please clean up the build and get all tests to pass. I believe this needs a |
|
@bachgarash can you regenerate the docs to take on the changes to the documentation templates? |
|
Hi there. I have added a basic prototype into Otel-demo as requested |
9679b6a to
0de4ab4
Compare
0de4ab4 to
bf07d66
Compare
Fixes #2986
This PR introduces a new attribute -
service.criticalityRelevant discussion has been made during Service and Deployment Semconv SIG meet
Meet notes https://docs.google.com/document/d/1Fy6yXfZqrwN_oHw95Bdfg_0hzUgzlk3VO5wA1invgkI/edit?tab=t.0
Recording can be found https://docs.google.com/spreadsheets/d/1SYKfjYhZdm2Wh2Cl6KVQalKg_m4NhTPZqq-8SzEVO6s/edit?gid=0#gid=0 (6th of November)
Changes
This PR introduces a new semantic convention attribute
service.criticalityto enable classification of services based on their operational importance. This attribute will allow observability platforms to implement criticality-aware tracing, and sampling strategies.Important
Pull requests acceptance are subject to the triage process as described in Issue and PR Triage Management.
PRs that do not follow the guidance above, may be automatically rejected and closed.
Merge requirement checklist
[chore]