Switch from using choco to curl in pipeline#337
Open
nmlud21 wants to merge 10 commits into
Open
Conversation
Signed-off-by: Nicholas Ludwig <nludwig@microsoft.com>
nmlud21
requested review from
Alan-Jowett,
LakshK98,
matthewige,
mikeagun,
mtfriesen,
poornagmsft,
saxena-anurag and
shankarseal
as code owners
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the reusable Windows test workflow to reduce CI flakiness by replacing Chocolatey-based installs with direct downloads for ProcDump and OpenCppCoverage.
Changes:
- Download ProcDump via
curl.exe, verify SHA256, extract to workspace, add toPATH, and pre-accept the Sysinternals EULA via registry. - Download OpenCppCoverage installer via
curl.exe, verify SHA256, silently install, and add install directory toPATH.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Agent-Logs-Url: https://github.com/microsoft/ntosebpfext/sessions/afd2dc19-27f4-4c5f-9a7c-f31ec41d4830 Co-authored-by: nmlud21 <79675390+nmlud21@users.noreply.github.com>
shankarseal
reviewed
May 4, 2026
shankarseal
reviewed
May 4, 2026
Signed-off-by: Nicholas Ludwig <nludwig@microsoft.com>
added 2 commits
May 7, 2026 14:53
…ndencies. Signed-off-by: Nicholas Ludwig <nludwig@microsoft.com>
Signed-off-by: Nicholas Ludwig <nludwig@microsoft.com>
shankarseal
reviewed
May 8, 2026
shankarseal
previously approved these changes
May 8, 2026
matthewige
reviewed
May 12, 2026
added 2 commits
May 12, 2026 16:30
Signed-off-by: Nicholas Ludwig <nludwig@microsoft.com>
…om/microsoft/ntosebpfext into user/nmlud21/opencpp_curl_download
LakshK98
approved these changes
May 18, 2026
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.
Description
Switch downloading of procdump and OpenCppCoverage from choco to curl to fix pipeline flakiness.
Fixes #323
Testing
No
Documentation
No
Installation
No