Skip to content

Disabling log collection at the tracer level#768

Merged
GSVarsha merged 5 commits into
mainfrom
log_disabling
Jul 31, 2025
Merged

Disabling log collection at the tracer level#768
GSVarsha merged 5 commits into
mainfrom
log_disabling

Conversation

@pvital
Copy link
Copy Markdown
Member

@pvital pvital commented Jul 29, 2025

We need to disable log collection at the tracer level to prevent duplication in the backend when both the application tracer and an OpenTelemetry collector are running on the same system.

This PR adds support for the Span Disabling feature, which allows the exclusion of specific traces or calls from tracing based on the category (technology) or type (frameworks, libraries, instrumentations) supported by the traces.

In addition, it refactors the initialization of the Instana module and the ConfigReader class.

Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
@pvital pvital requested a review from a team as a code owner July 29, 2025 22:03
@pvital pvital self-assigned this Jul 29, 2025
@pvital pvital added Review & Merge FUP Fair Usage Policy labels Jul 29, 2025
@pvital pvital force-pushed the log_disabling branch 2 times, most recently from e3c5c1c to 5bc5e21 Compare July 29, 2025 22:33
Comment thread tests/clients/test_logging.py Outdated
Comment thread tests/clients/test_logging.py
Comment thread tests/util/test_configuration-1.yaml Outdated
Comment thread tests/util/test_configuration-2.yaml Outdated
Copy link
Copy Markdown
Contributor

@GSVarsha GSVarsha left a comment

Choose a reason for hiding this comment

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

Just a few requests.

pvital added 3 commits July 30, 2025 09:52
The span disabling allows the exclusion of specific traces or calls from tracing based on the category (technology) or type (frameworks, libraries, instrumentations) supported by the traces.

This commit adds support to handle the configuration from the INSTANA_TRACING_DISABLE, INSTANA_CONFIG_PATH, and Agent-provided configuration.

Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
Disabling log spans collection at the tracer level to prevent duplication in the backend when both the application tracer and an OpenTelemetry collector are running on the same system.

Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
@pvital pvital requested a review from GSVarsha July 30, 2025 07:52
Comment thread src/instana/__init__.py Outdated
@pvital pvital added the feature label Jul 30, 2025
Comment thread src/instana/__init__.py Outdated
The `INSTANA_DISABLE` is not a general environment variable from Instana, and with the adoption of the new `INSTANA_TRACING_DISABLE` we can have the same effect. So this commit removes the legacy one.

Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
@pvital pvital requested a review from GSVarsha July 30, 2025 10:21
Copy link
Copy Markdown
Contributor

@GSVarsha GSVarsha left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@GSVarsha GSVarsha merged commit 4beed5f into main Jul 31, 2025
17 checks passed
@GSVarsha GSVarsha deleted the log_disabling branch July 31, 2025 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants