Skip to content

CHORE: Disable scheduled Azure SQL tests due to limited subscription availability#584

Merged
gargsaumya merged 1 commit into
mainfrom
saumya/disable-azuresql-tests
May 15, 2026
Merged

CHORE: Disable scheduled Azure SQL tests due to limited subscription availability#584
gargsaumya merged 1 commit into
mainfrom
saumya/disable-azuresql-tests

Conversation

@gargsaumya
Copy link
Copy Markdown
Contributor

@gargsaumya gargsaumya commented May 15, 2026

Temporarily disable Azure SQL Database tests that run on a 15-day schedule. The tests will be skipped until re-enabled.

To re-enable: Uncomment the conditional logic in CheckAzureSQLSchedule job.

This affects:

  • PytestOnWindows_AzureSQL job
  • Any other jobs dependent on CheckAzureSQLSchedule

Work Item / Issue Reference

AB#45033

GitHub Issue: #<ISSUE_NUMBER>


Summary

This pull request temporarily disables the execution of Azure SQL tests in the PR validation pipeline. The logic that conditionally enabled these tests based on the date or an environment variable has been commented out, and a message now clearly indicates that the tests are disabled for now.

  • Pipeline behavior change:
    • The conditional logic for running Azure SQL tests in eng/pipelines/pr-validation-pipeline.yml has been commented out, so the tests will always be skipped regardless of the day or the ENABLE_AZURE_SQL environment variable. An explanatory message is printed, and instructions are provided for re-enabling the tests in the future.

Copilot AI review requested due to automatic review settings May 15, 2026 03:58
@github-actions github-actions Bot added the pr-size: small Minimal code update label May 15, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR temporarily disables the Azure SQL Database test jobs in the PR validation pipeline by forcing the schedule gate to always set shouldRunAzureSQL=false, preventing dependent Azure SQL jobs from running while subscription capacity is limited.

Changes:

  • Force CheckAzureSQLSchedule to always output shouldRunAzureSQL=false.
  • Add inline messaging and commented instructions for how to re-enable the original conditional logic later.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread eng/pipelines/pr-validation-pipeline.yml Outdated
Comment thread eng/pipelines/pr-validation-pipeline.yml Outdated
Temporarily disable all Azure SQL Database test jobs including:
- CheckAzureSQLSchedule (schedule check job)
- PytestOnWindows_AzureSQL
- PytestOnMacOS_AzureSQL
- PytestOnLinux_AzureSQL

All jobs are commented out and will be skipped until re-enabled.

To re-enable: Uncomment all jobs in the Azure SQL section.
@gargsaumya gargsaumya force-pushed the saumya/disable-azuresql-tests branch from 0f9e339 to feb24dd Compare May 15, 2026 04:16
@github-actions github-actions Bot added pr-size: medium Moderate update size and removed pr-size: small Minimal code update labels May 15, 2026
@github-actions
Copy link
Copy Markdown

📊 Code Coverage Report

🔥 Diff Coverage

100%


🎯 Overall Coverage

25%


📈 Total Lines Covered: 7018 out of 27134
📁 Project: mssql-python


Diff Coverage

Diff: main...HEAD, staged and unstaged changes

No lines with coverage information in this diff.


📋 Files Needing Attention

📉 Files with overall lowest coverage (click to expand)
mssql_python.pybind.build._deps.simdutf-src.src.haswell.implementation.cpp: 0.4%
mssql_python.pybind.build._deps.simdutf-src.src.implementation.cpp: 6.7%
mssql_python.pybind.build._deps.simdutf-src.include.simdutf.implementation.h: 10.4%
mssql_python.pybind.build._deps.simdutf-src.include.simdutf.scalar.utf16_to_utf8.utf16_to_utf8.h: 25.3%
mssql_python.pybind.logger_bridge.cpp: 59.2%
mssql_python.pybind.ddbc_bindings.h: 59.7%
mssql_python.pybind.build._deps.simdutf-src.include.simdutf.internal.isadetection.h: 65.3%
mssql_python.row.py: 70.5%
mssql_python.pybind.logger_bridge.hpp: 70.8%
mssql_python.pybind.ddbc_bindings.cpp: 74.2%

🔗 Quick Links

⚙️ Build Summary 📋 Coverage Details

View Azure DevOps Build

Browse Full Coverage Report

@gargsaumya gargsaumya merged commit e8bb648 into main May 15, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-size: medium Moderate update size

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants