-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[receiver/azureeventhub] Better handling of timestamp parsing #27589
Comments
Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
I just noticed this, because I needed to use this receiver. I will look it up and see if we should copy the parser used in Azure's SDK or import it. Our code currently uses ISO8601, whereas they use RFC3339 and explicitly allow for a further variation with the T or Z. |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
This issue has been closed as inactive because it has been stale for 120 days with no activity. |
@nslaughter are you interested in fixing this issue? |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners: See Adding Labels via Comments if you do not have permissions to add labels yourself. |
Component(s)
receiver/azureeventhub
What happened?
Description
Trying to use the
azureventhub
receiver to read diagnostic logs from Azure but its receiving a lot of timestamp parsing errors. It seems like the diagnostic records in event hub are either missing timestamp or have a different format in some cases. Would be good with some better handling, for instance option to ignore and set current time.Steps to Reproduce
Set up diagnostic logs and send to event hub
Expected Result
Able to parse all logs
Actual Result
Many many error message with Invalid timestamp
Collector version
v0.85.0
Environment information
Environment
OS: Ubuntu 20.04
OpenTelemetry Collector configuration
Log output
Additional context
No response
The text was updated successfully, but these errors were encountered: