Skip to content

Conversation

@Bangarraju-Microsoft
Copy link
Contributor

…elerator Repositories changes

Purpose

* ...
This pull request introduces a utility function for managing Azure credentials and refactors multiple scripts to use it, enhancing maintainability and consistency. It also updates related shell scripts to ensure the new utility file is downloaded alongside other dependencies.

Azure Credential Management Refactor:

  • Added a new utility function get_azure_credential in infra/scripts/azure_credential_utils.py to handle Azure credential selection based on the environment (prod or dev). It uses ManagedIdentityCredential in production and DefaultAzureCredential in development.
  • Replaced direct usage of DefaultAzureCredential with get_azure_credential in the following scripts:
    • infra/scripts/aihub_scripts/create_ai_hub.py [1] [2]
    • infra/scripts/fabric_scripts/create_fabric_items.py
    • infra/scripts/index_scripts/create_articles_index.py [1] [2]
    • infra/scripts/index_scripts/create_drafts_index.py [1] [2]
    • infra/scripts/index_scripts/create_grants_index.py [1] [2]

Shell Script Updates:

  • Updated the following shell scripts to download the new azure_credential_utils.py file:
    • infra/scripts/run_create_aihub_scripts.sh
    • infra/scripts/run_create_index_scripts.sh
    • infra/scripts/run_fabric_items_scripts.sh

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

@Prajwal-Microsoft Prajwal-Microsoft merged commit 3dbeac9 into byoc-researcher Jul 31, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants