From b0dd8d044dd0c649f58191d3461846ec180b5ce0 Mon Sep 17 00:00:00 2001
From: Eric <xplzv@126.com>
Date: Thu, 24 Oct 2024 14:50:47 +0800
Subject: [PATCH] perf: add error msg when applet task failed

---
 .../automations/deploy_applet_host/install_applet.yml    | 7 +++++++
 .../terminal/automations/deploy_applet_host/playbook.yml | 9 ++++++++-
 .../automations/deploy_applet_host/uninstall_applet.yml  | 7 +++++++
 3 files changed, 22 insertions(+), 1 deletion(-)

diff --git a/apps/terminal/automations/deploy_applet_host/install_applet.yml b/apps/terminal/automations/deploy_applet_host/install_applet.yml
index de3c0fa49..d4970a73b 100644
--- a/apps/terminal/automations/deploy_applet_host/install_applet.yml
+++ b/apps/terminal/automations/deploy_applet_host/install_applet.yml
@@ -9,4 +9,11 @@
       ansible.windows.win_powershell:
         script: |
           tinkerd install --name {{ applet_name }}
+          $exitCode = $LASTEXITCODE
+          if ($exitCode -ne 0) {
+              Write-Host "Failed to install {{ applet_name }}"
+              Write-Host "Exit code: $exitCode"
+              $Ansible.Failed = $true
+              exit 1
+          }
       when: applet_name != 'all'
diff --git a/apps/terminal/automations/deploy_applet_host/playbook.yml b/apps/terminal/automations/deploy_applet_host/playbook.yml
index 5b097b164..86940202e 100644
--- a/apps/terminal/automations/deploy_applet_host/playbook.yml
+++ b/apps/terminal/automations/deploy_applet_host/playbook.yml
@@ -18,7 +18,7 @@
     PYTHON_VERSION: 3.11.10
     CHROME_VERSION: 118.0.5993.118
     CHROME_DRIVER_VERSION: 118.0.5993.70
-    TINKER_VERSION: v0.1.9
+    TINKER_VERSION: v0.2.0
 
   tasks:
     - block:
@@ -268,6 +268,13 @@
           ansible.windows.win_powershell:
             script: |
               tinkerd install all
+              $exitCode = $LASTEXITCODE
+              if ($exitCode -ne 0) {
+                  Write-Host "Failed to install applets"
+                  Write-Host "Exit code: $exitCode"
+                  $Ansible.Failed = $true
+                  exit 1
+              }
           register: sync_remote_applets
           when: INSTALL_APPLETS
 
diff --git a/apps/terminal/automations/deploy_applet_host/uninstall_applet.yml b/apps/terminal/automations/deploy_applet_host/uninstall_applet.yml
index 3da0b6d0e..57e448735 100644
--- a/apps/terminal/automations/deploy_applet_host/uninstall_applet.yml
+++ b/apps/terminal/automations/deploy_applet_host/uninstall_applet.yml
@@ -9,3 +9,10 @@
       ansible.windows.win_powershell:
         script: |
           tinkerd uninstall --name {{ applet_name }}
+          $exitCode = $LASTEXITCODE
+          if ($exitCode -ne 0) {
+              Write-Host "Failed to uninstall {{ applet_name }}"
+              Write-Host "Exit code: $exitCode"
+              $Ansible.Failed = $true
+              exit 1
+          }