Partially updated get/put/post for CompoundJob/Action methods.

wrench-project · Apr 8, 2024 · f186079 · f186079
1 parent 916a446
commit f186079
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/tools/wrench/wrench-daemon/doc/wrench-openapi.json b/tools/wrench/wrench-daemon/doc/wrench-openapi.json
@@ -2824,7 +2824,7 @@
       }
     },
     "/simulation/{simid}/compoundJobs/{compound_job_name}/addComputeAction": {
-      "get": {
+      "post": {
         "tags": [
           "WRENCH"
         ],

diff --git a/tools/wrench/wrench-daemon/src/SimulationController.cpp b/tools/wrench/wrench-daemon/src/SimulationController.cpp
@@ -1199,16 +1199,17 @@ json SimulationController::addSleepAction(json data) {
         std::string child_compound_job_name = data["compound_job"];
         std::string parent_compound_job_name = data["parent_compound_job"];
 
-        std::shared_ptr<CompoundJob> compound_job;
+        std::shared_ptr<CompoundJob> parent_compound_job;
+        std::shared_ptr<CompoundJob> child_compound_job;
 
-        if (not this->compound_job_registry.lookup(parent_compound_job_name, compound_job)) {
+        if (not this->compound_job_registry.lookup(parent_compound_job_name, parent_compound_job)) {
             throw std::runtime_error("Unknown compound job " + parent_compound_job_name);
     }
-        if (not this->compound_job_registry.lookup(child_compound_job_name, compound_job)) {
+        if (not this->compound_job_registry.lookup(child_compound_job_name, child_compound_job)) {
             throw std::runtime_error("Unknown compound job " + child_compound_job_name);
         }
 
-        child_compound_job_name->addParentJob(parent_compound_job_name);
+        child_compound_job->addParentJob(parent_compound_job);
         return;
 }