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

[receiver/awss3receiver] Add support RFC3339 format for starttime and endtime #36787

Merged
merged 1 commit into from
Dec 14, 2024

Conversation

sters
Copy link
Contributor

@sters sters commented Dec 11, 2024

Description

This PR adds support RFC3339 ("2006-01-02T15:04:05Z07:00") format for starttime and endtime fields.

Currently, those fields cannot specify timezone. It's depends on the situation.
This change makes clear the timezone.

Testing

This PR adds a new test data awss3/4 which tests specifying RFC3339.

Documentation

This PR changes README.md to describe time format.

@sters sters requested review from atoulme and a team as code owners December 11, 2024 16:17
Copy link

linux-foundation-easycla bot commented Dec 11, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: sters / name: sters (95a7f37)

Copy link
Contributor

@atoulme atoulme left a comment

Choose a reason for hiding this comment

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

LGTM, neat changes.

@sters
Copy link
Contributor Author

sters commented Dec 11, 2024

@atoulme

Thanks for quick review!

I've added a file under .chloggen. Changes are here 🙇

Copy link
Member

@SpencerTorres SpencerTorres left a comment

Choose a reason for hiding this comment

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

Great change, RFC3339 is the way to go for these time inputs 👍

@atoulme atoulme added the ready to merge Code review completed; ready to merge by maintainers label Dec 12, 2024
@dmitryax dmitryax merged commit 4c33430 into open-telemetry:main Dec 14, 2024
170 checks passed
@github-actions github-actions bot added this to the next release milestone Dec 14, 2024
sbylica-splunk pushed a commit to sbylica-splunk/opentelemetry-collector-contrib that referenced this pull request Dec 17, 2024
… endtime (open-telemetry#36787)

#### Description

This PR adds support RFC3339 (`"2006-01-02T15:04:05Z07:00"`) format for
`starttime` and `endtime` fields.

Currently, those fields cannot specify timezone. It's depends on the
situation.
This change makes clear the timezone.

#### Testing

This PR adds a new test data `awss3/4` which tests specifying RFC3339.
mterhar pushed a commit to mterhar/opentelemetry-collector-contrib that referenced this pull request Dec 19, 2024
… endtime (open-telemetry#36787)

#### Description

This PR adds support RFC3339 (`"2006-01-02T15:04:05Z07:00"`) format for
`starttime` and `endtime` fields.

Currently, those fields cannot specify timezone. It's depends on the
situation.
This change makes clear the timezone.

#### Testing

This PR adds a new test data `awss3/4` which tests specifying RFC3339.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Code review completed; ready to merge by maintainers receiver/awss3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants