ephraimbuddy commented on code in PR #74370: URL: https://github.com/apache/airflow/pull/74370#discussion_r4208902897
########## 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: dag_processor_session is provisioned externally and permits registration/renewal. dag_processor is issued for a specific Job and permits its lifecycle operations and parsing-token exchange. Maybe the first one should be named `dag_processor_registration`? -- 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]
