From 4889bee4fd3f7d3c79d82e35d7689a9cda94b0e3 Mon Sep 17 00:00:00 2001
From: Matthias Tafelmeier <matthias.tafelmeier@gmx.net>
Date: Thu, 8 Jun 2023 18:46:17 +0200
Subject: [PATCH] stop instrumenting api for create for now

We generate the dag id in the root dag file, that's not trivially
available to the api controller and it cannot respect that id in the api
dag run create call.

The templating is more important for now as the dags are triggered
anyhow via airflow engine.
---
 api/controller.py | 30 +++++++++++++++++-------------
 1 file changed, 17 insertions(+), 13 deletions(-)

diff --git a/api/controller.py b/api/controller.py
index 0bfde45d..1d025da8 100644
--- a/api/controller.py
+++ b/api/controller.py
@@ -35,6 +35,7 @@
 from airflow_client.client.model.connection import Connection
 
 from flask import abort
+from flask import Response
 
 from jinja2 import Environment, FileSystemLoader
 
@@ -193,19 +194,22 @@ def create_breeder(api_client, content):
 
         time.sleep(2) # wait as workaround until synchronous reload of dags implemented
 
-        dag_run = DAGRun(
-            dag_run_id=breeder_id ,
-            #state=DagState("queued"),
-            conf=breeder_config,
-        ) # DAGRun |
+        # Stop calling the API for now until decided
+        # if we template the breeder dags only or we really want to instrument the API.
 
-        try:
-            # Trigger a new DAG run
-            _api_response = api_instance.post_dag_run(breeder_id, dag_run)
-        except client.ApiException as e:
-            print("Exception when calling DAGRunApi->post_dag_run: %s\n" % e)
-            raise e
-        return _api_response
+        #dag_run = DAGRun(
+        #    dag_run_id=breeder_id ,
+        #    #state=DagState("queued"),
+        #    conf=breeder_config,
+        #) # DAGRun |
+
+        #try:
+        #    # Trigger a new DAG run
+        #    _api_response = api_instance.post_dag_run(breeder_id, dag_run)
+        #except client.ApiException as e:
+        #    print("Exception when calling DAGRunApi->post_dag_run: %s\n" % e)
+        #    raise e
+        #return _api_response
 
     with client.ApiClient(configuration) as api_client:
         # Do not create connection dynamically for now
@@ -213,7 +217,7 @@ def create_breeder(api_client, content):
         api_response['breeder'] = create_breeder(api_client, content).to_dict()
 
 
-    return api_response
+    return Response(dict(), status=200, mimetype='application/json')
 
 
 def breeders_put(content):  # noqa: E501