Skip to content

Rename tcplog receiver to tcp_log#1687

Merged
aphralG merged 1 commit into
nginx:release-3.10.1from
ciarams87:fix/tcp-collector
May 15, 2026
Merged

Rename tcplog receiver to tcp_log#1687
aphralG merged 1 commit into
nginx:release-3.10.1from
ciarams87:fix/tcp-collector

Conversation

@ciarams87
Copy link
Copy Markdown
Contributor

Proposed changes

Issue: The tcplog receiver has been renamed to tcp_log in open-telemetry/opentelemetry-collector-contrib#47369

Proposed fix: Rename tcplog -> tcp_log

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING document
  • I have run make install-tools and have attached any dependency changes to this pull request
  • If applicable, I have added tests that prove my fix is effective or that my feature works
  • If applicable, I have checked that any relevant tests pass after adding my changes
  • If applicable, I have updated any relevant documentation (README.md)
  • If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

@github-actions github-actions Bot added bug Something isn't working chore Pull requests for routine tasks labels May 15, 2026
@ciarams87 ciarams87 force-pushed the fix/tcp-collector branch from d5b47b1 to 3eae6a6 Compare May 15, 2026 08:34
@ciarams87 ciarams87 changed the base branch from main to release-3.10.1 May 15, 2026 09:27
@ciarams87 ciarams87 marked this pull request as ready for review May 15, 2026 09:32
@ciarams87 ciarams87 requested a review from a team as a code owner May 15, 2026 09:32
@codecov
Copy link
Copy Markdown

codecov Bot commented May 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.00%. Comparing base (0938f7c) to head (2287116).

Additional details and impacted files
@@                Coverage Diff                 @@
##           release-3.10.1    #1687      +/-   ##
==================================================
+ Coverage           84.86%   85.00%   +0.13%     
==================================================
  Files                 104      104              
  Lines               13579    13579              
==================================================
+ Hits                11524    11543      +19     
+ Misses               1542     1522      -20     
- Partials              513      514       +1     

see 6 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0938f7c...2287116. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ciarams87 ciarams87 changed the base branch from release-3.10.1 to main May 15, 2026 10:08
@ciarams87 ciarams87 changed the base branch from main to release-3.10.1 May 15, 2026 10:14
@ciarams87 ciarams87 force-pushed the fix/tcp-collector branch from 3eae6a6 to 1178484 Compare May 15, 2026 10:17
@ciarams87 ciarams87 force-pushed the fix/tcp-collector branch from 1178484 to 2287116 Compare May 15, 2026 10:22
@aphralG aphralG merged commit 3e73074 into nginx:release-3.10.1 May 15, 2026
52 of 57 checks passed
@ciarams87 ciarams87 deleted the fix/tcp-collector branch May 15, 2026 13:45
aphralG added a commit that referenced this pull request May 15, 2026
Co-authored-by: Ciara Stacke <18287516+ciarams87@users.noreply.github.com>
craigell pushed a commit that referenced this pull request May 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working chore Pull requests for routine tasks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants