-
Notifications
You must be signed in to change notification settings - Fork 525
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
Setting event timestamp from OTel observed timestamp #11935
Conversation
This pull request does not have a backport label. Could you fix it @LikeTheSalad? 🙏
NOTE: |
This pull request is now in conflicts. Could you fix it @LikeTheSalad? 🙏
|
@LikeTheSalad apologies for the review, I'll approve once the conflicts are fixed. |
# Conflicts: # go.mod # go.sum
We might to wait before merging this until we sort out otel partial success PR elastic/apm-data#156 |
This pull request is now in conflicts. Could you fix it @LikeTheSalad? 🙏
|
# Conflicts: # go.mod # go.sum
This pull request is now in conflicts. Could you fix it @LikeTheSalad? 🙏
|
# Conflicts: # go.mod # go.sum
|
This pull request is now in conflicts. Could you fix it @LikeTheSalad? 🙏
|
@LikeTheSalad the breaking change should be fixed after elastic/apm-data#172 We can merge this! Can you fix the conflicts ? |
# Conflicts: # go.mod # go.sum
Testing notes✔️ test-plan-ok Using a patched version of python otel exporter that sends observed_timestamp with no timestamp, send the following logs to apm-server 8.12
|
Motivation/summary
Setting event timestamp from OTel observed timestamp when needed.
apm-data: elastic/apm-data#166
Checklist
apmpackage
have been made)For functional changes, consider:
How to test these changes
Related issues