Skip to content

Conversation

Angith
Copy link
Contributor

@Angith Angith commented Sep 12, 2025

This PR enables Go Tracer to disable log spans at tracer level.

There are four ways to disable log spans:

  1. Using Code
  2. Using Environment Variables
  3. Using Configuration File
  4. Using agent configuration

@Angith Angith requested a review from a team as a code owner September 12, 2025 11:24
@Angith Angith marked this pull request as draft September 12, 2025 11:25
@Angith Angith self-assigned this Sep 15, 2025
@Angith Angith marked this pull request as ready for review September 16, 2025 07:10
@Angith Angith added the tekton_ci Add this label to start running Tekton pipelines label Sep 16, 2025
Copy link
Member

@sanojsubran sanojsubran left a comment

Choose a reason for hiding this comment

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

@Angith I have done one round of review. Kindly check my comments. We can discuss together.

@Angith Angith requested a review from sanojsubran September 24, 2025 06:05
Copy link
Member

@sanojsubran sanojsubran left a comment

Choose a reason for hiding this comment

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

@Angith The changes look good. I have added few suggestions. Kindly check.

Angith and others added 4 commits September 24, 2025 14:56
Co-authored-by: Sanoj Subran <sanoj.subran@ibm.com>
Co-authored-by: Sanoj Subran <sanoj.subran@ibm.com>
@Angith Angith requested a review from sanojsubran September 24, 2025 11:43
Copy link
Member

@nithinputhenveettil nithinputhenveettil left a comment

Choose a reason for hiding this comment

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

@Angith : Just a few minor changes/suggestions.

Angith and others added 2 commits September 26, 2025 11:05
Co-authored-by: Nithin Puthenveettil <nithin.puthenveettil@ibm.com>
Co-authored-by: Nithin Puthenveettil <nithin.puthenveettil@ibm.com>
Co-authored-by: Nithin Puthenveettil <nithin.puthenveettil@ibm.com>
Copy link
Member

@nithinputhenveettil nithinputhenveettil left a comment

Choose a reason for hiding this comment

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

:shipit:

@Angith Angith requested a review from sanojsubran September 29, 2025 11:04
@Angith Angith force-pushed the disabling-log-spans branch from e242207 to bfbee64 Compare September 29, 2025 11:08
@Angith Angith requested a review from sanojsubran September 29, 2025 12:22
Copy link
Member

@sanojsubran sanojsubran left a comment

Choose a reason for hiding this comment

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

Good job @Angith!

Copy link

@Angith Angith merged commit 5c262a9 into main Sep 29, 2025
5 checks passed
@Angith Angith deleted the disabling-log-spans branch September 29, 2025 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tekton_ci Add this label to start running Tekton pipelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants