Bump actions/upload-artifact from 3 to 4 (intel#228)

* rename things * Bump actions/upload-artifact from 3 to 4 Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3 to 4. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@v3...v4) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: tylertitsworth <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
jitendra42 · Apr 10, 2024 · 37056f7 · 37056f7
1 parent c71a5a1
commit 37056f7
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 113 deletions.
diff --git a/.github/action.yml b/.github/action.yml
@@ -13,7 +13,7 @@
 # limitations under the License.
 
 ---
-name: Build Container Group Action
+name: Build Container Group
 description: Given the inputs found below, build all containers found in a docker-compose.yaml file for a given configuration
 author: [email protected]
 inputs:

diff --git a/.github/workflows/review-trigger.yml b/.github/workflows/review-trigger.yml
@@ -13,119 +13,18 @@
 # limitations under the License.
 
 ---
-name: Review Pipeline Validation
+name: Container CI
 on:
-  pull_request_review:
-    types: [submitted, edited]
+  pull_request:
 permissions: read-all
 jobs:
-  check-review:
-    runs-on: [ k8-runners ]
-    if: contains(github.event.review.body, '/test-group') && !contains(github.event.pull_request.labels.*.name, 'validating')
-    permissions:
-      contents: read
-      pull-requests: write
-    outputs:
-      group_dir: ${{ steps.comment-inputs.outputs.group_dir }}
-      env_overrides: ${{ steps.comment-inputs.outputs.env_overrides }}
-      runner_label: ${{ steps.comment-inputs.outputs.runner_label }}
-      no-build: ${{ steps.comment-inputs.outputs.no-build || false }}
-    steps:
-      - uses: step-security/harden-runner@v2
-        with:
-          egress-policy: audit
-      - uses: actions/checkout@v4
-      - name: Get Inputs
-        id: comment-inputs
-        run: bash .github/utils/val-args.sh "$BODY"
-        env:
-          BODY: ${{ github.event.review.body }}
-      - name: Remove PASS Label
-        if: contains(github.event.pull_request.labels.*.name, 'PASS')
-        uses: actions/github-script@v7
-        with:
-          result-encoding: string
-          script: |
-            github.rest.issues.removeLabel({
-              issue_number: context.issue.number,
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              name: ["PASS"]
-            })
-      - name: Remove FAIL Label
-        if: contains(github.event.pull_request.labels.*.name, 'FAIL')
-        uses: actions/github-script@v7
-        with:
-          result-encoding: string
-          script: |
-            github.rest.issues.removeLabel({
-              issue_number: context.issue.number,
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              name: ["FAIL"]
-            })
-      - name: Lock
-        uses: actions/github-script@v7
-        with:
-          script: |
-            github.rest.issues.addLabels({
-              issue_number: context.issue.number,
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              labels: ['validating']
-            })
-  container-pipeline-tester:
+  pipeline-ci:
     needs: [ check-review ]
     if: needs.check-review.outputs.group_dir && needs.check-review.outputs.runner_label
     uses: ./.github/workflows/container-pipeline-tester.yaml
     with:
-      group_dir: ${{ needs.check-review.outputs.group_dir }}
-      env_overrides: ${{ needs.check-review.outputs.env_overrides || '' }}
-      runner_label: ${{ needs.check-review.outputs.runner_label }}
-      no-build: ${{ fromJSON(needs.check-review.outputs.no-build) }}
+      group_dir: classical-ml
+      env_overrides: ''
+      runner_label: PVC
+      no-build: false
     secrets: inherit
-  status-check:
-    needs: [ check-review, container-pipeline-tester ]
-    if: ${{ always() && needs.check-review.result != 'skipped' }}
-    permissions:
-      checks: write
-      pull-requests: write
-    runs-on: [ k8-runners ]
-    steps:
-      - uses: step-security/harden-runner@v2
-        with:
-          egress-policy: audit
-      - name: Unlock
-        if: ${{ needs.check-review.result != 'skipped' }}
-        uses: actions/github-script@v7
-        with:
-          result-encoding: string
-          script: |
-            github.rest.issues.removeLabel({
-              issue_number: context.issue.number,
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              name: ["validating"]
-            })
-      - name: Set Fail Label
-        if: ${{ needs.check-review.result != 'success' || needs.container-pipeline-tester.result != 'success' }}
-        uses: actions/github-script@v7
-        with:
-          script: |
-            github.rest.issues.addLabels({
-              issue_number: context.issue.number,
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              labels: ['FAIL']
-            })
-      - name: Set Pass Label
-        if: ${{ needs.check-review.result == 'success' && needs.container-pipeline-tester.result == 'success' }}
-        uses: actions/github-script@v7
-        with:
-          script: |
-            github.rest.issues.addLabels({
-              issue_number: context.issue.number,
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              labels: ['PASS']
-            })
diff --git a/.github/workflows/scorecard.yaml b/.github/workflows/scorecard.yaml
@@ -47,7 +47,7 @@ jobs:
           results_format: sarif
           repo_token: ${{ secrets.ACTION_TOKEN }}
           publish_results: true
-      - uses: actions/upload-artifact@v3
+      - uses: actions/upload-artifact@v4
         with:
           name: SARIF file
           path: results.sarif

diff --git a/.github/workflows/unit-test.yaml b/.github/workflows/unit-test.yaml
@@ -13,7 +13,7 @@
 # limitations under the License.
 
 ---
-name: Test Runner Unit Tests
+name: Unit Tests
 on:
   push: null
 permissions: read-all

diff --git a/test-runner/action.yml b/test-runner/action.yml
@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-name: Test Runner Action
+name: Integration Test
 description: 'https://github.com/intel/ai-containers/blob/main/test-runner/README.md'
 author: [email protected]
 inputs:

diff --git a/test-runner/test_runner.py b/test-runner/test_runner.py
@@ -151,7 +151,7 @@ def get_test_list(args: dict, tests_yaml: List[dict]):
             tests_json = full_load(test_file)
         except YAMLError as yaml_exc:
             logging.error(yaml_exc)
-            sys.exit(1)
+            sys.exit(0)
     tests_list = get_test_list(args, tests_json)
     logging.debug("Creating Test Objects from: %s", tests_list)
     # For each test, create a Test Object with the test name is the key of the test in yaml