ColtenOuO commented on PR #73381: URL: https://github.com/apache/airflow/pull/73381#issuecomment-5749765512
> One note on the description, since it has no inline anchor: the summary gives "for example from a template or configuration" as how an empty allow-list arrives. Jinja cannot be the source here. `toolsets` is not in `AgentOperator.template_fields` (`operators/agent.py:261-270`), `@task.agent` re-exports that same list without adding to it (`decorators/agent.py:61-64`), and no toolset declares `template_fields` of its own, so nothing inside `SQLToolset(...)` is ever rendered. > > The configuration half holds, though: a `Variable.get` or a YAML read in the Dag file can hand back `[]`. Worth dropping the template example so the rationale rests on the path that can actually happen. Yeah, I have checked them again and fixed the PR's description -- 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]
