Skip to content

Commit

Permalink
Update to disable bunyan and winston in non-AKS auto-attach.
Browse files Browse the repository at this point in the history
  • Loading branch information
JacksonWeber committed May 10, 2024
1 parent bee21e7 commit 5e9c80e
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 10 deletions.
3 changes: 0 additions & 3 deletions src/agent/agentLoader.ts
Original file line number Diff line number Diff line change
Expand Up @@ -69,9 +69,6 @@ export class AgentLoader {
redis: {
enabled: true
},
bunyan: {
enabled: true
},
}
};

Expand Down
7 changes: 5 additions & 2 deletions src/agent/aksLoader.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,11 @@ export class AKSLoader extends AgentLoader {
// Add OTLP if env variable is present
enabled: process.env["OTEL_EXPORTER_OTLP_METRICS_ENDPOINT"] ? true : false
};
(this._options.instrumentationOptions as InstrumentationOptions).console = {
enabled: true
(this._options.instrumentationOptions as InstrumentationOptions) = {
...this._options.instrumentationOptions,
console: { enabled: true },
bunyan: { enabled: true },
winston: { enabled: true },
}

let statusLogDir = '/var/log/applicationinsights/';
Expand Down
4 changes: 2 additions & 2 deletions src/shared/configuration/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -72,8 +72,8 @@ export class ApplicationInsightsConfig {
redis: { enabled: false },
redis4: { enabled: false },
console: { enabled: false },
bunyan: { enabled: true },
winston: { enabled: true },
bunyan: { enabled: false },
winston: { enabled: false },
};
this._resource = this._getDefaultResource();

Expand Down
3 changes: 0 additions & 3 deletions test/unitTests/agent/agentLoader.tests.ts
Original file line number Diff line number Diff line change
Expand Up @@ -42,9 +42,6 @@ describe("agent/agentLoader", () => {
redis: {
enabled: true
},
bunyan: {
enabled: true
},
}
};

Expand Down

0 comments on commit 5e9c80e

Please sign in to comment.