Skip to content

[DTOSS-12472] - feat(variables): enable statsbeat disablement across all environments#1189

Open
josielsouzanordcloud wants to merge 1 commit intomainfrom
DTOSS-12472-investigate-app-insights-region-config
Open

[DTOSS-12472] - feat(variables): enable statsbeat disablement across all environments#1189
josielsouzanordcloud wants to merge 1 commit intomainfrom
DTOSS-12472-investigate-app-insights-region-config

Conversation

@josielsouzanordcloud
Copy link
Contributor

@josielsouzanordcloud josielsouzanordcloud commented Mar 17, 2026

Description

This PR enables the global statsbeat collection feature for Application Insights to be disabled across all environments by introducing a new variable APPLICATIONINSIGHTS_STATSBEAT_DISABLED_ALL. This change is aimed at aligning the configuration and reducing unnecessary data collection in development, pre-production, production, and review environments.

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Infrastructure change
  • Pipeline / CI
  • Documentation update
  • Refactor

Jira link

DTOSS-12472

Review notes

  • Added APPLICATIONINSIGHTS_STATSBEAT_DISABLED_ALL: True to all environment variable files (dev, preprod, prod, review) to disable statsbeat collection globally.

Breaking changes

  • None

Impact

  • No impact on deployment or services.
  • Reduces data collection in Application Insights, which may reduce costs and improve focus on relevant telemetry data.
  • No security or privacy concerns introduced by this change.

Review checklist

  • No hardcoded secrets, credentials, or tokens
  • PR title follows correct format (with ticket prefix if ticket provided, without if not)
  • terraform plan reviewed — no unintended resource destruction
  • Mandatory tags on all new resources
  • Resource locks on critical resources
  • Variable changes propagated to all environment .tfvars files
  • State file impact assessed

Pipeline (*.yml, *.yaml):

  • No secrets hardcoded in pipeline YAML
  • GPG signed commits configured correctly
  • Environment approval gates in place for production stages

Application code (*.py, *.ts, *.js, *.cs):

  • Unit tests written or updated
  • Error handling covers failure paths
  • No debug logging left in production paths

@github-actions
Copy link

The review app is available at this URL:
https://pr-1189.manage-breast-screening.non-live.screening.nhs.uk
You must authenticate with HTTP basic authentication. Ask the team for credentials.

@josielsouzanordcloud josielsouzanordcloud changed the title [DTOSS-12472] - feat(variables): enable statsbeat disablement in review environment [DTOSS-12472] - feat(variables): enable statsbeat disablement across all environments Mar 19, 2026
@josielsouzanordcloud josielsouzanordcloud force-pushed the DTOSS-12472-investigate-app-insights-region-config branch from 56df6c5 to 51a6ced Compare March 19, 2026 15:41
@josielsouzanordcloud josielsouzanordcloud force-pushed the DTOSS-12472-investigate-app-insights-region-config branch from 51a6ced to e5f9f8a Compare March 19, 2026 18:01
  Hardcodes APPLICATIONINSIGHTS_STATSBEAT_DISABLED_ALL=true in the
  container-apps module's common environment variables to suppress
  statsbeat telemetry across all environments.

  Refs: DTOSS-12472
@josielsouzanordcloud josielsouzanordcloud force-pushed the DTOSS-12472-investigate-app-insights-region-config branch from e5f9f8a to 5d6aacc Compare March 19, 2026 18:01
@sonarqubecloud
Copy link

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