Skip to content

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