Skip to content

Commit 5b37778

Browse files
authored
address new lint errors (#843)
Signed-off-by: Srikanth Ramakrishna <[email protected]>
1 parent 22527f0 commit 5b37778

10 files changed

+48
-10
lines changed

.github/workflows/apptainer-ci.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ jobs:
3737
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
3838
with:
3939
fetch-depth: 0
40+
persist-credentials: false
4041
- name: Output Modified Group Directories
4142
id: group-list
4243
run: |
@@ -78,6 +79,8 @@ jobs:
7879
with:
7980
egress-policy: audit
8081
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
82+
with:
83+
persist-credentials: false
8184
- uses: eWaterCycle/setup-apptainer@4bb22c52d4f63406c49e94c804632975787312b3 # v2.0.0
8285
with:
8386
apptainer-version: 1.3.4
@@ -86,7 +89,7 @@ jobs:
8689
APPTAINER_DOCKER_USERNAME: ${{ secrets.REGISTRY_USER }}
8790
APPTAINER_DOCKER_PASSWORD: ${{ secrets.REGISTRY_TOKEN }}
8891
run: apptainer registry login -u ${{ secrets.REGISTRY_USER }} -p ${{ secrets.REGISTRY_TOKEN }} docker://${{ secrets.REGISTRY }}
89-
- uses: intel/ai-containers/apptainer@main
92+
- uses: intel/ai-containers/apptainer@22527f085f2accb495bf4fed7475d683b4120927
9093
with:
9194
registry: oras://${{ secrets.REGISTRY }}
9295
repo: ${{ secrets.REPO }}-apptainer

.github/workflows/chart-ci.yaml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,8 @@ jobs:
3030
with:
3131
egress-policy: audit
3232
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
33-
- uses: intel/ai-containers/workflows/charts@main
33+
with:
34+
persist-credentials: false
35+
- uses: intel/ai-containers/workflows/charts@22527f085f2accb495bf4fed7475d683b4120927
3436
with:
3537
kubeconfig_path: ${{ secrets.KUBECONFIG_PATH }}

.github/workflows/container-ci.yaml

Lines changed: 24 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -70,12 +70,18 @@ jobs:
7070
with:
7171
egress-policy: audit
7272
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
73+
with:
74+
persist-credentials: false
7375
- name: Set Matrix
7476
id: build-matrix
75-
run: echo "matrix=$(jq -c . < ${{ inputs.group_dir }}/.actions.json)" >> $GITHUB_OUTPUT
77+
run: echo "matrix=$(jq -c . < ${INPUTS_GROUP_DIR}/.actions.json)" >> $GITHUB_OUTPUT
78+
env:
79+
INPUTS_GROUP_DIR: ${{ inputs.group_dir }}
7680
- name: Print Inputs
7781
if: ${{ inputs.env_overrides }}
78-
run: echo "Overrides - ${{ inputs.env_overrides }}" >> $GITHUB_STEP_SUMMARY
82+
run: echo "Overrides - ${INPUTS_ENV_OVERRIDES}" >> $GITHUB_STEP_SUMMARY
83+
env:
84+
INPUTS_ENV_OVERRIDES: ${{ inputs.env_overrides }}
7985
build-containers:
8086
needs: [setup-build]
8187
env: ${{ matrix }}
@@ -89,6 +95,8 @@ jobs:
8995
- uses: jlumbroso/free-disk-space@54081f138730dfa15788a46383842cd2f914a1be # v1.3.1
9096
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
9197
if: ${{ !inputs.no_build }}
98+
with:
99+
persist-credentials: false
92100
- uses: azure/docker-login@15c4aadf093404726ab2ff205b2cdd33fa6d054c # v2
93101
with:
94102
login-server: ${{ secrets.REGISTRY }}
@@ -103,7 +111,7 @@ jobs:
103111
- name: Build Container Group
104112
if: ${{ !inputs.no_build }}
105113
id: build-group
106-
uses: intel/ai-containers/.github@main
114+
uses: intel/ai-containers/.github@22527f085f2accb495bf4fed7475d683b4120927
107115
with:
108116
group_dir: ${{ inputs.group_dir }}
109117
env_overrides: ${{ inputs.env_overrides || env.env_overrides || '' }}
@@ -130,7 +138,9 @@ jobs:
130138
path: matrix
131139
- name: Set Matrix
132140
id: scan-matrix
133-
run: echo "matrix=$(cat matrix/*-${{ needs.build-containers.outputs.group }}/*.txt | jq -R '.' | jq -sc '. | unique')" >> $GITHUB_OUTPUT
141+
run: echo "matrix=$(cat matrix/*-${NEEDS_BUILD_CONTAINERS_OUTPUTS_GROUP}/*.txt | jq -R '.' | jq -sc '. | unique')" >> $GITHUB_OUTPUT
142+
env:
143+
NEEDS_BUILD_CONTAINERS_OUTPUTS_GROUP: ${{ needs.build-containers.outputs.group }}
134144
scan-containers:
135145
needs: [setup-scan]
136146
if: ${{ !inputs.no_build && github.event_name == 'pull_request' }}
@@ -146,6 +156,8 @@ jobs:
146156
with:
147157
egress-policy: audit
148158
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
159+
with:
160+
persist-credentials: false
149161
- uses: azure/docker-login@15c4aadf093404726ab2ff205b2cdd33fa6d054c # v2
150162
with:
151163
login-server: ${{ secrets.REGISTRY }}
@@ -196,9 +208,13 @@ jobs:
196208
with:
197209
egress-policy: audit
198210
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
211+
with:
212+
persist-credentials: false
199213
- name: Get Recipes
200214
id: test-matrix
201-
run: echo "matrix=$(find ${{ inputs.group_dir }} -type f -name 'tests.yaml' -exec dirname {} \; | jq -R -s -c 'split("\n")[:-1]')" >> $GITHUB_OUTPUT
215+
run: echo "matrix=$(find ${INPUTS_GROUP_DIR} -type f -name 'tests.yaml' -exec dirname {} \; | jq -R -s -c 'split("\n")[:-1]')" >> $GITHUB_OUTPUT
216+
env:
217+
INPUTS_GROUP_DIR: ${{ inputs.group_dir }}
202218
test-containers:
203219
needs: [setup-build, setup-test]
204220
if: ${{ needs.setup-test.outputs.matrix != '[]' }}
@@ -213,6 +229,8 @@ jobs:
213229
with:
214230
egress-policy: audit
215231
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
232+
with:
233+
persist-credentials: false
216234
- uses: azure/docker-login@15c4aadf093404726ab2ff205b2cdd33fa6d054c # v2
217235
with:
218236
login-server: ${{ secrets.REGISTRY }}
@@ -224,7 +242,7 @@ jobs:
224242
# username: ${{ secrets.REGISTRY_USER }}
225243
# password: ${{ secrets.REGISTRY_TOKEN }}
226244
- name: Test Container Group
227-
uses: intel/ai-containers/test-runner@main
245+
uses: intel/ai-containers/test-runner@22527f085f2accb495bf4fed7475d683b4120927
228246
with:
229247
cache_registry: ${{ secrets.CACHE_REGISTRY }}
230248
recipe_dir: ${{ inputs.group_dir }}

.github/workflows/dependency-review.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,8 @@ jobs:
4343
api.securityscorecards.dev:443
4444
github.com:443
4545
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
46+
with:
47+
persist-credentials: false
4648
- uses: actions/dependency-review-action@bc41886e18ea39df68b1b1245f4184881938e050 # v4.7.2
4749
with:
4850
comment-summary-in-pr: true

.github/workflows/dockerhub-description.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ jobs:
3030
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
3131
with:
3232
fetch-depth: 2
33+
persist-credentials: false
3334
- name: Set Matrix data
3435
id: set-matrix
3536
run: |
@@ -55,6 +56,8 @@ jobs:
5556
with:
5657
egress-policy: audit
5758
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
59+
with:
60+
persist-credentials: false
5861
- uses: peter-evans/dockerhub-description@432a30c9e07499fd01da9f8a49f0faf9e0ca5b77 # v4.0.2
5962
with:
6063
username: ${{ secrets.DOCKERHUB_USERNAME }}

.github/workflows/docs.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,8 @@ jobs:
4141
github.com:443
4242
pypi.org:443
4343
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
44+
with:
45+
persist-credentials: false
4446
- uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 # v5.6.0
4547
with:
4648
python-version: 3.8

.github/workflows/integration-test.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ jobs:
3434
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
3535
with:
3636
fetch-depth: 0
37+
persist-credentials: false
3738
- name: Output Modified Group Directories
3839
id: group-list
3940
run: |

.github/workflows/lint.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ jobs:
3939
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
4040
with:
4141
fetch-depth: 0
42+
persist-credentials: false
4243
- uses: super-linter/super-linter/slim@5119dcd8011e92182ce8219d9e9efc82f16fddb6 # v8.0.0
4344
env:
4445
GITHUB_ACTIONS_COMMAND_ARGS: '-ignore SC.*'

.github/workflows/test-runner-ci.yaml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ on:
1919
- 'test-runner/**'
2020
push:
2121
branches:
22-
- main
22+
- 22527f085f2accb495bf4fed7475d683b4120927
2323
permissions: read-all
2424
concurrency:
2525
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
@@ -37,6 +37,8 @@ jobs:
3737
with:
3838
egress-policy: audit
3939
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
40+
with:
41+
persist-credentials: false
4042
- uses: docker/setup-buildx-action@e468171a9de216ec08956ac3ada2f0791b6bd435 # v3.11.1
4143
with:
4244
driver: docker
@@ -85,6 +87,8 @@ jobs:
8587
with:
8688
egress-policy: audit
8789
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
90+
with:
91+
persist-credentials: false
8892
- uses: docker/setup-buildx-action@e468171a9de216ec08956ac3ada2f0791b6bd435 # v3.11.1
8993
with:
9094
driver: docker
@@ -102,7 +106,7 @@ jobs:
102106
with:
103107
python-version: "3.10"
104108
- name: Test Container Group
105-
uses: intel/ai-containers/test-runner@main
109+
uses: intel/ai-containers/test-runner@22527f085f2accb495bf4fed7475d683b4120927
106110
with:
107111
cache_registry: ${{ secrets.CACHE_REGISTRY }}
108112
recipe_dir: test-runner

.github/workflows/weekly-test.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,8 @@ jobs:
3131
allowed-endpoints: >
3232
github.com:443
3333
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
34+
with:
35+
persist-credentials: false
3436
- name: Output Group Directories
3537
id: group-list
3638
run: |

0 commit comments

Comments
 (0)