This repository was archived by the owner on Jun 5, 2025. It is now read-only.
Hyper-V extension: Add "Ask for VM credentials" and "Wait for logon" Adaptive Cards to Hyper-V Configure command. #2289
Conversation
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
sshilov7
requested review from
SohamDas2021,
bbonaby,
huzaifa-d,
kanismohammed and
krschau
sshilov7
changed the base branch from
main
to
microsoftfeature/dev-environments
bbonaby
reviewed
Feb 22, 2024
Contributor
There was a problem hiding this comment.
Can you check if the line endings for this file are still set CRLF?
Member
Author
There was a problem hiding this comment.
I am not sure why some lines got wrong EOL. I think either Intellisence or CoPilot broke it. This one seems fine on my machine though.
HyperVExtension/src/DevSetupAgent/Requests/IsUserLoggedInRequest.cs
Outdated
Show resolved
Hide resolved
Contributor
There was a problem hiding this comment.
Can you check if the line endings for this class is still in CRLF?
Member
Author
There was a problem hiding this comment.
I think it got broken in previous change, now it's getting fixed.
Contributor
There was a problem hiding this comment.
Can you check if the line endings for this file are still set CRLF?
Member
Author
There was a problem hiding this comment.
Yeah, the new file is fixing inconsistent EOL.
HyperVExtension/src/HyperVExtension/CommunicationWithGuest/Responses/ResponseBase.cs
Show resolved
Hide resolved
HyperVExtension/src/HyperVExtension/Strings/en-US/Resources.resw
Outdated
Show resolved
Hide resolved
bbonaby
approved these changes
Feb 23, 2024
| } | ||
|
|
||
| private readonly ApplyConfigurationOperation _operation; | ||
| private readonly ManualResetEvent _sessionStatusChangedEvent = new(false); |
Contributor
There was a problem hiding this comment.
Oh so you're saying the UI that displays the adaptive card (the render in dev home) should attempt to disable the button? I think out the box the default render doesn't disable the button. But we can make our own custom render that disables it.
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/AdaptiveCards1
branch
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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
Add "Ask for VM credentials" and "Wait for logon" Adaptive Cards to Hyper-V Configure command in Hyper-V extension
Added Adaptive cards JSON to the code
Added Adaptive card session code to interact with DevHome
Added IsUserLoggedIn request/response to ask VM if there is a logged in user
Added an integration test for IsUserLoggedIn request
Modified manual "Configure" command test to test "Ask for VM credentials" and "Wait for logon" Adaptive Cards session code.
Validation steps performed
Ran new tests.
Manual debugging.
PR checklist