diff options
| author | LeaveMyYard <zhukovpavel2001@gmail.com> | 2024-04-30 18:54:43 +0300 |
|---|---|---|
| committer | LeaveMyYard <zhukovpavel2001@gmail.com> | 2024-04-30 18:54:43 +0300 |
| commit | d1ad17d01a818cad989280ab0274c8b230aa3286 (patch) | |
| tree | 4a8dd36e2d54ccc9f70491324f60a78dca5b65c3 /robusta_krr/core | |
| parent | 7124c80b9a61ede3f90d79e12f617f79c3f71457 (diff) | |
Minor additional logging improvements
Diffstat (limited to 'robusta_krr/core')
| -rw-r--r-- | robusta_krr/core/integrations/prometheus/connector.py | 11 | ||||
| -rw-r--r-- | robusta_krr/core/runner.py | 2 |
2 files changed, 5 insertions, 8 deletions
diff --git a/robusta_krr/core/integrations/prometheus/connector.py b/robusta_krr/core/integrations/prometheus/connector.py index d75050f..8020c2e 100644 --- a/robusta_krr/core/integrations/prometheus/connector.py +++ b/robusta_krr/core/integrations/prometheus/connector.py @@ -58,14 +58,9 @@ class PrometheusConnector: def connect(self, url: Optional[str] = None) -> None: """Connect to a Prometheus service using a URL.""" - try: - loader = PrometheusMetricsService(url=url) - self._connect(loader) - except Exception as e: - logger.warning(f"Unable to connect to Prometheus using the provided URL ({e})") - raise e - else: - logger.info(f"{loader.name()} connected successfully") + loader = PrometheusMetricsService(url=url) + self._connect(loader) + logger.info(f"{loader.name()} connected successfully") def _connect(self, loader: PrometheusMetricsService) -> None: service_name = loader.name() diff --git a/robusta_krr/core/runner.py b/robusta_krr/core/runner.py index 75c2191..a52f1be 100644 --- a/robusta_krr/core/runner.py +++ b/robusta_krr/core/runner.py @@ -247,6 +247,8 @@ class Runner: async def _collect_result(self) -> Result: clusters = await self.connector.list_clusters() + logger.info(f"Clusters available: {', '.join(clusters)}") + if clusters and len(clusters) > 1 and settings.prometheus_url: # this can only happen for multi-cluster querying a single centeralized prometheus # In this scenario we dont yet support determining |
