MATTER-7126 : Use docker for SLT CLI (#1126)

* MATTER-7126 : Use docker for SLT CLI (#1109)

Author: Sarthak-Shaha

.github/actions/cache-tools/action.yaml

@@ -31,23 +31,26 @@ runs:
           run: |
               echo "cache-key=silabs-zap-slt-tools-${{ runner.os }}-${{ runner.arch }}-${{ github.sha }}" >> $GITHUB_OUTPUT
 
-        - name: Restore ZAP and SLT tools cache
+        - name: Restore ZAP, SLT and Tools cache
           id: cache
           uses: actions/cache/restore@v5
           with:
               path: |
                   /tmp/zap-linux-x64
                   /home/runner/.silabs/slt
                   /home/runner/.local/bin/slt
+                  /home/runner/.local/bin/conan
+                  /home/runner/.local/bin/conan_engine
               key: ${{ steps.compute-cache-key.outputs.cache-key }}
 
         - name: Set env and paths from cache
           if: steps.cache.outputs.cache-hit == 'true'
           shell: bash
           run: |
-              echo "CONAN_HOME=~/.silabs/slt/installs/conan" >> $GITHUB_ENV
+              echo "CONAN_HOME=/home/runner/.silabs/slt/installs/conan" >> $GITHUB_ENV
               echo "STUDIO_ADAPTER_PACK_PATH=/tmp/zap-linux-x64" >> $GITHUB_ENV
               echo "SLT_PATH=/home/runner/.local/bin/slt" >> $GITHUB_ENV
+              echo "SLT_CI=true" >> $GITHUB_ENV
               echo "/tmp/zap-linux-x64" >> $GITHUB_PATH
               echo "/home/runner/.silabs/slt/bin" >> $GITHUB_PATH
               echo "/home/runner/.local/bin" >> $GITHUB_PATH
@@ -75,12 +78,14 @@ runs:
           if: steps.cache.outputs.cache-hit != 'true'
           uses: ./.github/actions/install-tools
 
-        - name: Save ZAP and SLT tools cache
+        - name: Save ZAP, SLT and Tools cache
           if: steps.cache.outputs.cache-hit != 'true'
           uses: actions/cache/save@v5
           with:
               path: |
                   /tmp/zap-linux-x64
                   /home/runner/.silabs/slt
                   /home/runner/.local/bin/slt
+                  /home/runner/.local/bin/conan
+                  /home/runner/.local/bin/conan_engine
               key: ${{ steps.compute-cache-key.outputs.cache-key }}

.github/actions/install-tools/action.yaml

@@ -19,7 +19,25 @@ runs:
     
     - name: Setup SLT
       id: setup-action
-      uses: SiliconLabsSoftware/action-setup-slt@v2
+      shell: bash
+      env:
+        SLT_IMAGE: ghcr.io/siliconlabssoftware/slt-cli:1.2.2
+        SLT_CI: "true"
+      run: |
+        INSTALL_PATH="$HOME/.local/bin"
+        mkdir -p "$INSTALL_PATH"
+        cid=$(docker create "$SLT_IMAGE")
+        docker cp "$cid:/opt/silabs/slt/slt" "$INSTALL_PATH/slt"
+        docker rm "$cid"
+        chmod +x "$INSTALL_PATH/slt"
+        echo "$INSTALL_PATH" >> $GITHUB_PATH
+        echo "SLT_CI=true" >> $GITHUB_ENV
+
+        "$INSTALL_PATH/slt" install conan
+        ln -fs "$HOME/.silabs/slt/engines/conan/conan_engine" "$INSTALL_PATH/conan_engine"
+        ln -fs "$HOME/.silabs/slt/engines/conan/conan/conan" "$INSTALL_PATH/conan"
+        echo "slt-path=$INSTALL_PATH/slt" >> $GITHUB_OUTPUT
+        echo "conan-path=$INSTALL_PATH/conan" >> $GITHUB_OUTPUT
     
     - name: Set Environment Variables
       shell: bash

requirements.txt

@@ -1,3 +1,2 @@
 python-dotenv
-dload
 pyyaml

slc/sl_setup_env.py

@@ -38,7 +38,6 @@
 import logging
 import os
 import sys
-import dload
 import stat
 import subprocess
 import shutil
@@ -140,16 +139,27 @@ def remove_tools_directory(self):
                 sys.exit(1)
         tools.mkdir(parents=True, exist_ok=True)
 
+    def _download_file(self, url, destination):
+        """Download url to destination with curl (www.silabs.com only allows curl for scripted downloads)."""
+        subprocess.run(["curl", "-fsSL", url, "-o", destination], check=True)
+
+    def _download_and_extract_zip(self, url, destination):
+        """Download a ZIP archive and extract it into destination."""
+        zip_path = destination + ".zip"
+        try:
+            self._download_file(url, zip_path)
+            with ZipFile(zip_path, 'r') as zObject:
+                zObject.extractall(path=destination)
+        finally:
+            if os.path.exists(zip_path):
+                os.remove(zip_path)
+
     def download_and_extract_slt_cli(self):
         """Download and extract SLT CLI tool."""
         if not os.path.isfile(self.slt_cli_path):
             logging.info(f"Downloading and unzipping slt-cli...")
-            slt_zip_path = os.path.join(self.tools_folder_path, "slt.zip")
             try:
-                dload.save(self.slt_cli_url, slt_zip_path)
-                with ZipFile(slt_zip_path, 'r') as zObject:
-                    zObject.extractall(path=self.tools_folder_path)
-                os.remove(slt_zip_path)
+                self._download_and_extract_zip(self.slt_cli_url, self.tools_folder_path)
                 if self.platform != "win32":
                     self._make_executable(self.slt_cli_path)
             except Exception as e:
@@ -178,18 +188,18 @@ def download_and_extract_zap(self, install_without_checking=False):
             try:
                 if self.platform == "darwin":
                     zap_zip = self.zap_path + ".zip"
-                    dload.save(zap_url, zap_zip)
+                    self._download_file(zap_url, zap_zip)
                     subprocess.run(["unzip", zap_zip, "-d", self.zap_path], check=True)
                     os.remove(zap_zip)
                     self._make_executable(zap_cli_path)
                     zap_root = os.path.join(self.zap_path, "zap.app", "Contents", "MacOS", "zap")
                     self._make_executable(zap_root)
                 elif self.platform == "linux":
-                    dload.save_unzip(zap_url, os.path.join(self.tools_folder_path, "zap"), delete_after=True)
+                    self._download_and_extract_zip(zap_url, self.zap_path)
                     self._make_executable(zap_cli_path)
-                    self._make_executable(os.path.join(self.tools_folder_path, "zap", "zap"))
+                    self._make_executable(os.path.join(self.zap_path, "zap"))
                 elif self.platform == "win32":
-                    dload.save_unzip(zap_url, os.path.join(self.tools_folder_path, "zap"), delete_after=True)
+                    self._download_and_extract_zip(zap_url, self.zap_path)
             except Exception as e:
                 logging.error(f"Failed to download/extract zap: {e}")
                 sys.exit(1)