The GitHub Actions job "Tests" on airflow.git/feature/dynamic-dag-bundle-config has failed. Run started by GitHub user tardunge (triggered by potiuk).
Head commit for run: 8abfb212b836f017d64dc445efb853980499c683 / Manoj Babu Katragadda <[email protected]> Document dag_bundle_config_path and tighten review feedback Adds a prose section to dag-bundles.rst describing the directory convention, JSON file format, hot-reload semantics, precedence over dag_bundle_config_list, and validation behaviour for the new dag_bundle_config_path option (the previous PR commits only added a config.yml description). Also addresses review correctness/robustness feedback on the DagBundlesManager directory mode: - Track every *.json file's mtime regardless of parse outcome, so invalid or duplicate-name files no longer cause check_config_path_changes to report "changed" on every poll. - Use st_mtime_ns instead of float st_mtime for reliable detection on filesystems with coarse-resolution timestamps. - Validate that the JSON "name" is a non-empty string before adding it to the duplicate-detection set; non-string names previously raised TypeError when added to a set. - Clear bundle_config and config_path_mtime on the early-return paths (missing directory, no valid configs found) so removed bundles get cleaned up instead of staying configured. - Replace time.sleep(0.01) in the test with an explicit os.utime mtime bump so the test doesn't depend on filesystem timestamp granularity. Report URL: https://github.com/apache/airflow/actions/runs/24998684674 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
