josh-fell commented on code in PR #29892: URL: https://github.com/apache/airflow/pull/29892#discussion_r1130018126
########## docs/apache-airflow/howto/connection.rst: ########## @@ -191,19 +191,23 @@ Passwords cannot be manipulated or read without the key. For information on conf Testing Connections ^^^^^^^^^^^^^^^^^^^ -Airflow Web UI & API allows to test connections. The test connection feature can be used from -:ref:`create <creating_connection_ui>` or :ref:`edit <editing_connection_ui>` connection page, or through calling -:doc:`Connections REST API </stable-rest-api-ref/>`. +Airflow Web UI, REST API, and CLI allow you to test connections. The test connection feature can be used from +:ref:`create <creating_connection_ui>` or :ref:`edit <editing_connection_ui>` connection page in the UI, through calling +:doc:`Connections REST API </stable-rest-api-ref/>`, or running the ``airflow connections test`` :ref:`CLI command <cli>`. -To test a connection Airflow calls out the ``test_connection`` method from the associated hook class and reports the -results of it. It may happen that the connection type does not have any associated hook or the hook doesn't have the -``test_connection`` method implementation, in either case the error message will throw the proper error message. +To test a connection, Airflow calls the ``test_connection`` method from the associated hook class and reports the +results. It may happen that the connection type does not have any associated hook or the hook doesn't have the +``test_connection`` method implementation, in either case an error message will be displayed or functionality +will be disabled (if you are testing in the UI). -One important point to note is that the connections will be tested from the webserver only, so this feature is +One important point to note is that when testing in the Airflow UI, the test executes from the webserver only. This feature is Review Comment: @msumit Made some small tweaks and explicitly added warning/note directives so these statement stand out. Let me know what you think when you get a chance. -- 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]
