dstandish commented on code in PR #37851:
URL: https://github.com/apache/airflow/pull/37851#discussion_r1510092522


##########
airflow/cli/commands/webserver_command.py:
##########
@@ -137,7 +137,9 @@ def ready_prefix_on_cmdline(proc):
             try:
                 cmdline = proc.cmdline()
                 if cmdline:
-                    return settings.GUNICORN_WORKER_READY_PREFIX in cmdline[0]
+                    return cmdline[0].contains(
+                        (settings.GUNICORN_WORKER_READY_PREFIX,)

Review Comment:
   this logic does not work for internal API cus it has a different string; if 
you turn on debug logging then you can see that it thinks workers are not ready 
when they are



-- 
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]

Reply via email to