chore: disable tracing on integration tests#88
Conversation
radu-mocanu
commented
Feb 16, 2026
radu-mocanu
commented
- disable tracing on integration tests
There was a problem hiding this comment.
Pull request overview
This PR aims to disable tracing during the integration-test jobs in the GitHub Actions workflows, likely to reduce noise/cost and avoid trace emission during CI runs.
Changes:
- Adds
UIPATH_TRACING_ENABLED: falseto the container environment for UiPath integration test jobs. - Adds the same environment variable to the container environment for UiPath Langchain integration test jobs.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| .github/workflows/test-uipath.yml | Sets UIPATH_TRACING_ENABLED=false for the integration-test container job. |
| .github/workflows/test-uipath-langchain.yml | Sets UIPATH_TRACING_ENABLED=false for the integration-test container job. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| env: | ||
| UIPATH_JOB_KEY: "3a03d5cb-fa21-4021-894d-a8e2eda0afe0" | ||
| UIPATH_TRACING_ENABLED: false |
There was a problem hiding this comment.
UIPATH_TRACING_ENABLED is set here, but in this repo it appears to be parsed and then never applied (see UiPathRuntimeContext.with_defaults in src/uipath/runtime/context.py, where tracing_enabled is computed but not assigned to the context or used to conditionally create/disable a trace manager). Unless another component reads this env var directly, this workflow change likely won’t actually disable tracing for the integration tests. Consider wiring the env var into trace manager initialization (or removing tracing setup when disabled) and/or updating the workflow to use the env var that the tracing implementation actually honors.
| env: | ||
| UIPATH_JOB_KEY: "3a03d5cb-fa21-4021-894d-a8e2eda0afe0" | ||
| UIPATH_TRACING_ENABLED: false |
There was a problem hiding this comment.
UIPATH_TRACING_ENABLED is set here, but in this repo it appears to be parsed and then never applied (see UiPathRuntimeContext.with_defaults in src/uipath/runtime/context.py, where tracing_enabled is computed but not assigned to the context or used to conditionally create/disable a trace manager). Unless another component reads this env var directly, this workflow change likely won’t actually disable tracing for the integration tests. Consider wiring the env var into trace manager initialization (or removing tracing setup when disabled) and/or updating the workflow to use the env var that the tracing implementation actually honors.
2 participants
