potiuk commented on PR #34331: URL: https://github.com/apache/airflow/pull/34331#issuecomment-1826808549
> If maintainers are not interested of merging PR which avoids producing wrong pod description, and instead prefer to do nothing with that for months - ok, but very pitty to hear. Well. Maintainers are interested in getting future looking solution that not only solve this problem but will prevent users from shooting themselves in their foot and then opening even more issues when the do and have weird problems. Sometimes putting a small patch where your blood is poisoned, will make you die anyway, but you will not realize it because you put a blind eye and pretend that tbe bigger problem does not exist. So -if this PR (which I hope it will) will start a discussion that will lead to a better solution, that might be the best outcome of this PR. Merging it now without discussion will simply let people put a blind eye on the real issue we have, so I woul strongly suggest to discuss alternatives and see what other proposals people have knowing the context. If you think we are doing nothing here, you are completely wrong: * I initially proposed to disable this combo: https://github.com/apache/airflow/pull/28822 - but then it was not well received * Then I closed in favour of documentation STRONG discouragement of using this combo: https://github.com/apache/airflow/pull/28822 Yet - people like you continue ignoring the strong discouragement we have and raise issues about some of the things not working with this combo - which of course are unrelated, but they are - again not realising they have a bigger problem and that what they do is really poisonous blood in their organism that they are trying to put patch on. So yeah. I am not thinking about doing nothing. I am looking for other's comments and suggestions what we do in this situation - including taking your suggestion about warning (which I have no idea how to do even if I tried in the original PR). It's a pity that some users are complaining, when the maintainers actually go to a great length to solve the actual problems rather than let their users to patch and put blind eye on the bigger problems they have. I'd say it would be an easy solution to just merge the PR. yet I've chosen a much more demanding approach that could help to solve bigger problem if we have people helping here. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
