Skip to content

Add Timestamps, Time Zones, Time Ranges, and Date Formats for OpenTelemetry Collector #4758

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

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

JV0812
Copy link
Collaborator

@JV0812 JV0812 commented Nov 14, 2024

Purpose of this pull request

This pull request is to create OTel Timestamps Date Formats docs.

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

Create a page with "Supported timestamp and date format"

@JV0812 JV0812 requested review from kimsauce, jpipkin1, mafsumo and a team as code owners November 14, 2024 11:56
@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Nov 14, 2024
@JV0812 JV0812 self-assigned this Nov 14, 2024
@JV0812 JV0812 marked this pull request as draft November 14, 2024 11:56
@JV0812 JV0812 added the do-not-merge Pull requests that should not be merged label Nov 14, 2024
@JV0812 JV0812 marked this pull request as ready for review November 15, 2024 03:56
@jpipkin1 jpipkin1 requested a review from amee-sumo as a code owner December 4, 2024 18:09
Copy link
Collaborator

@jpipkin1 jpipkin1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, @JV0812. I'm sorry I missed seeing this one earlier!

Copy link
Collaborator

@kimsauce kimsauce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few suggestions!

@soc-service-agent
Copy link

soc-service-agent commented Apr 25, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@kimsauce kimsauce self-assigned this Apr 25, 2025
@kimsauce
Copy link
Collaborator

@kimsauce kimsauce requested a review from kevin-sumo April 25, 2025 17:35
@kimsauce kimsauce changed the title OTel Timestamps Date Formats docs added Add Timestamps, Time Zones, Time Ranges, and Date Formats for OpenTelemetry Collector Apr 25, 2025
@kimsauce kimsauce requested a review from sumoanema April 25, 2025 19:16
@kimsauce
Copy link
Collaborator

@kevin-sumo @sumoanema , I've tagged you both for review/feedback.

@kimsauce kimsauce requested a review from echlebek April 25, 2025 22:26
The timestamp is the part of a log message that marks the time that an event occurred. During ingestion, we can detect the message timestamp, convert it to Unix epoch time (the number of milliseconds since midnight, January 1, 1970 UTC), and index it. The timestamp is parsed either using the default timestamp parsing settings, or a custom format that you specify.

### OTel automated timestamp parsing

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per @echlebek , it would be useful to include another column that describes the name of the format, e.g., ISO8601

@kimsauce kimsauce removed their assignment May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed Contributor approved, listed in .clabot file do-not-merge Pull requests that should not be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants