The GitHub Actions job "Tests" on airflow.git has failed.
Run started by GitHub user potiuk (triggered by potiuk).

Head commit for run:
c14a381aa3f21e5deeef266364519ddca2678bce / Jarek Potiuk <[email protected]>
Switch configuration to use built-in defaults rather than config file

When Airflow start without airflow.cfg, the default configuration file
is generated and so far it contained all the defaults.

However, this caused some problems - the configuration values
modified by the users were not easily distinguishable from those from
default values.

While having comments, sections and descriptions in the generated
config file was useful, having the values there was problematic,
mainly when defaults have cahnged or when configuration sections
or keys were renamed - because we could not easily change defaults
for those users who already had their config file generated.

This PR changes the approach for defaults:

* instead of reading defaults from the config file, we read the
  defaults from internal python dictionary (generated from the
  config yaml single-source-of truth

* the config file when missing is still generated, but all values
  in there are commented out. This has the nice effect that you
  can still use just the config file to know what configuration
  options you have, and easiky change them (by just uncommenting
  and changing value) - but the "default" value is now really part
  of the airflow application not the generated configuration file

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

With regards,
GitHub Actions via GitBox


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

Reply via email to