Added fix for Update-AzMaintenanceConfiguration (#25126)

* Added potential fix for Update-AzMaintenanceConfiguration

* Added test for bug fix

* Reverted small change

* Updated ChangeLog

* Updated ChangeLog

* Update ChangeLog.md

---------

Co-authored-by: NoriZC <[email protected]>

Author: adana-popescu

src/Maintenance/Maintenance.Test/ScenarioTests/MaintenanceTests.ps1

@@ -172,6 +172,7 @@ function Test-AzMaintenanceConfigurationInGuestPatch
         $maintenanceConfigurationNameInstance = $allMaintenanceConfigInSubscription | ?{ $_.Name -eq $maintenanceConfigurationName}
 
         $maintenanceConfigurationNameInstance.LinuxParameterPackageNameMaskToInclude.Add("package3")
+        $maintenanceConfigurationNameInstance.InstallPatchRebootSetting = "AlwaysReboot"
 
         # Act
         Update-AzMaintenanceConfiguration -ResourceGroupName $resourceGroupName -Name $maintenanceConfigurationName -Configuration  $maintenanceConfigurationNameInstance
@@ -182,6 +183,7 @@ function Test-AzMaintenanceConfigurationInGuestPatch
         Assert-True { $updatedMRPConfig.LinuxParameterPackageNameMaskToInclude.Contains("apt") }
         Assert-True { $updatedMRPConfig.LinuxParameterPackageNameMaskToInclude.Contains("httpd") }
         Assert-True { $updatedMRPConfig.LinuxParameterPackageNameMaskToInclude.Contains("package3") }
+        Assert-AreEqual $updatedMRPConfig.InstallPatchRebootSetting "AlwaysReboot"
 
         # Default patch config
         $maintenanceConfigurationName2 = Get-RandomMaintenanceConfigurationName

src/Maintenance/Maintenance.Test/SessionRecords/Microsoft.Azure.Commands.Maintenance.Test.ScenarioTests.MaintenanceTests/TestAzConfigurationAssignmentDynamicGroupForResourceGroup.json

@@ -19,6 +19,7 @@
 -->
 
 ## Upcoming Release
+* Fixed bug where rebootSettings property wasn't updating.
 
 ## Version 1.4.1
 * Removed outdated upcoming breaking change warning

src/Maintenance/Maintenance.Test/SessionRecords/Microsoft.Azure.Commands.Maintenance.Test.ScenarioTests.MaintenanceTests/TestAzConfigurationAssignmentDynamicGroupForSubscription.json

@@ -87,7 +87,6 @@ private static void Initialize()
                     .ForMember(dest => dest.WindowParameterExcludeKbRequiringReboot, opt => opt.MapFrom(src => src.InstallPatches.WindowsParameters.ExcludeKbsRequiringReboot))
                     .ForMember(dest => dest.WindowParameterKbNumberToExclude, opt => opt.MapFrom(src => src.InstallPatches.WindowsParameters.KbNumbersToExclude))
                     .ForMember(dest => dest.WindowParameterKbNumberToInclude, opt => opt.MapFrom(src => src.InstallPatches.WindowsParameters.KbNumbersToInclude))
-                    .ForMember(dest => dest.InstallPatchRebootSetting, opt => opt.MapFrom(src => src.InstallPatches.RebootSetting))
                     .ForMember(dest => dest.PreTask, opt => opt.Ignore())
                     .ForMember(dest => dest.PostTask, opt => opt.Ignore())
                     .ForSourceMember(src => src.SystemData, s => s.Ignore());
@@ -99,6 +98,7 @@ private static void Initialize()
                     .ForPath(dest => dest.InstallPatches.WindowsParameters.ExcludeKbsRequiringReboot, src => src.MapFrom(o => o.WindowParameterExcludeKbRequiringReboot))
                     .ForPath(dest => dest.InstallPatches.WindowsParameters.KbNumbersToExclude, src => src.MapFrom(o => o.WindowParameterKbNumberToExclude))
                     .ForPath(dest => dest.InstallPatches.WindowsParameters.KbNumbersToInclude, src => src.MapFrom(o => o.WindowParameterKbNumberToInclude))
+                    .ForPath(dest => dest.InstallPatches.RebootSetting, src => src.MapFrom(o => o.InstallPatchRebootSetting))
                     .AfterMap((src, dst) => {
                         if (string.IsNullOrEmpty(src.InstallPatchRebootSetting) &&
                             !src.WindowParameterExcludeKbRequiringReboot.HasValue &&