From 1c42c8ddcc79f8417afd87dbb9ed5821d58a3e83 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christoph=20Fr=C3=B6hlich?=
 <christophfroehlich@users.noreply.github.com>
Date: Thu, 15 Feb 2024 17:48:14 +0100
Subject: [PATCH] [CI] reviewer lottery and fail-fast (#1047)

---
 .github/reviewer-lottery.yml                  | 33 -------------------
 .github/workflows/humble-binary-build.yml     |  1 +
 .../workflows/humble-semi-binary-build.yml    |  1 +
 .github/workflows/iron-binary-build.yml       |  1 +
 .github/workflows/iron-semi-binary-build.yml  |  1 +
 .github/workflows/reviewer_lottery.yml        |  1 +
 .github/workflows/rolling-binary-build.yml    |  1 +
 .../workflows/rolling-semi-binary-build.yml   |  1 +
 8 files changed, 7 insertions(+), 33 deletions(-)
 delete mode 100644 .github/reviewer-lottery.yml

diff --git a/.github/reviewer-lottery.yml b/.github/reviewer-lottery.yml
deleted file mode 100644
index c6580eacd4..0000000000
--- a/.github/reviewer-lottery.yml
+++ /dev/null
@@ -1,33 +0,0 @@
-groups:
-  # Default reviewers for all pull requests.
-  # Usually, at least on of the maintainers should approve PR before merging.
-  # The best is if two maintainers do that.
-  - name: maintainers # name of the group
-    reviewers: 2 # how many reviewers do you want to assign?
-    internal_reviewers: 1 # how many reviewers do you want to assign when the PR author belongs to this group?
-    usernames: # github usernames of the reviewers
-      - bmagyar
-      - destogl
-
-  # Reviewers group to get broader feedback.
-  - name: reviewers
-    reviewers: 5
-    usernames:
-      - aprotyas
-      - arne48
-      - bijoua29
-      - christophfroehlich
-      - DasRoteSkelett
-      - duringhof
-      - erickisos
-      - fmauch
-      - jaron-l
-      - livanov93
-      - mcbed
-      - moriarty
-      - olivier-stasse
-      - peterdavidfagan
-      - progtologist
-      - saikishor
-      - VanshGehlot
-      - VX792
diff --git a/.github/workflows/humble-binary-build.yml b/.github/workflows/humble-binary-build.yml
index 989805733f..df449caecb 100644
--- a/.github/workflows/humble-binary-build.yml
+++ b/.github/workflows/humble-binary-build.yml
@@ -18,6 +18,7 @@ jobs:
   binary:
     uses: ros-controls/ros2_control_ci/.github/workflows/reusable-industrial-ci-with-cache.yml@master
     strategy:
+      fail-fast: false
       matrix:
         ROS_REPO: [main, testing]
     with:
diff --git a/.github/workflows/humble-semi-binary-build.yml b/.github/workflows/humble-semi-binary-build.yml
index 1d9d3bd1fb..aaed9c8ca2 100644
--- a/.github/workflows/humble-semi-binary-build.yml
+++ b/.github/workflows/humble-semi-binary-build.yml
@@ -17,6 +17,7 @@ jobs:
   semi_binary:
     uses: ros-controls/ros2_control_ci/.github/workflows/reusable-industrial-ci-with-cache.yml@master
     strategy:
+      fail-fast: false
       matrix:
         ROS_REPO: [main, testing]
     with:
diff --git a/.github/workflows/iron-binary-build.yml b/.github/workflows/iron-binary-build.yml
index 146d5a8c69..1510fac859 100644
--- a/.github/workflows/iron-binary-build.yml
+++ b/.github/workflows/iron-binary-build.yml
@@ -20,6 +20,7 @@ jobs:
   binary:
     uses: ros-controls/ros2_control_ci/.github/workflows/reusable-industrial-ci-with-cache.yml@master
     strategy:
+      fail-fast: false
       matrix:
         ROS_REPO: [main, testing]
     with:
diff --git a/.github/workflows/iron-semi-binary-build.yml b/.github/workflows/iron-semi-binary-build.yml
index ab508dfc50..38ca4fe490 100644
--- a/.github/workflows/iron-semi-binary-build.yml
+++ b/.github/workflows/iron-semi-binary-build.yml
@@ -19,6 +19,7 @@ jobs:
   semi_binary:
     uses: ros-controls/ros2_control_ci/.github/workflows/reusable-industrial-ci-with-cache.yml@master
     strategy:
+      fail-fast: false
       matrix:
         ROS_REPO: [main, testing]
     with:
diff --git a/.github/workflows/reviewer_lottery.yml b/.github/workflows/reviewer_lottery.yml
index ed28964e01..5784b2f836 100644
--- a/.github/workflows/reviewer_lottery.yml
+++ b/.github/workflows/reviewer_lottery.yml
@@ -12,3 +12,4 @@ jobs:
     - uses: uesteibar/reviewer-lottery@v3
       with:
         repo-token: ${{ secrets.GITHUB_TOKEN }}
+        config: ros-controls/ros2_control_ci/.github/reviewer-lottery.yml
diff --git a/.github/workflows/rolling-binary-build.yml b/.github/workflows/rolling-binary-build.yml
index 5831f0147f..d0be23f076 100644
--- a/.github/workflows/rolling-binary-build.yml
+++ b/.github/workflows/rolling-binary-build.yml
@@ -20,6 +20,7 @@ jobs:
   binary:
     uses: ros-controls/ros2_control_ci/.github/workflows/reusable-industrial-ci-with-cache.yml@master
     strategy:
+      fail-fast: false
       matrix:
         ROS_REPO: [main, testing]
     with:
diff --git a/.github/workflows/rolling-semi-binary-build.yml b/.github/workflows/rolling-semi-binary-build.yml
index edc55ebfb7..4784654db8 100644
--- a/.github/workflows/rolling-semi-binary-build.yml
+++ b/.github/workflows/rolling-semi-binary-build.yml
@@ -19,6 +19,7 @@ jobs:
   semi_binary:
     uses: ros-controls/ros2_control_ci/.github/workflows/reusable-industrial-ci-with-cache.yml@master
     strategy:
+      fail-fast: false
       matrix:
         ROS_REPO: [main, testing]
     with: