MK8S-25: Add detailed error reporting for directory cleanup failures

Enhance Mkdir cleanup to show tree view of directory contents when removal fails:
- Custom _clean_with_details method for better error diagnosis
- Uses 'tree -a' command for detailed directory tree view
- Falls back to 'ls -la' if tree command is not available
- Shows exactly what files are preventing directory removal
- Helps diagnose "_build_root - cannot remove (it is not empty)" errors

This will make debugging cleanup failures much easier by showing
the exact files that are blocking directory removal.

Author: rdebay-scality

buildchain/buildchain/targets/directory.py

@@ -4,9 +4,11 @@
 """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
 
@@ -35,6 +37,7 @@ def task(self) -> types.TaskDict:
             {
                 "title": utils.title_with_target1("MKDIR"),
                 "actions": [(self._run, [task["targets"][0]])],
+                "clean": [self._clean_with_details],
                 "uptodate": [True],
             }
         )
@@ -43,3 +46,43 @@ 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,
+                    )
+                    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,
+                    )
+                    print(result.stdout)
+            except Exception as list_exc:
+                print(f"Could not list directory contents: {list_exc}")
+            # Re-raise the original exception
+            raise