The GitHub Actions job "Tests (AMD)" on 
airflow.git/fix/supervisor-comms-import-error has succeeded.
Run started by GitHub user andreahlert (triggered by andreahlert).

Head commit for run:
e8ba49a1ed939d93021f6df58ca8ae626298f442 / AndrĂ© Ahlert <[email protected]>
Address review: fix Variable.keys() ImportError and hoist Variable.set() guard

- _get_variable_keys: replace `from ... import SUPERVISOR_COMMS` (raises
  ImportError, bare annotation) with `task_runner` namespace + hasattr
  guard; previously Variable.keys() outside task context hit the same
  opaque failure this PR is trying to wrap.
- _set_variable: hoist hasattr guard above the secrets-backend
  conflict-check loop so the misleading "API Server will be updated"
  warning is not emitted when the write cannot happen.

Matches the pattern already applied to _get_variable and _delete_variable.

Signed-off-by: AndrĂ© Ahlert <[email protected]>

Report URL: https://github.com/apache/airflow/actions/runs/26375126114

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to