Skip to content

NPA-7041: Add monitoring only for int and prod - #384

Merged
ellie-bound1-NHSD merged 2 commits into
masterfrom
release/NPA-7041
Aug 5, 2026
Merged

NPA-7041: Add monitoring only for int and prod#384
ellie-bound1-NHSD merged 2 commits into
masterfrom
release/NPA-7041

Conversation

@ellie-bound1-NHSD

Copy link
Copy Markdown
Contributor

Pull Request

🧾 Ticket Link

https://nhsd-jira.digital.nhs.uk/browse/NPA-7041


📄 Description/Summary of Changes

  • Disable monitoring in lower envs

🧪 Developer Testing Carried Out


📋 PR Principles

  • Keep PRs Small and Focused: Ensure the PR addresses a single task or feature to make it easier to review.
  • Multiple PRs for one Ticket: When splitting work into multiple PRs, clearly describe what this PR addresses and outline the remaining work to complete the ticket.
  • Ensure Tests Are Included: Add or update unit, integration, or end-to-end tests to cover the changes made.
  • Follow Coding Standards: Ensure the code adheres to the team's coding guidelines and best practices.
  • Resolve Comments Promptly: If you raise a comment, ensure you follow up and resolve it before approving the PR to maintain clarity and ensure comments are addressed.
  • Foster Learning: PR reviews are an opportunity to share knowledge, provide constructive feedback, and encourage a collaborative environment.

🏷️ Naming Conventions Reminder

Please ensure the following naming conventions are followed:

  • PR title follows the format: NPA-XXXX: <short-description>
  • Branch name follows the convention: <type>/NPA-XXXX/<short-description>
  • Commit messages follow the template: NPA-XXXX: <short-description>

Copilot AI review requested due to automatic review settings August 4, 2026 15:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Azure release pipeline’s Apigee deployment configuration to ensure monitoring is only enabled for the int and prod environments, aligning with the ticket goal of disabling monitoring in lower environments.

Changes:

  • Added enable_monitoring and enable_status_monitoring flags to each apigee_deployments entry.
  • Explicitly disabled both monitoring flags for internal-dev, internal-dev-sandbox, internal-qa, internal-qa-sandbox, and sandbox.
  • Explicitly enabled both monitoring flags for int and prod.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ellie-bound1-NHSD
ellie-bound1-NHSD merged commit 792edb3 into master Aug 5, 2026
18 checks passed
@ellie-bound1-NHSD
ellie-bound1-NHSD deleted the release/NPA-7041 branch August 5, 2026 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants