Update AzPowerShellVersion to 14.0 for Azure SQL deployment scripts

[Copilot]

Description

Updates AzPowerShellVersion from "10.0" to "14.0" in AzureSqlServerResource.cs to address Bicep linter warning:

sql-roles.module.bicep(22,1) : Warning use-recent-az-powershell-version:
Deployment script is using AzPowerShell version '10.0' which is below the recommended minimum version '11.0'.
Consider upgrading to version 11.0 or higher to avoid EOL Ubuntu 20.04 LTS.

Changes:

• Updated AzPowerShellVersion from "10.0" to "14.0" (latest supported version)
• Added inline comments referencing the linter warning and version rationale
• Updated snapshot file RoleAssignmentTests.SqlSupport.verified.bicep

Checklist

• Is this feature complete?
  • Yes. Ready to ship.
  • No. Follow-up changes expected.
• Are you including unit tests for the changes and scenario tests if relevant?
  • Yes
  • No
• Did you add public API?
  • Yes
    • If yes, did you have an API Review for it?
      • Yes
      • No
    • Did you add <remarks /> and <code /> elements on your triple slash comments?
      • Yes
      • No
  • No
• Does the change make any security assumptions or guarantees?
  • Yes
    • If yes, have you done a threat model and had a security review?
      • Yes
      • No
  • No
• Does the change require an update in our Aspire docs?
  • Yes
    • Link to aspire-docs issue (consider using one of the following templates):
      • New (or update) doc-idea template
      • New breaking-change template
      • New diagnostic template
  • No

Original prompt

Update the AzPowerShellVersion in src/Aspire.Hosting.Azure.Sql/AzureSqlServerResource.cs from "10.0" to "14.0" (the latest supported version). This addresses the following Bicep linter warning:

sql-roles.module.bicep(22,1) : Warning use-recent-az-powershell-version:
Deployment script is using AzPowerShell version '10.0' which is below the recommended minimum version '11.0'. Consider upgrading to version 11.0 or higher to avoid EOL Ubuntu 20.04 LTS. [https://aka.ms/bicep/linter-diagnostics#use-recent-az-powershell-version]

Also, add a test verifying that AzPowerShellVersion is set to "14.0" in the relevant deployment script/resource generation for AzureSqlServerResource. The test should fail if the version is < 14.0 or not set. Include comments referencing the warning and reason for the version bump.

This pull request was created as a result of the following prompt from Copilot chat.

Update the AzPowerShellVersion in src/Aspire.Hosting.Azure.Sql/AzureSqlServerResource.cs from "10.0" to "14.0" (the latest supported version). This addresses the following Bicep linter warning:

sql-roles.module.bicep(22,1) : Warning use-recent-az-powershell-version:
Deployment script is using AzPowerShell version '10.0' which is below the recommended minimum version '11.0'. Consider upgrading to version 11.0 or higher to avoid EOL Ubuntu 20.04 LTS. [https://aka.ms/bicep/linter-diagnostics#use-recent-az-powershell-version]

Also, add a test verifying that AzPowerShellVersion is set to "14.0" in the relevant deployment script/resource generation for AzureSqlServerResource. The test should fail if the version is < 14.0 or not set. Include comments referencing the warning and reason for the version bump.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[github-actions]

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 13308

Or

• Run remotely in PowerShell:

iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 13308"

[eerhardt]

Do we need this test when we have the above test that baselines the azPowerShellVersion in the bicep file?

[radical]

@copilot drop this new test

[Copilot]

Dropped the new test in 9b667bb. The existing SqlSupport baseline test already verifies the azPowerShellVersion in the snapshot file.

[eerhardt]

This looks good, but it is still in draft. Is that intentional?

[radical]

This looks good, but it is still in draft. Is that intentional?

Fixed it.

[Copilot]

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[Copilot]

[nitpick] The PR description mentions "Also, add a test verifying that AzPowerShellVersion is set to '14.0'" and states that the test should fail if the version is < 14.0 or not set. However, only the snapshot test was updated, and no new explicit test was added.

While the existing snapshot test (RoleAssignmentTests.SqlSupport) will catch changes to the generated Bicep and verify the version is "14.0", consider adding an explicit unit test that directly asserts the AzPowerShellVersion property is set to "14.0" in the AzureSqlServerResource.AddRoleAssignments method. This would make the version requirement more explicit and easier to identify if changed in the future.

For example, you could add a test in AzureSqlExtensionsTests.cs or RoleAssignmentTests.cs that verifies the PowerShell version property directly.

[radical]

@eerhardt thoughts? ^