-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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/tcpcheck] Add New Receiver#34458 #37556
base: main
Are you sure you want to change the base?
[receiver/tcpcheck] Add New Receiver#34458 #37556
Conversation
@yanfeng1992 To add you as a code owner for running pipeline checks, could you create a new PR for membership? Thanks! Here’s an example: link |
I am sorry for the mistake earlier. I've cleaned up my commits and reopened the PR. However, I accidentally included other commits, which unintentionally involved multiple people. |
Codecov ReportAttention: Patch coverage is
❌ Your patch check has failed because the patch coverage (53.54%) is below the target coverage (95.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #37556 +/- ##
==========================================
+ Coverage 79.58% 79.62% +0.03%
==========================================
Files 2292 2314 +22
Lines 213536 216777 +3241
==========================================
+ Hits 169939 172601 +2662
- Misses 37893 38415 +522
- Partials 5704 5761 +57 ☔ View full report in Codecov by Sentry. |
27d7b03
to
9121225
Compare
[receiver/tcpcheck] Change CODEOWNERS#34458 [receiver/tcpcheck] Change CODEOWNERS#34458 [receiver/tcpcheck] Make Fmt#34458 [receiver/tcpcheck] Fix Version And Test#34458 [receiver/tcpcheck] Make Fmt#34458 [receiver/tcpcheck] Temporarily Change Owner#34458 [receiver/tcpcheck] Change Version#34458 [receiver/tcpcheck] Make Fmt#34458 [receiver/tcpcheck] Rebase#34458 [receiver/tcpcheck] Fix Test#34458 [receiver/tcpcheck] Make Tidy#34458
d37e646
to
dd615eb
Compare
Signed-off-by: huangyanfeng [email protected], chengchuanpeng [email protected],
This PR builds upon the work from @yanfeng1992. We really appreciate the effort and thought he put into the original implementation, which laid the groundwork for this update. While there were some changes needed, his contribution was instrumental in guiding the direction of this PR
Description:
To monitor the availability and performance of TCP endpoints.
Use case: TCP network detection plug-in, usually used to monitor whether a TCP port on the local machine is listening, or whether a remote port can be connected
Link to tracking Issue:
#34414
Testing:
(1) PR Includes coverage ~85% in the package with the scraper.
(2) Also includes coverage ~80% in the internal/configtcp package
Documentation:
The new receiver includes documentation in receiver/tcpcheckreceiver/README.md and receiver/tcpcheckreceiver/documentation.md.