[CI] retire TensorRT from default CI and remove Go TensorRT test#79444
Conversation
There was a problem hiding this comment.
Pull request overview
This PR retires TensorRT from the default CI “must-pass” path to reduce coupling to fixed TensorRT environments during CUDA/Windows/image upgrades, while still allowing TensorRT to be enabled explicitly when needed.
Changes:
- Default-disable
WITH_TENSORRTin Linux CI build scripts and Windows CI workflows/scripts, removing defaultTENSORRT_ROOTwiring. - Stop installing TensorRT by default in the manylinux CUDA image install flow (keeping install helpers for optional/manual paths).
- Gate TensorRT-related test scheduling in CMake by
WITH_TENSORRTto avoid entering TRT test sets when TRT is not enabled.
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tools/dockerfile/manylinux/common/install_cuda.sh | Remove default TensorRT install steps from CUDA install functions. |
| test/tensorrt/CMakeLists.txt | Gate TensorRT Python tests on WITH_TENSORRT as well as TENSORRT_FOUND. |
| test/ir/inference/CMakeLists.txt | Add WITH_TENSORRT condition around TensorRT-related IR pass tests. |
| test/cpp/inference/api/CMakeLists.txt | Add WITH_TENSORRT condition around TensorRT inference API test assets/properties. |
| test/cpp/fluid/CMakeLists.txt | Gate adding TensorRT C++ fluid test subdir on WITH_TENSORRT. |
| ci/windows/test_fluid_library.bat | Default WITH_TENSORRT=OFF and pass TENSORRT_ROOT safely as an argument. |
| ci/windows/config_env.bat | Default WITH_TENSORRT=OFF and stop exporting a default TENSORRT_ROOT. |
| ci/windows/build.bat | Only prepend TensorRT to PATH when explicitly enabled and TENSORRT_ROOT is set. |
| ci/utils.sh | Switch CI’s default CMake flag -DWITH_TENSORRT to OFF and default demo-ci TRT flag OFF. |
| ci/run_setup.sh | Switch printed/exported default WITH_TENSORRT to OFF. |
| ci/inference_build.sh | Switch inference build default WITH_TENSORRT to OFF (still overridable). |
| .github/workflows/_Windows-Inference.yml | Remove default TENSORRT_ROOT and set WITH_TENSORRT=OFF. |
| .github/workflows/_Windows-GPU.yml | Remove default TENSORRT_ROOT and set WITH_TENSORRT=OFF. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| CUDNN_VERSION=9.1.1.17 | ||
| NCCL=2.21.5 | ||
| TensorRT_VERSION=10.5 | ||
| echo "Installing CUDA 12.4.1 and cuDNN ${CUDNN_VERSION} and NCCL ${NCCL_VERSION} and TensorRT ${TensorRT_VERSION} and cuSparseLt-0.6.2" | ||
| echo "Installing CUDA 12.4.1 and cuDNN ${CUDNN_VERSION} and NCCL ${NCCL_VERSION} and cuSparseLt-0.6.2" |
risemeup1111
left a comment
risemeup1111
left a comment
There was a problem hiding this comment.
发现一处需要修复后再合入的问题，细节已放在行级评论中。当前已完成的检查项为成功，仍有部分构建测试在运行中。
| if(WITH_GPU | ||
| AND WITH_TENSORRT | ||
| AND TENSORRT_FOUND) |
There was a problem hiding this comment.
问题： 默认关闭 WITH_TENSORRT 后，这个文件里非 TensorRT 专属的 inference IR 测试仍会被上面的 if(WITH_ONEDNN AND TENSORRT_FOUND AND WITH_GPU) 卡住。cmake/tensorrt.cmake 在 NOT WITH_GPU OR NOT WITH_TENSORRT 时会直接 return()，所以新默认 WITH_TENSORRT=OFF 下 TENSORRT_FOUND 不会被置为 ON，即使机器上安装了 TensorRT，TEST_INFERENCE_IR_PASSES 也不会注册。
影响： GPU+oneDNN 默认 CI 会静默丢掉一批非 TRT 覆盖（例如 test_inference_predictor_run*、test_layer_norm_fuse_pass、test_yolo_box_post 等），与本 PR 只退场 TensorRT 专属测试的目标不一致。
处理要求：请针对该评论修复并提交新的 commit。
建议： 把通用 inference IR 测试注册条件与 TensorRT 解耦，同时保留真正依赖 TensorRT 的测试在 WITH_TENSORRT AND TENSORRT_FOUND 下运行。可以显式拆出/维护一份 TRT 依赖列表，再只对剩余通用列表使用 WITH_ONEDNN AND WITH_GPU：
set(TEST_TRT_DEPENDENT_IR_PASSES
test_adaptive_pool2d_convert_global_pass_autoscan
test_conv_bn_fuse_pass
test_conv_elementwise_add_fuse_pass
# ... include the non-test_trt_* tests that call create_trt_inference_config,
# enable_tensorrt_engine, or other TRT-only APIs.
)
foreach(TEST_TRT_DEPENDENT_IR_PASS ${TEST_TRT_DEPENDENT_IR_PASSES})
list(REMOVE_ITEM TEST_INFERENCE_IR_PASSES ${TEST_TRT_DEPENDENT_IR_PASS})
endforeach()
if(WITH_ONEDNN AND WITH_GPU)
foreach(target ${TEST_INFERENCE_IR_PASSES})
py_test_modules(${target} MODULES ${target})
set_tests_properties(${target} PROPERTIES LABELS "RUN_TYPE=INFER")
endforeach()
endif()
if(WITH_GPU AND WITH_TENSORRT AND TENSORRT_FOUND)
foreach(target ${TEST_TRT_DEPENDENT_IR_PASSES})
py_test_modules(${target} MODULES ${target})
set_tests_properties(${target} PROPERTIES LABELS "RUN_TYPE=INFER")
endforeach()
endif()There was a problem hiding this comment.
补充当前 head 4c836cc8978620fd34c681c3936a3cad11ef277e 的状态：上一版对这处的修复被回退了。当前 test/ir/inference/CMakeLists.txt 又在 WITH_ONEDNN AND TENSORRT_FOUND AND WITH_GPU 下注册 TEST_INFERENCE_IR_PASSES；而 cmake/tensorrt.cmake 在 WITH_TENSORRT=OFF 时会直接 return()，不会设置 TENSORRT_FOUND。因此默认无 TensorRT 构建仍会漏注册通用 inference IR 测试，原问题重新出现。
处理要求：请针对该评论修复并提交新的 commit。
建议仍是把通用 inference IR 测试与 TensorRT 探测解耦，只让真正依赖 TensorRT 的测试继续受 WITH_TENSORRT AND TENSORRT_FOUND 保护：
if(WITH_ONEDNN AND WITH_GPU)
foreach(target ${TEST_INFERENCE_IR_PASSES})
py_test_modules(${target} MODULES ${target})
set_tests_properties(${target} PROPERTIES LABELS "RUN_TYPE=INFER")
endforeach()
endif()
if(WITH_GPU AND WITH_TENSORRT AND TENSORRT_FOUND)
# register TRT-only tests here
endif()
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
CI报告基于以下代码生成（30分钟更新一次）: 1 Required任务 : 49/50 通过
2 失败详情🔴 CE-Framework / Infer — PR问题（置信度: 高）失败用例: TRT Python API 推理用例
关键日志:
修复建议:
关联变更: |
|
@ShigureNyako 看看 ci 咋回事，重试那么多次都不行 |
|
我看了一下当前 check rollup 和
建议下一步先展开 |
|
补充：我拿到完整
对应命令在 建议不要继续只重试 failed build job；需要重新跑包含 |
|
@ShigureNyako review 一下，看看还有没有哪没清理到的 |
ShigureNyako
left a comment
ShigureNyako
left a comment
There was a problem hiding this comment.
结论：当前 head 51d322e763153a2e90cbb66c0d76cfe21aa7409e 仍需修改。
Linux/Windows 默认构建参数、Go API 门控、TensorRT 专属 CMake 目录以及 manylinux 安装流程的调整整体与“从默认 CI 退场 TensorRT”的目标一致；但现有的 inference IR 测试门控问题 仍未解决，因此这里不重复新开行级评论：
test/ir/inference/CMakeLists.txt:166-173仍要求WITH_ONEDNN AND TENSORRT_FOUND AND WITH_GPU才注册通用TEST_INFERENCE_IR_PASSES。cmake/tensorrt.cmake:1-3在新默认WITH_TENSORRT=OFF时直接返回，TENSORRT_FOUND不会置为真；于是test_inference_predictor_run、test_layer_norm_fuse_pass、test_yolo_box_post等非 TRT 专属测试会被静默漏注册。CI 全绿无法发现这种测试集合缩减。
需要把通用 inference IR 测试注册与 TENSORRT_FOUND 解耦，并仅对实际依赖 TensorRT 的测试保留 WITH_TENSORRT AND TENSORRT_FOUND 门控。
本地聚焦验证：改动脚本 bash -n、三个 workflow 的 YAML 解析、Go 文件 gofmt、git diff-tree --check 均通过；另用 cmake -P 验证了 WITH_TENSORRT=OFF 时上述通用注册条件确实为 false。当前 check rollup 为 83/83 success。
SigureMo
left a comment
SigureMo
left a comment
There was a problem hiding this comment.
这个 PR 是把所有流水线的 tensorrt 测试都跳过了吗？
| docker exec -t ${{ env.container_name }} /bin/bash -c ' | ||
| export PATH=$(pwd)/run_env:${PATH} | ||
| cd /PaddleTest/inference/python_api_test | ||
| find . -name run_parallel.sh -exec sed -i "/_trt_/d" {} + |
There was a problem hiding this comment.
这是什么行为？是将 trt 单测排除出去了吗？这些单测具体是在哪里？PaddleTest repo 还是？如果确定不支持了，是否可以直接移除源码而不是通过 hack 跳过？否则的话，就算 CI 能通过，CE 回归仍然会挂掉
There was a problem hiding this comment.
我核对了这行的实际作用范围：这里处理的是运行时下载的 PaddleTest 内容，不是 Paddle 仓内单测源码；在当前 PaddleTest develop（793ccafd）中，find 只会命中 test_class_model/run_parallel.sh 和 test_det_model/run_parallel.sh，分别删掉 5 个和 4 个 _trt_ case。
所以这不是“所有流水线统一跳过 TRT”，而是仅在 _CE-Framework.yml 这个 job 内临时改写 9 个 case 的清单。PaddleTest 的其他入口（例如 test_class_model/run.sh、test_nlp_model/run.sh）仍列有 TRT case；任何未执行这条 sed 的 CE/回归入口，在使用本 PR 默认生成的无 TRT wheel 时仍可能失败。
这与此前 Paddle 仓内通用 inference IR 测试被漏注册的问题是独立的 CE 测试选择问题。这里需要把 skip/选择规则落实到 PaddleTest 的共享入口或显式配置中，并让 CI 与 CE 回归复用，而不是只在单个 workflow 中运行时改写下载产物。
There was a problem hiding this comment.
问题： 这次提交删除了 _CE-Framework.yml 里运行前过滤 _trt_ 用例的命令，但当前 workflow 仍安装由默认 WITH_TENSORRT=OFF 构建出的 wheel，并直接执行 /PaddleTest/inference/python_api_test/parallel_run.sh 2。我核对了该 workflow 下载的 https://xly-devops.bj.bcebos.com/PaddleTest/PaddleTest.tar.gz：其中 test_class_model/run_parallel.sh 仍包含 5 个 _trt_ case，test_det_model/run_parallel.sh 仍包含 4 个 _trt_ case。
影响： 默认无 TensorRT 的 wheel 会继续在 CE Infer 入口执行 TRT 模型用例，CI/CE 回归仍可能因为缺少 TensorRT 支持失败；这与本 PR “默认 CI 退场 TensorRT”的目标还没有闭环。
处理要求：请针对该评论修复并提交新的 commit。
建议： 合入前需要让 PaddleTest 的共享入口/下载产物支持在无 TensorRT wheel 下跳过 TRT case，并让 _CE-Framework.yml 使用这个共享机制；或者在本 PR 中保留一个显式且可复用的跳过配置，直到 PaddleTest 侧更新完成。修复形态可以类似下面这样，而不是依赖后续人工更新：
# _CE-Framework.yml 调用 PaddleTest 前显式声明当前 wheel 不含 TensorRT
PADDLETEST_SKIP_TRT=1 bash parallel_run.sh 2
# PaddleTest 的 run_parallel.sh / shared entrypoint 中统一处理
if [ "${PADDLETEST_SKIP_TRT:-0}" = "1" ]; then
cases=$(printf "%s\n" "${cases}" | grep -v '_trt_')
fiThere was a problem hiding this comment.
当前 head 的 CE-Framework / Infer 已直接验证本线程的风险，并非基础设施故障：安装默认无 TensorRT 的 wheel 后，PaddleTest 两个 run_parallel.sh 中的 9 个 TRT case 全部失败，日志统一报：
PreconditionNotMetError: To use Paddle-TensorRT, please compile with TENSORRT first.
最终清单正好是 class 侧 5 个、det 侧 4 个，job 以 exit code 8 结束；同一清单中的 GPU/oneDNN 非 TRT case 可正常通过。需要先让 CE 使用的 PaddleTest 共享入口在无 TRT wheel 下跳过这些 case，再运行 CI 验证。
gouzil
changed the title
gouzil
changed the title
risemeup1111
left a comment
risemeup1111
left a comment
There was a problem hiding this comment.
已复查当前提交：此前 Nyanpasu 提到的 test/ir/inference/CMakeLists.txt 通用 inference IR 测试门控问题已经修复。不过 CE/PaddleTest 这条路径仍有一个需要提交新 commit 处理的阻塞问题，细节已补充在对应的行级讨论中。
当前检查还有任务在运行中，建议在修复后再观察完整 CI 结果。
risemeup1111
left a comment
risemeup1111
left a comment
There was a problem hiding this comment.
已复查最新提交。test/ir/inference/CMakeLists.txt 的通用 inference IR 测试门控问题在当前 head 又恢复到依赖 TENSORRT_FOUND 的状态，我已在原行级讨论里补充了当前证据；此前 CE/PaddleTest 的无 TensorRT wheel 仍运行 TRT case 的问题也仍由原讨论覆盖。
当前仍需要提交新的 commit 处理这些 P1 阻塞项后再复查。
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
ShigureNyako
left a comment
ShigureNyako
left a comment
There was a problem hiding this comment.
结论：当前 head 4c836cc8978620fd34c681c3936a3cad11ef277e 仍需修改。
此前两个 P1 blocker 在原线程已有针对当前 head 的证据，这里不重复展开：
- 通用 inference IR 注册仍依赖
TENSORRT_FOUND；中间提交的拆分修复已在最新提交中回退。 - CE/PaddleTest 仍会对默认无 TRT wheel 执行 TRT case；workflow 内临时过滤也已删除，PaddleTest 共享入口尚未更新。
本轮另发现一个独立阻塞问题并已行级标注：无 TRT 时用 GOFLAGS 跳过整个 TestNewConfig，会同时丢掉大量非 TRT Go Config API 覆盖。应只门控 TRT 调用/独立 TRT 测试，保留通用 Config 测试。
聚焦静态验证通过：改动 shell 脚本 bash -n、Windows workflow YAML 解析、git diff-tree --check。CI 仍在运行，不能以当前部分成功替代上述测试选择审查。
| if [[ "${WITH_TENSORRT:-OFF}" == "ON" ]]; then | ||
| bash test.sh | ||
| else | ||
| GOFLAGS='-run=^(TestOnednn|TestONNXRuntime)$' bash test.sh |
There was a problem hiding this comment.
TestNewConfig 只有 config_test.go:40-63 是 TensorRT 调用；其余还覆盖 SetProgFile/SetParamsFile、GPU 配置、IR 优化、多流、内存优化、Profile、Pass 管理和 Summary 等通用接口。当前 GOFLAGS 在 WITH_TENSORRT=OFF 时只选择另外两个测试，导致这些通用行为全部不再执行，而不只是退场 TRT 覆盖。
请把 TRT 调用拆成独立测试或在测试内按构建能力门控，并继续无条件运行通用 TestNewConfig；不要通过过滤整个测试来绕过中间的 TRT 调用。
There was a problem hiding this comment.
已按这个方向修改：删除 TestNewConfig 中的 TensorRT 专属调用，恢复 CI 无条件执行完整 Go 测试。通用 Config API 覆盖不再被过滤。提交：81946d02d5。
There was a problem hiding this comment.
我去啥时候偷摸用我账号做的回复。go 的这个 TensorRT 单测直接删掉了
There was a problem hiding this comment.
我这里只依据 GitHub 返回的平台元数据判断身份，也没有权限使用你的账号。此前 “已按这个方向修改…” 这条回复及其 COMMENTED review，API 都记录为 user.login=gouzil、提交时间 2026-07-11T17:47:19Z。平台元数据无法说明背后的实际操作者；既然你明确否认，这应视为账号或自动化来源异常，不能再把那条回复当作你的已确认说明。
技术结论不依赖该回复：当前 head 的 diff 确实直接删除了 Go TensorRT 测试段，这正是 现有 blocker 的证据，而不是对 blocker 的解决。PR 仍声明 default-off 且保留显式 TRT 支持路径，因此此前的 范围澄清 仍然适用。
There was a problem hiding this comment.
明白，是我误解了上一句的指代：你说的是绑定在 gouzil 账号下的 @codex 自动回复，不是在说 @ShigureNyako 使用了他的账号。上条关于“我没有权限使用账号”和账号异常的展开不适用，我撤回这部分推断；只能保留 GitHub API 将该评论归属为 gouzil 这一平台记录。
这项更正不改变代码审查结论：当前 head 仍是直接删除 Go TRT 测试，且没有新增代码来处理另外两个 blocker。
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
ShigureNyako
left a comment
ShigureNyako
left a comment
There was a problem hiding this comment.
人工复查结论：当前 head 81946d02d5d71b2dbb9ff92a3e3b148479a8968e 仍需修改。
本次提交恢复了无 TRT 构建对完整 TestNewConfig 的执行，解决了“过滤整个 Go 通用测试”的问题；但修复方式改为永久删除全部 Go TensorRT Config API 检查，导致显式 WITH_TENSORRT=ON 的专项路径也失去这些覆盖。我已在删除处新增行级意见，要求拆分/门控 TRT 专属测试，而不是删除覆盖。
另外两个既有问题在当前 head 未发生变化，不重复开新线程：
- 通用 inference IR 注册仍依赖
TENSORRT_FOUND。 - CE/PaddleTest 仍会用默认无 TRT wheel 执行 9 个 TRT case；当前 PaddleTest
develop和动态 tarball 清单仍未更新。
聚焦静态验证通过：shell bash -n、Windows workflow YAML 解析、Go gofmt、git diff-tree --check。CI 仍在运行，当前未见失败。
| t.Logf("use_gpu:%+v, gpu_id:%+v", config.UseGpu(), config.GpuDeviceId()) | ||
| t.Logf("MemoryPoolInitSizeMb:%+v, FractionOfGpuMemoryForPool:%+v", config.MemoryPoolInitSizeMb(), config.FractionOfGpuMemoryForPool()) | ||
|
|
||
| config.EnableTensorRtEngine(1024, 16, 3, PrecisionFloat32, false, false) |
There was a problem hiding this comment.
这次修改恢复了通用 TestNewConfig 覆盖，但把 Go Config 的 TensorRT API 检查从测试源码中永久删除了；因此即使专项构建显式设置 WITH_TENSORRT=ON，EnableTensorRtEngine、动态 shape、OSS、DLA 和禁用算子等接口也不再被 Go 测试执行。
本 PR 的范围是让 TensorRT 退出默认 CI，同时说明仍保留显式开启路径。请将这段拆成独立的 TensorRT 测试并仅在 WITH_TENSORRT=ON 时选择执行，或在测试内按构建能力门控；不应通过删除测试覆盖来修复默认无 TRT 构建。
There was a problem hiding this comment.
如果我的最终目的是完全删除 TensorRT, 那我觉得为了 ci 通过是可以这么干的
There was a problem hiding this comment.
如果范围改成“完整删除 TensorRT”，那么对应功能/API 测试当然可以随实现一起删除；但当前 PR 还不是这个状态：标题和描述仍是退出默认 CI，明确保留 WITH_TENSORRT=ON 手动路径与 install_trt_* helper，diff 也没有删除 TensorRT 实现、CMake option、Go/C API 或用户接口。
因此这里只删除 Go API 检查，会形成“功能和显式构建路径仍在，但测试先消失”的中间状态，不能视为完整删除。若确实决定完整移除，需要在同一套可验证变更中协调处理实现/构建选项/API 与兼容性、文档、PaddleTest/CE 清单；当前 CE 失败也证明仅移除默认依赖但保留 TRT case 无法闭环。
另外，完整移除的意图也不会消除通用 inference IR blocker：那些非 TRT 测试必须先从 TENSORRT_FOUND 解耦，否则 TensorRT 删除后它们会永久漏注册。
There was a problem hiding this comment.
位置就在这个行级线程：当前 diff 删除的是 TestNewConfig 里 EnableTensorRtEngine、SetTRTDynamicShapeInfo、EnableTensorRtDLA、DisableTensorRtOPs 这一整段 Go Config API 检查。
我认可“默认 CI 不再支持/不再必测 TensorRT”这个方向；分歧点是当前 PR 还没有完整移除 TensorRT。PR 描述仍写着 Linux CI 保留手动 WITH_TENSORRT=ON 路径，manylinux 也保留 install_trt_* helper；当前代码里 Go API 的 EnableTensorRtEngine、SetTRTDynamicShapeInfo、DisableTensorRtOPs、EnableTensorRtDLA 也还存在。因此这里只删 Go 测试会变成“API 和显式构建入口还在，但对应 Go 覆盖先消失”。
如果项目结论是 Paddle 后续完全不支持 TensorRT，那这条可以通过完整移除 TensorRT 实现/API/构建入口/文档/下游测试清单来解决；如果本 PR 仍是退出默认 CI，那建议把这段 Go TRT 测试拆成独立测试并只在显式 WITH_TENSORRT=ON 的专项路径运行。
There was a problem hiding this comment.
我之前保留这项 blocker 的原因是：当前改动只是把默认路径切成 WITH_TENSORRT=OFF，显式 WITH_TENSORRT=ON 和 Go TRT API 仍存在，而这 25 行删除会让 engine、dynamic shape、OSS/varseqlen、DLA、disabled-op 在显式 TRT 路径也失去 Go 覆盖；按这个理解，处理方式应是拆分测试或仅在显式 TRT 构建下门控。
不过你现在已经明确确认“删除这段 Go TRT 覆盖”本身属于本 PR 可接受范围，这是项目范围决定。我撤回这一项 blocker，不再要求本 PR 拆分/门控。通用 inference-IR 注册仍依赖 TENSORRT_FOUND 是另一项独立问题，不受这里的范围结论影响。
There was a problem hiding this comment.
因此这里只删 Go 测试会变成“API 和显式构建入口还在，但对应 Go 覆盖先消失”。
没问题，这不超过范围，清理符合预期 @risemeup1111
There was a problem hiding this comment.
收到。既然已经明确这是项目预期范围，我撤回这项 Go TensorRT 覆盖 blocker，不再要求本 PR 拆分或门控这段 Go 测试。
PaddlePaddle-bot
left a comment
PaddlePaddle-bot
left a comment
There was a problem hiding this comment.
🤖 Paddle-CI-Agent | pr_review |
2026-07-12 02:16:52
📋 Review 摘要
PR 概述：将默认 CI、镜像和 Windows workflow 的 TensorRT 构建/测试依赖切到默认关闭，保留显式开启路径。
变更范围：Windows GPU/Inference workflows、CI build scripts、manylinux CUDA image install script、Go inference config test。
影响面 Tag：Inference Environment Adaptation
问题
未发现阻塞性问题。PR 规范问题在下面章节报,不要在这里重复
📝 PR 规范检查
标题当前使用 [CI]，不在 Paddle PR 模板的官方 Tag 列表中；描述结构和“是否引起精度变化”字段符合规范。
标题建议（可直接复制）：
[Environment Adaptation] retire TensorRT from default CI总体评价
本轮按风险优先覆盖了 A8 Inference/TensorRT 和 A10 Build/CI 相关路径，检查了默认关闭后的 CMake 参数、Windows PATH 注入、demo_ci/infer_ut 门禁、manylinux 安装函数和 Go API 测试范围。未发现需要阻塞合入的代码问题；只读核验中 bash -n、gofmt -d、git diff --check 通过，当前环境缺少 ruby，未能复跑 workflow YAML 解析。
gouzil
changed the title
|
你的PR提交成功，感谢你对开源项目的贡献! |
|
@risemeup1111 大哥，你是大哥，你怎么不来 review 了啊，不 review 这 PR 合入不了啊 |
risemeup1111
left a comment
risemeup1111
left a comment
There was a problem hiding this comment.
已复查当前 head。之前指出的 Go TensorRT 覆盖删除问题仍未修复，细节仍在现有行级评论中；本轮没有新增 inline comment。请先补新的 commit 处理该阻塞项后再复查。
@risemeup1111 在哪？而且不是说了不支持 TensorRT 了吗？Go 的测试删了咋了？ |
risemeup1111
left a comment
risemeup1111
left a comment
There was a problem hiding this comment.
已复查当前 head。关于 Go TensorRT 测试删除，已按线程中的范围确认撤回此前 blocker；PaddleTest 侧 TensorRT 清理已合入，当前 CE-Framework/Infer 也已通过。本轮没有新增 inline comment，未发现需要继续阻塞合入的问题。
PR Category
Environment Adaptation
PR Types
Improvements
Description
本 PR 将 TensorRT 从默认 CI 必测链路中退场，避免 CUDA、Windows 和镜像升级继续依赖固定 TensorRT 环境。
主要改动：
WITH_TENSORRT，并移除默认TENSORRT_ROOT配置。WITH_TENSORRT=OFF，仍保留手动显式传入WITH_TENSORRT=ON的路径。install_trt_xxx，但保留这些安装函数供手动或专项路径复用。TestNewConfig删除 TensorRT engine、dynamic shape、OSS、DLA 和禁用算子检查；通用 Config API 测试仍由 CI 完整执行。本 PR 不修改 Paddle 功能源码或 CMake 测试注册。PaddleTest 中 TRT case 的删除不在本 PR 处理，后续在 PaddleTest 仓库统一调整。
验证：
prekgofmt -d paddle/fluid/inference/goapi/config_test.gobash -n ci/inference_build.sh ci/run_setup.sh ci/utils.sh tools/dockerfile/manylinux/common/install_cuda.shruby -e 'require \"yaml\"; ARGV.each { |f| YAML.load_file(f) }' .github/workflows/_Windows-GPU.yml .github/workflows/_Windows-Inference.ymlgit diff --checkopenspec validate retire-tensorrt-ci --strict是否引起精度变化
否。本 PR 仅调整 CI 默认构建、镜像安装和 TensorRT 测试覆盖，不修改 Paddle TensorRT 功能实现或模型计算逻辑。