> @@ -135,100 +148,129 @@ public SoftLayerComputeServiceAdapter(SoftLayerApi 
> client,
>  
>        checkState(orderInSystem, "order for guest %s doesn't have login 
> details within %sms", result,
>              Long.toString(guestLoginDelay));

I know this check hasn't changed, but a failure at this point could leave the 
virtual guest created but the ComputeService won't return gracefully. Worth 
changing it to leave a log and not failing, or rolling back the server if it is 
not accessible? Just trying to avoid an inconsistent state or one where the 
user might think that the operation failed whilst the server remains created.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/296/files#r10276329

Reply via email to