Add system metrics clarification around containers#2388
Add system metrics clarification around containers#2388trask wants to merge 2 commits intoopen-telemetry:mainfrom
Conversation
tigrannajaryan
left a comment
There was a problem hiding this comment.
This clarification matches my understanding.
|
Thanks for the clarification, this would also match my understanding. |
Co-authored-by: Armin Ruech <[email protected]>
bogdandrutu
left a comment
There was a problem hiding this comment.
I read few times description/change. I feel that the motivation for this change is wrong, the change itself is ok. I do believe that we should not offer this in the java instrumentation until we have lots of requests for this. This is one of the easiest way to make things inconsistent and then blame that java or collector or whatever does inconsistent things etc.
|
I believe the principle here that when a process wishes to observe metrics on behalf of the system it's running in, we're defining that "system" as the inner-most container. I believe @bogdandrutu's concern is that we should not expect a process to observe metrics on behalf of the system it's running in because it leads to duplicated code, but that doesn't change the principle stated above. If there is not another entity observing the system you are running in, you should feel free to do that yourself. 👍 |
@jmacd not only duplicate code, but also confusion. Imagine how confusing would be for a Java application owner, that uses the collector as well. What does |
|
@bogdandrutu are you ok if these |
Prove me with numbers / requests for this. The "we cannot force them" can be translated "if you want system metrics" here is a small binary that does this for you. Otherwise you just create huge amount of work for all the language sigs. |
I didn't read the PR description carefully. I agree with what Bogdan wrote. I would definitely not expect that a Java application emits system metrics information (unless that Java application is purpose-built to be a system metric Collector). |
For compatibility with the Java ecosystem (e.g. JFR, JMX, Micrometer) there are a couple of common "system" level metrics that are provided by the JVM that we would like to map to the OpenTelemetry world. Can you clarify if the objection is specific to reusing the |
Yes. It looks surprising that the application emits metrics in the |
Thanks, I think this is the clarification we needed to move forward with JVM metrics specification. |
The JVM Metrics working group would like to emit a couple key system metrics from inside the JVM (e.g. for users who aren't using the collector/agent/other mechanism), and we'd like to add clarification somewhere that these metrics should be interpreted as container-level metrics and not host-level metrics.
cc: @jonatan-ivanov @jack-berg @kittylyst