Skip to content

[Cloud] az cloud show: Add missing synapseAnalyticsResourceId for AzureBleuCloud - #34188

Open
Yeming Liu (isra-fel) wants to merge 1 commit into
Azure:devfrom
isra-fel:fix/bleu-synapse-resource-id-34059
Open

Yeming Liu (isra-fel) wants to merge 1 commit into
Azure:devfrom
isra-fel:fix/bleu-synapse-resource-id-34059

Conversation

@isra-fel

Copy link
Copy Markdown
Member

Related command
az cloud show --name AzureBleuCloud

Description
Fixes #34059.

AzureBleuCloud defined synapse_analytics_endpoint (.dev.azuresynapse.sovcloud-api.fr) but was missing its paired synapse_analytics_resource_id. As a result, az cloud show --name AzureBleuCloud returned "synapseAnalyticsResourceId": null even though Synapse is available in Bleu.

Every other known cloud pairs the two values consistently:

Cloud synapse_analytics_resource_id synapse_analytics_endpoint
Public https://dev.azuresynapse.net .dev.azuresynapse.net
China https://dev.azuresynapse.azure.cn .dev.azuresynapse.azure.cn
US Gov https://dev.azuresynapse.usgovcloudapi.net .dev.azuresynapse.usgovcloudapi.net
Bleu (before) (missing) .dev.azuresynapse.sovcloud-api.fr
Bleu (after) https://dev.azuresynapse.sovcloud-api.fr .dev.azuresynapse.sovcloud-api.fr

This PR adds synapse_analytics_resource_id='https://dev.azuresynapse.sovcloud-api.fr' to the Bleu cloud definition, following the established pattern.

Scope note: #34059 lists several other properties (Data Lake, App Insights, Attestation, Log Analytics, etc.) that are unset for Bleu. As the reporter notes, those fall into "never/only-public/only-public+national" buckets and require a per-RP availability decision, so they are intentionally left out of this focused fix. The Synapse one is an unambiguous bug because its suffix counterpart is already present.

The broader sovereign-cloud onboarding (Delos + GovSG) tracked in #34129 is not included here: the authoritative endpoints table currently publishes only placeholder values ("replace fr suffix with de") for several required Delos/GovSG endpoints, so those clouds cannot yet be added with verified values.

Testing Guide

az cloud show --name AzureBleuCloud --query endpoints.synapseAnalyticsResourceId
# -> "https://dev.azuresynapse.sovcloud-api.fr"

Unit tests: azdev test --src-file src/azure-cli-core/azure/cli/core/tests/test_cloud.py (14 passed, 1 skipped).

History Notes
[Cloud] az cloud show: Add missing synapseAnalyticsResourceId for AzureBleuCloud


This checklist is used to make sure that common guidelines for a pull request are followed.

AzureBleuCloud defined synapse_analytics_endpoint
(.dev.azuresynapse.sovcloud-api.fr) but was missing its paired
synapse_analytics_resource_id, so 'az cloud show --name AzureBleuCloud'
returned synapseAnalyticsResourceId: null. Every other cloud pairs the
two values (https://dev.azuresynapse.<suffix> and
.dev.azuresynapse.<suffix>); apply the same pattern for Bleu.

Fixes Azure#34059

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 7707d164-8d38-41f6-9884-04c670e74640

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.

🟡 Changes recommended

A focused regression test is needed to prevent the Bleu resource ID from silently returning to null.

1 open finding
What changed in this PR

Adds the missing Synapse authentication resource ID for Azure Bleu Cloud.

Changes:

  • Configures the Bleu Synapse resource ID.
  • Documents the fix in release history.
File Description
src/​azure-cli-core/​azure/​cli/​core/​cloud.py Adds the Bleu Synapse resource ID.
src/​azure-cli-core/​HISTORY.rst Records the cloud metadata fix.

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

vm_image_alias_doc='https://azcliprod.blob.core.windows.net/cli/vm/aliases_master.json',
media_resource_id='https://rest.media.sovcloud-api.fr',
ossrdbms_resource_id='https://ossrdbms-aad.database.sovcloud-api.fr',
synapse_analytics_resource_id='https://dev.azuresynapse.sovcloud-api.fr',
@yonzhan

Copy link
Copy Markdown
Collaborator

Add missing synapseAnalyticsResourceId for AzureBleuCloud

This branch has not been deployed

No deployments
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.

Missing sovereign cloud metadata (Bleu + Blackforest)

3 participants