fix: azd template validation pipeline issue (#113)

* pipeline issue updated

* pipeline issue updated

* psr rule file added

* removed psl rule file

* changed the path

* changed the path

* readme changes

* readme changes

* readme changes

* readme changes

* variable fabric capacity

* variable fabric capacity

* setting fabriccapacity admin to my id

* bicepparams changes to read env variable

* bicepparams changes to read env variable

* readme changes

* readme changes

* path changes

* path changes

* removed pull request

* run pipeline on dev merge

* accept array for admin value

* accept array for admin value

* remove run on merge

* updated doc for fabric capacity admin value

Author: Priyanka-Microsoft

.github/workflows/azd-template-validation.yml

@@ -29,6 +29,6 @@ jobs:
           AZURE_ENV_NAME: ${{ vars.AZURE_ENV_NAME }}
           AZURE_LOCATION: ${{ vars.AZURE_LOCATION }}
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-
+          fabricCapacityMode: 'none'
       - name: print result
         run: cat ${{ steps.validation.outputs.resultFile }}

README.md

@@ -48,6 +48,10 @@ This accelerator extends the [AI Landing Zone](https://github.com/Azure/ai-landi
 <!-------------------------------------------->
 <!-- KEY FEATURES                            -->
 <!-------------------------------------------->
+
+## Features
+
+
 ### Key features
 <details open>
   <summary>Click to learn more about the key features this solution enables</summary>
@@ -73,6 +77,9 @@ This accelerator extends the [AI Landing Zone](https://github.com/Azure/ai-landi
 <!-------------------------------------------->
 <!-- QUICK DEPLOY                            -->
 <!-------------------------------------------->
+
+## Getting Started
+
 <h2><img src="./docs/images/readme/quick-deploy.png" width="48" />
 Quick deploy
 </h2>
@@ -213,6 +220,11 @@ After deployment, you'll have a complete, enterprise-ready platform that unifies
 <!------------------------------------------>
 <!-- SUPPORTING DOCUMENTATION                -->
 <!------------------------------------------>
+
+
+## Guidance
+
+
 <h2><img src="./docs/images/readme/supporting-documentation.png" width="48" />
 Supporting documentation
 </h2>
@@ -251,6 +263,10 @@ Supporting documentation
 
 </details>
 
+
+## Resources
+
+
 ### Cross references
 Check out similar solution accelerators
 | Solution Accelerator | Description |

docs/DeploymentGuide.md

@@ -156,7 +156,7 @@ Edit `infra/main.bicepparam` or set environment variables:
 | `fabricCapacityMode` | Fabric capacity mode: `create`, `byo`, or `none` | `create` |
 | `fabricWorkspaceMode` | Fabric workspace mode: `create`, `byo`, or `none` | `create` |
 | `fabricCapacitySku` | Fabric capacity SKU (only used when `fabricCapacityMode=create`) | `F8` (default) |
-| `fabricCapacityAdmins` | Fabric capacity admin principals (UPN emails or Entra object IDs) (required when `fabricCapacityMode=create`) | `["user@contoso.com"]` |
+| `fabricCapacityAdmins` | Fabric capacity admin principals (UPN emails or Entra object IDs) (required when `fabricCapacityMode=create`) | `user@contoso.com,user2@contoso.com` |
 | `fabricCapacityResourceId` | Existing Fabric capacity ARM resource ID (required when `fabricCapacityMode=byo`) | `/subscriptions/.../providers/Microsoft.Fabric/capacities/...` |
 | `fabricWorkspaceId` | Existing Fabric workspace ID (GUID) (required when `fabricWorkspaceMode=byo`) | `00000000-0000-0000-0000-000000000000` |
 | `fabricWorkspaceName` | Existing Fabric workspace name (used when `fabricWorkspaceMode=byo`) | `my-existing-workspace` |

infra/main.bicepparam

@@ -99,7 +99,7 @@ param aiSearchAdditionalAccessObjectIds = []
 // - Full setup: fabricCapacityPreset='create', fabricWorkspacePreset='create'
 // - No Fabric:  fabricCapacityPreset='none',   fabricWorkspacePreset='none'
 // - BYO both:   fabricCapacityPreset='byo',    fabricWorkspacePreset='byo'
-var fabricCapacityPreset = 'create'
+var fabricCapacityPreset = readEnvironmentVariable('fabricCapacityMode', 'create')
 var fabricWorkspacePreset = fabricCapacityPreset
 
 // Legacy toggle retained for back-compat with older docs/scripts
@@ -118,7 +118,8 @@ param fabricWorkspaceName = '' // optional (helpful for naming/UX)
 param fabricCapacitySku = 'F8'
 
 // Fabric capacity admin members (email addresses or object IDs).
-param fabricCapacityAdmins = []
+var fabricAdminValue = readEnvironmentVariable('fabricCapacityAdmins', '')
+param fabricCapacityAdmins = empty(fabricAdminValue) ? [] : split(fabricAdminValue, ',')
 
 // ========================================
 // PURVIEW PARAMETERS (Optional)