Update ci (#1315)

* Remove github.job from cache identifier

It is the same for all of them, anyway

* Remove semi-binary-testing builds

They don't make a lot of sense. We don't need to know whether our direct
dependencies fail on the testing repos.

* Do not require setting an upstream workspace

It could potentially just be empty. With that we can remove the empty
not-released files

* Add hardware_interface_testing as test_depend for ur_controllers

The test_actuator is used in the load tests

Author: urfeex

.github/workflows/coverage-build.yml

@@ -29,8 +29,6 @@ jobs:
             ur_calibration
             ur_controllers
             ur_robot_driver
-          vcs-repo-file-url: |
-            https://raw.githubusercontent.com/${{ github.repository }}/${{ github.sha }}/Universal_Robots_ROS2_Driver-not-released.${{ env.ROS_DISTRO }}.repos
           colcon-defaults: |
             {
               "build": {

.github/workflows/humble-binary-main.yml

@@ -11,5 +11,4 @@ jobs:
     with:
       ros_distro: humble
       ros_repo: main
-      upstream_workspace: Universal_Robots_ROS2_Driver-not-released.humble.repos
       ref_for_scheduled_build: humble

.github/workflows/humble-binary-testing.yml

@@ -12,5 +12,4 @@ jobs:
     with:
       ros_distro: humble
       ros_repo: testing
-      upstream_workspace: Universal_Robots_ROS2_Driver-not-released.humble.repos
       ref_for_scheduled_build: humble

.github/workflows/humble-semi-binary-testing.yml

@@ -17,5 +17,4 @@ jobs:
     with:
       ros_distro: jazzy
       ros_repo: main
-      upstream_workspace: Universal_Robots_ROS2_Driver-not-released.jazzy.repos
       ref_for_scheduled_build: main

.github/workflows/jazzy-binary-main.yml

@@ -17,5 +17,4 @@ jobs:
     with:
       ros_distro: jazzy
       ros_repo: testing
-      upstream_workspace: Universal_Robots_ROS2_Driver-not-released.jazzy.repos
       ref_for_scheduled_build: main

.github/workflows/jazzy-binary-testing.yml

@@ -12,7 +12,8 @@ on:
 
       upstream_workspace:
         description: 'UPSTREAM_WORKSPACE variable for industrial_ci. Usually path to local .repos file.'
-        required: true
+        required: false
+        default: ''
         type: string
       ros_distro:
         description: 'ROS_DISTRO variable for industrial_ci'
@@ -41,7 +42,7 @@ jobs:
     env:
       DOCKER_RUN_OPTS: '-v /var/run/docker.sock:/var/run/docker.sock --network ursim_net'
       CCACHE_DIR: ${{ github.workspace }}/${{ inputs.ccache_dir }}
-      CACHE_PREFIX: ${{ inputs.ros_distro }}-${{ inputs.upstream_workspace }}-${{ inputs.ros_repo }}-${{ github.job }}
+      CACHE_PREFIX: ${{ inputs.ros_distro }}-${{ inputs.upstream_workspace }}-${{ inputs.ros_repo }}
     steps:
       - name: Checkout ${{ github.ref_name }} since build is not scheduled
         if: ${{ github.event_name != 'schedule' }}

.github/workflows/jazzy-semi-binary-testing.yml

@@ -17,5 +17,4 @@ jobs:
     with:
       ros_distro: rolling
       ros_repo: main
-      upstream_workspace: Universal_Robots_ROS2_Driver-not-released.rolling.repos
       ref_for_scheduled_build: main

.github/workflows/reusable_ici.yml

@@ -17,5 +17,4 @@ jobs:
     with:
       ros_distro: rolling
       ros_repo: testing
-      upstream_workspace: Universal_Robots_ROS2_Driver-not-released.rolling.repos
       ref_for_scheduled_build: main