chore: dev to main merge

[Saswato-Microsoft]

Purpose

This pull request focuses on improving the reliability and maintainability of the Azure Event Hub integration, particularly for Fabric SAS connections, and includes several code cleanups and minor dependency adjustments. The most significant change is the normalization of Event Hub endpoints to ensure compatibility and prevent connection issues. Additionally, the pull request introduces targeted tag overrides for Event Hub resources and removes unused imports for cleaner scripts.

Event Hub Integration Improvements

• Added a _normalize_eventhub_endpoint static method in fabric_api.py to standardize Event Hub endpoints, ensuring that both bare namespaces and full endpoints are converted to the expected host format. This normalization is now used in both connection creation and update flows. [1] [2] [3] [4]
• Set singleSignOnType to "None" in the Event Hub connection payloads to explicitly disable SSO, aligning with the required authentication method for SAS connections. [1] [2]

Infrastructure Tagging

• Introduced eventHubTags in main.bicep to override security policy controls by setting the SecurityControl tag to 'Ignore' for Event Hub resources, and applied these tags to the Event Hub namespace module. [1] [2]

Dependency and Import Cleanups

• Removed unused imports (e.g., argparse, sys, ClientRequestProperties, and certain typing annotations) from various Python scripts to improve code clarity and reduce linting noise. [1] [2] [3] [4] [5] [6] [7] [8] [9]

Tooling Version Control

• Updated the azd required version constraint in azure.yaml to explicitly exclude version 1.23.9 due to compatibility or known issues.

Does this introduce a breaking change?

• Yes
• No

How to Test

• Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]

• Test the code

What to Check

Verify that the following are valid

• ...

Other Information

[Copilot]

Pull request overview

This PR improves the Microsoft Fabric ↔ Azure Event Hub integration (especially for SAS-based connections) by normalizing Event Hub endpoints and aligning connection payloads, while also introducing an Event Hub–specific tag override in the infra deployment and cleaning up unused imports/tooling constraints.

Changes:

• Add Event Hub endpoint normalization and apply it during Event Hub connection create/update flows, including explicitly setting singleSignOnType to "None".
• Add eventHubTags in Bicep to apply a targeted SecurityControl: 'Ignore' tag override to the Event Hub namespace module.
• Remove unused Python imports across Fabric helper scripts and adjust azd version constraints in azure.yaml.

Reviewed changes

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

Show a summary per file

File	Description
infra/scripts/fabric/graph_api.py	Removes unused typing imports.
infra/scripts/fabric/fabric_workspace.py	Removes unused sys import.
infra/scripts/fabric/fabric_eventstream_definition.py	Removes unused typing imports.
infra/scripts/fabric/fabric_eventhub.py	Removes unused sys import.
infra/scripts/fabric/fabric_eventhouse.py	Removes unused typing imports.
infra/scripts/fabric/fabric_database.py	Removes unused imports and drops unused Kusto request properties import.
infra/scripts/fabric/fabric_common_utils.py	Removes unused argparse import.
infra/scripts/fabric/fabric_api.py	Adds endpoint normalization + updates Event Hub connection create/update payloads.
infra/scripts/fabric/fabric_activator_definition.py	Removes unused typing imports.
infra/main.bicep	Adds Event Hub–specific tag overrides and applies them to the namespace module.
azure.yaml	Updates azd required version constraint to exclude a specific version.

Comments suppressed due to low confidence (1)

infra/scripts/fabric/fabric_api.py:785

• In create_eventhub_connection, allowConnectionUsageInGateway is set to the string "false" while update_eventhub_connection uses a boolean False. Since this payload is JSON-encoded from a Python dict, sending a string here can cause the API to reject the request or interpret the value incorrectly; it should be a boolean for consistency with the update flow.

            "displayName": name,
            "connectivityType": "ShareableCloud",
            "allowConnectionUsageInGateway": "false",
            "connectionDetails": {
                "type": "EventHub",

---

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

[github-actions]

🎉 This PR is included in version 1.3.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀