From fa0db1286311d0b735e015e090059724f6eeb253 Mon Sep 17 00:00:00 2001 From: Matthias Tafelmeier 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