holmesgpt 0.13.3a0__py3-none-any.whl → 0.14.1__py3-none-any.whl

holmes/plugins/toolsets/prometheus/prometheus.py

@@ -1,10 +1,9 @@
 import json
 import logging
 import os
-import re
 import time
 import dateutil.parser
-from typing import Any, Dict, List, Optional, Tuple, Type, Union
+from typing import Any, Dict, Optional, Tuple, Type, Union
 from urllib.parse import urljoin
 
 import requests  # type: ignore
@@ -17,11 +16,12 @@ from holmes.core.tools import (
     StructuredToolResult,
     Tool,
     ToolParameter,
-    ToolResultStatus,
+    StructuredToolResultStatus,
     Toolset,
     ToolsetTag,
 )
 from holmes.plugins.toolsets.consts import STANDARD_END_DATETIME_TOOL_PARAM_DESCRIPTION
+from holmes.plugins.toolsets.prometheus.utils import parse_duration_to_seconds
 from holmes.plugins.toolsets.service_discovery import PrometheusDiscovery
 from holmes.plugins.toolsets.utils import (
     get_param_or_raise,
@@ -38,23 +38,64 @@ from holmes.plugins.toolsets.logging_utils.logging_api import (
 from holmes.utils.keygen_utils import generate_random_key
 
 PROMETHEUS_RULES_CACHE_KEY = "cached_prometheus_rules"
+PROMETHEUS_METADATA_API_LIMIT = 100  # Default limit for Prometheus metadata APIs (series, labels, metadata) to prevent overwhelming responses
+# Default timeout values for PromQL queries
+DEFAULT_QUERY_TIMEOUT_SECONDS = 20
+MAX_QUERY_TIMEOUT_SECONDS = 180
+# Default character limit for query responses to prevent token limit issues
+DEFAULT_QUERY_RESPONSE_SIZE_LIMIT = 20000
+# Default timeout for metadata API calls (discovery endpoints)
+DEFAULT_METADATA_TIMEOUT_SECONDS = 20
+MAX_METADATA_TIMEOUT_SECONDS = 60
+# Default time window for metadata APIs (in hours)
+DEFAULT_METADATA_TIME_WINDOW_HRS = 1
+# Sample size for data summaries when results are too large
+DATA_SUMMARY_SAMPLE_SIZE = 10
 
 
 class PrometheusConfig(BaseModel):
     # URL is optional because it can be set with an env var
     prometheus_url: Optional[str]
     healthcheck: str = "-/healthy"
-    # Setting to None will remove the time window from the request for labels
-    metrics_labels_time_window_hrs: Union[int, None] = 48
-    # Setting to None will disable the cache
-    metrics_labels_cache_duration_hrs: Union[int, None] = 12
-    fetch_labels_with_labels_api: bool = False
-    fetch_metadata_with_series_api: bool = False
+
+    # New config for default time window for metadata APIs
+    default_metadata_time_window_hrs: int = DEFAULT_METADATA_TIME_WINDOW_HRS  # Default: only show metrics active in the last hour
+
+    # Query timeout configuration
+    default_query_timeout_seconds: int = (
+        DEFAULT_QUERY_TIMEOUT_SECONDS  # Default timeout for PromQL queries
+    )
+    max_query_timeout_seconds: int = (
+        MAX_QUERY_TIMEOUT_SECONDS  # Maximum allowed timeout for PromQL queries
+    )
+
+    # Metadata API timeout configuration
+    default_metadata_timeout_seconds: int = (
+        DEFAULT_METADATA_TIMEOUT_SECONDS  # Default timeout for metadata/discovery APIs
+    )
+    max_metadata_timeout_seconds: int = (
+        MAX_METADATA_TIMEOUT_SECONDS  # Maximum allowed timeout for metadata APIs
+    )
+
+    # DEPRECATED: These config values are deprecated and will be removed in a future version
+    # Using None as default so we can detect if user explicitly set them
+    metrics_labels_time_window_hrs: Optional[int] = (
+        None  # DEPRECATED - use default_metadata_time_window_hrs instead
+    )
+    metrics_labels_cache_duration_hrs: Optional[int] = (
+        None  # DEPRECATED - no longer used
+    )
+    fetch_labels_with_labels_api: Optional[bool] = None  # DEPRECATED - no longer used
+    fetch_metadata_with_series_api: Optional[bool] = None  # DEPRECATED - no longer used
+
     tool_calls_return_data: bool = True
     headers: Dict = Field(default_factory=dict)
-    rules_cache_duration_seconds: Union[int, None] = 1800  # 30 minutes
+    rules_cache_duration_seconds: Optional[int] = 1800  # 30 minutes
     additional_labels: Optional[Dict[str, str]] = None
     prometheus_ssl_enabled: bool = True
+    query_response_size_limit: Optional[int] = (
+        DEFAULT_QUERY_RESPONSE_SIZE_LIMIT  # Limit the max number of characters in a query result to proactively prevent token limit issues (roughly 5-6k tokens)
+    )
 
     @field_validator("prometheus_url")
     def ensure_trailing_slash(cls, v: Optional[str]) -> Optional[str]:
@@ -64,6 +105,26 @@ class PrometheusConfig(BaseModel):
 
     @model_validator(mode="after")
     def validate_prom_config(self):
+        # Check for deprecated config values and print warnings
+        deprecated_configs = []
+        if self.metrics_labels_time_window_hrs is not None:  # Check if explicitly set
+            deprecated_configs.append(
+                "metrics_labels_time_window_hrs (use default_metadata_time_window_hrs instead)"
+            )
+        if (
+            self.metrics_labels_cache_duration_hrs is not None
+        ):  # Check if explicitly set
+            deprecated_configs.append("metrics_labels_cache_duration_hrs")
+        if self.fetch_labels_with_labels_api is not None:  # Check if explicitly set
+            deprecated_configs.append("fetch_labels_with_labels_api")
+        if self.fetch_metadata_with_series_api is not None:  # Check if explicitly set
+            deprecated_configs.append("fetch_metadata_with_series_api")
+
+        if deprecated_configs:
+            logging.warning(
+                f"WARNING: The following Prometheus config values are deprecated and will be removed in a future version: "
+                f"{', '.join(deprecated_configs)}. These configs no longer affect behavior."
+            )
         # If openshift is enabled, and the user didn't configure auth headers, we will try to load the token from the service account.
         if IS_OPENSHIFT:
             if self.healthcheck == "-/healthy":
@@ -160,6 +221,8 @@ def do_request(
 
     if isinstance(config, AMPConfig):
         client = config.get_aws_client()  # cached AWSPrometheusConnect
+        # Note: timeout parameter is not supported by prometrix's signed_request
+        # AWS/AMP requests will not respect the timeout setting
         return client.signed_request(  # type: ignore
             method=method,
             url=url,
@@ -181,99 +244,6 @@ def do_request(
     )
 
 
-def filter_metrics_by_type(metrics: Dict, expected_type: str):
-    return {
-        metric_name: metric_data
-        for metric_name, metric_data in metrics.items()
-        if expected_type in metric_data.get("type", "")
-        or metric_data.get("type", "") == "?"
-    }
-
-
-def filter_metrics_by_name(metrics: Dict, pattern: str) -> Dict:
-    regex = re.compile(pattern)
-    return {
-        metric_name: metric_data
-        for metric_name, metric_data in metrics.items()
-        if regex.search(metric_name)
-    }
-
-
-METRICS_SUFFIXES_TO_STRIP = ["_bucket", "_count", "_sum"]
-
-
-def fetch_metadata(
-    prometheus_url: str,
-    headers: Optional[Dict],
-    config,
-    verify_ssl: bool = True,
-) -> Dict:
-    metadata_url = urljoin(prometheus_url, "api/v1/metadata")
-    metadata_response = do_request(
-        config=config,
-        url=metadata_url,
-        headers=headers,
-        timeout=60,
-        verify=verify_ssl,
-        method="GET",
-    )
-    metadata_response.raise_for_status()
-
-    metadata = metadata_response.json()["data"]
-
-    metrics = {}
-    for metric_name, meta_list in metadata.items():
-        if meta_list:
-            metric_type = meta_list[0].get("type", "unknown")
-            metric_description = meta_list[0].get("help", "unknown")
-            metrics[metric_name] = {
-                "type": metric_type,
-                "description": metric_description,
-                "labels": set(),
-            }
-
-    return metrics
-
-
-def fetch_metadata_with_series_api(
-    prometheus_url: str,
-    metric_name: str,
-    headers: Dict,
-    config,
-    verify_ssl: bool = True,
-) -> Dict:
-    url = urljoin(prometheus_url, "api/v1/series")
-    params: Dict = {"match[]": f'{{__name__=~".*{metric_name}.*"}}', "limit": "10000"}
-
-    response = do_request(
-        config=config,
-        url=url,
-        headers=headers,
-        params=params,
-        timeout=60,
-        verify=verify_ssl,
-        method="GET",
-    )
-    response.raise_for_status()
-    metrics = response.json()["data"]
-
-    metadata: Dict = {}
-    for metric_data in metrics:
-        metric_name = metric_data.get("__name__")
-        if not metric_name:
-            continue
-
-        metric = metadata.get(metric_name)
-        if not metric:
-            metric = {"description": "?", "type": "?", "labels": set()}
-            metadata[metric_name] = metric
-
-        labels = {k for k in metric_data.keys() if k != "__name__"}
-        metric["labels"].update(labels)
-
-    return metadata
-
-
 def result_has_data(result: Dict) -> bool:
     data = result.get("data", {})
     if len(data.get("result", [])) > 0:
@@ -284,33 +254,58 @@ def result_has_data(result: Dict) -> bool:
 def adjust_step_for_max_points(
     start_timestamp: str,
     end_timestamp: str,
-    step: float,
+    step: Optional[float] = None,
+    max_points_override: Optional[float] = None,
 ) -> float:
     """
     Adjusts the step parameter to ensure the number of data points doesn't exceed max_points.
-    Max points is controlled by the PROMETHEUS_MAX_GRAPH_POINTS environment variable (default: 300).
 
     Args:
         start_timestamp: RFC3339 formatted start time
         end_timestamp: RFC3339 formatted end time
-        step: The requested step duration in seconds
+        step: The requested step duration in seconds (None for auto-calculation)
+        max_points_override: Optional override for max points (must be <= MAX_GRAPH_POINTS)
 
     Returns:
         Adjusted step value in seconds that ensures points <= max_points
     """
+    # Use override if provided and valid, otherwise use default
+    max_points = MAX_GRAPH_POINTS
+    if max_points_override is not None:
+        if max_points_override > MAX_GRAPH_POINTS:
+            logging.warning(
+                f"max_points override ({max_points_override}) exceeds system limit ({MAX_GRAPH_POINTS}), using {MAX_GRAPH_POINTS}"
+            )
+            max_points = MAX_GRAPH_POINTS
+        elif max_points_override < 1:
+            logging.warning(
+                f"max_points override ({max_points_override}) is invalid, using default {MAX_GRAPH_POINTS}"
+            )
+            max_points = MAX_GRAPH_POINTS
+        else:
+            max_points = max_points_override
+            logging.debug(f"Using max_points override: {max_points}")
 
     start_dt = dateutil.parser.parse(start_timestamp)
     end_dt = dateutil.parser.parse(end_timestamp)
 
     time_range_seconds = (end_dt - start_dt).total_seconds()
 
+    # If no step provided, calculate a reasonable default
+    # Aim for ~60 data points across the time range (1 per minute for hourly, etc)
+    if step is None:
+        step = max(1, time_range_seconds / 60)
+        logging.debug(
+            f"No step provided, defaulting to {step}s for {time_range_seconds}s range"
+        )
+
     current_points = time_range_seconds / step
 
     # If current points exceed max, adjust the step
-    if current_points > MAX_GRAPH_POINTS:
-        adjusted_step = time_range_seconds / MAX_GRAPH_POINTS
+    if current_points > max_points:
+        adjusted_step = time_range_seconds / max_points
         logging.info(
-            f"Adjusting step from {step}s to {adjusted_step}s to limit points from {current_points:.0f} to {MAX_GRAPH_POINTS}"
+            f"Adjusting step from {step}s to {adjusted_step}s to limit points from {current_points:.0f} to {max_points}"
         )
         return adjusted_step
 
@@ -324,168 +319,97 @@ def add_prometheus_auth(prometheus_auth_header: Optional[str]) -> Dict[str, Any]
     return results
 
 
-def fetch_metrics_labels_with_series_api(
-    prometheus_url: str,
-    headers: Dict[str, str],
-    cache: Optional[TTLCache],
-    metrics_labels_time_window_hrs: Union[int, None],
-    metric_name: str,
-    config=None,
-    verify_ssl: bool = True,
-) -> dict:
-    """This is a slow query. Takes 5+ seconds to run"""
-    cache_key = f"metrics_labels_series_api:{metric_name}"
-    if cache:
-        cached_result = cache.get(cache_key)
-        if cached_result:
-            return cached_result
-
-    series_url = urljoin(prometheus_url, "api/v1/series")
-    params: dict = {"match[]": f'{{__name__=~".*{metric_name}.*"}}', "limit": "10000"}
-
-    if metrics_labels_time_window_hrs is not None:
-        params["end"] = int(time.time())
-        params["start"] = params["end"] - (metrics_labels_time_window_hrs * 60 * 60)
-
-    series_response = do_request(
-        config=config,
-        url=series_url,
-        headers=headers,
-        params=params,
-        timeout=60,
-        verify=verify_ssl,
-        method="GET",
-    )
-    series_response.raise_for_status()
-    series = series_response.json()["data"]
-
-    metrics_labels: dict = {}
-    for serie in series:
-        metric_name = serie["__name__"]
-        # Add all labels except __name__
-        labels = {k for k in serie.keys() if k != "__name__"}
-        if metric_name in metrics_labels:
-            metrics_labels[metric_name].update(labels)
-        else:
-            metrics_labels[metric_name] = labels
-    if cache:
-        cache.set(cache_key, metrics_labels)
-
-    return metrics_labels
-
-
-def fetch_metrics_labels_with_labels_api(
-    prometheus_url: str,
-    cache: Optional[TTLCache],
-    metrics_labels_time_window_hrs: Union[int, None],
-    metric_names: List[str],
-    headers: Dict,
-    config=None,
-    verify_ssl: bool = True,
-) -> dict:
-    metrics_labels = {}
-
-    for metric_name in metric_names:
-        cache_key = f"metrics_labels_labels_api:{metric_name}"
-        if cache:
-            cached_result = cache.get(cache_key)
-            if cached_result:
-                metrics_labels[metric_name] = cached_result
-
-        url = urljoin(prometheus_url, "api/v1/labels")
-        params: dict = {
-            "match[]": f'{{__name__="{metric_name}"}}',
-        }
-        if metrics_labels_time_window_hrs is not None:
-            params["end"] = int(time.time())
-            params["start"] = params["end"] - (metrics_labels_time_window_hrs * 60 * 60)
+def create_data_summary_for_large_result(
+    result_data: Dict, query: str, data_size_chars: int, is_range_query: bool = False
+) -> Dict[str, Any]:
+    """
+    Create a summary for large Prometheus results instead of returning full data.
 
-        response = do_request(
-            config=config,
-            url=url,
-            headers=headers,
-            params=params,
-            timeout=60,
-            verify=verify_ssl,
-            method="GET",
-        )
-        response.raise_for_status()
-        labels = response.json()["data"]
-        filtered_labels = {label for label in labels if label != "__name__"}
-        metrics_labels[metric_name] = filtered_labels
-
-        if cache:
-            cache.set(cache_key, filtered_labels)
-
-    return metrics_labels
-
-
-def fetch_metrics(
-    prometheus_url: str,
-    cache: Optional[TTLCache],
-    metrics_labels_time_window_hrs: Union[int, None],
-    metric_name: str,
-    should_fetch_labels_with_labels_api: bool,
-    should_fetch_metadata_with_series_api: bool,
-    headers: Dict,
-    config=None,
-    verify_ssl: bool = True,
-) -> dict:
-    metrics = None
-    should_fetch_labels = True
-    if should_fetch_metadata_with_series_api:
-        metrics = fetch_metadata_with_series_api(
-            prometheus_url=prometheus_url,
-            metric_name=metric_name,
-            headers=headers,
-            config=config,
-            verify_ssl=verify_ssl,
+    Args:
+        result_data: The Prometheus data result
+        query: The original PromQL query
+        data_size_chars: Size of the data in characters
+        is_range_query: Whether this is a range query (vs instant query)
+
+    Returns:
+        Dictionary with summary information and suggestions
+    """
+    if is_range_query:
+        series_list = result_data.get("result", [])
+        num_items = len(series_list)
+
+        # Calculate exact total data points across all series
+        total_points = 0
+        for series in series_list:  # Iterate through ALL series for exact count
+            points = len(series.get("values", []))
+            total_points += points
+
+        # Analyze label keys and their cardinality
+        label_cardinality: Dict[str, set] = {}
+        for series in series_list:
+            metric = series.get("metric", {})
+            for label_key, label_value in metric.items():
+                if label_key not in label_cardinality:
+                    label_cardinality[label_key] = set()
+                label_cardinality[label_key].add(label_value)
+
+        # Convert sets to counts for the summary
+        label_summary = {
+            label: len(values) for label, values in label_cardinality.items()
+        }
+        # Sort by cardinality (highest first) for better insights
+        label_summary = dict(
+            sorted(label_summary.items(), key=lambda x: x[1], reverse=True)
         )
-        should_fetch_labels = False  # series API returns the labels
+
+        return {
+            "message": f"Data too large to return ({data_size_chars:,} characters). Query returned {num_items} time series with {total_points:,} total data points.",
+            "series_count": num_items,
+            "total_data_points": total_points,
+            "data_size_characters": data_size_chars,
+            "label_cardinality": label_summary,
+            "suggestion": f'Consider using topk({min(5, num_items)}, {query}) to limit results to the top {min(5, num_items)} series. To also capture remaining data as \'other\': topk({min(5, num_items)}, {query}) or label_replace((sum({query}) - sum(topk({min(5, num_items)}, {query}))), "pod", "other", "", "")',
+        }
     else:
-        metrics = fetch_metadata(
-            prometheus_url=prometheus_url,
-            headers=headers,
-            config=config,
-            verify_ssl=verify_ssl,
+        # Instant query
+        result_type = result_data.get("resultType", "")
+        result_list = result_data.get("result", [])
+        num_items = len(result_list)
+
+        # Analyze label keys and their cardinality
+        instant_label_cardinality: Dict[str, set] = {}
+        for item in result_list:
+            if isinstance(item, dict):
+                metric = item.get("metric", {})
+                for label_key, label_value in metric.items():
+                    if label_key not in instant_label_cardinality:
+                        instant_label_cardinality[label_key] = set()
+                    instant_label_cardinality[label_key].add(label_value)
+
+        # Convert sets to counts for the summary
+        label_summary = {
+            label: len(values) for label, values in instant_label_cardinality.items()
+        }
+        # Sort by cardinality (highest first) for better insights
+        label_summary = dict(
+            sorted(label_summary.items(), key=lambda x: x[1], reverse=True)
         )
-        metrics = filter_metrics_by_name(metrics, metric_name)
-
-    if should_fetch_labels:
-        metrics_labels = {}
-        if should_fetch_labels_with_labels_api:
-            metrics_labels = fetch_metrics_labels_with_labels_api(
-                prometheus_url=prometheus_url,
-                cache=cache,
-                metrics_labels_time_window_hrs=metrics_labels_time_window_hrs,
-                metric_names=list(metrics.keys()),
-                headers=headers,
-                config=config,
-                verify_ssl=verify_ssl,
-            )
-        else:
-            metrics_labels = fetch_metrics_labels_with_series_api(
-                prometheus_url=prometheus_url,
-                cache=cache,
-                metrics_labels_time_window_hrs=metrics_labels_time_window_hrs,
-                metric_name=metric_name,
-                headers=headers,
-                config=config,
-                verify_ssl=verify_ssl,
-            )
-
-        for metric_name in metrics:
-            if metric_name in metrics_labels:
-                metrics[metric_name]["labels"] = metrics_labels[metric_name]
 
-    return metrics
+        return {
+            "message": f"Data too large to return ({data_size_chars:,} characters). Query returned {num_items} results.",
+            "result_count": num_items,
+            "result_type": result_type,
+            "data_size_characters": data_size_chars,
+            "label_cardinality": label_summary,
+            "suggestion": f'Consider using topk({min(5, num_items)}, {query}) to limit results. To also capture remaining data as \'other\': topk({min(5, num_items)}, {query}) or label_replace((sum({query}) - sum(topk({min(5, num_items)}, {query}))), "instance", "other", "", "")',
+        }
 
 
 class ListPrometheusRules(BasePrometheusTool):
     def __init__(self, toolset: "PrometheusToolset"):
         super().__init__(
             name="list_prometheus_rules",
-            description="List all defined prometheus rules. Will show the prometheus rules description, expression and annotations",
+            description="List all defined Prometheus rules (api/v1/rules). Will show the Prometheus rules description, expression and annotations",
             parameters={},
             toolset=toolset,
         )
@@ -496,13 +420,13 @@ class ListPrometheusRules(BasePrometheusTool):
     ) -> StructuredToolResult:
         if not self.toolset.config or not self.toolset.config.prometheus_url:
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error="Prometheus is not configured. Prometheus URL is missing",
                 params=params,
             )
         if self.toolset.config.is_amp():
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error="Tool not supported in AMP",
                 params=params,
             )
@@ -515,7 +439,7 @@ class ListPrometheusRules(BasePrometheusTool):
                     logging.debug("rules returned from cache")
 
                     return StructuredToolResult(
-                        status=ToolResultStatus.SUCCESS,
+                        status=StructuredToolResultStatus.SUCCESS,
                         data=cached_rules,
                         params=params,
                     )
@@ -528,7 +452,7 @@ class ListPrometheusRules(BasePrometheusTool):
                 config=self.toolset.config,
                 url=rules_url,
                 params=params,
-                timeout=180,
+                timeout=40,
                 verify=self.toolset.config.prometheus_ssl_enabled,
                 headers=self.toolset.config.headers,
                 method="GET",
@@ -539,28 +463,28 @@ class ListPrometheusRules(BasePrometheusTool):
             if self._cache:
                 self._cache.set(PROMETHEUS_RULES_CACHE_KEY, data)
             return StructuredToolResult(
-                status=ToolResultStatus.SUCCESS,
+                status=StructuredToolResultStatus.SUCCESS,
                 data=data,
                 params=params,
             )
         except requests.Timeout:
             logging.warning("Timeout while fetching prometheus rules", exc_info=True)
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error="Request timed out while fetching rules",
                 params=params,
             )
         except RequestException as e:
             logging.warning("Failed to fetch prometheus rules", exc_info=True)
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error=f"Network error while fetching rules: {str(e)}",
                 params=params,
             )
         except Exception as e:
             logging.warning("Failed to process prometheus rules", exc_info=True)
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error=f"Unexpected error: {str(e)}",
                 params=params,
             )
@@ -569,120 +493,563 @@ class ListPrometheusRules(BasePrometheusTool):
         return f"{toolset_name_for_one_liner(self.toolset.name)}: Fetch Rules"
 
 
-class ListAvailableMetrics(BasePrometheusTool):
+class GetMetricNames(BasePrometheusTool):
+    """Thin wrapper around /api/v1/label/__name__/values - the fastest way to discover metric names"""
+
     def __init__(self, toolset: "PrometheusToolset"):
         super().__init__(
-            name="list_available_metrics",
-            description="List all the available metrics to query from prometheus, including their types (counter, gauge, histogram, summary) and available labels.",
+            name="get_metric_names",
+            description=(
+                "Get list of metric names using /api/v1/label/__name__/values. "
+                "FASTEST method for metric discovery when you need to explore available metrics. "
+                f"Returns up to {PROMETHEUS_METADATA_API_LIMIT} unique metric names (limit={PROMETHEUS_METADATA_API_LIMIT}). If {PROMETHEUS_METADATA_API_LIMIT} results returned, more may exist - use a more specific filter. "
+                f"ALWAYS use match[] parameter to filter metrics - without it you'll get random {PROMETHEUS_METADATA_API_LIMIT} metrics which is rarely useful. "
+                "Note: Does not return metric metadata (type, description, labels). "
+                "By default returns metrics active in the last 1 hour (configurable via default_metadata_time_window_hrs)."
+            ),
             parameters={
-                "type_filter": ToolParameter(
-                    description="Optional filter to only return a specific metric type. Can be one of counter, gauge, histogram, summary",
+                "match": ToolParameter(
+                    description=(
+                        "REQUIRED: PromQL selector to filter metrics. Use regex OR (|) to check multiple patterns in one call - much faster than multiple calls! Examples: "
+                        "'{__name__=~\"node_cpu.*|node_memory.*|node_disk.*\"}' for all node resource metrics, "
+                        "'{__name__=~\"container_cpu.*|container_memory.*|container_network.*\"}' for all container metrics, "
+                        "'{__name__=~\"kube_pod.*|kube_deployment.*|kube_service.*\"}' for multiple Kubernetes object metrics, "
+                        "'{__name__=~\".*cpu.*|.*memory.*|.*disk.*\"}' for all resource metrics, "
+                        "'{namespace=~\"kube-system|default|monitoring\"}' for metrics from multiple namespaces, "
+                        "'{job=~\"prometheus|node-exporter|kube-state-metrics\"}' for metrics from multiple jobs."
+                    ),
+                    type="string",
+                    required=True,
+                ),
+                "start": ToolParameter(
+                    description="Start timestamp (RFC3339 or Unix). Default: 1 hour ago",
                     type="string",
                     required=False,
                 ),
-                "name_filter": ToolParameter(
-                    description="Only the metrics partially or fully matching this name will be returned",
+                "end": ToolParameter(
+                    description="End timestamp (RFC3339 or Unix). Default: now",
                     type="string",
-                    required=True,
+                    required=False,
                 ),
             },
             toolset=toolset,
         )
-        self._cache = None
 
     def _invoke(
         self, params: dict, user_approved: bool = False
     ) -> StructuredToolResult:
         if not self.toolset.config or not self.toolset.config.prometheus_url:
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error="Prometheus is not configured. Prometheus URL is missing",
                 params=params,
             )
-        if not self._cache and self.toolset.config.metrics_labels_cache_duration_hrs:
-            self._cache = TTLCache(
-                self.toolset.config.metrics_labels_cache_duration_hrs * 3600  # type: ignore
-            )
         try:
-            prometheus_url = self.toolset.config.prometheus_url
-            metrics_labels_time_window_hrs = (
-                self.toolset.config.metrics_labels_time_window_hrs
+            match_param = params.get("match")
+            if not match_param:
+                return StructuredToolResult(
+                    status=StructuredToolResultStatus.ERROR,
+                    error="Match parameter is required to filter metrics",
+                    params=params,
+                )
+
+            url = urljoin(
+                self.toolset.config.prometheus_url, "api/v1/label/__name__/values"
             )
+            query_params = {
+                "limit": str(PROMETHEUS_METADATA_API_LIMIT),
+                "match[]": match_param,
+            }
+
+            # Add time parameters - use provided values or defaults
+            if params.get("end"):
+                query_params["end"] = params["end"]
+            else:
+                query_params["end"] = str(int(time.time()))
+
+            if params.get("start"):
+                query_params["start"] = params["start"]
+            elif self.toolset.config.default_metadata_time_window_hrs:
+                # Use default time window
+                query_params["start"] = str(
+                    int(time.time())
+                    - (self.toolset.config.default_metadata_time_window_hrs * 3600)
+                )
+
+            response = do_request(
+                config=self.toolset.config,
+                url=url,
+                params=query_params,
+                timeout=self.toolset.config.default_metadata_timeout_seconds,
+                verify=self.toolset.config.prometheus_ssl_enabled,
+                headers=self.toolset.config.headers,
+                method="GET",
+            )
+            response.raise_for_status()
+            data = response.json()
+
+            # Check if results were truncated
+            if (
+                "data" in data
+                and isinstance(data["data"], list)
+                and len(data["data"]) == PROMETHEUS_METADATA_API_LIMIT
+            ):
+                data["_truncated"] = True
+                data["_message"] = (
+                    f"Results truncated at limit={PROMETHEUS_METADATA_API_LIMIT}. Use a more specific match filter to see additional metrics."
+                )
+
+            return StructuredToolResult(
+                status=StructuredToolResultStatus.SUCCESS,
+                data=data,
+                params=params,
+            )
+        except Exception as e:
+            return StructuredToolResult(
+                status=StructuredToolResultStatus.ERROR,
+                error=str(e),
+                params=params,
+            )
+
+    def get_parameterized_one_liner(self, params) -> str:
+        return f"{toolset_name_for_one_liner(self.toolset.name)}: Get Metric Names"
+
+
+class GetLabelValues(BasePrometheusTool):
+    """Get values for a specific label across all metrics"""
+
+    def __init__(self, toolset: "PrometheusToolset"):
+        super().__init__(
+            name="get_label_values",
+            description=(
+                "Get all values for a specific label using /api/v1/label/{label}/values. "
+                "Use this to discover pods, namespaces, jobs, instances, etc. "
+                f"Returns up to {PROMETHEUS_METADATA_API_LIMIT} unique values (limit={PROMETHEUS_METADATA_API_LIMIT}). If {PROMETHEUS_METADATA_API_LIMIT} results returned, more may exist - use match[] to filter. "
+                "Supports optional match[] parameter to filter. "
+                "By default returns values from metrics active in the last 1 hour (configurable via default_metadata_time_window_hrs)."
+            ),
+            parameters={
+                "label": ToolParameter(
+                    description="Label name to get values for (e.g., 'pod', 'namespace', 'job', 'instance')",
+                    type="string",
+                    required=True,
+                ),
+                "match": ToolParameter(
+                    description=(
+                        "Optional PromQL selector to filter (e.g., '{__name__=~\"kube.*\"}', "
+                        "'{namespace=\"default\"}')."
+                    ),
+                    type="string",
+                    required=False,
+                ),
+                "start": ToolParameter(
+                    description="Start timestamp (RFC3339 or Unix). Default: 1 hour ago",
+                    type="string",
+                    required=False,
+                ),
+                "end": ToolParameter(
+                    description="End timestamp (RFC3339 or Unix). Default: now",
+                    type="string",
+                    required=False,
+                ),
+            },
+            toolset=toolset,
+        )
 
-            name_filter = params.get("name_filter")
-            if not name_filter:
+    def _invoke(
+        self, params: dict, user_approved: bool = False
+    ) -> StructuredToolResult:
+        if not self.toolset.config or not self.toolset.config.prometheus_url:
+            return StructuredToolResult(
+                status=StructuredToolResultStatus.ERROR,
+                error="Prometheus is not configured. Prometheus URL is missing",
+                params=params,
+            )
+        try:
+            label = params.get("label")
+            if not label:
                 return StructuredToolResult(
-                    status=ToolResultStatus.ERROR,
-                    error="Error: cannot run tool 'list_available_metrics'. The param 'name_filter' is required but is missing.",
+                    status=StructuredToolResultStatus.ERROR,
+                    error="Label parameter is required",
                     params=params,
                 )
 
-            metrics = fetch_metrics(
-                prometheus_url=prometheus_url,
-                cache=self._cache,
-                metrics_labels_time_window_hrs=metrics_labels_time_window_hrs,
-                metric_name=name_filter,
-                should_fetch_labels_with_labels_api=self.toolset.config.fetch_labels_with_labels_api,
-                should_fetch_metadata_with_series_api=self.toolset.config.fetch_metadata_with_series_api,
+            url = urljoin(
+                self.toolset.config.prometheus_url, f"api/v1/label/{label}/values"
+            )
+            query_params = {"limit": str(PROMETHEUS_METADATA_API_LIMIT)}
+            if params.get("match"):
+                query_params["match[]"] = params["match"]
+
+            # Add time parameters - use provided values or defaults
+            if params.get("end"):
+                query_params["end"] = params["end"]
+            else:
+                query_params["end"] = str(int(time.time()))
+
+            if params.get("start"):
+                query_params["start"] = params["start"]
+            elif self.toolset.config.default_metadata_time_window_hrs:
+                # Use default time window
+                query_params["start"] = str(
+                    int(time.time())
+                    - (self.toolset.config.default_metadata_time_window_hrs * 3600)
+                )
+
+            response = do_request(
+                config=self.toolset.config,
+                url=url,
+                params=query_params,
+                timeout=self.toolset.config.default_metadata_timeout_seconds,
+                verify=self.toolset.config.prometheus_ssl_enabled,
                 headers=self.toolset.config.headers,
+                method="GET",
+            )
+            response.raise_for_status()
+            data = response.json()
+
+            # Check if results were truncated
+            if (
+                "data" in data
+                and isinstance(data["data"], list)
+                and len(data["data"]) == PROMETHEUS_METADATA_API_LIMIT
+            ):
+                data["_truncated"] = True
+                data["_message"] = (
+                    f"Results truncated at limit={PROMETHEUS_METADATA_API_LIMIT}. Use match[] parameter to filter label '{label}' values."
+                )
+
+            return StructuredToolResult(
+                status=StructuredToolResultStatus.SUCCESS,
+                data=data,
+                params=params,
+            )
+        except Exception as e:
+            return StructuredToolResult(
+                status=StructuredToolResultStatus.ERROR,
+                error=str(e),
+                params=params,
+            )
+
+    def get_parameterized_one_liner(self, params) -> str:
+        label = params.get("label", "")
+        return f"{toolset_name_for_one_liner(self.toolset.name)}: Get {label} Values"
+
+
+class GetAllLabels(BasePrometheusTool):
+    """Get all label names that exist in Prometheus"""
+
+    def __init__(self, toolset: "PrometheusToolset"):
+        super().__init__(
+            name="get_all_labels",
+            description=(
+                "Get list of all label names using /api/v1/labels. "
+                "Use this to discover what labels are available across all metrics. "
+                f"Returns up to {PROMETHEUS_METADATA_API_LIMIT} label names (limit={PROMETHEUS_METADATA_API_LIMIT}). If {PROMETHEUS_METADATA_API_LIMIT} results returned, more may exist - use match[] to filter. "
+                "Supports optional match[] parameter to filter. "
+                "By default returns labels from metrics active in the last 1 hour (configurable via default_metadata_time_window_hrs)."
+            ),
+            parameters={
+                "match": ToolParameter(
+                    description=(
+                        "Optional PromQL selector to filter (e.g., '{__name__=~\"kube.*\"}', "
+                        "'{job=\"prometheus\"}')."
+                    ),
+                    type="string",
+                    required=False,
+                ),
+                "start": ToolParameter(
+                    description="Start timestamp (RFC3339 or Unix). Default: 1 hour ago",
+                    type="string",
+                    required=False,
+                ),
+                "end": ToolParameter(
+                    description="End timestamp (RFC3339 or Unix). Default: now",
+                    type="string",
+                    required=False,
+                ),
+            },
+            toolset=toolset,
+        )
+
+    def _invoke(
+        self, params: dict, user_approved: bool = False
+    ) -> StructuredToolResult:
+        if not self.toolset.config or not self.toolset.config.prometheus_url:
+            return StructuredToolResult(
+                status=StructuredToolResultStatus.ERROR,
+                error="Prometheus is not configured. Prometheus URL is missing",
+                params=params,
+            )
+        try:
+            url = urljoin(self.toolset.config.prometheus_url, "api/v1/labels")
+            query_params = {"limit": str(PROMETHEUS_METADATA_API_LIMIT)}
+            if params.get("match"):
+                query_params["match[]"] = params["match"]
+
+            # Add time parameters - use provided values or defaults
+            if params.get("end"):
+                query_params["end"] = params["end"]
+            else:
+                query_params["end"] = str(int(time.time()))
+
+            if params.get("start"):
+                query_params["start"] = params["start"]
+            elif self.toolset.config.default_metadata_time_window_hrs:
+                # Use default time window
+                query_params["start"] = str(
+                    int(time.time())
+                    - (self.toolset.config.default_metadata_time_window_hrs * 3600)
+                )
+
+            response = do_request(
                 config=self.toolset.config,
-                verify_ssl=self.toolset.config.prometheus_ssl_enabled,
+                url=url,
+                params=query_params,
+                timeout=self.toolset.config.default_metadata_timeout_seconds,
+                verify=self.toolset.config.prometheus_ssl_enabled,
+                headers=self.toolset.config.headers,
+                method="GET",
             )
+            response.raise_for_status()
+            data = response.json()
+
+            # Check if results were truncated
+            if (
+                "data" in data
+                and isinstance(data["data"], list)
+                and len(data["data"]) == PROMETHEUS_METADATA_API_LIMIT
+            ):
+                data["_truncated"] = True
+                data["_message"] = (
+                    f"Results truncated at limit={PROMETHEUS_METADATA_API_LIMIT}. Use match[] parameter to filter labels."
+                )
 
-            type_filter = params.get("type_filter")
-            if type_filter:
-                metrics = filter_metrics_by_type(metrics, type_filter)
+            return StructuredToolResult(
+                status=StructuredToolResultStatus.SUCCESS,
+                data=data,
+                params=params,
+            )
+        except Exception as e:
+            return StructuredToolResult(
+                status=StructuredToolResultStatus.ERROR,
+                error=str(e),
+                params=params,
+            )
 
-            output = ["Metric | Description | Type | Labels"]
-            output.append("-" * 100)
+    def get_parameterized_one_liner(self, params) -> str:
+        return f"{toolset_name_for_one_liner(self.toolset.name)}: Get All Labels"
+
+
+class GetSeries(BasePrometheusTool):
+    """Get time series matching a selector"""
 
-            for metric, info in sorted(metrics.items()):
-                labels_str = (
-                    ", ".join(sorted(info["labels"])) if info["labels"] else "none"
+    def __init__(self, toolset: "PrometheusToolset"):
+        super().__init__(
+            name="get_series",
+            description=(
+                "Get time series using /api/v1/series. "
+                "Returns label sets for all time series matching the selector. "
+                "SLOWER than other discovery methods - use only when you need full label sets. "
+                f"Returns up to {PROMETHEUS_METADATA_API_LIMIT} series (limit={PROMETHEUS_METADATA_API_LIMIT}). If {PROMETHEUS_METADATA_API_LIMIT} results returned, more series exist - use more specific selector. "
+                "Requires match[] parameter with PromQL selector. "
+                "By default returns series active in the last 1 hour (configurable via default_metadata_time_window_hrs)."
+            ),
+            parameters={
+                "match": ToolParameter(
+                    description=(
+                        "PromQL selector to match series (e.g., 'up', 'node_cpu_seconds_total', "
+                        "'{__name__=~\"node.*\"}', '{job=\"prometheus\"}', "
+                        '\'{__name__="up",job="prometheus"}\').'
+                    ),
+                    type="string",
+                    required=True,
+                ),
+                "start": ToolParameter(
+                    description="Start timestamp (RFC3339 or Unix). Default: 1 hour ago",
+                    type="string",
+                    required=False,
+                ),
+                "end": ToolParameter(
+                    description="End timestamp (RFC3339 or Unix). Default: now",
+                    type="string",
+                    required=False,
+                ),
+            },
+            toolset=toolset,
+        )
+
+    def _invoke(
+        self, params: dict, user_approved: bool = False
+    ) -> StructuredToolResult:
+        if not self.toolset.config or not self.toolset.config.prometheus_url:
+            return StructuredToolResult(
+                status=StructuredToolResultStatus.ERROR,
+                error="Prometheus is not configured. Prometheus URL is missing",
+                params=params,
+            )
+        try:
+            match = params.get("match")
+            if not match:
+                return StructuredToolResult(
+                    status=StructuredToolResultStatus.ERROR,
+                    error="Match parameter is required",
+                    params=params,
                 )
-                output.append(
-                    f"{metric} | {info['description']} | {info['type']} | {labels_str}"
+
+            url = urljoin(self.toolset.config.prometheus_url, "api/v1/series")
+            query_params = {
+                "match[]": match,
+                "limit": str(PROMETHEUS_METADATA_API_LIMIT),
+            }
+
+            # Add time parameters - use provided values or defaults
+            if params.get("end"):
+                query_params["end"] = params["end"]
+            else:
+                query_params["end"] = str(int(time.time()))
+
+            if params.get("start"):
+                query_params["start"] = params["start"]
+            elif self.toolset.config.default_metadata_time_window_hrs:
+                # Use default time window
+                query_params["start"] = str(
+                    int(time.time())
+                    - (self.toolset.config.default_metadata_time_window_hrs * 3600)
                 )
 
-            table_output = "\n".join(output)
+            response = do_request(
+                config=self.toolset.config,
+                url=url,
+                params=query_params,
+                timeout=self.toolset.config.default_metadata_timeout_seconds,
+                verify=self.toolset.config.prometheus_ssl_enabled,
+                headers=self.toolset.config.headers,
+                method="GET",
+            )
+            response.raise_for_status()
+            data = response.json()
+
+            # Check if results were truncated
+            if (
+                "data" in data
+                and isinstance(data["data"], list)
+                and len(data["data"]) == PROMETHEUS_METADATA_API_LIMIT
+            ):
+                data["_truncated"] = True
+                data["_message"] = (
+                    f"Results truncated at limit={PROMETHEUS_METADATA_API_LIMIT}. Use a more specific match selector to see additional series."
+                )
+
+            return StructuredToolResult(
+                status=StructuredToolResultStatus.SUCCESS,
+                data=data,
+                params=params,
+            )
+        except Exception as e:
             return StructuredToolResult(
-                status=ToolResultStatus.SUCCESS,
-                data=table_output,
+                status=StructuredToolResultStatus.ERROR,
+                error=str(e),
                 params=params,
             )
 
-        except requests.Timeout:
-            logging.warn("Timeout while fetching prometheus metrics", exc_info=True)
+    def get_parameterized_one_liner(self, params) -> str:
+        return f"{toolset_name_for_one_liner(self.toolset.name)}: Get Series"
+
+
+class GetMetricMetadata(BasePrometheusTool):
+    """Get metadata (type, description, unit) for metrics"""
+
+    def __init__(self, toolset: "PrometheusToolset"):
+        super().__init__(
+            name="get_metric_metadata",
+            description=(
+                "Get metric metadata using /api/v1/metadata. "
+                "Returns type, help text, and unit for metrics. "
+                "Use after discovering metric names to get their descriptions. "
+                f"Returns up to {PROMETHEUS_METADATA_API_LIMIT} metrics (limit={PROMETHEUS_METADATA_API_LIMIT}). If {PROMETHEUS_METADATA_API_LIMIT} results returned, more may exist - filter by specific metric name. "
+                "Supports optional metric name filter."
+            ),
+            parameters={
+                "metric": ToolParameter(
+                    description=(
+                        "Optional metric name to filter (e.g., 'up', 'node_cpu_seconds_total'). "
+                        "If not provided, returns metadata for all metrics."
+                    ),
+                    type="string",
+                    required=False,
+                ),
+            },
+            toolset=toolset,
+        )
+
+    def _invoke(
+        self, params: dict, user_approved: bool = False
+    ) -> StructuredToolResult:
+        if not self.toolset.config or not self.toolset.config.prometheus_url:
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
-                error="Request timed out while fetching metrics",
+                status=StructuredToolResultStatus.ERROR,
+                error="Prometheus is not configured. Prometheus URL is missing",
                 params=params,
             )
-        except RequestException as e:
-            logging.warn("Failed to fetch prometheus metrics", exc_info=True)
+        try:
+            url = urljoin(self.toolset.config.prometheus_url, "api/v1/metadata")
+            query_params = {"limit": str(PROMETHEUS_METADATA_API_LIMIT)}
+
+            if params.get("metric"):
+                query_params["metric"] = params["metric"]
+
+            response = do_request(
+                config=self.toolset.config,
+                url=url,
+                params=query_params,
+                timeout=self.toolset.config.default_metadata_timeout_seconds,
+                verify=self.toolset.config.prometheus_ssl_enabled,
+                headers=self.toolset.config.headers,
+                method="GET",
+            )
+            response.raise_for_status()
+            data = response.json()
+
+            # Check if results were truncated (metadata endpoint returns a dict, not a list)
+            if (
+                "data" in data
+                and isinstance(data["data"], dict)
+                and len(data["data"]) == PROMETHEUS_METADATA_API_LIMIT
+            ):
+                data["_truncated"] = True
+                data["_message"] = (
+                    f"Results truncated at limit={PROMETHEUS_METADATA_API_LIMIT}. Use metric parameter to filter by specific metric name."
+                )
+
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
-                error=f"Network error while fetching metrics: {str(e)}",
+                status=StructuredToolResultStatus.SUCCESS,
+                data=data,
                 params=params,
             )
         except Exception as e:
-            logging.warn("Failed to process prometheus metrics", exc_info=True)
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
-                error=f"Unexpected error: {str(e)}",
+                status=StructuredToolResultStatus.ERROR,
+                error=str(e),
                 params=params,
             )
 
     def get_parameterized_one_liner(self, params) -> str:
-        name_filter = params.get("name_filter", "")
-        return f"{toolset_name_for_one_liner(self.toolset.name)}: Search Metrics ({name_filter})"
+        metric = params.get("metric", "all")
+        return (
+            f"{toolset_name_for_one_liner(self.toolset.name)}: Get Metadata ({metric})"
+        )
 
 
 class ExecuteInstantQuery(BasePrometheusTool):
     def __init__(self, toolset: "PrometheusToolset"):
         super().__init__(
             name="execute_prometheus_instant_query",
-            description="Execute an instant PromQL query",
+            description=(
+                f"Execute an instant PromQL query (single point in time). "
+                f"Default timeout is {DEFAULT_QUERY_TIMEOUT_SECONDS} seconds "
+                f"but can be increased up to {MAX_QUERY_TIMEOUT_SECONDS} seconds for complex/slow queries."
+            ),
             parameters={
                 "query": ToolParameter(
                     description="The PromQL query",
@@ -694,6 +1061,15 @@ class ExecuteInstantQuery(BasePrometheusTool):
                     type="string",
                     required=True,
                 ),
+                "timeout": ToolParameter(
+                    description=(
+                        f"Query timeout in seconds. Default: {DEFAULT_QUERY_TIMEOUT_SECONDS}. "
+                        f"Maximum: {MAX_QUERY_TIMEOUT_SECONDS}. "
+                        f"Increase for complex queries that may take longer."
+                    ),
+                    type="number",
+                    required=False,
+                ),
             },
             toolset=toolset,
         )
@@ -703,7 +1079,7 @@ class ExecuteInstantQuery(BasePrometheusTool):
     ) -> StructuredToolResult:
         if not self.toolset.config or not self.toolset.config.prometheus_url:
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error="Prometheus is not configured. Prometheus URL is missing",
                 params=params,
             )
@@ -715,12 +1091,24 @@ class ExecuteInstantQuery(BasePrometheusTool):
 
             payload = {"query": query}
 
+            # Get timeout parameter and enforce limits
+            default_timeout = self.toolset.config.default_query_timeout_seconds
+            max_timeout = self.toolset.config.max_query_timeout_seconds
+            timeout = params.get("timeout", default_timeout)
+            if timeout > max_timeout:
+                timeout = max_timeout
+                logging.warning(
+                    f"Timeout requested ({params.get('timeout')}) exceeds maximum ({max_timeout}s), using {max_timeout}s"
+                )
+            elif timeout < 1:
+                timeout = default_timeout  # Min 1 second, but use default if invalid
+
             response = do_request(
                 config=self.toolset.config,
                 url=url,
                 headers=self.toolset.config.headers,
                 data=payload,
-                timeout=60,
+                timeout=timeout,
                 verify=self.toolset.config.prometheus_ssl_enabled,
                 method="POST",
             )
@@ -743,12 +1131,44 @@ class ExecuteInstantQuery(BasePrometheusTool):
                     "query": query,
                 }
 
+                # Check if data should be included based on size
                 if self.toolset.config.tool_calls_return_data:
-                    response_data["data"] = data.get("data")
+                    result_data = data.get("data", {})
+
+                    # Estimate the size of the data
+                    data_str_preview = json.dumps(result_data)
+                    data_size_chars = len(data_str_preview)
+
+                    # Provide summary if data is too large
+                    if (
+                        self.toolset.config.query_response_size_limit
+                        and data_size_chars
+                        > self.toolset.config.query_response_size_limit
+                    ):
+                        response_data["data_summary"] = (
+                            create_data_summary_for_large_result(
+                                result_data,
+                                query,
+                                data_size_chars,
+                                is_range_query=False,
+                            )
+                        )
+                        logging.info(
+                            f"Prometheus instant query returned large dataset: "
+                            f"{response_data['data_summary'].get('result_count', 0)} results, "
+                            f"{data_size_chars:,} characters (limit: {self.toolset.config.query_response_size_limit:,}). "
+                            f"Returning summary instead of full data."
+                        )
+                        # Also add character info to the summary for debugging
+                        response_data["data_summary"]["_debug_info"] = (
+                            f"Data size: {data_size_chars:,} chars exceeded limit of {self.toolset.config.query_response_size_limit:,} chars"
+                        )
+                    else:
+                        response_data["data"] = result_data
 
                 data_str = json.dumps(response_data, indent=2)
                 return StructuredToolResult(
-                    status=ToolResultStatus.SUCCESS,
+                    status=StructuredToolResultStatus.SUCCESS,
                     data=data_str,
                     params=params,
                 )
@@ -764,14 +1184,14 @@ class ExecuteInstantQuery(BasePrometheusTool):
                 except json.JSONDecodeError:
                     pass
                 return StructuredToolResult(
-                    status=ToolResultStatus.ERROR,
+                    status=StructuredToolResultStatus.ERROR,
                     error=f"Query execution failed. HTTP {response.status_code}: {error_msg}",
                     params=params,
                 )
 
             # For other status codes, just return the status code and content
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error=f"Query execution failed with unexpected status code: {response.status_code}. Response: {str(response.content)}",
                 params=params,
             )
@@ -779,14 +1199,14 @@ class ExecuteInstantQuery(BasePrometheusTool):
         except RequestException as e:
             logging.info("Failed to connect to Prometheus", exc_info=True)
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error=f"Connection error to Prometheus: {str(e)}",
                 params=params,
             )
         except Exception as e:
             logging.info("Failed to connect to Prometheus", exc_info=True)
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error=f"Unexpected error executing query: {str(e)}",
                 params=params,
             )
@@ -800,7 +1220,12 @@ class ExecuteRangeQuery(BasePrometheusTool):
     def __init__(self, toolset: "PrometheusToolset"):
         super().__init__(
             name="execute_prometheus_range_query",
-            description="Generates a graph and Execute a PromQL range query",
+            description=(
+                f"Generates a graph and Execute a PromQL range query. "
+                f"Default timeout is {DEFAULT_QUERY_TIMEOUT_SECONDS} seconds "
+                f"but can be increased up to {MAX_QUERY_TIMEOUT_SECONDS} seconds for complex/slow queries. "
+                f"Default time range is last 1 hour."
+            ),
             parameters={
                 "query": ToolParameter(
                     description="The PromQL query",
@@ -827,13 +1252,32 @@ class ExecuteRangeQuery(BasePrometheusTool):
                 "step": ToolParameter(
                     description="Query resolution step width in duration format or float number of seconds",
                     type="number",
-                    required=True,
+                    required=False,
                 ),
                 "output_type": ToolParameter(
                     description="Specifies how to interpret the Prometheus result. Use 'Plain' for raw values, 'Bytes' to format byte values, 'Percentage' to scale 0–1 values into 0–100%, or 'CPUUsage' to convert values to cores (e.g., 500 becomes 500m, 2000 becomes 2).",
                     type="string",
                     required=True,
                 ),
+                "timeout": ToolParameter(
+                    description=(
+                        f"Query timeout in seconds. Default: {DEFAULT_QUERY_TIMEOUT_SECONDS}. "
+                        f"Maximum: {MAX_QUERY_TIMEOUT_SECONDS}. "
+                        f"Increase for complex queries that may take longer."
+                    ),
+                    type="number",
+                    required=False,
+                ),
+                "max_points": ToolParameter(
+                    description=(
+                        f"Maximum number of data points to return. Default: {int(MAX_GRAPH_POINTS)}. "
+                        f"Can be reduced to get fewer data points (e.g., 50 for simpler graphs). "
+                        f"Cannot exceed system limit of {int(MAX_GRAPH_POINTS)}. "
+                        f"If your query would return more points than this limit, the step will be automatically adjusted."
+                    ),
+                    type="number",
+                    required=False,
+                ),
             },
             toolset=toolset,
         )
@@ -843,7 +1287,7 @@ class ExecuteRangeQuery(BasePrometheusTool):
     ) -> StructuredToolResult:
         if not self.toolset.config or not self.toolset.config.prometheus_url:
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error="Prometheus is not configured. Prometheus URL is missing",
                 params=params,
             )
@@ -857,12 +1301,17 @@ class ExecuteRangeQuery(BasePrometheusTool):
                 end_timestamp=params.get("end"),
                 default_time_span_seconds=DEFAULT_GRAPH_TIME_SPAN_SECONDS,
             )
-            step = params.get("step", "")
+            step = parse_duration_to_seconds(params.get("step"))
+            max_points = params.get(
+                "max_points"
+            )  # Get the optional max_points parameter
 
+            # adjust_step_for_max_points handles None case and converts to float
             step = adjust_step_for_max_points(
                 start_timestamp=start,
                 end_timestamp=end,
-                step=float(step) if step else MAX_GRAPH_POINTS,
+                step=step,
+                max_points_override=max_points,
             )
 
             description = params.get("description", "")
@@ -874,12 +1323,24 @@ class ExecuteRangeQuery(BasePrometheusTool):
                 "step": step,
             }
 
+            # Get timeout parameter and enforce limits
+            default_timeout = self.toolset.config.default_query_timeout_seconds
+            max_timeout = self.toolset.config.max_query_timeout_seconds
+            timeout = params.get("timeout", default_timeout)
+            if timeout > max_timeout:
+                timeout = max_timeout
+                logging.warning(
+                    f"Timeout requested ({params.get('timeout')}) exceeds maximum ({max_timeout}s), using {max_timeout}s"
+                )
+            elif timeout < 1:
+                timeout = default_timeout  # Min 1 second, but use default if invalid
+
             response = do_request(
                 config=self.toolset.config,
                 url=url,
                 headers=self.toolset.config.headers,
                 data=payload,
-                timeout=120,
+                timeout=timeout,
                 verify=self.toolset.config.prometheus_ssl_enabled,
                 method="POST",
             )
@@ -906,12 +1367,42 @@ class ExecuteRangeQuery(BasePrometheusTool):
                     "output_type": output_type,
                 }
 
+                # Check if data should be included based on size
                 if self.toolset.config.tool_calls_return_data:
-                    response_data["data"] = data.get("data")
+                    result_data = data.get("data", {})
+
+                    # Estimate the size of the data
+                    data_str_preview = json.dumps(result_data)
+                    data_size_chars = len(data_str_preview)
+
+                    # Provide summary if data is too large
+                    if (
+                        self.toolset.config.query_response_size_limit
+                        and data_size_chars
+                        > self.toolset.config.query_response_size_limit
+                    ):
+                        response_data["data_summary"] = (
+                            create_data_summary_for_large_result(
+                                result_data, query, data_size_chars, is_range_query=True
+                            )
+                        )
+                        logging.info(
+                            f"Prometheus range query returned large dataset: "
+                            f"{response_data['data_summary'].get('series_count', 0)} series, "
+                            f"{data_size_chars:,} characters (limit: {self.toolset.config.query_response_size_limit:,}). "
+                            f"Returning summary instead of full data."
+                        )
+                        # Also add character info to the summary for debugging
+                        response_data["data_summary"]["_debug_info"] = (
+                            f"Data size: {data_size_chars:,} chars exceeded limit of {self.toolset.config.query_response_size_limit:,} chars"
+                        )
+                    else:
+                        response_data["data"] = result_data
+
                 data_str = json.dumps(response_data, indent=2)
 
                 return StructuredToolResult(
-                    status=ToolResultStatus.SUCCESS,
+                    status=StructuredToolResultStatus.SUCCESS,
                     data=data_str,
                     params=params,
                 )
@@ -926,13 +1417,13 @@ class ExecuteRangeQuery(BasePrometheusTool):
                 except json.JSONDecodeError:
                     pass
                 return StructuredToolResult(
-                    status=ToolResultStatus.ERROR,
+                    status=StructuredToolResultStatus.ERROR,
                     error=f"Query execution failed. HTTP {response.status_code}: {error_msg}",
                     params=params,
                 )
 
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error=f"Query execution failed with unexpected status code: {response.status_code}. Response: {str(response.content)}",
                 params=params,
             )
@@ -940,14 +1431,14 @@ class ExecuteRangeQuery(BasePrometheusTool):
         except RequestException as e:
             logging.info("Failed to connect to Prometheus", exc_info=True)
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error=f"Connection error to Prometheus: {str(e)}",
                 params=params,
             )
         except Exception as e:
             logging.info("Failed to connect to Prometheus", exc_info=True)
             return StructuredToolResult(
-                status=ToolResultStatus.ERROR,
+                status=StructuredToolResultStatus.ERROR,
                 error=f"Unexpected error executing query: {str(e)}",
                 params=params,
             )
@@ -969,7 +1460,11 @@ class PrometheusToolset(Toolset):
             prerequisites=[CallablePrerequisite(callable=self.prerequisites_callable)],
             tools=[
                 ListPrometheusRules(toolset=self),
-                ListAvailableMetrics(toolset=self),
+                GetMetricNames(toolset=self),
+                GetLabelValues(toolset=self),
+                GetAllLabels(toolset=self),
+                GetSeries(toolset=self),
+                GetMetricMetadata(toolset=self),
                 ExecuteInstantQuery(toolset=self),
                 ExecuteRangeQuery(toolset=self),
             ],
@@ -1060,13 +1555,8 @@ class PrometheusToolset(Toolset):
                     f"Failed to connect to Prometheus at {url}: HTTP {response.status_code}",
                 )
 
-        except RequestException:
-            return (
-                False,
-                f"Failed to initialize using url={url}",
-            )
         except Exception as e:
-            logging.exception("Failed to initialize Prometheus")
+            logging.exception("Failed to initialize Prometheus", exc_info=True)
             return (
                 False,
                 f"Failed to initialize using url={url}. Unexpected error: {str(e)}",