Build DevSetupAgent separately from the main Dev Home solution and add DevSetupAgent_*.zip to MSIX package. #2325
Conversation
This file contains 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
Create DevSetupAgent zip for different VM architectures. Fixed build scripts and HyperVExtension.csproj to include DevSetupAgent zip file into Dev Home MSIX package.
sshilov7
requested review from
huzaifa-d,
EricJohnson327,
SohamDas2021 and
bbonaby
bbonaby
approved these changes
Feb 28, 2024
|
|
||
| if (-not $BypassWarning) { | ||
| Write-Host @" | ||
| This script is not meant to be run directly. To build DevSetupAgent, please run the following from the root directory: |
There was a problem hiding this comment.
Do we need to say to run it in a visual studio powershell terminal?
There was a problem hiding this comment.
Not really, it works from regular PS window (the same message is in the SDK build script), but it's supposed to be called from main Build.ps1. (unless you know that you can pass "secret" -BypassWarning parameter).
|
|
||
| & $msbuildPath $msbuildArgs | ||
|
|
||
| $binariesOutputPath = (Join-Path $env:Build_RootDirectory "HyperVExtension\src\DevSetupAgent\bin\$Platform\$Configuration\net8.0-windows10.0.22000.0\win10-$Platform\*") |
There was a problem hiding this comment.
When we I move the hyper-v extension to a top level "Extensions" folder we'll need to change these. I'll make a note of this
There was a problem hiding this comment.
Yes, unfortunately there are hardcoded paths in a couple of places. Perhaps we can improve this later with some kind of variable that can be used here and from VS. (And then we'd need not to forget to update it.). It would be nice to be able to ask Msbuild: "Give me output location for this project", but I haven't found a way to do it so far.
bbonaby
reviewed
Feb 28, 2024
There was a problem hiding this comment.
Approved but we'll wait for @EricJohnson327 to check the build script
| if ($Platform -ieq "arm64") { | ||
| HyperVExtension\BuildDevSetupAgentHelper.ps1 -Platform $Platform -Configuration $configuration -VersionOfSDK $env:sdk_version -SDKNugetSource $SDKNugetSource -AzureBuildingBranch $AzureBuildingBranch -IsAzurePipelineBuild $IsAzurePipelineBuild -BypassWarning | ||
| } | ||
| else { |
There was a problem hiding this comment.
This is a minor issue since we very rarely build more than 1 platform at a time. If someone is builing both x86 and x64, this will build DevSetupAgent x86 twice.
| @@ -26,7 +26,7 @@ Description: | |||
| Options: | |||
There was a problem hiding this comment.
Script looks good. However, the official builds won't pick this up without changes to build\azure-pipelines.yml
EricJohnson327
approved these changes
Feb 29, 2024
bbonaby
merged commit 8353204
into
microsoftfeature/dev-environments
1 of 4 checks passed
bbonaby
added a commit
that referenced
this pull request
Mar 7, 2024
* add initial SDK changes (#2209) * Add UX code from private ADO branch to public feature branch (#2241) * Add initial code from private branch that will be shared between the setup flow and the Dev environments tool page. PRs: from private ADO repo: https://dev.azure.com/microsoft/Dart/_git/DevHome?version=GBDevEnv * add changes to the setup flow for dev environment configuration PRS: from private ADO repo: https://dev.azure.com/microsoft/Dart/_git/DevHome?version=GBDevEnv * Add dev environments management tool page from private ADO repo: https://dev.azure.com/microsoft/Dart/_git/DevHome?version=GBDevEnv * fix conflicts and stylecop errors due to update from merge with main --------- Co-authored-by: Huzaifa Danish <[email protected]> * Move Hyper-V extension code from private repository to public dev environments feature branch (#2246) * Add Hyper-v extension to Dev Home from Private Hyper-v extension branch: PRS https://github.com/microsoft/DevHomeHyperVExtension * Merge changes from Dev Home main and fix style cop errors * Initial implementation of IComputeSystem::ApplyConfiguration for HyperV extension. (#2258) * Add environments to experimentation (#2260) * Add PowerShell script and a helper class to deploy DevSetupAgent service to a Hyper-V VM. (#2261) * Hyper-V extension: Add "Ask for VM credentials" and "Wait for logon" Adaptive Cards to Hyper-V Configure command. (#2289) * Changes to make DevSetupEngine work with .NET 8 (#2308) * Build DevSetupAgent separately from the main Dev Home solution and add DevSetupAgent_*.zip to MSIX package. (#2325) * Add WaitForLogin and Credentials Adaptive Cards * Address review comments. * New VS solution for DevSetupAgent * Add BuildDevSetupAgentHelper script * Fix x86 DevSetupAgent to work on x64 OS. Create DevSetupAgent zip for different VM architectures. Fixed build scripts and HyperVExtension.csproj to include DevSetupAgent zip file into Dev Home MSIX package. * Remove old comment. * Fix a comment. * Update setup target flow to allow for configuration of a dev environment. (#2321) * initial code * update messaging and adaptive cards * remove added method * update strings and names * update based on initial comments and update IsHyperVModuleLoaded with new work around that doesn't involve installing using Install-Module which installs from PsGallery * improve wording * fix merge conflicts * update dev environments feature branch with latest idl changes (#2334) * update feature branch to latest sdk * update InputJson to inputJson * update with latest changes * Update environments page UX (#2320) * add updates to ui * fix crashes due to resource name mismatch * update feature branch to latest sdk * update InputJson to inputJson * update with latest changes * update * update to remove duplicate, resize shimmer, remove id from winget file since its not needed, and add comments * Re-add correct adaptive host file for correct theming, address some initial comments, update loading page to allow scrolling when there are tasks and actions in the action center. Fix error message spelling. * improve setup target loading page progress messaging * Fix git clone's dependsOn Id to match our new Ids for setup target flow * Update build scripts to build DevSetupAgent in Azure official build. (#2339) * update dev home to use new SDK version and other projects to use win app sdk v1.5 to prevent build issues (#2344) * Fix PS DevSetupAgent deployment script. (#2345) * remove classes that don't need to be added * Change all tabs to spaces * Update tools directory to use crlf * Update HyperVExtension directory to use crlf * Update common/Environments directory to use crlf * Update more in common directory to use crlf * [Hyper-V extension]: Fixes for DevSetupEngine registration, Configure progress reporting, and GitHub build. (#2348) * remove s that was added from a previous commit causing build to fail * fix tests for build * update malformed configuration string to allow arm64 to build --------- Co-authored-by: Huzaifa Danish <[email protected]> Co-authored-by: sshilov7 <[email protected]> Co-authored-by: Kristen Schau <[email protected]>
sshilov7
deleted the
user/sergeys/microsoftfeature/dev-environments/BuildDevSetupAgent
branch
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.
Summary of the pull request
Validation steps performed
Ran manual Hyper-V Configure E2E test.
Built all architectures and verified that DevSetupAgent_*.zip is included into Dev Home MSIX package.
PR checklist