Skip to content

test: Remove migrated initializer_as_input directory (TRI-1441) - #346

Merged
Vinya567 merged 1 commit into
mainfrom
vinyak/tri-1441-delete-migrated-test
Jun 29, 2026
Merged

test: Remove migrated initializer_as_input directory (TRI-1441)#346
Vinya567 merged 1 commit into
mainfrom
vinyak/tri-1441-delete-migrated-test

Conversation

@Vinya567

Copy link
Copy Markdown
Contributor

What does the PR do?

Removes the now-orphaned test/initializer_as_input/ directory.

The initializer_as_input ONNX test was migrated to the upstream server
repo at qa/L0_initializer_as_input/ in
triton-inference-server/server#8850 and is no
longer consumed from this repo by any CI flow. With this PR, the test/
directory becomes empty and is removed entirely.

Checklist

  • PR title reflects the change and is of format <commit_type>: <Title>
  • Changes are described in the pull request.
  • Related issues are referenced.
  • Populated github labels field
  • Added test plan and verified test passes.
  • Verified that the PR passes existing CI.
  • Verified copyright is correct on all changed files.
  • Added succinct git squash message before merging ref.
  • All template sections are filled out.
  • Optional: Additional screenshots for behavior/output changes with before/after.

Commit Type:

  • build
  • ci
  • docs
  • feat
  • fix
  • perf
  • refactor
  • revert
  • style
  • test

Related PRs:

  • test: Move initializer_as_input ONNX test into qa/ and drop cpdir server#8850 (merged) — moved the test to server/qa/L0_initializer_as_input/ + dropped the cpdir in build.py
  • GitLab dl/dgx/tritonserver !1812 (merged) — renamed CI job definitions to L0_initializer_as_input--*
  • GitLab dl/dgx/tritonserver !1814 (merged) — dropped the dead clone of this repo + the corresponding COPY in Dockerfile.QA.rhel

Where should the reviewer start?

Just confirm test/initializer_as_input/ deletion. There's no other file to look at — the directory is the entire change. After deletion, test/ itself is empty and goes away.

Test plan:

No new test cases required — this PR removes only orphaned files. The migrated test continues to run from its new home and was validated green on the GitLab pipeline as part of the prior cleanup MR.

  • CI Pipeline ID: 55977040

Caveats:

This PR is the final piece of TRI-1441 and must merge AFTER the GitLab companion MRs (!1812 + !1814). Both prerequisites are already merged, so this PR can land safely now.

Background

Part of TRI-1441 — consolidating the single ONNX QA test under the upstream server/qa/ tree so the RHEL build no longer needs a cross-repo COPY of test files from this repo. With this PR merged, the test/ directory in this repo is empty and removed; nothing inside this repo (or anywhere else in the triton-inference-server org) references test/initializer_as_input/ (verified via repo-wide and org-wide GitHub code searches: 0 matches).

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • Resolves TRI-1441

The initializer_as_input ONNX test was migrated to the upstream server
repo at qa/L0_initializer_as_input/ in
triton-inference-server/server#8850 and is no
longer consumed from this repo by any CI flow.

Companion to:
- triton-inference-server/server#8850 (merged) - upstream test move
- GitLab dl/dgx/tritonserver!1812 (merged) - yaml job rename
- GitLab dl/dgx/tritonserver!1814 (merged) - dead clone+COPY removal

With this PR, test/ becomes empty and is removed entirely. Verified
zero references exist across the triton-inference-server GitHub org.

Resolves TRI-1441
@Vinya567
Vinya567 requested review from mc-nv, whoisj and yinggeh June 29, 2026 19:19
@Vinya567
Vinya567 marked this pull request as ready for review June 29, 2026 19:19
@Vinya567
Vinya567 merged commit 00fd3e8 into main Jun 29, 2026
3 checks passed
@Vinya567
Vinya567 deleted the vinyak/tri-1441-delete-migrated-test branch June 29, 2026 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants