Skip to content

Commit e7152cc

Browse files
committed
Ignore server status during polling checks when the TaskState property is not null
1 parent ea7d492 commit e7152cc

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

src/corelib/Providers/Rackspace/CloudServersProvider.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -331,7 +331,7 @@ public Server WaitForServerState(string serverId, ServerState[] expectedStates,
331331

332332
int count = 0;
333333
int currentProgress = -1;
334-
while (!expectedStates.Contains(serverDetails.Status) && !errorStates.Contains(serverDetails.Status) && count < refreshCount)
334+
while ((serverDetails.TaskState != null || (!expectedStates.Contains(serverDetails.Status) && !errorStates.Contains(serverDetails.Status))) && count < refreshCount)
335335
{
336336
if (progressUpdatedCallback != null)
337337
{

src/testing/integration/Providers/Rackspace/UserServerTests.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -211,7 +211,7 @@ public void TestRescueServer()
211211

212212
string rescueResult = provider.RescueServer(_server.Id);
213213
Assert.IsFalse(string.IsNullOrEmpty(rescueResult));
214-
Server rescueServer = provider.WaitForServerState(_server.Id, ServerState.Rescue, new[] { ServerState.Error });
214+
Server rescueServer = provider.WaitForServerState(_server.Id, ServerState.Rescue, new[] { ServerState.Active, ServerState.Error, ServerState.Unknown, ServerState.Suspended });
215215
Assert.AreEqual(ServerState.Rescue, rescueServer.Status);
216216

217217
bool unrescueResult = provider.UnRescueServer(_server.Id);
@@ -325,7 +325,7 @@ public void TestConfirmServerResize()
325325

326326
bool resized = provider.ResizeServer(_server.Id, serverName, flavor.Id);
327327
Assert.IsTrue(resized);
328-
Server resizedServer = provider.WaitForServerState(_server.Id, ServerState.VerifyResize, new[] { ServerState.Error, ServerState.Unknown, ServerState.Suspended });
328+
Server resizedServer = provider.WaitForServerState(_server.Id, ServerState.VerifyResize, new[] { ServerState.Active, ServerState.Error, ServerState.Unknown, ServerState.Suspended });
329329
Assert.AreEqual(ServerState.VerifyResize, resizedServer.Status);
330330
_server = resizedServer;
331331

@@ -351,7 +351,7 @@ public void TestRevertServerResize()
351351

352352
bool resized = provider.ResizeServer(_server.Id, serverName, flavor.Id);
353353
Assert.IsTrue(resized);
354-
Server resizedServer = provider.WaitForServerState(_server.Id, ServerState.VerifyResize, new[] { ServerState.Error, ServerState.Unknown, ServerState.Suspended });
354+
Server resizedServer = provider.WaitForServerState(_server.Id, ServerState.VerifyResize, new[] { ServerState.Active, ServerState.Error, ServerState.Unknown, ServerState.Suspended });
355355
Assert.AreEqual(ServerState.VerifyResize, resizedServer.Status);
356356
_server = resizedServer;
357357

0 commit comments

Comments
 (0)