Skip to content

Handle RD Infra Agent and Bootloader install strings containing spaces gracefully#677

Open
murajput wants to merge 1 commit intomasterfrom
murajput/RDSTemplatesFixInstallStrings
Open

Handle RD Infra Agent and Bootloader install strings containing spaces gracefully#677
murajput wants to merge 1 commit intomasterfrom
murajput/RDSTemplatesFixInstallStrings

Conversation

@murajput
Copy link
Contributor

  • When the RDAgentBootLoader name contains a space in the title (exp. Microsoft.RDInfra.RDAgentBootLoader.Installer-x64 (7).msi), it causes the DSC Installation to hang in a transitioning phase as the msiexec command can't recognize the install.
  • Encapsulated all /i $Path strings so they look like /i '$Path'.

@murajput murajput requested a review from lintFan March 17, 2022 10:43

Write-Log -Message "Installing RD Infra Agent on VM $AgentInstaller"
RunMsiWithRetry -programDisplayName "RD Infra Agent" -argumentList @("/i $AgentInstaller", "/quiet", "/qn", "/norestart", "/passive", "REGISTRATIONTOKEN=$RegistrationToken") -msiOutputLogPath "C:\Users\AgentInstall.txt" -msiLogVerboseOutput:$EnableVerboseMsiLogging
RunMsiWithRetry -programDisplayName "RD Infra Agent" -argumentList @("/i '$AgentInstaller'", "/quiet", "/qn", "/norestart", "/passive", "REGISTRATIONTOKEN=$RegistrationToken") -msiOutputLogPath "C:\Users\AgentInstall.txt" -msiLogVerboseOutput:$EnableVerboseMsiLogging
Copy link
Contributor

Choose a reason for hiding this comment

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

We should first do this change in portal repo and test it and put it here

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