molcay commented on PR #74305: URL: https://github.com/apache/airflow/pull/74305#issuecomment-6019919411
Hi @shahar1, First off, I want to sincerely apologize if my previous comment came across as harsh—that wasn't my intention at all. My main goal was to highlight a broader, ongoing challenge we are facing on our end, rather than making a strict point about this specific PR. I completely understand your perspective. You are absolutely right that requiring manual system tests and waiting for our team's explicit approval can create bottlenecks, slow down the release cycle, and put an unfair burden on maintainers and release managers. At the same time, when untested changes make it to the RC phase, it creates significant downstream problems and triples the workload to catch and fix issues late in the game. Currently, we are trying to make internal arrangements to review PRs more quickly. I think this situation is a great signal that we need to find a better middle ground. We should work together to establish a sustainable process that works for everyone involved—our team, the Airflow maintainers, and the contributors. Thank you for taking ownership of running the tests for this PR in the meantime; I really appreciate it. Let's definitely keep the conversation going on the dev list about how we can better automate and integrate system tests into the CI/CD. Until automation is complete, maybe we can find another way to share this load somehow. Ultimately, we all share the exact same goal: making Airflow and all providers (not only the google provider) as stable and reliable as possible. Thanks again for your understanding and collaboration! -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
