fix: Merging dev to main

[Roopan-Microsoft]

Purpose

• This pull request updates the deployment script to improve robustness and compatibility when retrieving deployment output values, and to streamline the Azure login process. The main changes include introducing a helper function for output value retrieval with support for both legacy and new output key names, and optimizing the login flow to avoid redundant authentication.

Deployment Output Retrieval Improvements

• Added a new Get-DeploymentOutputValue helper function to retrieve deployment output values using both legacy and new key names, ensuring backward compatibility with different output conventions. This function is now used throughout the DeploymentResult class for all resource property assignments. [1] [2]

Azure Login Flow Optimization

• Introduced a $script:alreadyLoggedIn flag and updated the LoginAzure function to set this flag after a successful login, preventing redundant logins during deployment. [1] [2]
• Modified the deployment logic to perform an initial login with empty credentials and only re-login with specific tenant and subscription IDs if not already authenticated.

Error Handling and Messaging

• Improved error messaging to clarify when required deployment output values are missing, now referencing both legacy and new key names for easier troubleshooting.

Does this introduce a breaking change?

• Yes
• No

Golden Path Validation

• I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

• I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

• ...

Other Information