XD-DENG commented on a change in pull request #12689:
URL: https://github.com/apache/airflow/pull/12689#discussion_r532112938
##########
File path: docs/modules_management.rst
##########
@@ -232,29 +232,45 @@ Below is the sample output of the ``airflow info``
command:
.. code-block:: none
- Apache Airflow [1.10.11]
-
- Platform: [Linux, x86_64] uname_result(system='Linux', node='Shekhar',
release='5.4.0-42-generic', version='#46-Ubuntu SMP Fri Jul 10 00:24:02 UTC
2020', machine='x86_64', processor='x86_64')
- Locale: ('en_US', 'UTF-8')
- Python Version: [3.8.2 (default, Jul 16 2020, 14:00:26) [GCC 9.3.0]]
- Python Location: [/home/rootcss/venvs/airflow/bin/python3]
-
- git: [git version 2.25.1]
- ssh: [OpenSSH_8.2p1 Ubuntu-4ubuntu0.1, OpenSSL 1.1.1f 31 Mar 2020]
- kubectl: [NOT AVAILABLE]
- gcloud: [NOT AVAILABLE]
- cloud_sql_proxy: [NOT AVAILABLE]
- mysql: [NOT AVAILABLE]
- sqlite3: [3.31.1 2020-01-27 19:55:54
3bfa9cc97da10598521b342961df8f5f68c7388fa117345eeb516eaa837balt1]
- psql: [psql (PostgreSQL) 12.2 (Ubuntu 12.2-4)]
-
- Airflow Home: [/home/rootcss/airflow]
- System PATH:
[/home/rootcss/venvs/airflow/bin:/home/rootcss/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/home/rootcss/.rvm/bin:/usr/local/go/bin:/home/rootcss/.rvm/bin]
- Python PATH:
[/home/rootcss/venvs/airflow/bin:/usr/lib/python38.zip:/usr/lib/python3.8:/usr/lib/python3.8/lib-dynload:/home/rootcss/venvs/airflow/lib/python3.8/site-packages:/home/rootcss/airflow/dags:/home/rootcss/airflow/config:/home/rootcss/airflow/plugins]
- airflow on PATH: [True]
-
- Executor: [SequentialExecutor]
- SQL Alchemy Conn: [sqlite:////home/rootcss/airflow/airflow.db]
- DAGS Folder: [/home/rootcss/airflow/dags]
- Plugins Folder: [/home/rootcss/airflow/plugins]
- Base Log Folder: [/home/rootcss/airflow/logs]
+ Apache Airflow: 2.0.0b3
+
+ System info
+ OS | Linux
+ architecture | x86_64
+ uname | uname_result(system='Linux', node='85cd7ab7018e',
release='4.19.76-linuxkit', version='#1 SMP Tue May 26 11:42:35 UTC 2020',
machine='x86_64', processor='')
+ locale | ('en_US', 'UTF-8')
+ python_version | 3.8.6 (default, Nov 25 2020, 02:47:44) [GCC 8.3.0]
+ python_location | /usr/local/bin/python
+
+ Tools info
+ git | git version 2.20.1
+ ssh | OpenSSH_7.9p1 Debian-10+deb10u2, OpenSSL 1.1.1d 10 Sep
2019
+ kubectl | NOT AVAILABLE
+ gcloud | NOT AVAILABLE
+ cloud_sql_proxy | NOT AVAILABLE
+ mysql | mysql Ver 8.0.22 for Linux on x86_64 (MySQL Community
Server - GPL)
+ sqlite3 | 3.27.2 2019-02-25 16:06:06
bd49a8271d650fa89e446b42e513b595a717b9212c91dd384aab871fc1d0alt1
+ psql | psql (PostgreSQL) 11.9 (Debian 11.9-0+deb10u1)
+
+ Paths info
+ airflow_home | /root/airflow
+ system_path |
/opt/bats/bin:/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
+ python_path |
/usr/local/bin:/opt/airflow:/files/plugins:/usr/local/lib/python38.zip:/usr/local/lib/python3.8:/usr/
+ |
local/lib/python3.8/lib-dynload:/usr/local/lib/python3.8/site-packages:/files/dags:/root/airflow/conf
+ | ig:/root/airflow/plugins
+ airflow_on_path | True
+
+ Config info
+ executor | LocalExecutor
+ task_logging_handler | airflow.utils.log.file_task_handler.FileTaskHandler
+ sql_alchemy_conn |
postgresql+psycopg2://postgres:airflow@postgres/airflow
Review comment:
My bad that I missed the `--anonymize` flag.
Regarding whether to keep `sql_alchemy_conn` in the output of `airflow
info`, I still keep my view given it's (almost) the only sensitive item in the
output. *But I will not insist in this opinion in this PR*.
OR, at least, let's make anonymisation default. Currently it's `store_true`,
i.e. no anonymisation by default. Let me know your thoughts?
https://github.com/apache/airflow/blob/07a2ed5a2ab6edf3fc38c94186fe2204a47f44b7/airflow/cli/cli_parser.py#L649-L653
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
