kacpermuda commented on PR #69212: URL: https://github.com/apache/airflow/pull/69212#issuecomment-4916067262
> the fixed time.sleep(15) in the auth_headers fixture is a touch fragile given compose.up(wait=True) already gates readiness, and wait_for_airflow_api() uses a bare requests.get while its siblings use the retry-mounted self.session. Neither needs to hold up merge. Agreed, this is probably to many safety checks for that. I've ported some of the code from the actual e2e tests I've been running on 3rd party infra for some time, where there is much more instability of infra causing flakiness, so I might have been too cautious here. Let's keep it for now if it's okay, will be happy to adjust soon if we see these tests taking up to much time or just during my next update to this framework (I'll leave a note to myself about cleaning this) -- 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]
