Skip to content

use cred to make remote pwsh session #4118

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

Merged
merged 2 commits into from
Feb 7, 2024
Merged

use cred to make remote pwsh session #4118

merged 2 commits into from
Feb 7, 2024

Conversation

ProjectsByJackHe
Copy link
Contributor

Description

WS2025 exhibits weird behavior when starting remote PWSH sessions. This is a workaround.

Testing

CI

Documentation

N/A

@ProjectsByJackHe ProjectsByJackHe requested a review from a team as a code owner February 7, 2024 18:38
@@ -98,7 +98,14 @@ $useXDP = ($io -eq "xdp" -or $io -eq "qtip")
# Set up the connection to the peer over remote powershell.
Write-Host "Connecting to $RemoteName"
if ($isWindows) {
$Session = New-PSSession -ComputerName $RemoteName -ConfigurationName PowerShell.7
if ($os -eq "windows-2025") {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we simply just always do this? If it works for 2022, I think I'd rather have the simpler script.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets try and see if this works for WS 2022

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it looks like it works. Lab failures are unrelated, currently doing some tweaking to diagnose poor performance.

@nibanks nibanks merged commit 19b2ce8 into main Feb 7, 2024
@nibanks nibanks deleted the jackhe/ws-2025 branch February 7, 2024 20:49
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