feat: AZD AI Template validation - devcontainers + azure-dev action#7
Merged
Conversation
…onality for eventhub connection removal; add helper functions for API interactions
…al script to provide detailed summary of operations and handle failures gracefully
- Deleted the fabric_rti_helper.py file as its functionality has been refactored and distributed across multiple new modules. - Introduced fabric_auth.py for authentication handling with the Fabric API. - Created fabric_common_utils.py for common utility functions used across the Fabric scripts. - Modularized connection deletion into fabric_connection_delete.py, providing functions to delete connections by name or ID. - Modularized workspace deletion into fabric_workspace_delete.py, providing functions to look up and delete workspaces by name or ID. - Updated remove_fabric_rti.py to utilize the new modular structure, improving readability and maintainability. - Enhanced error handling and logging across all modules for better debugging and user feedback.
- Updated error messages across various scripts to provide clearer context and consistency. - Simplified authentication and connection logic in `fabric_auth.py`, `fabric_connection_delete.py`, and `fabric_workspace_delete.py`. - Enhanced user feedback in `fabric_eventhub.py`, `fabric_eventhouse.py`, and `fabric_eventstream.py` to indicate success or failure of operations. - Consolidated redundant code and improved readability in multiple scripts. - Added detailed module descriptions and usage examples for better documentation.
… and improve modularity
- Updated `create_activator`, `update_activator_definition`, `setup_eventhouse`, `create_eventstream`, `update_eventstream_definition`, and `setup_real_time_dashboard` functions to accept `FabricWorkspaceApiClient` instances instead of workspace IDs. - Introduced `authenticate_workspace` function in `fabric_auth.py` to handle workspace-specific authentication. - Added a new script `fabric_workspace_admins.py` for managing workspace administrators, supporting both UPNs and object IDs. - Enhanced error handling and logging across scripts for better clarity and debugging. - Removed unused parameters and streamlined API client initialization.
- Renamed output variable for Fabric Capacity Admin members in Bicep file for clarity. - Removed deprecated Python script for adding Fabric workspace administrators and replaced it with a new script that supports CSV input for admin identifiers. - Updated deployment script to reflect changes in output variable names. - Consolidated workspace administrator setup logic to streamline the process and improve error handling. - Added new script for deleting Fabric workspaces, ensuring proper orchestration of removal functions. - Updated PowerShell script to reference the new delete Fabric RTI script instead of the removed one.
…mprove Bicep template validation
…Azure Dev workflow
alguadam
requested review from
DocGailZhou,
Yamini-Microsoft and
sethsteenken
sethsteenken
reviewed
Dec 1, 2025
…rating workspace authentication in eventstream and dashboard scripts
sethsteenken
approved these changes
Dec 1, 2025
|
🎉 This PR is included in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
This pull request addresses all requirements to pass AZD AI Template Validation requirements
CI/CD and Workflow Changes:
.github/workflows/azure-dev.ymland.github/workflows/azd-ai-template-validation.ymlto use suffixes (dfor dev,valfor validation) and simplified environment variable usage.Configuration and Template Updates:
FABRIC_WORKSPACE_ADMINISTRATORSas an environment variable, allowing users to provide a comma-separated list of workspace administrators.azure.yamlto "real-time-intelligence-operations-solution-accelerator" and added template metadata for clarity and discoverability.azure.yamlto usedelete_fabric_rti.pyinstead ofremove_fabric_rti.pyfor environment teardown.Documentation Improvements:
docs/DeploymentGuide.mdto point to the correct GitHub repository and reflect the new configuration options, including workspace administrators.FABRIC_WORKSPACE_ADMINISTRATORSvariable and the new cleanup script.Does this introduce a breaking change?
How to Test
Locally:
Manual trigger of all available actions
What to Check
Verify that the following are valid
Other Information