Skip to content

Commit 9cc49ff

Browse files
authored
Revert "[ML] Run allowlist validation in PyTorch edge pipeline (#2989)" (#3005)
This reverts commit a6afb0d. "[ML] Run allowlist validation in PyTorch edge pipeline (#2989)"
1 parent 47010b2 commit 9cc49ff

1 file changed

Lines changed: 1 addition & 24 deletions

File tree

.buildkite/scripts/steps/run_tests.sh

Lines changed: 1 addition & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -105,27 +105,6 @@ else
105105
-P cmake/run-all-tests-parallel.cmake || TEST_OUTCOME=$?
106106
fi
107107

108-
# --- PyTorch allowlist validation ---
109-
# When triggered from the PyTorch edge pipeline, run the Python-based
110-
# allowlist validation which traces live HuggingFace models with the
111-
# new PyTorch version and verifies every op is in ALLOWED_OPERATIONS.
112-
VALIDATION_OUTCOME=0
113-
if [[ "${GITHUB_PR_COMMENT_VAR_ACTION:-}" == "run_pytorch_tests" ]]; then
114-
echo "--- Validating PyTorch allowlist against HuggingFace models"
115-
cmake \
116-
-DSOURCE_DIR="$(pwd)" \
117-
-DVALIDATE_CONFIG="$(pwd)/dev-tools/extract_model_ops/validation_models.json" \
118-
-DVALIDATE_PT_DIR="$(pwd)/dev-tools/extract_model_ops/es_it_models" \
119-
-DVALIDATE_VERBOSE=TRUE \
120-
-P cmake/run-validation.cmake || VALIDATION_OUTCOME=$?
121-
122-
if [[ $VALIDATION_OUTCOME -ne 0 ]]; then
123-
echo "^^^ +++"
124-
echo "Allowlist validation failed — the new PyTorch version may introduce ops not in ALLOWED_OPERATIONS."
125-
echo "See dev-tools/extract_model_ops/README.md for how to update the allowlist."
126-
fi
127-
fi
128-
129108
# Upload test results
130109
echo "--- Uploading test results"
131110
TEST_RESULTS_ARCHIVE=${OS}-${HARDWARE_ARCH}-unit_test_results.tgz
@@ -138,6 +117,4 @@ else
138117
echo "No test results archive created"
139118
fi
140119

141-
if [[ $TEST_OUTCOME -ne 0 || $VALIDATION_OUTCOME -ne 0 ]]; then
142-
exit 1
143-
fi
120+
exit $TEST_OUTCOME

0 commit comments

Comments
 (0)