feat: Add check for held ROS packages during installation

Signed-off-by: Naophis <nao1288stusj@gmail.com>
autowarefoundation · May 27, 2024 · e22777e · e22777e
1 parent 0cb9ef4
commit e22777e
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/ansible/roles/rmw_implementation/tasks/main.yaml b/ansible/roles/rmw_implementation/tasks/main.yaml
@@ -3,14 +3,24 @@
   register: rmw_implementation__dash_case_rmw_implementation
   changed_when: false
 
+- name: Hold check of ros-{{ rosdistro + '-' + rmw_implementation__dash_case_rmw_implementation.stdout }}
+  ansible.builtin.command: apt-mark showhold
+  register: held_ros_packages
+  changed_when: false
+
 - name: Install ros-{{ rosdistro + '-' + rmw_implementation__dash_case_rmw_implementation.stdout }}
   become: true
   ansible.builtin.apt:
     name: ros-{{ rosdistro }}-{{ rmw_implementation__dash_case_rmw_implementation.stdout }}
     state: latest
-    update_cache: true
-  register: apt_install_result
-  failed_when: "'Held packages were changed' not in apt_install_result.msg"
+  when: "'ros-' + rosdistro + '-' + rmw_implementation__dash_case_rmw_implementation.stdout not in held_ros_packages.stdout"
+  register: install_result
+  failed_when: false
+
+- name: Display warning if ROS 2 RMW package is held
+  ansible.builtin.debug:
+    msg: ROS 2 RMW package 'ros-{{ rosdistro + '-' + rmw_implementation__dash_case_rmw_implementation.stdout }}' is apt-mark hold. Skipping installation.
+  when: not install_result.changed
 
 - name: Add RMW_IMPLEMENTATION to .bashrc
   ansible.builtin.lineinfile: