(main) Update CI to consider latest URL and installation steps#497
Merged
amitkarsale merged 1 commit intomainfrom Feb 25, 2026
Merged
(main) Update CI to consider latest URL and installation steps#497amitkarsale merged 1 commit intomainfrom
amitkarsale merged 1 commit intomainfrom
Conversation
shubhamshinde360
approved these changes
Feb 24, 2026
span786
approved these changes
Feb 24, 2026
There was a problem hiding this comment.
Pull request overview
This PR updates CI workflows to align with infrastructure changes and improve the setup process for SQL Server testing on Windows nodes. The changes update the provisioning service URL, add DNS configuration fixes for Twingate connectivity, introduce Chocolatey-based installation of the MSOLEDBSQL driver, and standardize bolt command targets from ssh_nodes to all.
Changes:
- Updated SERVICE_URL from the maintenance configuration endpoint to the release endpoint
- Added DNS fix step after Twingate installation to properly configure DNS resolution for internal artifact servers
- Added new step to install Chocolatey and MSOLEDBSQL driver using Chocolatey package manager
- Changed bolt command targets from
ssh_nodestoallfor consistency across authentication, download, and environment setup steps
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/nightly.yml | Updated SERVICE_URL, added DNS fix step, added Chocolatey/MSOLEDBSQL installation step, changed bolt targets from ssh_nodes to all |
| .github/workflows/ci.yml | Applied identical changes as nightly.yml to maintain consistency between CI and nightly workflows |
Comments suppressed due to low confidence (1)
.github/workflows/nightly.yml:129
- The step name has a typo: "Authenitcate" should be "Authenticate". Note that the corresponding step in ci.yml (line 132) uses the correct spelling.
- name: Authenitcate with GCP
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updated CI to consider updated
SERVICE_URL, fixed the DNS, used chocolaty to installMSOLEDBSQLand fixed existing steps in the CI for both nightly and ci files