diff mbox series

[ovs-dev,v3] ci: Update crun in GitHub actions runner.

Message ID 20240228071809.1959349-1-mheib@redhat.com
State Accepted
Delegated to: Dumitru Ceara
Headers show
Series [ovs-dev,v3] ci: Update crun in GitHub actions runner. | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/github-robot-_ovn-kubernetes success github build: passed

Commit Message

Mohammad Heib Feb. 28, 2024, 7:18 a.m. UTC
There are some issues with older crun and newer kernel
combination [0]. Unfortunately Ubuntu 22.04 falls into this
category and the jobs will often time fail. Temoprarily use
newer crun from git that has the fix [1].

[0] https://github.com/containers/crun/pull/1309
[1] https://noobient.com/2023/11/15/fixing-ubuntu-containers-failing-to-start-with-systemd/

Signed-off-by: Mohammad Heib <mheib@redhat.com>
---
 .github/workflows/ovn-fake-multinode-tests.yml | 8 ++++++++
 .github/workflows/test.yml                     | 8 ++++++++
 2 files changed, 16 insertions(+)

Comments

Dumitru Ceara Feb. 28, 2024, 10:55 a.m. UTC | #1
On 2/28/24 08:18, Mohammad Heib wrote:
> There are some issues with older crun and newer kernel
> combination [0]. Unfortunately Ubuntu 22.04 falls into this
> category and the jobs will often time fail. Temoprarily use
> newer crun from git that has the fix [1].
> 
> [0] https://github.com/containers/crun/pull/1309
> [1] https://noobient.com/2023/11/15/fixing-ubuntu-containers-failing-to-start-with-systemd/
> 
> Signed-off-by: Mohammad Heib <mheib@redhat.com>
> ---

Thanks, Mohammad for the new version!

I applied this to main and backported it to all affected branches down
to 23.06.

Regards,
Dumitru
diff mbox series

Patch

diff --git a/.github/workflows/ovn-fake-multinode-tests.yml b/.github/workflows/ovn-fake-multinode-tests.yml
index 179c1d662..79b6c4253 100644
--- a/.github/workflows/ovn-fake-multinode-tests.yml
+++ b/.github/workflows/ovn-fake-multinode-tests.yml
@@ -137,6 +137,14 @@  jobs:
         sudo systemctl start openvswitch-switch
         sudo ovs-vsctl show
 
+    # XXX This should be removed when native crun >=1.9.1
+    - name: update crun script
+      run: |
+        crun --version
+        sudo curl -L "https://github.com/containers/crun/releases/download/1.14.1/crun-1.14.1-linux-amd64" -o /usr/bin/crun
+        sudo chmod +x /usr/bin/crun
+        echo "New crun version: "$(crun --version)
+
     - name: Start basic cluster
       run: |
         sudo -E ./ovn_cluster.sh start
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 2def39531..0cb981775 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -217,6 +217,14 @@  jobs:
         path: /tmp/image.tar
         key: ${{ github.sha }}
 
+    # XXX This should be removed when native crun >=1.9.1
+    - name: update crun script
+      run: |
+        crun --version
+        sudo curl -L "https://github.com/containers/crun/releases/download/1.14.1/crun-1.14.1-linux-amd64" -o /usr/bin/crun
+        sudo chmod +x /usr/bin/crun
+        echo "New crun version: "$(crun --version)
+
     - name: load image
       run: |
         sudo podman load -i /tmp/image.tar