Set scenarioStatus to ready when copied (#48)

* tests for clusters option before using it * uses _hasVms value inside GetStackstormVms * sets scenarioStatus to ready when scenario is copied
cmu-sei · Dec 13, 2022 · 4e54346 · 4e54346
1 parent d7a453d
commit 4e54346
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/Steamfitter.Api/Services/ScenarioService.cs b/Steamfitter.Api/Services/ScenarioService.cs
@@ -252,6 +252,7 @@ public ScenarioService(SteamfitterContext context,
             newScenarioEntity.CreatedBy = _user.GetId();
             newScenarioEntity.Name = $"{oldScenarioEntity.Name} - {_user.Claims.FirstOrDefault(c => c.Type == "name").Value}";
             newScenarioEntity.OnDemand = true;
+            newScenarioEntity.Status = ScenarioStatus.ready;
 
             _context.Scenarios.Add(newScenarioEntity);
             await _context.SaveChangesAsync(ct);

diff --git a/Steamfitter.Api/Services/StackStormService.cs b/Steamfitter.Api/Services/StackStormService.cs
@@ -160,6 +160,11 @@ public async STT.Task GetStackstormVms()
             var apiParameters = _options.ApiParameters;
             try
             {
+                if (!_hasVms)
+                {
+                    return;
+                }
+
                 var clusters = apiParameters["clusters"].ToString().Split(",");
                 var vmListResult = await _stackStormConnector.VSphere.GetVmsWithUuid(clusters);
                 // add VM's to _vmList