MK8S-25: Revert unnecessary Python buildchain changes

Revert buildchain/buildchain/targets/directory.py and repository.py to their
original state. These files were modified in earlier commits to support
index.html file creation, but that approach has been replaced with nginx
location directives. The Python changes are no longer needed.

Author: rdebay-scality

buildchain/buildchain/targets/directory.py

@@ -4,11 +4,9 @@
 """Provides operations on directories."""
 
 
-import subprocess
 from pathlib import Path
 from typing import Any
 
-from buildchain import coreutils
 from buildchain import types
 from buildchain import utils
 
@@ -37,7 +35,6 @@ def task(self) -> types.TaskDict:
             {
                 "title": utils.title_with_target1("MKDIR"),
                 "actions": [(self._run, [task["targets"][0]])],
-                "clean": [self._clean_with_details],
                 "uptodate": [True],
             }
         )
@@ -46,45 +43,3 @@ def task(self) -> types.TaskDict:
     @staticmethod
     def _run(directory: Path) -> None:
         directory.mkdir(exist_ok=True)
-
-    def _clean_with_details(self) -> None:
-        """Clean directory with detailed error reporting if removal fails."""
-        directory = self.targets[0]
-        if not directory.exists():
-            return
-
-        try:
-            # Try to remove the directory
-            coreutils.rm_rf(directory)
-        except Exception as exc:
-            # If removal fails, show detailed tree view of what's still there
-            print(f"ERROR: Cannot remove directory '{directory}' - {exc}")
-            print("Directory contents:")
-            try:
-                # Use the 'tree' command if available, otherwise use 'ls -la'
-                try:
-                    result = subprocess.run(
-                        ["tree", "-a", str(directory)],
-                        capture_output=True,
-                        text=True,
-                        timeout=30,
-                        check=False,
-                    )
-                    if result.returncode == 0:
-                        print(result.stdout)
-                    else:
-                        raise subprocess.CalledProcessError(result.returncode, "tree")
-                except (subprocess.CalledProcessError, FileNotFoundError):
-                    # Fall back to ls -la if tree is not available
-                    result = subprocess.run(
-                        ["ls", "-la", str(directory)],
-                        capture_output=True,
-                        text=True,
-                        timeout=30,
-                        check=False,
-                    )
-                    print(result.stdout)
-            except Exception as list_exc:
-                print(f"Could not list directory contents: {list_exc}")
-            # Re-raise the original exception
-            raise

buildchain/buildchain/targets/repository.py

@@ -33,14 +33,13 @@
 import abc
 import operator
 from pathlib import Path
-from typing import Any, List, Optional, Sequence, Tuple
+from typing import Any, List, Optional, Sequence
 
 from buildchain import config
 from buildchain import coreutils
 from buildchain import constants
 from buildchain import types
 from buildchain import utils
-from buildchain import versions
 from buildchain import docker_command
 
 from . import base
@@ -207,13 +206,6 @@ def clean() -> None:
             task["file_dep"].extend([self.get_rpm_path(pkg) for pkg in self.packages])
         return task
 
-    @property
-    def execution_plan(self) -> List[types.TaskDict]:
-        tasks = [self.build_repo()]
-        if self._packages:
-            tasks.extend(self.build_packages())
-        return tasks
-
     def build_packages(self) -> List[types.TaskDict]:
         """Build the RPMs from SRPMs."""
         tasks = [self._mkdir_repo_root(), self._mkdir_repo_arch()]