Skip to content

Commit

Permalink
Revert "Added force remove application cmdlet in case of failure of r…
Browse files Browse the repository at this point in the history
…emove ap… (#6431)"

This reverts commit 34d5dde.
  • Loading branch information
vinodkumar3 committed Feb 15, 2018
1 parent 34d5dde commit 320a33f
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -209,12 +209,17 @@
Write-Host (Get-VstsLocString -Key SFSDK_AppAlreadyExistsInfo -ArgumentList @($ApplicationName, $app.ApplicationTypeName, $app.ApplicationTypeVersion))

try
{
$app | Remove-ServiceFabricApplication -Force
}
catch [System.TimeoutException]
{
# Catch operation timeout and continue with force remove replica.
}

foreach ($node in Get-ServiceFabricNode)
{
$app | Remove-ServiceFabricApplication -Force
}
catch [System.TimeoutException]
{
$app | Remove-ServiceFabricApplication -Force -ForceRemove
[void](Get-ServiceFabricDeployedReplica -NodeName $node.NodeName -ApplicationName $ApplicationName | Remove-ServiceFabricReplica -NodeName $node.NodeName -ForceRemove)
}

if($OverwriteBehavior.Equals("Always"))
Expand Down
2 changes: 1 addition & 1 deletion Tasks/ServiceFabricDeploy/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
"version": {
"Major": 1,
"Minor": 6,
"Patch": 1
"Patch": 0
},
"demands": [
"Cmd"
Expand Down
2 changes: 1 addition & 1 deletion Tasks/ServiceFabricDeploy/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
"version": {
"Major": 1,
"Minor": 6,
"Patch": 1
"Patch": 0
},
"demands": [
"Cmd"
Expand Down

0 comments on commit 320a33f

Please sign in to comment.