dheerajturaga commented on code in PR #50425:
URL: https://github.com/apache/airflow/pull/50425#discussion_r2083465992


##########
providers/edge3/src/airflow/providers/edge3/cli/edge_command.py:
##########
@@ -787,6 +803,13 @@ def remote_worker_request_shutdown(args) -> None:
     help="State of the edge worker",
 )
 
+ARG_DAEMON = Arg(
+    ("-D", "--daemon"), help="Daemonize instead of running in the foreground", 
action="store_true"
+)
+ARG_STDERR = Arg(("--stderr",), help="Redirect stderr to this file")
+ARG_STDOUT = Arg(("--stdout",), help="Redirect stdout to this file")
+ARG_LOG_FILE = Arg(("-l", "--log-file"), help="Location of the log file")

Review Comment:
   the `run_command_with_daemon_option` works as such. When -D is unspecified 
the -l/stderr/out are ignored. When running as a daemonized process. If the log 
files are unspecified, it will create `airflow-edge-worker.pid,log,out,err` 
files in $AIRFLOW_HOME. The difference between -l and --stdout is that -l only 
captures output from the logger object, however --stdout will capture entire 
stdout ( including prints)



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

Reply via email to