The GitHub Actions job "Tests (AMD)" on airflow.git/guard-get-config-in-the-key-vault-backend has succeeded. Run started by GitHub user potiuk (triggered by potiuk).
Head commit for run: d41d9dab6fc19cb2902295b4f6cb86460ec1fa8d / Jarek Potiuk <[email protected]> Apply the ambiguous-id refusal to get_config in the Key Vault backend Follow-up to #70876. The check used to live in _get_secret, which all three getters share. Moving it up into get_conn_value and get_variable left get_config reaching _get_secret unguarded, so the helper's claim that such an id is refused for every lookup stopped being true. Not a cross-team read -- get_config never receives a team_name -- but two getters and the docstring disagreed with the third. The same gap was found in review of the Amazon sibling (#70878) and fixed there; this is the Azure half, which had already merged by then. _names_a_team_namespace stays an instance method here, unlike Amazon's: it builds the candidate path with self.build_path and self.sep. The refusal-logging assertion uses getMessage() rather than msg, since how a record carries its payload differs between the Airflow version this runs on and the ones the provider compat tests use. Report URL: https://github.com/apache/airflow/actions/runs/30704257086 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
