removed unused import, initialized variable, updated log

Author: aish1331

inference_perf/metrics/prometheus_client.py

@@ -13,7 +13,6 @@
 # limitations under the License.
 import time
 from typing import cast
-from urllib.error import HTTPError
 import requests
 from inference_perf.client.base import ModelServerMetrics, ModelServerPrometheusMetric
 from inference_perf.config import PrometheusClientConfig
@@ -142,11 +141,14 @@ def collect_model_server_metrics(self, runtime_parameters: PerfRuntimeParameters
         for summary_metric_name in metrics_metadata:
             summary_metric_metadata = metrics_metadata.get(summary_metric_name)
             if summary_metric_metadata is None:
-                print("Metric metadata is not present for metric: %s" % (summary_metric_name))
+                print("Metric metadata is not present for metric: %s. Skipping this metric." % (summary_metric_name))
                 continue
             summary_metric_metadata = cast(ModelServerPrometheusMetric, summary_metric_metadata)
             if summary_metric_metadata is None:
-                print("Metric metadata is not present for metric: %s" % (summary_metric_name))
+                print(
+                    "Metric metadata for %s is missing or has an incorrect format. Skipping this metric."
+                    % (summary_metric_name)
+                )
                 continue
 
             query_builder = PrometheusQueryBuilder(summary_metric_metadata, duration)

inference_perf/reportgen/mock_reportgen.py

@@ -19,7 +19,6 @@
 class MockReportGenerator(ReportGenerator):
     def __init__(self, metrics_client: MetricsClient | None) -> None:
         super().__init__(metrics_client=metrics_client)
-        self.metrics_client = metrics_client
 
     async def generate_report(self, runtime_parameters: PerfRuntimeParameters) -> None:
         print("\n\nGenerating Report ..")