-
Notifications
You must be signed in to change notification settings - Fork 16.4k
Add note about Official Docker Image default for load_examples #60723
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
2b3f57e to
c039006
Compare
jroachgolf84
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! I'm not a committer, so my review does not hold much weight. But this LG!
choo121600
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, LGTM 😉
Same comment as before — could we also update DAGs to Dags here for consistency?
1c0d5aa to
b5bca4e
Compare
I’ve updated it for consistency. Thank you :) |
Related Issue
closes: #60722
Summary
Adds a note to the
load_examplesconfiguration documentation clarifying that the Official Airflow Docker Image sets this value toFalseby default, which overrides the defaultTruevalue.Problem
The Configuration Reference mentions that the default value for
load_examplesisTrue, but it doesn't mention that this is overridden in the official Docker image. This can be confusing for users who expect example DAGs to be present when they first run the official image but find them missing.Solution
Added a note in the
load_examplesconfiguration description inconfig.ymlto inform users about this behavior in the official Docker image and how to enable example DAGs.Was generative AI tooling used to co-author this PR?
{pr_number}.significant.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.