Skip to content

Commit

Permalink
Fix Az.ServiceFabric XML Comments (#18586)
Browse files Browse the repository at this point in the history
  • Loading branch information
v-yuzhichen authored Jun 22, 2022
1 parent 5bd637e commit e3aed55
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -108,14 +108,14 @@ public override void ExecuteCmdlet()
{
cluster.NodeTypes.Remove(existingNodeType);

/**
/*
* * Pulled this out after discussion with Justin. Opened Issue #6246 to track the Null Ptr Exception.
cluster.UpgradeDescription.DeltaHealthPolicy = new ClusterUpgradeDeltaHealthPolicy()
{
MaxPercentDeltaUnhealthyApplications = 0,
MaxPercentDeltaUnhealthyNodes = 0,
MaxPercentUpgradeDomainDeltaUnhealthyNodes = 0
};**/
};*/

var patchRequest = new ClusterUpdateParameters
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ public class UpdateAzServiceFabricVmImage : ServiceFabricClusterCmdlet
/// <summary>
/// VmImage sets the Service Fabric runtime package delivery hint for the cluster, which results in dropping the appropriately-mapped package for the VM image OS.
/// Accepted values: Windows, Linux, Ubuntu, Ubuntu18_04
/// Note: Linux & Ubuntu both map to Ubuntu 16.04
/// Note: Linux &amp; Ubuntu both map to Ubuntu 16.04
/// </summary>
[Parameter(Mandatory = true, ValueFromPipeline = true,
HelpMessage = "VM Image hint for determining SF runtime package delivery: i.e. .cab/.deb")]
Expand Down

0 comments on commit e3aed55

Please sign in to comment.