App Settings Logs formatting fixes (#1387)

* added space and delimiter for parsing

* updated test

* flake issues

Author: hallvictoria

azure_functions_worker/dispatcher.py

@@ -268,9 +268,9 @@ async def _handle__worker_init_request(self, request):
         logger.info('Received WorkerInitRequest, '
                     'python version %s, '
                     'worker version %s, '
-                    'request ID %s.'
-                    'App Settings state: %s.'
-                    ' To enable debug level logging, please refer to '
+                    'request ID %s. '
+                    'App Settings state: %s. '
+                    'To enable debug level logging, please refer to '
                     'https://aka.ms/python-enable-debug-logging',
                     sys.version,
                     VERSION,
@@ -554,9 +554,9 @@ async def _handle__function_environment_reload_request(self, request):
         """
         try:
             logger.info('Received FunctionEnvironmentReloadRequest, '
-                        'request ID: %s,'
-                        'App Settings state: %s.'
-                        ' To enable debug level logging, please refer to '
+                        'request ID: %s, '
+                        'App Settings state: %s. '
+                        'To enable debug level logging, please refer to '
                         'https://aka.ms/python-enable-debug-logging',
                         self.request_id,
                         get_python_appsetting_state())

azure_functions_worker/utils/app_setting_manager.py

@@ -24,7 +24,7 @@ def get_python_appsetting_state():
          FUNCTIONS_WORKER_SHARED_MEMORY_DATA_TRANSFER_ENABLED]
 
     app_setting_states = "".join(
-        f"{app_setting}: {current_vars[app_setting]} "
+        f"{app_setting}: {current_vars[app_setting]} | "
         for app_setting in python_specific_settings
         if app_setting in current_vars
     )
@@ -38,6 +38,6 @@ def get_python_appsetting_state():
         else:
             app_setting_states += \
                 (f"{PYTHON_ENABLE_WORKER_EXTENSIONS}: "
-                 f"{str(PYTHON_ENABLE_WORKER_EXTENSIONS_DEFAULT)} ")
+                 f"{str(PYTHON_ENABLE_WORKER_EXTENSIONS_DEFAULT)}")
 
     return app_setting_states

tests/unittests/test_app_setting_manager.py

@@ -88,6 +88,7 @@ async def test_initialize_worker_logging(self):
 
     def test_get_python_appsetting_state(self):
         app_setting_state = get_python_appsetting_state()
-        self.assertIn("PYTHON_THREADPOOL_THREAD_COUNT: 20 ", app_setting_state)
-        self.assertIn("PYTHON_ENABLE_DEBUG_LOGGING: 1 ", app_setting_state)
+        self.assertIn("PYTHON_THREADPOOL_THREAD_COUNT: 20 | ",
+                      app_setting_state)
+        self.assertIn("PYTHON_ENABLE_DEBUG_LOGGING: 1 | ", app_setting_state)
         self.assertIn("PYTHON_ENABLE_WORKER_EXTENSIONS: ", app_setting_state)