From c53471674f9c346cea2e2b75c2f785d6d5f080d3 Mon Sep 17 00:00:00 2001
From: Mohammad Torkashvand <mohammad.torkashvand@geant.org>
Date: Tue, 5 Nov 2024 15:15:00 +0100
Subject: [PATCH] remove unnecessary begin steps

---
 gso/oss-params-example.json | 3 ++-
 gso/utils/workflow_steps.py | 6 +++---
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/gso/oss-params-example.json b/gso/oss-params-example.json
index 95ad8c02..63de6f89 100644
--- a/gso/oss-params-example.json
+++ b/gso/oss-params-example.json
@@ -125,6 +125,7 @@
     "DSN": "https://sentry-dsn-url"
   },
   "MOODI": {
-    "host": "moodi.test.gap.geant.org"
+    "host": "moodi.test.gap.geant.org",
+    "moodi_enabled": false
   }
 }
diff --git a/gso/utils/workflow_steps.py b/gso/utils/workflow_steps.py
index 3ce208a8..01a64e58 100644
--- a/gso/utils/workflow_steps.py
+++ b/gso/utils/workflow_steps.py
@@ -7,7 +7,7 @@ from orchestrator import inputstep, step
 from orchestrator.config.assignee import Assignee
 from orchestrator.types import State, UUIDstr
 from orchestrator.utils.json import json_dumps
-from orchestrator.workflow import StepList, begin, conditional
+from orchestrator.workflow import StepList, conditional
 from pydantic import ConfigDict
 from pydantic_forms.core import FormPage
 from pydantic_forms.types import FormGenerator
@@ -407,7 +407,7 @@ def start_moodi() -> StepList:
             "extra_vars": {"subscription": subscription},
         }
 
-    return begin >> _is_moodi_enabled(begin >> lso_interaction(_start_moodi))
+    return _is_moodi_enabled(lso_interaction(_start_moodi))
 
 
 def stop_moodi() -> StepList:
@@ -422,4 +422,4 @@ def stop_moodi() -> StepList:
             "extra_vars": None,
         }
 
-    return begin >> _is_moodi_enabled(begin >> lso_interaction(_stop_moodi))
+    return _is_moodi_enabled(lso_interaction(_stop_moodi))
-- 
GitLab