Stabilize macOS agent setup

[paulmedynski]

Description

This PR makes a variety of small changes in an attempt to improve the stability of the macOS test runs:

• Removed unused and obsolete pipeline template parameters throughout.
• Increased test job timeout since flaky tests can add 30+ minutes to test runs 😞
• Skipped one test that isfailing on macOS 50% of the time.
• Many small changes due to Copilot suggestions.

Testing

The usual PR/CI runs will validate.

[Copilot]

Pull request overview

This PR refactors SQL Server configuration in Azure DevOps pipelines by centralizing password management and adding diagnostic capabilities for macOS agent setup.

Changes:

• Removed password parameter passing through template layers, replacing it with direct use of the $(Password) variable from the ADO Library "ADO Test Configuration Properties"
• Removed temporary password generation/verification steps from the test job template
• Added documentation comments explaining the Password variable source across all SQL Server configuration step templates
• Enhanced macOS SQL Server setup with timestamp-based diagnostic output and improved error message clarity
• Cleaned up trailing whitespace throughout the pipeline files

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
eng/pipelines/common/templates/steps/configure-sql-server-win-step.yml	Removed password parameter, replaced all parameter references with $(Password) variable, added documentation comment, cleaned up trailing whitespace
eng/pipelines/common/templates/steps/configure-sql-server-step.yml	Removed password parameter definition and parameter passing to child templates
eng/pipelines/common/templates/steps/configure-sql-server-macos-step.yml	Removed password parameter, replaced with $(Password) variable, added documentation, added PS4 timestamp prompt for diagnostics, improved error log message
eng/pipelines/common/templates/steps/configure-sql-server-linux-step.yml	Removed password parameter, replaced with $(Password) variable, added documentation comment, cleaned up trailing whitespace
eng/pipelines/common/templates/jobs/ci-run-tests-job.yml	Removed password generation and verification steps that created a temporary GUID-based password

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.24%. Comparing base (104a401) to head (8bf46d0).
⚠️ Report is 5 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (104a401) and HEAD (8bf46d0). Click for more details.

HEAD has 1 upload less than BASE

Flag	BASE (104a401)	HEAD (8bf46d0)
addons	1	0

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3928      +/-   ##
==========================================
- Coverage   74.91%   67.24%   -7.67%     
==========================================
  Files         269      260       -9     
  Lines       43279    65704   +22425     
==========================================
+ Hits        32422    44183   +11761     
- Misses      10857    21521   +10664     

Flag	Coverage Δ
addons	?
netcore	67.55% <ø> (-7.39%)	⬇️
netfx	66.12% <ø> (-8.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 6 comments.

[benrr101]

Since this is no longer a work-in-progress, can you update the description to explain what the PR does? That'll be the first landing place if anyone wants to investigate the PR, so it'll be helpful to have it.