Skip to content

Commit aa4cbe2

Browse files
chore: enable telemetry and upgrade deps (#125)
1 parent aa4bd5d commit aa4cbe2

5 files changed

Lines changed: 10 additions & 10 deletions

File tree

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ readme = { file = "README.md", content-type = "text/markdown" }
66
requires-python = ">=3.11"
77
dependencies = [
88
"uipath==2.5.15",
9-
"uipath-langchain[vertex,bedrock]==0.4.12",
9+
"uipath-langchain[vertex,bedrock]==0.4.14",
1010
"langgraph>=1.0.0,<2.0.0",
1111
"azure-monitor-opentelemetry>=1.7.0,<2.0.0",
1212
]

src/uipath_agents/_observability/tracing.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ def configure_telemetry(trace_manager: UiPathTraceManager | None = None) -> None
117117

118118
def _get_azure_exporter() -> AzureMonitorTraceExporter | None:
119119
"""Get Azure Monitor trace exporter if connection string is configured."""
120-
connection_string = os.getenv("APPLICATIONINSIGHTS_CONNECTION_STRING")
120+
connection_string = os.getenv("TELEMETRY_CONNECTION_STRING")
121121
if not connection_string:
122122
logger.debug("Azure Monitor exporter not configured - no connection string")
123123
return None

tests/e2e/test_telemetry.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,9 @@ def app_insights_url(self) -> str:
2424
"""Get Application Insights API URL from connection string."""
2525
import os
2626

27-
conn_str = os.getenv("APPLICATIONINSIGHTS_CONNECTION_STRING")
27+
conn_str = os.getenv("TELEMETRY_CONNECTION_STRING")
2828
if not conn_str:
29-
pytest.skip("APPLICATIONINSIGHTS_CONNECTION_STRING not configured")
29+
pytest.skip("TELEMETRY_CONNECTION_STRING not configured")
3030

3131
# Parse connection string to get Application ID
3232
parts = {}
@@ -181,7 +181,7 @@ def test_telemetry_events_sent_to_app_insights(
181181
# If we get here, check if telemetry is enabled but instrumentation key is invalid
182182
import os
183183

184-
conn_str = os.getenv("APPLICATIONINSIGHTS_CONNECTION_STRING", "")
184+
conn_str = os.getenv("TELEMETRY_CONNECTION_STRING", "")
185185
if "your-key-here" in conn_str:
186186
pytest.skip(
187187
"Telemetry test requires valid Application Insights instrumentation key"

tests/unit/observability/test_tracing.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ def test_with_connection_string_returns_exporter(self):
106106
with (
107107
patch.dict(
108108
os.environ,
109-
{"APPLICATIONINSIGHTS_CONNECTION_STRING": conn_str},
109+
{"TELEMETRY_CONNECTION_STRING": conn_str},
110110
clear=True,
111111
),
112112
patch.object(tracing, "AzureMonitorTraceExporter") as mock_exporter,

uv.lock

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)