Skip to content

Conversation

@steventux
Copy link
Contributor

@steventux steventux commented Dec 2, 2025

Description

We get a lot of debug logging around azure sdk operations which involved http requests, eg. fetching MI tokens or performing blob storage operations, azure monitoring also produces a lot of verbose output.
These are often multiline statements with little value to us in normal running operations.
For the purposes of tracing execution of container apps & jobs these log entries obscure useful messaging so set their log levels to reduce the unneeded output.

eg.:

image

Jira link

https://nhsd-jira.digital.nhs.uk/browse/DTOSS-11544

Review notes

azure.core.pipeline.policies.http_logging_policy is particularly verbose hence the warning level. At info level we see a lot of request debug output.

Review checklist

  • Check database queries are correctly scoped to current_provider

@github-actions
Copy link

github-actions bot commented Dec 2, 2025

The review app at this URL has been deleted:
https://pr-796.manage-breast-screening.non-live.screening.nhs.uk

@steventux steventux force-pushed the reduce-azure-identity-logging-verbosity branch from c914ae1 to 5f0077a Compare December 2, 2025 16:32
@steventux steventux force-pushed the reduce-azure-identity-logging-verbosity branch from 5f0077a to e1636be Compare December 2, 2025 16:43
@steventux steventux force-pushed the reduce-azure-identity-logging-verbosity branch from e1636be to 02afdda Compare December 3, 2025 13:38
@steventux steventux force-pushed the reduce-azure-identity-logging-verbosity branch from 02afdda to ce67079 Compare December 3, 2025 15:33
@steventux steventux force-pushed the reduce-azure-identity-logging-verbosity branch from ce67079 to 8fa3bd0 Compare December 3, 2025 16:01
@steventux steventux force-pushed the reduce-azure-identity-logging-verbosity branch from 8fa3bd0 to 4f75262 Compare December 3, 2025 16:22
@steventux steventux force-pushed the reduce-azure-identity-logging-verbosity branch from 4f75262 to a35d80b Compare December 3, 2025 16:47
@steventux steventux force-pushed the reduce-azure-identity-logging-verbosity branch from a35d80b to f740fe6 Compare December 4, 2025 12:06
@steventux steventux force-pushed the reduce-azure-identity-logging-verbosity branch 2 times, most recently from 1fd4044 to b91c21f Compare December 4, 2025 12:34
@steventux steventux changed the title Reduce azure.identity logging verbosity Reduce azure token cache, monitoring and http request logging verbosity Dec 4, 2025
@steventux steventux force-pushed the reduce-azure-identity-logging-verbosity branch from b91c21f to 15da69a Compare December 4, 2025 12:48
Suppress a lot of DEBUG and some overly verbose INFO level output around
azure sdk http requests for monitoring, token retrieval and blob storage operations.
@steventux steventux force-pushed the reduce-azure-identity-logging-verbosity branch from 15da69a to 845c8ec Compare December 4, 2025 13:00
@steventux steventux marked this pull request as ready for review December 4, 2025 13:00
@steventux steventux requested review from a team, MatMoore and gpeng December 4, 2025 13:00
@steventux steventux merged commit 53fc401 into main Dec 8, 2025
13 checks passed
@steventux steventux deleted the reduce-azure-identity-logging-verbosity branch December 8, 2025 12:59
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