Skip to content
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

Get ARP entries from MSI database #550

Merged
merged 4 commits into from
Oct 11, 2024
Merged

Conversation

vedantmgoyal9
Copy link
Contributor

@vedantmgoyal9 vedantmgoyal9 commented Aug 14, 2024

@vedantmgoyal9 vedantmgoyal9 requested a review from a team as a code owner August 14, 2024 11:29
@vedantmgoyal9 vedantmgoyal9 requested review from yao-msft and ryfu-msft and removed request for a team August 14, 2024 11:29
@ryfu-msft
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vedantmgoyal9 vedantmgoyal9 changed the title Update UpgradeCode from MSI database Update ProductCode and UpgradeCode inside ARP entries Aug 24, 2024
@vedantmgoyal9 vedantmgoyal9 force-pushed the patch-1 branch 2 times, most recently from 139d2a6 to 2f91e41 Compare September 5, 2024 21:48
@vedantmgoyal9 vedantmgoyal9 changed the title Update ProductCode and UpgradeCode inside ARP entries Get ARP entries from MSI database Sep 5, 2024
@ryfu-msft
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vedantmgoyal9
Copy link
Contributor Author

The failed tests are due to some issue with GitHub App private key 😕

@ryfu-msft
Copy link
Contributor

The failed tests are due to some issue with GitHub App private key 😕

Lets get this checked in. I will need to address the broken tests in another PR since we're no longer allowed to use private keys. Thanks for the contribution :)

@ryfu-msft ryfu-msft merged commit eb8207d into microsoft:main Oct 11, 2024
3 of 5 checks passed
@vedantmgoyal9 vedantmgoyal9 deleted the patch-1 branch October 11, 2024 19:50
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