Skip to content
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

[exporter/<logs>] Update timestamp processing logic #9130

Closed
djaglowski opened this issue Apr 9, 2022 · 3 comments · Fixed by #9369
Closed

[exporter/<logs>] Update timestamp processing logic #9130

djaglowski opened this issue Apr 9, 2022 · 3 comments · Fixed by #9369
Assignees

Comments

@djaglowski
Copy link
Member

djaglowski commented Apr 9, 2022

The log data model was recently update to include a second timestamp. In order for logging exporters to maintain their current functionality, they must be updated to use Timestamp when it is present, and ObservedTimestamp otherwise.

A quick search appears to show that the following exporters may be affected:

  • azuremonitorexporter
  • observiqexporter
  • lokiexporter

This issue is dependent upon collector#5171 being completed first.

@jpkrohling
Copy link
Member

cc @mar4uk

@mar4uk
Copy link
Contributor

mar4uk commented Apr 14, 2022

@djaglowski Do you plan to work on this issue by yourself? Or I can assign it to myself and do it?

@djaglowski
Copy link
Member Author

@mar4uk, your help would be very appreciated, thank you! ! I'll assign it to you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants