Skip to content

Commit ceabc9b

Browse files
authored
Revert "[ML] Harden pytorch_inference with TorchScript model graph validation (#2999)" (#3006)
This reverts commit d3df09c.
1 parent 9cc49ff commit ceabc9b

44 files changed

Lines changed: 181 additions & 3449 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.buildkite/pipeline.json.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,6 @@ def main():
5252
if config.build_x86_64:
5353
pipeline_steps.append(pipeline_steps.generate_step("Upload ES tests x86_64 runner pipeline",
5454
".buildkite/pipelines/run_es_tests_x86_64.yml.sh"))
55-
pipeline_steps.append(pipeline_steps.generate_step("Upload ES inference tests x86_64 runner pipeline",
56-
".buildkite/pipelines/run_es_inference_tests_x86_64.yml.sh"))
5755
# We only use linux x86_64 builds for QA tests.
5856
if config.run_qa_tests:
5957
pipeline_steps.append(pipeline_steps.generate_step("Upload QA tests runner pipeline",

.buildkite/pipelines/run_es_inference_tests_x86_64.yml.sh

Lines changed: 0 additions & 34 deletions
This file was deleted.

.buildkite/pipelines/run_pytorch_tests.yml.sh

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,6 @@
88
# compliance with the Elastic License 2.0 and the foregoing additional
99
# limitation.
1010

11-
SAFE_MESSAGE=$(printf '%s' "${BUILDKITE_MESSAGE}" | head -1 | sed -e 's/\\/\\\\/g' -e 's/"/\\"/g')
12-
1311
cat <<EOL
1412
steps:
1513
- label: "Trigger Appex PyTorch Tests :test_tube:"
@@ -24,7 +22,7 @@ steps:
2422
- trigger: appex-qa-stateful-custom-ml-cpp-build-testing
2523
async: false
2624
build:
27-
message: "${SAFE_MESSAGE}"
25+
message: "${BUILDKITE_MESSAGE}"
2826
env:
2927
QAF_TESTS_TO_RUN: "pytorch_tests"
3028
EOL

.buildkite/pipelines/run_qa_tests.yml.sh

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,6 @@
88
# compliance with the Elastic License 2.0 and the foregoing additional
99
# limitation.
1010

11-
SAFE_MESSAGE=$(printf '%s' "${BUILDKITE_MESSAGE}" | head -1 | sed -e 's/\\/\\\\/g' -e 's/"/\\"/g')
12-
1311
cat <<EOL
1412
steps:
1513
- label: "Trigger Appex QA Tests :test_tube:"
@@ -24,7 +22,13 @@ steps:
2422
- trigger: appex-qa-stateful-custom-ml-cpp-build-testing
2523
async: false
2624
build:
27-
message: "${SAFE_MESSAGE}"
25+
message: |
26+
EOL
27+
28+
# Output the message with proper indentation for YAML literal block scalar
29+
printf '%s\n' "${BUILDKITE_MESSAGE}" | sed 's/^/ /'
30+
31+
cat <<EOL
2832
env:
2933
QAF_TESTS_TO_RUN: "${QAF_TESTS_TO_RUN:-ml_cpp_pr}"
3034
EOL

.buildkite/scripts/steps/run_es_inference_tests.sh

Lines changed: 0 additions & 31 deletions
This file was deleted.

bin/pytorch_inference/CMakeLists.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,7 @@ ml_add_executable(pytorch_inference
3535
CBufferedIStreamAdapter.cc
3636
CCmdLineParser.cc
3737
CCommandParser.cc
38-
CModelGraphValidator.cc
3938
CResultWriter.cc
40-
CSupportedOperations.cc
4139
CThreadSettings.cc
4240
)
4341

bin/pytorch_inference/CModelGraphValidator.cc

Lines changed: 0 additions & 115 deletions
This file was deleted.

bin/pytorch_inference/CModelGraphValidator.h

Lines changed: 0 additions & 91 deletions
This file was deleted.

0 commit comments

Comments
 (0)