fix python issue and slack bug

Author: aantn

pyproject.toml

@@ -23,7 +23,7 @@ plugins = "numpy.typing.mypy_plugin,pydantic.mypy"
 krr = "robusta_krr.main:run"
 
 [tool.poetry.dependencies]
-python = ">=3.9,<=3.12.3"
+python = ">=3.9,<=3.12.9"
 typer = { extras = ["all"], version = "^0.7.0" }
 pydantic = "^1.10.7"
 kubernetes = "^26.1.0"

robusta_krr/core/runner.py

@@ -11,6 +11,7 @@
 from prometrix import PrometheusNotFound
 from rich.console import Console
 from slack_sdk import WebClient
+from slack_sdk.errors import SlackApiError
 from tenacity import retry, stop_after_attempt, wait_exponential, retry_if_exception_type
 import requests
 import json
@@ -135,14 +136,50 @@ def _process_result(self, result: Result) -> None:
             if settings.slack_output:
                 client = WebClient(os.environ["SLACK_BOT_TOKEN"])
                 warnings.filterwarnings("ignore", category=UserWarning)
-                client.files_upload(
-                    channels=f"#{settings.slack_output}",
+                
+                # Resolve channel name to ID if needed (files_upload_v2 requires channel ID)
+                channel_id = self._resolve_slack_channel_id(client, settings.slack_output)
+                
+                client.files_upload_v2(
+                    channels=channel_id,
                     title="KRR Report",
                     file=f"./{file_name}",
                     initial_comment=f'Kubernetes Resource Report for {(" ".join(settings.namespaces))}',
                 )
                 os.remove(file_name)
 
+    def _resolve_slack_channel_id(self, client: WebClient, channel_input: str) -> str:
+        """Resolve channel name to ID if needed. files_upload_v2 requires channel ID."""
+        # If it already looks like a channel ID (starts with C), return as-is
+        if channel_input.startswith('C'):
+            return channel_input
+        
+        # Remove # if present
+        channel_name = channel_input.lstrip('#')
+        
+        # Search through all pages of channels
+        cursor = None
+        while True:
+            response = client.conversations_list(
+                types="public_channel,private_channel",
+                cursor=cursor
+            )
+            
+            # Check channels on this page
+            for channel in response.get("channels", []):
+                if channel.get("name") == channel_name:
+                    return channel["id"]
+            
+            # Move to next page if available
+            cursor = response.get("response_metadata", {}).get("next_cursor")
+            if not cursor:
+                break
+        
+        raise ValueError(
+            f"Channel '{channel_input}' not found. "
+            f"Ensure: 1) Channel exists, 2) Bot is added to channel, 3) Bot has 'channels:read' permission"
+        )
+
     def __get_resource_minimal(self, resource: ResourceType) -> float:
         if resource == ResourceType.CPU:
             return 1 / 1000 * settings.cpu_min_value

robusta_krr/main.py

@@ -263,7 +263,7 @@ def run_strategy(
                 slack_output: Optional[str] = typer.Option(
                     None,
                     "--slackoutput",
-                    help="Send to output to a slack channel, must have SLACK_BOT_TOKEN",
+                    help="Send to output to a slack channel, must have SLACK_BOT_TOKEN with permissions: channels:read, groups:read, chat:write, chat:write.public, files:write. Bot must be added to the channel.",
                     rich_help_panel="Output Settings",
                 ),
                 publish_scan_url: Optional[str] = typer.Option(