Skip to content

ci: implement quota check in CI Pipeline to ensure resource availability in supported region #1680

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 24, 2025

Conversation

Priyanka-Microsoft
Copy link
Contributor

Purpose

  • ...
    This pull request introduces a quota check mechanism to the CI pipeline and updates the related scripts and workflow files. The most important changes include adding a script to check Azure quotas, modifying the CI workflow to run this script, and handling quota check failures.

Quota check implementation:

  • .github/workflows/ci.yml: Added steps to run the quota check script, send notifications on quota failure, and fail the pipeline if the quota check fails.
  • scripts/checkquota.sh: Added a new script to check Azure quotas for specific regions and models, ensuring that the required capacities are available before proceeding with the deployment.

Workflow environment variable update:

  • .github/workflows/ci.yml: Updated the AZURE_LOCATION environment variable to use the value set by the quota check script.

Does this introduce a breaking change?

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

…gion and prevent failures due to insufficient quota.
@Roopan-Microsoft Roopan-Microsoft added this pull request to the merge queue Feb 24, 2025
Merged via the queue into main with commit ebfbddf Feb 24, 2025
9 checks passed
@Priyanka-Microsoft Priyanka-Microsoft deleted the task/ci-pipeline-quota-check branch April 3, 2025 12:21
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.

2 participants