diff mbox series

[ovs-dev] github: Update versions of action dependencies (Node.js 20).

Message ID 20240126135924.56238-1-i.maximets@ovn.org
State Superseded
Headers show
Series [ovs-dev] github: Update versions of action dependencies (Node.js 20). | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/intel-ovs-compilation success test: success

Commit Message

Ilya Maximets Jan. 26, 2024, 1:59 p.m. UTC 
checkout@v3, cache@v3 and setup-python@v4 are using outdated Node.js 16
which is now deprecated in GHA [1], so these actions will 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 version to the latest version.
Removed versions from the comment as the general behavior of this
action regarding symlinks and wildcards doesn't seem to change between
versions much.

Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
---
 .github/workflows/build-and-test.yml | 38 ++++++++++++++--------------
 1 file changed, 19 insertions(+), 19 deletions(-)

Comments

Ilya Maximets Jan. 26, 2024, 2:06 p.m. UTC | #1 
On 1/26/24 14:59, Ilya Maximets wrote:
> checkout@v3, cache@v3 and setup-python@v4 are using outdated Node.js 16
> which is now deprecated in GHA [1], so these actions will 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 version to the latest version.
> Removed versions from the comment as the general behavior of this
> action regarding symlinks and wildcards doesn't seem to change between
> versions much.
> 
> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
> ---
>  .github/workflows/build-and-test.yml | 38 ++++++++++++++--------------
>  1 file changed, 19 insertions(+), 19 deletions(-)

Missed the clang-analyze use of sub-actions of cache.
Will send v2.
diff mbox series

Patch

diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
index 3807e5f13..8d90f9798 100644
--- a/.github/workflows/build-and-test.yml
+++ b/.github/workflows/build-and-test.yml
@@ -17,7 +17,7 @@  jobs:
 
     steps:
     - name: checkout
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
 
     - name: update PATH
       run:  |
@@ -45,14 +45,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'
 
@@ -207,7 +207,7 @@  jobs:
 
     steps:
     - name: checkout
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
 
     - name: update PATH
       run:  |
@@ -215,13 +215,13 @@  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.9'
 
     - name: cache
       if:   matrix.dpdk != '' || matrix.dpdk_shared != ''
-      uses: actions/cache@v3
+      uses: actions/cache@v4
       with:
         path: dpdk-dir
         key:  ${{ needs.build-dpdk.outputs.dpdk_key }}
@@ -247,9 +247,9 @@  jobs:
     - name: copy logs on failure
       if: failure() || cancelled()
       run: |
-        # upload-artifact@v2 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@v2 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 config.log ./logs/
@@ -259,7 +259,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.*, '-') }}
         path: logs.tgz
@@ -279,7 +279,7 @@  jobs:
 
     steps:
     - name: checkout
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
       with:
         fetch-depth: 0
 
@@ -338,13 +338,13 @@  jobs:
 
     - name: check for analyzer result cache
       id: clang_cache
-      uses: actions/cache@v3
+      uses: actions/cache@v4
       with:
         path: base-clang-analyzer-results
         key:  ${{ steps.cache_key.outputs.key }}
 
     - name: set up python
-      uses: actions/setup-python@v4
+      uses: actions/setup-python@v5
       with:
         python-version: '3.9'
 
@@ -391,13 +391,13 @@  jobs:
 
     steps:
     - name: checkout
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
     - name: update PATH
       run:  |
         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.9'
     - name: install dependencies
@@ -408,7 +408,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
@@ -432,7 +432,7 @@  jobs:
 
     steps:
     - name: checkout
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
 
     - name: update PATH
       run:  |
@@ -454,7 +454,7 @@  jobs:
       run:  ./.ci/linux-build.sh
 
     - name: upload deb packages
-      uses: actions/upload-artifact@v3
+      uses: actions/upload-artifact@v4
       with:
         name: deb-packages-${{ matrix.dpdk }}-dpdk
         path: '/home/runner/work/ovs/*.deb'
@@ -470,7 +470,7 @@  jobs:
 
     steps:
     - name: checkout
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
     - name: install dependencies
       run: |
         dnf install -y rpm-build dnf-plugins-core
@@ -489,7 +489,7 @@  jobs:
       run:  dnf install -y rpm/rpmbuild/RPMS/*/*.rpm
 
     - name: upload rpm packages
-      uses: actions/upload-artifact@v3
+      uses: actions/upload-artifact@v4
       with:
         name: rpm-packages
         path: |