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/envoyals] init new receiver #37193

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

Conversation

zirain
Copy link
Contributor

@zirain zirain commented Jan 14, 2025

Description

Add a new receiver for the Envoy ALS (Access Log Service)

Link to tracking issue

Fixes #36464

cc @evan-bradley

@zirain zirain requested a review from a team as a code owner January 14, 2025 06:37
@zirain zirain requested a review from ChrsMark January 14, 2025 06:37
@github-actions github-actions bot added the cmd/otelcontribcol otelcontribcol command label Jan 14, 2025
@zirain zirain force-pushed the anvoy-als-receiver branch from bea3704 to a5d1e52 Compare January 14, 2025 06:38
@zirain
Copy link
Contributor Author

zirain commented Jan 14, 2025

wait #37194

@zirain zirain force-pushed the anvoy-als-receiver branch from afaca7d to 02fd711 Compare January 15, 2025 01:13
Signed-off-by: zirain <[email protected]>
Signed-off-by: zirain <[email protected]>
Signed-off-by: zirain <[email protected]>
Signed-off-by: zirain <[email protected]>
alpha: [logs]
distributions: [contrib, k8s]
codeowners:
active: [evan-bradley]
Copy link
Contributor

Choose a reason for hiding this comment

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

don't you want to be a codeowner as well? Are you an OpenTelemetry member?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm happy to be a codeowner, but right now I'm not an OpenTelemetry member.

Copy link
Contributor

Choose a reason for hiding this comment

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

Please discuss with your component sponsor. Please see to open an issue under https://github.com/open-telemetry/community/issues/new?template=membership.md

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I read it earlier and I will update my id here after I become a member.

Signed-off-by: zirain <[email protected]>
Signed-off-by: zirain <[email protected]>
@zirain zirain requested review from atoulme and joaopgrassi January 18, 2025 04:37
@atoulme
Copy link
Contributor

atoulme commented Jan 18, 2025

Please fix the conflict.

@zirain
Copy link
Contributor Author

zirain commented Jan 18, 2025

/retest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New component: Envoy access log receiver
4 participants