Skip to content

Commit 4875dd5

Browse files
Update security runs to also support AL2 on node20 (#1002)
* Make test security github action also support AL2 on node20 Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
1 parent 9d71188 commit 4875dd5

File tree

2 files changed

+17
-13
lines changed

2 files changed

+17
-13
lines changed

.github/workflows/test_aggregations.yml

+10-8
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,6 @@ on:
1010
branches:
1111
- "*"
1212
- "feature/**"
13-
env:
14-
ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION: true
15-
1613
jobs:
1714
Get-CI-Image-Tag:
1815
uses: opensearch-project/opensearch-build/.github/workflows/get-ci-image-tag.yml@main
@@ -33,16 +30,20 @@ jobs:
3330
# this image tag is subject to change as more dependencies and updates will arrive over time
3431
image: ${{ needs.Get-CI-Image-Tag.outputs.ci-image-version-linux }}
3532
# need to switch to root so that github actions can install runner binary on container without permission issues.
36-
options: --user root
33+
options: ${{ needs.Get-CI-Image-Tag.outputs.ci-image-start-options }}
3734

3835

3936
steps:
37+
- name: Run start commands
38+
run: ${{ needs.Get-CI-Image-Tag.outputs.ci-image-start-command }}
39+
4040
- name: Checkout neural-search
41-
uses: actions/checkout@v1
41+
uses: actions/checkout@v4
4242

4343
- name: Setup Java ${{ matrix.java }}
44-
uses: actions/setup-java@v1
44+
uses: actions/setup-java@v4
4545
with:
46+
distribution: 'temurin'
4647
java-version: ${{ matrix.java }}
4748

4849
- name: Run tests
@@ -61,11 +62,12 @@ jobs:
6162

6263
steps:
6364
- name: Checkout neural-search
64-
uses: actions/checkout@v1
65+
uses: actions/checkout@v4
6566

6667
- name: Setup Java ${{ matrix.java }}
67-
uses: actions/setup-java@v1
68+
uses: actions/setup-java@v4
6869
with:
70+
distribution: 'temurin'
6971
java-version: ${{ matrix.java }}
7072

7173
- name: Run tests

.github/workflows/test_security.yml

+7-5
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,6 @@ on:
1010
branches:
1111
- "*"
1212
- "feature/**"
13-
env:
14-
ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION: true
1513

1614
jobs:
1715
Get-CI-Image-Tag:
@@ -32,17 +30,21 @@ jobs:
3230
# this image tag is subject to change as more dependencies and updates will arrive over time
3331
image: ${{ needs.Get-CI-Image-Tag.outputs.ci-image-version-linux }}
3432
# need to switch to root so that github actions can install runner binary on container without permission issues.
35-
options: --user root
33+
options: ${{ needs.Get-CI-Image-Tag.outputs.ci-image-start-options }}
3634

3735
steps:
36+
- name: Run start commands
37+
run: ${{ needs.Get-CI-Image-Tag.outputs.ci-image-start-command }}
38+
3839
- name: Checkout neural-search
39-
uses: actions/checkout@v1
40+
uses: actions/checkout@v4
4041
with:
4142
submodules: true
4243

4344
- name: Setup Java ${{ matrix.java }}
44-
uses: actions/setup-java@v1
45+
uses: actions/setup-java@v4
4546
with:
47+
distribution: 'temurin'
4648
java-version: ${{ matrix.java }}
4749

4850
- name: Run tests

0 commit comments

Comments
 (0)