You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/07/08 19:50:29 UTC
[GitHub] [airflow] stephsamson opened a new issue #16895: Chart:
stephsamson opened a new issue #16895:
URL: https://github.com/apache/airflow/issues/16895
<!--
Welcome to Apache Airflow! For a smooth issue process, try to answer the following questions.
Don't worry if they're not all applicable; just try to include what you can :-)
If you need to include code snippets or logs, please put them in fenced code
blocks. If they're super-long, please use the details tag like
<details><summary>super-long log</summary> lots of stuff </details>
Please delete these comment blocks before submitting the issue.
-->
<!--
IMPORTANT!!!
PLEASE CHECK "SIMILAR TO X EXISTING ISSUES" OPTION IF VISIBLE
NEXT TO "SUBMIT NEW ISSUE" BUTTON!!!
PLEASE CHECK IF THIS ISSUE HAS BEEN REPORTED PREVIOUSLY USING SEARCH!!!
Please complete the next sections or the issue will be closed.
These questions are the first thing we need to know to understand the context.
-->
**Apache Airflow version**: 2.0.2
**Kubernetes version (if you are using kubernetes)** (use `kubectl version`): v1.19.10
**Environment**: GKE
- **Cloud provider or hardware configuration**: GCP
- **OS** (e.g. from /etc/os-release): Debian GNU/Linux 10 (buster)
- **Kernel** (e.g. `uname -a`): x86_64 GNU/Linux
- **Install tools**: Helm
- **Others**:
**What happened**: Configured `values.yaml` to enable remote logging for task pods (running KubernetesExecutor)
with Google Stackdriver [using docs here](https://airflow.apache.org/docs/apache-airflow-providers-google/stable/logging/stackdriver.html) but running `airflow show` shows the `task_logging_handler` to still be `FileTaskHandler` instead of the `StackdriverHandler`.
Relevant `values.yaml` section below:
```
...
config:
core:
executor: KubernetesExecutor
logging:
remote_logging: "True"
remote_base_log_folder: stackdriver://airflow
....
```
When I check my configmap, these are the results:
```
Data
====
airflow.cfg:
----
...
[logging]
colored_console_log = False
remote_base_log_folder = stackdriver://airflow
remote_logging = True
...
```
But when I exec into a scheduler or webserver pod and check the values there with `airflow config list`, I get:
```
[logging]
base_log_folder = /opt/airflow/logs
remote_logging = False
remote_log_conn_id =
google_key_path =
remote_base_log_folder = stackdriver://airflow
```
or `airflow config get-value logging remote_logging`, I get `False`.
**What you expected to happen**:
`airflow config get-value logging remote_logging` should return `True`.
**How to reproduce it**:
In your `values.yaml`, add the following section:
```
config:
core:
executor: KubernetesExecutor
logging:
remote_logging: "True"
remote_base_log_folder: stackdriver://airflow
```
Then run ` helm install --generate-name airflow apache-airflow/airflow --namespace airflow --debug -f values.yaml`
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] kaxil closed issue #16895: Chart: remote_logging keeps evaluating to False
Posted by GitBox <gi...@apache.org>.
kaxil closed issue #16895:
URL: https://github.com/apache/airflow/issues/16895
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] jedcunningham commented on issue #16895: Chart: remote_logging keeps evaluating to False
Posted by GitBox <gi...@apache.org>.
jedcunningham commented on issue #16895:
URL: https://github.com/apache/airflow/issues/16895#issuecomment-878670958
Can you look in your `$AIRFLOW_HOME\airflow.cfg` instead? Are you using version 1.0.0 of the chart?
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] stephsamson edited a comment on issue #16895: Chart: remote_logging keeps evaluating to False
Posted by GitBox <gi...@apache.org>.
stephsamson edited a comment on issue #16895:
URL: https://github.com/apache/airflow/issues/16895#issuecomment-879449682
With the enabled `env`.
```
[logging]
colored_console_log = False
remote_base_log_folder = stackdriver://airflow
remote_logging = True
```
yes, I am using version 1.0.0 of the chart. I can try checking the config file later with the environment variables disabled.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] stephsamson commented on issue #16895: Chart: remote_logging keeps evaluating to False
Posted by GitBox <gi...@apache.org>.
stephsamson commented on issue #16895:
URL: https://github.com/apache/airflow/issues/16895#issuecomment-876988231
@jedcunningham I do not but as a workaround for now I am using the following as a workaround:
```
env:
- name: AIRFLOW__LOGGING__REMOTE_LOGGING
value: 'True'
- name: AIRFLOW__LOGGING__REMOTE_BASE_LOG_FOLDER
value: 'stackdriver://airflow'
```
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] jedcunningham commented on issue #16895: Chart: remote_logging keeps evaluating to False
Posted by GitBox <gi...@apache.org>.
jedcunningham commented on issue #16895:
URL: https://github.com/apache/airflow/issues/16895#issuecomment-889524824
With Airflow config, the first one wins out of: env var, config file, commands, secrets, default config
Env var still seems like the most likely cause. Config file seems like it is being found because `remote_base_log_folder` is correct. We see `remote_logging = True` in your config file as well, so env var? Have you had a chance to check?
For me, standing up a new release with these values does attempt to use remote logging (it fails to come up because of missing credentials, but it is trying):
```
$ cat test-remote.yaml
config:
core:
executor: KubernetesExecutor
logging:
remote_logging: "True"
remote_base_log_folder: stackdriver://airflow
```
Anything else in your values file?
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] jedcunningham commented on issue #16895: Chart: remote_logging keeps evaluating to False
Posted by GitBox <gi...@apache.org>.
jedcunningham commented on issue #16895:
URL: https://github.com/apache/airflow/issues/16895#issuecomment-876761407
@stephsamson, can you check if you have an `AIRFLOW__LOGGING__REMOTE_LOGGING` env var set? That would be the easy explanation.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] stephsamson edited a comment on issue #16895: Chart: remote_logging keeps evaluating to False
Posted by GitBox <gi...@apache.org>.
stephsamson edited a comment on issue #16895:
URL: https://github.com/apache/airflow/issues/16895#issuecomment-876988231
@jedcunningham I did not when I was reporting this issue but as a workaround for now I am using the following:
```
env:
- name: AIRFLOW__LOGGING__REMOTE_LOGGING
value: 'True'
- name: AIRFLOW__LOGGING__REMOTE_BASE_LOG_FOLDER
value: 'stackdriver://airflow'
```
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] stephsamson commented on issue #16895: Chart: remote_logging keeps evaluating to False
Posted by GitBox <gi...@apache.org>.
stephsamson commented on issue #16895:
URL: https://github.com/apache/airflow/issues/16895#issuecomment-879449682
It is currently the following because I have enabled the configuration above with `env`:
```
[logging]
colored_console_log = False
remote_base_log_folder = stackdriver://airflow
remote_logging = True
```
and yes, I am using version 1.0.0 of the chart. I can try checking the config file later with the environment variables disabled.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org