-
Notifications
You must be signed in to change notification settings - Fork 38
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
Auto agent fails to export histogram without boundaries #359
Comments
@nioncode Debugging logs (Java Agent 1.33.0)
As you can see getBoundaries is empty. Debugging logs (Java Agent 2.5.0)
The boudaries are present here. Possible Workaround:
Note: There were breaking changes between 1.x and 2.x of the Java Agent. |
Thanks for the quick troubleshooting! I guess we'll filter out the warning then, since we cannot update to Java Agent v2, since Google Cloud Trace seems to only support v1 properly (I think it only parses the old http attributes from the traces and not the new opentelemetry attributes), but maybe we're just doing things wrong here and Cloud Trace should already support Java Agent v2 (and the latest opentelemetry http spec)? |
@nioncode could you explain a bit more what you mean by this? We can pass on feedback to the Cloud Trace team if you can share specifically what is wrong |
@aabmass I did a few requests to my service with java agent v1 and v2 and here are the attributes that were recorded in Cloud Trace for the top-level http post request: Java Agent v1.33.4
Java Agent v2.6.0
Only the trace from the v1 agent is properly labeled with a red error in Cloud Trace (I assume it parses the Apart from that it also seems like that for both, javaagent v1 and v2, sometimes some fields are missing (I did the same request 3 times and I always see a different set of attributes in Cloud Trace). Maybe there is some kind of maximum trace size that can be recorded and that's why random fields are dropped (depending on the order in the json payload I assume)? |
Hi @nioncode, You mentioned in your comment that "Cloud Trace only support v1 properly" - is there any other issue you faced with v2 except that the spans with an error did not render with the proper UI ? |
Good catch. Can we control which attributes are recorded by default in the agent via an environment variable to remove some of those that we don't need? Alternatively, can you increase the limit in Cloud Trace to support all the default attributes (plus a few more as a buffer)? Otherwise it is a bit confusing for customers why the default settings don't work correctly with Cloud Trace.
No, I think the only problem that we faced is that the UI does not visualize the http status for v2 correctly and then we assumed that it only works 'properly' with v1 (without checking/knowing if there are any other downsides). |
Hi @nioncode , |
Closing based on #359 (comment) |
@dashpole is there any issue that we can subscribe to to get notified once Cloud Trace supports the new opentelemetry convention and / or supports more span attributes? |
We are aware of this, but i'm afraid I can't share anything publicly right now. When changes are announced, it will be posted to the Cloud Trace release notes: https://cloud.google.com/trace/docs/release-notes. |
Important
Originally posted by @nioncode in open-telemetry/opentelemetry-collector-contrib#34250
Description
We are using auto instrumentation via the opentelemetry-javaagent (v1.33.4) and the auto exporter (
exporter-auto-0.31.0-alpha-shaded
). We instrument our app with the following command:When deploying our app to GCP, it seems to collect metrics correctly (at least some statistics show up for
http.server.duration
in Google Cloud Monitoring), but every minute our app prints a warning that it failed to write some TimeSeries (see below for the exact stack trace).How can we find out which metric could not be written (since at least the
http.server.duration
seems to work fine) and/or how can we avoid the warning?Steps to Reproduce
Create a java app, instrument it with the agent + auto exporter and deploy it to GCP. Wait a couple of minutes until the app has started and check its logs.
Expected Result
There should be no warnings.
Actual Result
We get warnings in the logs that some TimeSeries cannot be exported:
Environment information
Environment
JDK: Temurin 17
OS: Ubuntu 24.04
Undertow 2.3.12
Originally posted by @nioncode in open-telemetry/opentelemetry-collector-contrib#34250
The text was updated successfully, but these errors were encountered: