Skip to content

Commit

Permalink
fix(trace-action): remove azidentity (#1068)
Browse files Browse the repository at this point in the history
  • Loading branch information
sduranc authored Oct 30, 2024
1 parent 06872e0 commit 90c729e
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 264 deletions.
254 changes: 0 additions & 254 deletions actions/send-ci-cd-trace/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 0 additions & 1 deletion actions/send-ci-cd-trace/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
"dependencies": {
"@actions/core": "^1.11.1",
"@actions/github": "^6.0.0",
"@azure/identity": "^4.5.0",
"@azure/monitor-opentelemetry": "^1.8.0",
"@azure/monitor-opentelemetry-exporter": "^1.0.0-beta.27",
"@opentelemetry/api": "^1.9.0",
Expand Down
9 changes: 0 additions & 9 deletions actions/send-ci-cd-trace/src/index.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
import * as core from "@actions/core";
import * as github from "@actions/github";
import { DefaultAzureCredential } from "@azure/identity";
import { ReadableSpan, Span, SpanProcessor } from "@opentelemetry/sdk-trace-base";
import {
trace,
Expand Down Expand Up @@ -57,17 +56,9 @@ async function run() {
}
}

let credential;
try {
credential = new DefaultAzureCredential();
} catch (error) {
console.warn("Azure credential not available, proceeding without it.");
}

const options: AzureMonitorOpenTelemetryOptions = {
azureMonitorExporterOptions: {
connectionString: connectionString,
...(credential && { credential: credential }),
},
spanProcessors: [new SpanEnrichingProcessor()]
};
Expand Down

0 comments on commit 90c729e

Please sign in to comment.