ephraimbuddy commented on code in PR #74370: URL: https://github.com/apache/airflow/pull/74370#discussion_r4228166533
########## airflow-core/src/airflow/api_fastapi/execution_api/datamodels/token.py: ########## @@ -17,31 +17,60 @@ from __future__ import annotations -from typing import Literal +from typing import Annotated, Literal from uuid import UUID -from pydantic import ConfigDict +from pydantic import ConfigDict, Field, model_validator from airflow.api_fastapi.core_api.base import BaseModel +from airflow.typing_compat import Self -TokenScope = Literal["execution", "workload", "callback"] +TokenScope = Literal[ + "execution", "workload", "callback", "dag_processor_session", "dag_processor", "dag_parse" Review Comment: On a second thought, I think we should keep the dag_processor_session and not rename it as *_registration because dag_processor_session makes the distinction clear against job's registration_id -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
