potiuk commented on a change in pull request #17424:
URL: https://github.com/apache/airflow/pull/17424#discussion_r682970446
##########
File path: airflow/utils/log/file_task_handler.py
##########
@@ -187,10 +187,14 @@ def _read(self, ti, try_number, metadata=None):
response.encoding = "utf-8"
if response.status_code == 403:
- log += "*** !!!! Please make sure that all your webservers
and workers have" \
- " the same 'secret_key' configured in 'webserver'
section !!!!!\n***"
- log += "*** See more at
https://airflow.apache.org/docs/apache-airflow/"; \
- "stable/configurations-ref.html#secret-key\n***"
+ log += (
+ "*** !!!! Please make sure that all your webservers
and workers have"
Review comment:
Right. The If above is only for K8S executor. I always associated
LocalExecutor with running everything on one machine and reading logs from
filesystem, but indeed webserver might run separately. I am not sure though if
Log server actually runs for Scheduler in this case ?
##########
File path: airflow/utils/log/file_task_handler.py
##########
@@ -187,10 +187,14 @@ def _read(self, ti, try_number, metadata=None):
response.encoding = "utf-8"
if response.status_code == 403:
- log += "*** !!!! Please make sure that all your webservers
and workers have" \
- " the same 'secret_key' configured in 'webserver'
section !!!!!\n***"
- log += "*** See more at
https://airflow.apache.org/docs/apache-airflow/"; \
- "stable/configurations-ref.html#secret-key\n***"
+ log += (
+ "*** !!!! Please make sure that all your webservers
and workers have"
Review comment:
Ah yeah. It runs actually for both Sequential and Local Executor :)
--
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
