SNOW-680774 Check CLIENT_LOG_DIR_PATH_DOCKER for ENABLE_TELEMETRY_LOG (#1288)

* SNOW-680774 Check CLIENT_LOG_DIR_PATH_DOCKER for ENABLE_TELEMETRY_LOG

Description

test_util is supposed to only be used by snowflake regression test,
but currently it runs for all Jenkins linux hosts, which is not
desired. A quick fix is to also check CLIENT_LOG_DIR_PATH_DOCKER,
which presumbly is not used by most customers if not all.

Testing

existing tests

* fix fix_lint

* address comment

Co-authored-by: Mark Keller <[email protected]>

Author: sfc-gh-sfan

.pre-commit-config.yaml

@@ -61,12 +61,12 @@ repos:
     hooks:
         - id: pyupgrade
           args: [--py37-plus]
--   repo: https://gitlab.com/pycqa/flake8
-    rev: 3.9.2
+-   repo: https://github.com/PyCQA/flake8
+    rev: 5.0.4
     hooks:
       - id: flake8
         additional_dependencies:
-          - flake8-bugbear == 20.11.1
+          - flake8-bugbear
 -   repo: https://github.com/psf/black
     rev: 22.3.0
     hooks:

src/snowflake/connector/test_util.py

@@ -11,8 +11,8 @@
 from .compat import IS_LINUX
 
 RUNNING_ON_JENKINS = os.getenv("JENKINS_HOME") is not None
-REGRESSION_TEST_LOG_DIR = os.getenv("CLIENT_LOG_DIR_PATH_DOCKER", "/tmp")
-ENABLE_TELEMETRY_LOG = RUNNING_ON_JENKINS and IS_LINUX
+REGRESSION_TEST_LOG_DIR = os.getenv("CLIENT_LOG_DIR_PATH_DOCKER")
+ENABLE_TELEMETRY_LOG = RUNNING_ON_JENKINS and REGRESSION_TEST_LOG_DIR and IS_LINUX
 rt_plain_logger = None