You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/12/12 20:03:09 UTC

[GitHub] [airflow] evgenyslab commented on issue #28310: TriggerDagRunOperator Clears multiple runs instead of specific run_id due to date selection instead of run_id selection

evgenyslab commented on issue #28310:
URL: https://github.com/apache/airflow/issues/28310#issuecomment-1347222493

   I was able to work around this by creating a custom TriggerDagRunOperator by hacking together the dag_bag's internal clearing mechanism and explicitly specifying a run_id...
   
   ```
   #
   # Licensed to the Apache Software Foundation (ASF) under one
   # or more contributor license agreements.  See the NOTICE file
   # distributed with this work for additional information
   # regarding copyright ownership.  The ASF licenses this file
   # to you under the Apache License, Version 2.0 (the
   # "License"); you may not use this file except in compliance
   # with the License.  You may obtain a copy of the License at
   #
   #   http://www.apache.org/licenses/LICENSE-2.0
   #
   # Unless required by applicable law or agreed to in writing,
   # software distributed under the License is distributed on an
   # "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
   # KIND, either express or implied.  See the License for the
   # specific language governing permissions and limitations
   # under the License.
   
   import datetime
   import json
   import time
   from typing import Dict, List, Optional, Union
   
   from airflow import settings
   from airflow.api.common.experimental.trigger_dag import trigger_dag
   from airflow.exceptions import AirflowException, DagNotFound, DagRunAlreadyExists
   from airflow.models import BaseOperator, BaseOperatorLink, DagBag, DagModel, DagRun
   from airflow.utils import timezone
   from airflow.utils.helpers import build_airflow_url_with_query
   from airflow.utils.state import State
   from airflow.utils.types import DagRunType
   from airflow.utils.state import DagRunState, State
   from airflow.models.taskinstance import clear_task_instances
   
   
   class TriggerDagRunLink(BaseOperatorLink):
       """
       Operator link for TriggerDagRunOperator. It allows users to access
       DAG triggered by task using TriggerDagRunOperator.
       """
   
       name = 'Triggered DAG'
   
       def get_link(self, operator, dttm):
           query = {"dag_id": operator.trigger_dag_id, "execution_date": dttm.isoformat()}
           return build_airflow_url_with_query(query)
   
   
   class TriggerDagRunOperator(BaseOperator):
       """
       Triggers a DAG run for a specified ``dag_id``
   
       :param trigger_dag_id: The dag_id to trigger (templated).
       :type trigger_dag_id: str
       :param trigger_run_id: The run ID to use for the triggered DAG run (templated).
           If not provided, a run ID will be automatically generated.
       :type trigger_run_id: str
       :param conf: Configuration for the DAG run.
       :type conf: dict
       :param execution_date: Execution date for the dag (templated).
       :type execution_date: str or datetime.datetime
       :param reset_dag_run: Whether or not clear existing dag run if already exists.
           This is useful when backfill or rerun an existing dag run.
           When reset_dag_run=False and dag run exists, DagRunAlreadyExists will be raised.
           When reset_dag_run=True and dag run exists, existing dag run will be cleared to rerun.
       :type reset_dag_run: bool
       :param wait_for_completion: Whether or not wait for dag run completion. (default: False)
       :type wait_for_completion: bool
       :param poke_interval: Poke interval to check dag run status when wait_for_completion=True.
           (default: 60)
       :type poke_interval: int
       :param allowed_states: List of allowed states, default is ``['success']``.
       :type allowed_states: list
       :param failed_states: List of failed or dis-allowed states, default is ``None``.
       :type failed_states: list
       """
   
       template_fields = ("trigger_dag_id", "trigger_run_id", "execution_date", "conf")
       template_fields_renderers = {"conf": "py"}
       ui_color = "#ffefeb"
   
       @property
       def operator_extra_links(self):
           """Return operator extra links"""
           return [TriggerDagRunLink()]
   
       def __init__(
           self,
           *,
           trigger_dag_id: str,
           trigger_run_id: Optional[str] = None,
           conf: Optional[Dict] = None,
           execution_date: Optional[Union[str, datetime.datetime]] = None,
           reset_dag_run: bool = False,
           wait_for_completion: bool = False,
           poke_interval: int = 60,
           allowed_states: Optional[List] = None,
           failed_states: Optional[List] = None,
           **kwargs,
       ) -> None:
           super().__init__(**kwargs)
           self.trigger_dag_id = trigger_dag_id
           self.trigger_run_id = trigger_run_id
           self.conf = conf
           self.reset_dag_run = reset_dag_run
           self.wait_for_completion = wait_for_completion
           self.poke_interval = poke_interval
           self.allowed_states = allowed_states or [State.SUCCESS]
           self.failed_states = failed_states or [State.FAILED]
   
           if not isinstance(execution_date, (str, datetime.datetime, type(None))):
               raise TypeError(
                   "Expected str or datetime.datetime type for execution_date."
                   "Got {}".format(type(execution_date))
               )
   
           self.execution_date: Optional[datetime.datetime] = execution_date  # type: ignore
   
           try:
               json.dumps(self.conf)
           except TypeError:
               raise AirflowException("conf parameter should be JSON Serializable")
   
       def execute(self, context: Dict):
           if isinstance(self.execution_date, datetime.datetime):
               execution_date = self.execution_date
           elif isinstance(self.execution_date, str):
               execution_date = timezone.parse(self.execution_date)
               self.execution_date = execution_date
           else:
               execution_date = timezone.utcnow()
   
           if self.trigger_run_id:
               run_id = self.trigger_run_id
           else:
               run_id = DagRun.generate_run_id(DagRunType.MANUAL, execution_date)
   
           try:
               dag_run = trigger_dag(
                   dag_id=self.trigger_dag_id,
                   run_id=run_id,
                   conf=self.conf,
                   execution_date=self.execution_date,
                   replace_microseconds=False,
               )
           except DagRunAlreadyExists as e:
               if self.reset_dag_run:
                   self.log.info("Clearing %s on %s", self.trigger_dag_id, self.execution_date)
   
                   # Get target dag object and call clear()
   
                   dag_model = DagModel.get_current(self.trigger_dag_id)
                   if dag_model is None:
                       raise DagNotFound(f"Dag id {self.trigger_dag_id} not found in DagModel")
   
                   dag_bag = DagBag(dag_folder=dag_model.fileloc, read_dags_from_db=True)
                   dag = dag_bag.get_dag(self.trigger_dag_id)
                   # dag.clear(start_date=self.execution_date, end_date=self.execution_date)
                   session = settings.Session()  # type: ignore
                   tis = dag._get_task_instances(
                           task_ids=None,
                           start_date=self.execution_date,
                           end_date=self.execution_date,
                           run_id=run_id,
                           state=[],
                           include_subdags=True,
                           include_parentdag=True,
                           include_dependent_dags=True,  # compat, yes this is not a typo
                           as_pk_tuple=False,
                           session=session,
                           dag_bag=dag_bag,
                           exclude_task_ids=frozenset({}),
                       )
                   
                   clear_task_instances(
                       tis,
                       session,
                       dag=dag,
                       dag_run_state= DagRunState.QUEUED,
                       )
                   
                   dag_run = DagRun.find(dag_id=dag.dag_id, run_id=run_id)[0]
               else:
                   raise e
   
           if self.wait_for_completion:
               # wait for dag to complete
               while True:
                   self.log.info(
                       'Waiting for %s on %s to become allowed state %s ...',
                       self.trigger_dag_id,
                       dag_run.execution_date,
                       self.allowed_states,
                   )
                   time.sleep(self.poke_interval)
   
                   dag_run.refresh_from_db()
                   state = dag_run.state
                   if state in self.failed_states:
                       raise AirflowException(f"{self.trigger_dag_id} failed with failed states {state}")
                   if state in self.allowed_states:
                       self.log.info("%s finished with allowed state %s", self.trigger_dag_id, state)
                       return
   
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org