-
Notifications
You must be signed in to change notification settings - Fork 174
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 kernel related semantic conventions #66
Comments
Some notes:
I think a reasonable start is to specify the values for the kernels from
|
@ChrsMark @AlexanderWert @trisch-me would be interested to know your thoughts about this one. Unfortunately |
Hi @mx-psi, I have checked we are using os.kernel alone in security in around 100 different integrations. |
Hey @mx-psi Taking this into account and also following our soon to be merged guidelines I propose to use new naming for this namespace. We could use something like Additional consideration: |
Sure, we can defer up until that point, please ping me in the PR :) I will think about possible alternatives for the namespace from my side as well |
hey @mx-psi I think we can bring this issue up again. I have checked and there are no other fields for OS except So I will not create any new PR because these are exact fields we need to make agreement on |
@trisch-me Okay! To me, the main question here is what namespace to use if cc @open-telemetry/semconv-system-approvers |
Do we have examples of how other operating systems will populate these attributes? If this detailed info is only for unix based systems we could use sth like I also wonder if the usage of UpdateOn a mac we have: ~ ❯ uname -r
23.4.0
~ ❯ uname -s
Darwin
~ ❯ uname -v
Darwin Kernel Version 23.4.0: Fri Mar 15 00:10:42 PDT 2024; root:x-10063.101.17~1/RELEASE_ARM64_T6000
~ ❯ uname -a
Darwin H471B7HXWP 23.4.0 Darwin Kernel Version 23.4.0: Fri Mar 15 00:10:42 PDT 2024; root:x-10063.101.17~1/RELEASE_ARM64_T6000 arm64 |
From what I understand in ECS |
I would like to add three new resource-level semantic conventions to the
os.*
namespace that include information about the kernel used by the system. Roughly, they could be defined as follows:os.kernel.name
uname -s
.Linux
os.kernel.release
uname -r
5.19.0-42-generic
os.kernel.version
uname -v
#43~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Fri Apr 21 16:51:08 UTC 2
These would be useful to identify a resource as coming from a machine with a specific kernel version, which can be useful in debugging and troubleshooting systems.
It's been a while since I have contributed to semantic conventions so I have a few open questions, please bear with me :)
Open questions
os.kernel
as "Operating system kernel version as a raw string." and gives4.4.0-112-generic
as an example value (so, effectively what I called hereos.kernel.release
). I think we should use the more specificos.kernel.release
for this instead, sinceos.kernel
seems more like a namespace to me, and there's several meanings for 'version'. Is that okay?os.kernel.name
have a pre-defined set of values? If so, what values are in scope?uname
output and flags consistent enough to use it as a standard? Based on the description ofhost.id
it sounds like it should be okay, but I am happy to consider a different definition if something is suggested.The text was updated successfully, but these errors were encountered: