-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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
chore: clarify non existence or not defined pod model file #47406
base: main
Are you sure you want to change the base?
Conversation
providers/cncf/kubernetes/src/airflow/providers/cncf/kubernetes/pod_generator.py
Dismissed
Show resolved
Hide resolved
18ec5a4
to
31fd30b
Compare
@enchant3dmango Continuing the conversation from this discussion, I tried running the Ultimately, it's
I should note, this happens on a fresh Docker container with no custom configuration even mounted, and I was able to get the source version of 2.10.4 installed and working the same way, so I'm guessing something in In any case, I think the combination of the clearer log message and setting it to |
@shivshav I've changed my mind. In that case, I think we can open another PR. Let's keep it like this for now. |
@enchant3dmango No worries! Sounds good with me, I appreciate all the help you've given so far so thank you for that 🙏🏾 I can take a look at figuring out the |
Update ambiguous logging.
Related discussions:
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named
{pr_number}.significant.rst
or{issue_number}.significant.rst
, in newsfragments.