From patchwork Tue Jan 30 12:07:57 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ilya Maximets X-Patchwork-Id: 1892803 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@legolas.ozlabs.org Authentication-Results: legolas.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=openvswitch.org (client-ip=2605:bc80:3010::138; helo=smtp1.osuosl.org; envelope-from=ovs-dev-bounces@openvswitch.org; receiver=patchwork.ozlabs.org) Received: from smtp1.osuosl.org (smtp1.osuosl.org [IPv6:2605:bc80:3010::138]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (secp384r1) server-digest SHA384) (No client certificate requested) by legolas.ozlabs.org (Postfix) with ESMTPS id 4TPP7D1cbnz23fD for ; Tue, 30 Jan 2024 23:07:34 +1100 (AEDT) Received: from localhost (localhost [127.0.0.1]) by smtp1.osuosl.org (Postfix) with ESMTP id 0D88D83B8B; Tue, 30 Jan 2024 12:07:31 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp1.osuosl.org 0D88D83B8B X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp1.osuosl.org ([127.0.0.1]) by localhost (smtp1.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id q3aYzX3Y3Bvv; Tue, 30 Jan 2024 12:07:29 +0000 (UTC) Received: from lists.linuxfoundation.org (lf-lists.osuosl.org [140.211.9.56]) by smtp1.osuosl.org (Postfix) with ESMTPS id CF7BB83B52; Tue, 30 Jan 2024 12:07:28 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp1.osuosl.org CF7BB83B52 Received: from lf-lists.osuosl.org (localhost [127.0.0.1]) by lists.linuxfoundation.org (Postfix) with ESMTP id A27BAC0072; Tue, 30 Jan 2024 12:07:28 +0000 (UTC) X-Original-To: ovs-dev@openvswitch.org Delivered-To: ovs-dev@lists.linuxfoundation.org Received: from smtp1.osuosl.org (smtp1.osuosl.org [IPv6:2605:bc80:3010::138]) by lists.linuxfoundation.org (Postfix) with ESMTP id D1004C0037 for ; Tue, 30 Jan 2024 12:07:26 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp1.osuosl.org (Postfix) with ESMTP id 9A11983B72 for ; Tue, 30 Jan 2024 12:07:26 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp1.osuosl.org 9A11983B72 X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp1.osuosl.org ([127.0.0.1]) by localhost (smtp1.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id IMPHqH1VMEmc for ; Tue, 30 Jan 2024 12:07:25 +0000 (UTC) Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [217.70.183.193]) by smtp1.osuosl.org (Postfix) with ESMTPS id 0E5EB83B52 for ; Tue, 30 Jan 2024 12:07:24 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp1.osuosl.org 0E5EB83B52 Received: by mail.gandi.net (Postfix) with ESMTPSA id C3CA6240004; Tue, 30 Jan 2024 12:07:21 +0000 (UTC) From: Ilya Maximets To: ovs-dev@openvswitch.org Date: Tue, 30 Jan 2024 13:07:57 +0100 Message-ID: <20240130120803.50200-1-i.maximets@ovn.org> X-Mailer: git-send-email 2.43.0 MIME-Version: 1.0 X-GND-Sasl: i.maximets@ovn.org Cc: Ilya Maximets , Dumitru Ceara Subject: [ovs-dev] [PATCH ovn] github: Update versions of action dependencies (Node.js 20). X-BeenThere: ovs-dev@openvswitch.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: ovs-dev-bounces@openvswitch.org Sender: "dev" checkout@v3, cache@v3, setup-python@v4 and setup-go@v3 are using outdated Node.js 16 which is now deprecated in GHA [1], so these actions may stop working soon. Updating to most recent major versions with Node.js 20. This stops GHA from throwing warnings in every build. [1] https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/ While at it also updating upload-artifact and download-artifact to the latest versions. Removing versions from the upload-artifact comment, since the behavior doesn't seem to change much between versions. New setup-go@v5 attempts to cache dependencies by default. However, the default path it uses is go.sum in the root directory. This triggers a warning, since the file doesn't exist: Restore cache failed: Dependencies file is not found in /home/runner/work/ovn-kubernetes/ovn-kubernetes. Supported file pattern: go.sum Specify a path to all .sum files we have in the repository to make the setup-go happy. This should in theory make the builds a touch faster. This change is in line with recent changes in ovn-kubernetes itself. Signed-off-by: Ilya Maximets Acked-by: Ales Musil --- .github/workflows/containers.yml | 2 +- .../workflows/ovn-fake-multinode-tests.yml | 26 +++++++------- .github/workflows/ovn-kubernetes.yml | 19 +++++----- .github/workflows/test.yml | 36 +++++++++---------- 4 files changed, 42 insertions(+), 41 deletions(-) diff --git a/.github/workflows/containers.yml b/.github/workflows/containers.yml index bdd118087..87e28d645 100644 --- a/.github/workflows/containers.yml +++ b/.github/workflows/containers.yml @@ -20,7 +20,7 @@ jobs: matrix: distro: [ fedora, ubuntu ] steps: - - uses: actions/checkout@v3 + - uses: actions/checkout@v4 - name: Update APT cache run: sudo apt update diff --git a/.github/workflows/ovn-fake-multinode-tests.yml b/.github/workflows/ovn-fake-multinode-tests.yml index b3ba82a30..179c1d662 100644 --- a/.github/workflows/ovn-fake-multinode-tests.yml +++ b/.github/workflows/ovn-fake-multinode-tests.yml @@ -26,7 +26,7 @@ jobs: XDG_RUNTIME_DIR: '' steps: - name: Check out ovn-fake-multi-node - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: repository: 'ovn-org/ovn-fake-multinode' path: 'ovn-fake-multinode' @@ -36,14 +36,14 @@ jobs: # ovn-fake-multinode builds and installs ovs from ovn-fake-multinode/ovs # and it builds and installs ovn from ovn-fake-multinode/ovn. It uses the ovs submodule for ovn compilation. - name: Check out ovs master - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: path: 'ovn-fake-multinode/ovs' repository: 'openvswitch/ovs' ref: 'master' - name: Check out ovn ${{ matrix.cfg.branch }} - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: path: 'ovn-fake-multinode/ovn' submodules: recursive @@ -63,7 +63,7 @@ jobs: sudo podman save ovn/ovn-multi-node:${{ matrix.cfg.branch }} > /tmp/_output/ovn_${{ matrix.cfg.branch }}_image.tar working-directory: ovn-fake-multinode - - uses: actions/upload-artifact@v3 + - uses: actions/upload-artifact@v4 with: name: test-${{ matrix.cfg.branch }}-image path: /tmp/_output/ovn_${{ matrix.cfg.branch }}_image.tar @@ -100,7 +100,7 @@ jobs: steps: - name: Check out ovn - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: install required dependencies run: | @@ -112,11 +112,11 @@ jobs: . .ci/linux-util.sh free_up_disk_space_ubuntu - - uses: actions/download-artifact@v3 + - uses: actions/download-artifact@v4 with: name: test-main-image - - uses: actions/download-artifact@v3 + - uses: actions/download-artifact@v4 with: name: test-branch-22.03-image @@ -126,7 +126,7 @@ jobs: sudo podman load --input ovn_branch-22.03_image.tar - name: Check out ovn-fake-multi-node - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: repository: 'ovn-org/ovn-fake-multinode' path: 'ovn-fake-multinode' @@ -156,12 +156,12 @@ jobs: echo "$HOME/.local/bin" >> $GITHUB_PATH - name: set up python - uses: actions/setup-python@v4 + uses: actions/setup-python@v5 with: python-version: '3.12' - name: Check out ovn - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: path: 'ovn' submodules: recursive @@ -190,9 +190,9 @@ jobs: - name: copy logs on failure if: failure() || cancelled() run: | - # upload-artifact@v3 throws exceptions if it tries to upload socket + # upload-artifact throws exceptions if it tries to upload socket # files and we could have some socket files in testsuite.dir. - # Also, upload-artifact@v3 doesn't work well enough with wildcards. + # Also, upload-artifact doesn't work well enough with wildcards. # So, we're just archiving everything here to avoid any issues. mkdir logs cp ovn/config.log ./logs/ @@ -203,7 +203,7 @@ jobs: - name: upload logs on failure if: failure() || cancelled() - uses: actions/upload-artifact@v3 + uses: actions/upload-artifact@v4 with: name: logs-linux-${{ join(matrix.cfg.*, '-') }} path: logs.tgz diff --git a/.github/workflows/ovn-kubernetes.yml b/.github/workflows/ovn-kubernetes.yml index 1689396d6..0f2b30497 100644 --- a/.github/workflows/ovn-kubernetes.yml +++ b/.github/workflows/ovn-kubernetes.yml @@ -32,12 +32,12 @@ jobs: sudo service docker restart - name: Check out ovn - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: submodules: recursive - name: Check out ovn-kubernetes - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: path: src/github.com/ovn-org/ovn-kubernetes repository: ovn-org/ovn-kubernetes @@ -54,7 +54,7 @@ jobs: mkdir /tmp/_output docker save ovn-daemonset-f:dev > /tmp/_output/image.tar - - uses: actions/upload-artifact@v3 + - uses: actions/upload-artifact@v4 with: name: test-image path: /tmp/_output/image.tar @@ -96,7 +96,7 @@ jobs: steps: - name: Check out ovn - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: Free up disk space run: | @@ -104,7 +104,7 @@ jobs: free_up_disk_space_ubuntu - name: Check out ovn-kubernetes - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: path: src/github.com/ovn-org/ovn-kubernetes repository: ovn-org/ovn-kubernetes @@ -114,9 +114,10 @@ jobs: .ci/ovn-kubernetes/prepare.sh src/github.com/ovn-org/ovn-kubernetes $GITHUB_ENV - name: Set up Go - uses: actions/setup-go@v3 + uses: actions/setup-go@v5 with: go-version: ${{ env.GO_VERSION }} + cache-dependency-path: "**/*.sum" id: go - name: Set up GOPATH @@ -131,7 +132,7 @@ jobs: run: | sudo ufw disable - - uses: actions/download-artifact@v3 + - uses: actions/download-artifact@v4 with: name: test-image @@ -155,7 +156,7 @@ jobs: - name: Upload Junit Reports if: always() - uses: actions/upload-artifact@v3 + uses: actions/upload-artifact@v4 with: name: kind-junit-${{ env.JOB_NAME }}-${{ github.run_id }} path: 'src/github.com/ovn-org/ovn-kubernetes/test/_artifacts/*.xml' @@ -169,7 +170,7 @@ jobs: - name: Upload logs if: always() - uses: actions/upload-artifact@v3 + uses: actions/upload-artifact@v4 with: name: kind-logs-${{ env.JOB_NAME }}-${{ github.run_id }} path: /tmp/kind/logs diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 2503d87d0..2def39531 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -26,7 +26,7 @@ jobs: steps: - name: checkout - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: update PATH run: | @@ -54,14 +54,14 @@ jobs: - name: cache id: dpdk_cache - uses: actions/cache@v3 + uses: actions/cache@v4 with: path: dpdk-dir key: ${{ steps.gen_dpdk_key.outputs.key }} - name: set up python if: steps.dpdk_cache.outputs.cache-hit != 'true' - uses: actions/setup-python@v4 + uses: actions/setup-python@v5 with: python-version: '3.9' @@ -96,7 +96,7 @@ jobs: runs-on: ubuntu-22.04 steps: - - uses: actions/checkout@v3 + - uses: actions/checkout@v4 - name: Update APT cache run: sudo apt update @@ -128,7 +128,7 @@ jobs: - name: Cache image id: image_cache - uses: actions/cache@v3 + uses: actions/cache@v4 with: path: /tmp/image.tar key: ${{ github.sha }} @@ -177,20 +177,20 @@ jobs: steps: - name: checkout if: github.event_name == 'push' || github.event_name == 'pull_request' - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: submodules: recursive # For weekly runs, don't update submodules - name: checkout without submodule if: github.event_name == 'schedule' - uses: actions/checkout@v3 + uses: actions/checkout@v4 # Weekly runs test using the tip of the most recent stable OVS branch # instead of the submodule. - name: checkout OVS if: github.event_name == 'schedule' - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: repository: 'openvswitch/ovs' fetch-depth: 0 @@ -206,13 +206,13 @@ jobs: - name: cache dpdk if: matrix.cfg.dpdk != '' - uses: actions/cache@v3 + uses: actions/cache@v4 with: path: dpdk-dir key: ${{ needs.build-dpdk.outputs.dpdk_key }} - name: image cache - uses: actions/cache@v3 + uses: actions/cache@v4 with: path: /tmp/image.tar key: ${{ github.sha }} @@ -233,7 +233,7 @@ jobs: - name: upload logs on failure if: failure() || cancelled() - uses: actions/upload-artifact@v3 + uses: actions/upload-artifact@v4 with: name: logs-linux-${{ join(matrix.cfg.*, '-') }} path: logs.tgz @@ -252,18 +252,18 @@ jobs: steps: - name: checkout if: github.event_name == 'push' || github.event_name == 'pull_request' - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: submodules: recursive # For weekly runs, don't update submodules - name: checkout without submodule if: github.event_name == 'schedule' - uses: actions/checkout@v3 + uses: actions/checkout@v4 # Weekly runs test using the tip of the most recent stable OVS branch # instead of the submodule. - name: checkout OVS if: github.event_name == 'schedule' - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: repository: 'openvswitch/ovs' fetch-depth: 0 @@ -282,7 +282,7 @@ jobs: echo "$HOME/bin" >> $GITHUB_PATH echo "$HOME/.local/bin" >> $GITHUB_PATH - name: set up python - uses: actions/setup-python@v4 + uses: actions/setup-python@v5 with: python-version: '3.12' - name: prepare @@ -291,7 +291,7 @@ jobs: run: ./.ci/osx-build.sh - name: upload logs on failure if: failure() - uses: actions/upload-artifact@v3 + uses: actions/upload-artifact@v4 with: name: logs-osx-clang---disable-ssl path: config.log @@ -310,7 +310,7 @@ jobs: run: dnf install -y dnf-plugins-core git rpm-build - name: checkout - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: submodules: recursive @@ -338,7 +338,7 @@ jobs: run: make rpm-fedora - name: upload rpm packages - uses: actions/upload-artifact@v3 + uses: actions/upload-artifact@v4 with: name: rpm-packages path: |