Skip to content

Conversation

@nikhilNava
Copy link
Contributor

Task
Only support prod observability cluster and scope

Copilot AI review requested due to automatic review settings November 14, 2025 20:59
@nikhilNava nikhilNava requested a review from a team as a code owner November 14, 2025 20:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR simplifies the observability configuration to only support production endpoints, removing support for environment-specific test and preprod clusters. The changes consolidate all non-production environments to use the production Power Platform API endpoints.

Key changes:

  • Removed environment-specific observability scopes and cluster categories (test, preprod)
  • Simplified get_observability_authentication_scope() to always return production scope
  • Updated cluster-to-suffix mapping to route dev/test/preprod to production endpoints
  • Removed get_observability_cluster_category() function entirely

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
power_platform_api_discovery.py Updated cluster-to-suffix mapping to route dev/test/preprod clusters to production API endpoints (api.powerplatform.com)
environment_utils.py Removed environment-specific constants and simplified authentication scope function to always return production scope; removed cluster category function

fpfp100
fpfp100 previously approved these changes Nov 14, 2025
@nikhilNava nikhilNava merged commit 9a68182 into main Nov 14, 2025
8 checks passed
@nikhilNava nikhilNava deleted the nikhilc/useProd branch November 14, 2025 23:21
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.

6 participants