From 4a290da3def4dc13c4b9caa550b8afa9b37f55d8 Mon Sep 17 00:00:00 2001
From: Brandon Shien <44730413+bshien@users.noreply.github.com>
Date: Tue, 3 Dec 2024 11:19:25 -0800
Subject: [PATCH] Remove dco and update artifact action to v4 (#1467)

Signed-off-by: Brandon Shien <bshien@amazon.com>
---
 .github/workflows/build-and-test.yml |  2 +-
 .github/workflows/bwc.yml            |  2 +-
 .github/workflows/dco-check.yml      | 18 ------------------
 .github/workflows/security-tests.yml |  2 +-
 4 files changed, 3 insertions(+), 21 deletions(-)
 delete mode 100644 .github/workflows/dco-check.yml

diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
index e74f59461..735e253f2 100644
--- a/.github/workflows/build-and-test.yml
+++ b/.github/workflows/build-and-test.yml
@@ -36,7 +36,7 @@ jobs:
         run: |
           ./gradlew clean release -D"build.snapshot=true"
       - name: Upload failed logs
-        uses: actions/upload-artifact@v2
+        uses: actions/upload-artifact@v4
         if: failure()
         with:
           name: logs
diff --git a/.github/workflows/bwc.yml b/.github/workflows/bwc.yml
index 6d3d07c4d..f35d6155d 100644
--- a/.github/workflows/bwc.yml
+++ b/.github/workflows/bwc.yml
@@ -29,7 +29,7 @@ jobs:
           ./gradlew mixedClusterTask --stacktrace
           ./gradlew fullRestartClusterTask --stacktrace
       - name: Upload failed logs
-        uses: actions/upload-artifact@v2
+        uses: actions/upload-artifact@v4
         if: failure()
         with:
           name: logs
diff --git a/.github/workflows/dco-check.yml b/.github/workflows/dco-check.yml
deleted file mode 100644
index cf30ea89d..000000000
--- a/.github/workflows/dco-check.yml
+++ /dev/null
@@ -1,18 +0,0 @@
-name: Developer Certificate of Origin Check
-
-on: [pull_request]
-
-jobs:
-  check:
-    runs-on: ubuntu-latest
-
-    steps:
-      - name: Get PR Commits
-        id: 'get-pr-commits'
-        uses: tim-actions/get-pr-commits@v1.1.0
-        with:
-          token: ${{ secrets.GITHUB_TOKEN }}
-      - name: DCO Check
-        uses: tim-actions/dco@v1.1.0
-        with:
-          commits: ${{ steps.get-pr-commits.outputs.commits }}
diff --git a/.github/workflows/security-tests.yml b/.github/workflows/security-tests.yml
index 9383a9359..bd9d19b89 100644
--- a/.github/workflows/security-tests.yml
+++ b/.github/workflows/security-tests.yml
@@ -39,7 +39,7 @@ jobs:
           ls -al src/test/resources/security/plugin
           ./gradlew clean release -Dbuild.snapshot=true -Dopensearch.version=1.3.19-SNAPSHOT -PnumNodes=1 -Psecurity=true
       - name: Upload failed logs
-        uses: actions/upload-artifact@v2
+        uses: actions/upload-artifact@v4
         if: failure()
         with:
           name: logs