Fixing PR review comments

components/seldon-request-logger/app/log_helper.py

@@ -34,14 +34,12 @@ def extract_request_id(headers):
         request_id = headers.get(CLOUD_EVENT_ID)
     return request_id
 
-def build_index_name(headers, prefix = None, suffix = True, name_override = None):
+def build_index_name(headers, prefix = "inference", suffix = True, name_override = None):
     # use a fixed index name if user chooses to do so
     index_name = os.getenv("INDEX_NAME")
     if index_name:
         return index_name
 
-    if prefix is None:
-        prefix = "inference"
     # Adding seldon_environment (dev/test/staging/prod) to index_name if defined as a environment variable
     seldon_environment = os.getenv("SELDON_ENVIRONMENT")
     if seldon_environment:

components/seldon-request-logger/app/log_mapping.py

@@ -190,7 +190,6 @@ def fetch_metadata(namespace, serving_engine, inferenceservice_name, predictor_n
 
     if runtime_metadata is not None and runtime_metadata and \
             runtime_metadata.runtime_metadata is not None and runtime_metadata.runtime_metadata:
-        # print(runtime_metadata.runtime_metadata)
         if len(runtime_metadata.runtime_metadata) == 0:
             print('no runtime metadata for '+namespace+'/'+inferenceservice_name)
             return None
@@ -206,7 +205,6 @@ def fetch_metadata(namespace, serving_engine, inferenceservice_name, predictor_n
             return None
 
         print('prediction schema for '+namespace+'/'+inferenceservice_name)
-        # print(model_metadata.models[0].prediction_schema)
         if model_metadata.models[0].prediction_schema:
             return model_metadata.models[0].prediction_schema.to_dict()
         else: