Merge pull request #336 from Akhil-Nair-ML/makefile

Fixed issue due to make is not installed.
maplelabs · Feb 7, 2023 · 412c9b2 · 412c9b2
2 parents eaa04ab + 1a6fffc
commit 412c9b2
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/install_scaling_manager.yaml b/install_scaling_manager.yaml
@@ -32,12 +32,15 @@
     become: yes
     command: dpkg-query -W -f='${Status}' make | grep 'install ok installed'
     register: is_installed
-   - name: Install make only if installed
-      apt:
-        name: make
-        state: latest
-        update_cache: yes
-      when: is_installed.rc != 0
+    failed_when: no
+    changed_when: no
+  - name: Install make if not installed
+    become: yes
+    apt:
+      name: make
+      state: latest
+      update_cache: yes
+    when: is_installed.rc != 0
   - name: Installing Scalling Manager
     shell:
     args:
@@ -82,4 +85,4 @@
       cmd: make uninstall
       chdir: /tmp
     become: yes
-  tags: uninstall
+  tags: uninstall