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

Add tcpcheckreceiver To Contrib #865

Conversation

chengchuanpeng
Copy link
Contributor

@chengchuanpeng chengchuanpeng requested a review from a team as a code owner March 7, 2025 00:09
@chengchuanpeng chengchuanpeng force-pushed the chengchuanp-promte_tcpcheck_receiver_to_alpha branch from d1a9a8b to f5cb21e Compare March 8, 2025 02:58
@atoulme atoulme changed the title Add tcpcheckrecvier To Contrib Add tcpcheckreceiver To Contrib Mar 8, 2025
@atoulme
Copy link
Contributor

atoulme commented Mar 8, 2025

It's not working because it's not released yet. It was added after 0.121.0 was shipped. BTW, please add it to cmd/otelcontribcol/manifest.yaml in contrib so it's tested.

@chengchuanpeng
Copy link
Contributor Author

It's not working because it's not released yet. It was added after 0.121.0 was shipped. BTW, please add it to cmd/otelcontribcol/manifest.yaml in contrib so it's tested.

Hi @atoulme , Thanks!
I have a question, I don't see manifest.yaml in cmd/otelcontribcol in contrib repo, is it builder-config.yaml ?

@atoulme
Copy link
Contributor

atoulme commented Mar 10, 2025

yes, that's right.

@chengchuanpeng chengchuanpeng force-pushed the chengchuanp-promte_tcpcheck_receiver_to_alpha branch from faba014 to d66996a Compare March 11, 2025 16:13
@chengchuanpeng
Copy link
Contributor Author

@atoulme
Copy link
Contributor

atoulme commented Mar 18, 2025

Please fix conflicts, rebasing off main, and we can get this in.

@chengchuanpeng chengchuanpeng force-pushed the chengchuanp-promte_tcpcheck_receiver_to_alpha branch from 74d4496 to 797110e Compare March 18, 2025 18:00
chengchuanpeng and others added 2 commits March 18, 2025 12:00
@chengchuanpeng chengchuanpeng force-pushed the chengchuanp-promte_tcpcheck_receiver_to_alpha branch from 797110e to 6b6c3f9 Compare March 18, 2025 18:00
@chengchuanpeng
Copy link
Contributor Author

@atoulme done, would you helo me approve? Thanks!

@atoulme atoulme enabled auto-merge March 18, 2025 19:06
@atoulme atoulme added this pull request to the merge queue Mar 18, 2025
@atoulme atoulme removed this pull request from the merge queue due to a manual request Mar 18, 2025
@atoulme atoulme enabled auto-merge March 18, 2025 19:07
@atoulme atoulme added this pull request to the merge queue Mar 18, 2025
Merged via the queue into open-telemetry:main with commit c3d70c3 Mar 18, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants