shivaam commented on PR #63512: URL: https://github.com/apache/airflow/pull/63512#issuecomment-4061594242
Hey @potiuk, this is a great addition for keeping the queue clean! I really like the systemic failure detection, but I have a thought on the 4-hour grace period for PRs with active human engagement. In an asynchronous open-source environment, 4 hours is a very narrow window. Many of us are balancing this with other work or are in different time zones. If an author pushes a fix and hits an isolated flake or minor linting error, the bot auto-drafts it while they are offline. By the time the requester can fix it, the maintainer (who may only check PRs every few days) has already seen the "Draft" status and moved on. This kills momentum on PRs that were already 99% there. Could we make the auto-triage more "context-aware"? Engagement Lenience: If a PR has a maintainer approval or recent comments, could we extend the grace period (e.g., 24-48 hours)? This respects the human review state and the reality that people aren't monitoring CI 24/7. "Previously Green" Logic: If a PR has passed CI in the past, we could treat new failures as "needs a fix" rather than "needs to be drafted" for a longer period. Local Preview Mode: A way for committers to run the review mode locally to see suggestions and logs without the tool auto-drafting or posting to the PR. -- 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]
