Skip to content

Commit 7b72d7b

Browse files
committed
fix for workflow ssh issue and PR comments fix
Signed-off-by: Sukhobochenko, Anton <anton.sukhobochenko@intel.com>
1 parent ed2d2fb commit 7b72d7b

2 files changed

Lines changed: 18 additions & 26 deletions

File tree

.github/workflows/job_npu_tests.yml

Lines changed: 18 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,6 @@ on:
1717
type: string
1818
required: false
1919
default: null
20-
test-device:
21-
description: 'OpenVINO device to test against'
22-
type: string
23-
required: false
24-
default: 'NPU'
2520
npu-platforms:
2621
description: 'JSON list of NPU platforms to compile for (offline), each runs on its own runner. e.g. ["3720","4000","5010","5020"]'
2722
type: string
@@ -57,7 +52,7 @@ jobs:
5752
INSTALL_DIR: "${{ github.workspace }}\\install"
5853
INSTALL_TEST_DIR: "${{ github.workspace }}\\install\\tests"
5954
INSTALL_WHEELS_DIR: "${{ github.workspace }}\\install\\wheels"
60-
TEST_DEVICE: ${{ inputs.test-device }}
55+
TEST_DEVICE: 'NPU'
6156
NPU_PLATFORM: ${{ matrix.npu-platform }}
6257
MODEL_SCOPE: ${{ inputs.model-scope }}
6358
steps:
@@ -104,12 +99,13 @@ jobs:
10499
wheels-dir-path: ${{ env.INSTALL_WHEELS_DIR }}
105100
wheels-to-install: 'openvino'
106101

107-
- name: Set SSL_CERT_FILE for model downloading
108-
run: echo SSL_CERT_FILE=$(python3 -m certifi) >> $env:GITHUB_ENV
109-
110102
- name: Install TF Models tests requirements
111103
run: python3 -m pip install -r ${{ env.INSTALL_TEST_DIR }}\requirements_tensorflow
112104

105+
# certifi is pulled in by the requirements above, so this must run after the install.
106+
- name: Set SSL_CERT_FILE for model downloading
107+
run: echo SSL_CERT_FILE=$(python3 -m certifi) >> $env:GITHUB_ENV
108+
113109
- name: TensorFlow Models Tests - NPU compile-only (convert_model)
114110
shell: cmd
115111
run: |
@@ -128,7 +124,6 @@ jobs:
128124
set PYTHONPATH=${{ env.INSTALL_TEST_DIR }}\model_hub_tests;%PYTHONPATH%
129125
python3 -m pytest ${{ env.INSTALL_TEST_DIR }}\model_hub_tests\tensorflow\test_tf_read_model.py -m "%MODEL_SCOPE%" ^
130126
--html=${{ env.INSTALL_TEST_DIR }}\TEST-tf_fe_npu_read_model_%NPU_PLATFORM%_%MODEL_SCOPE%.html --self-contained-html -v -s
131-
132127
# To debug locally/manually, uncomment:
133128
# env:
134129
# OV_NPU_LOG_LEVEL: LOG_INFO
@@ -159,7 +154,7 @@ jobs:
159154
INSTALL_DIR: "${{ github.workspace }}\\install"
160155
INSTALL_TEST_DIR: "${{ github.workspace }}\\install\\tests"
161156
INSTALL_WHEELS_DIR: "${{ github.workspace }}\\install\\wheels"
162-
TEST_DEVICE: ${{ inputs.test-device }}
157+
TEST_DEVICE: 'NPU'
163158
NPU_PLATFORM: ${{ matrix.npu-platform }}
164159
MODEL_SCOPE: ${{ inputs.model-scope }}
165160
steps:
@@ -206,20 +201,22 @@ jobs:
206201
wheels-dir-path: ${{ env.INSTALL_WHEELS_DIR }}
207202
wheels-to-install: 'openvino'
208203

209-
- name: Set SSL_CERT_FILE for model downloading
210-
run: echo SSL_CERT_FILE=$(python3 -m certifi) >> $env:GITHUB_ENV
211-
212204
- name: Install JAX Models tests requirements
213205
run: python3 -m pip install -r ${{ env.INSTALL_TEST_DIR }}\requirements_jax
214206

207+
# certifi is pulled in by the requirements above, so this must run after the install.
208+
- name: Set SSL_CERT_FILE for model downloading
209+
run: echo SSL_CERT_FILE=$(python3 -m certifi) >> $env:GITHUB_ENV
210+
215211
- name: JAX/Flax Models Tests - NPU compile-only
216212
shell: cmd
217213
run: |
218214
set PYTHONPATH=${{ env.INSTALL_TEST_DIR }}\model_hub_tests;%PYTHONPATH%
219215
python3 -m pytest ${{ env.INSTALL_TEST_DIR }}\model_hub_tests\jax\test_hf_transformers.py -m "%MODEL_SCOPE%" ^
220216
--html=${{ env.INSTALL_TEST_DIR }}\TEST-jax_npu_model_%NPU_PLATFORM%_%MODEL_SCOPE%.html --self-contained-html -v -s
221-
222-
# To debug locally/manually, uncomment:
217+
# TODO: NPU plugin logging is disabled for now (LOG_INFO is very verbose).
218+
# Pending decision with the pipeline owners on how to expose it (input param
219+
# vs. leave off). To debug locally/manually, uncomment:
223220
# env:
224221
# OV_NPU_LOG_LEVEL: LOG_INFO
225222

@@ -251,7 +248,7 @@ jobs:
251248
INSTALL_DIR: "${{ github.workspace }}\\install"
252249
INSTALL_TEST_DIR: "${{ github.workspace }}\\install\\tests"
253250
INSTALL_WHEELS_DIR: "${{ github.workspace }}\\install\\wheels"
254-
TEST_DEVICE: ${{ inputs.test-device }}
251+
TEST_DEVICE: 'NPU'
255252
NPU_PLATFORM: ${{ matrix.npu-platform }}
256253
MODEL_SCOPE: ${{ inputs.model-scope }}
257254
steps:
@@ -298,9 +295,6 @@ jobs:
298295
wheels-dir-path: ${{ env.INSTALL_WHEELS_DIR }}
299296
wheels-to-install: 'openvino'
300297

301-
- name: Set SSL_CERT_FILE for model downloading
302-
run: echo SSL_CERT_FILE=$(python3 -m certifi) >> $env:GITHUB_ENV
303-
304298
# Group A: torchvision + timm + edsr share compatible deps (installed together).
305299
- name: "Install PyTorch tests requirements (group A: torchvision/timm/edsr)"
306300
run: |
@@ -309,6 +303,10 @@ jobs:
309303
-r ${{ env.INSTALL_TEST_DIR }}\model_hub_tests\pytorch\envs\timm.txt `
310304
-r ${{ env.INSTALL_TEST_DIR }}\model_hub_tests\pytorch\envs\edsr.txt
311305
306+
# certifi is pulled in by the requirements above, so this must run after the install.
307+
- name: Set SSL_CERT_FILE for model downloading
308+
run: echo SSL_CERT_FILE=$(python3 -m certifi) >> $env:GITHUB_ENV
309+
312310
- name: PyTorch Models Tests - NPU compile-only (group A)
313311
shell: cmd
314312
run: |

.github/workflows/windows_vs2022_npu.yml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,6 @@ on:
99
description: 'Target branch for the build; taken from event context by default'
1010
type: string
1111
required: false
12-
test-device:
13-
description: 'OpenVINO device to test against'
14-
type: string
15-
required: false
16-
default: 'NPU'
1712
npu-platforms:
1813
description: 'JSON list of NPU platforms to compile for (offline), each on its own runner. e.g. ["3720","4000","5010","5020"]'
1914
type: string
@@ -106,7 +101,6 @@ jobs:
106101
with:
107102
runner: 'aks-win-8-cores-16gb-test'
108103
affected-components: ${{ needs.smart_ci.outputs.affected_components }}
109-
test-device: ${{ inputs.test-device || 'NPU' }}
110104
npu-platforms: ${{ inputs.npu-platforms || '["3720","4000","5010","5020"]' }}
111105
model-scope: ${{ inputs.model-scope || 'precommit' }}
112106

0 commit comments

Comments
 (0)