Skip to content

Conversation

@Dhruvkumar-Microsoft
Copy link
Contributor

Purpose

This pull request primarily refactors deployment scripts and infrastructure templates to improve maintainability, security, and clarity. The main changes include replacing echo with printf for more consistent output formatting in deployment scripts, updating how administrator credentials are handled for virtual machines, and adjusting logging environment variables for the backend and frontend deployments.

Deployment script improvements:

  • Switched from echo to printf for post-deployment messaging in both azure.yaml and azure_custom.yaml, resulting in more consistent and readable output formatting. [1] [2]

Infrastructure parameter and security updates:

  • Changed virtualMachineAdminUsername and virtualMachineAdminPassword parameters in both infra/main.bicep and infra/main_custom.bicep from default-generated values to optional parameters, allowing explicit credential specification. [1] [2]
  • Updated the virtual machine module to use fallback default credentials only if parameters are not provided, improving flexibility and clarity in deployments. [1] [2]

Environment variable and logging configuration:

  • Added logging-related environment variables (AZURE_BASIC_LOGGING_LEVEL, AZURE_PACKAGE_LOGGING_LEVEL, AZURE_LOGGING_PACKAGES) to the backend container app configuration for both standard and custom deployments, while removing them from the frontend/web site configuration to avoid redundancy. [1] [2] [3] [4]

Python logging initialization:

  • Removed the global logging.basicConfig(level=logging.INFO) initialization from src/backend/common/utils/utils_kernel.py to prevent unintended side effects and allow more controlled logging configuration elsewhere.

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]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Roopan-Microsoft Roopan-Microsoft merged commit caac92a into main Dec 8, 2025
10 checks passed
@github-actions
Copy link

github-actions bot commented Dec 8, 2025

🎉 This PR is included in version 3.1.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants