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

Create extension/encoding/awscloudwatchmetricstreamsencoding #38388

Conversation

axw
Copy link
Contributor

@axw axw commented Mar 5, 2025

Description

Create the component skeleton and basic config tests. Unmarshallers will be implemented in a follow up PR.

Link to tracking issue

First part of #37870

Testing

Added basic unit tests.

Documentation

Added README.

@axw axw force-pushed the awscloudwatchmetricstreamsencodingextension-skeleton branch from dd31b2f to b4e40d7 Compare March 5, 2025 05:17
Create the component skeleton and basic config tests.
Unmarshallers will be implemented in a follow up PR.
@axw axw force-pushed the awscloudwatchmetricstreamsencodingextension-skeleton branch from b4e40d7 to 6ee2df3 Compare March 5, 2025 05:29
@axw axw marked this pull request as ready for review March 5, 2025 05:52
@axw axw requested a review from a team as a code owner March 5, 2025 05:52
@atoulme
Copy link
Contributor

atoulme commented Mar 5, 2025

Approved as sponsor.

@atoulme atoulme added the Accepted Component New component has been sponsored label Mar 5, 2025
@atoulme atoulme merged commit 7219cc4 into open-telemetry:main Mar 5, 2025
170 checks passed
@github-actions github-actions bot added this to the next release milestone Mar 5, 2025
atoulme pushed a commit that referenced this pull request Mar 11, 2025
#### Description

I would like to become a codeowner of the awsfirehosereceiver. Elastic
(where I work) intends to use this receiver heavily, and I would like
to:
 - help spread out the maintenance load
- ensure we have a path to progressing beyond the current Alpha
stability

Assuming
open-telemetry/opentelemetry-collector#11864
goes through, it will be necessary to have multiple codeowners to
progress to Beta and preferable to have codeowners from multiple
employers, which is the case for @Aneurysm9 (AWS) and me (Elastic).

Relevant changes:
-
#37111
(superseded by #37361)
-
#37262
-
#37361
-
#38388
(will extract from awsfirehosereceiver)
-
#38445

#### Link to tracking issue

N/A

#### Testing

N/A

#### Documentation

N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Component New component has been sponsored extension/encoding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants