🌐 US-Proxy
class="logged-out env-production page-responsive" style="word-wrap: break-word;" >
Skip to content

[TRTLLMINF-250][infra] Migrate BoltProfileGen JNLP image to Artifactory - #18791

Merged
chzblych merged 1 commit into
NVIDIA:mainfrom
chzblych:fix/trtllminf-250-bolt-jnlp-image
Sep 7, 2026
Merged

[TRTLLMINF-250][infra] Migrate BoltProfileGen JNLP image to Artifactory#18791
chzblych merged 1 commit into
NVIDIA:mainfrom
chzblych:fix/trtllminf-250-bolt-jnlp-image

Conversation

@chzblych

@chzblych chzblych commented Sep 7, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Collaborator

Description

Migrate the BoltProfileGen Jenkins inbound-agent image from the legacy
registry endpoint to the standard Artifactory endpoint. The existing amd64
image variant, tag, and node selector are unchanged.

Tracking: TRTLLMINF-250.

Test Coverage

  • git diff --check
  • Static verification that jenkins/BoltProfileGen.groovy contains exactly one
    standard JNLP image reference and no legacy custom JNLP image reference
  • Verified the existing kubernetes.io/arch: amd64 selector remains present

PR Checklist

  • Please check this after reviewing the above items as appropriate for this PR.

GitHub Bot Help

To see a list of available CI bot commands, please comment /bot help.

Dev Engineer Review

  • Updated the BoltProfileGen JNLP image from urm.nvidia.com to artifactory.pdx.nvidia.com.
  • Preserved the amd64 image variant, tag, pod configuration, and kubernetes.io/arch: amd64 selector.
  • No API, performance, error-handling, or scope changes identified.
  • git diff --check and static reference checks passed.

Verdict: sufficient.

QA Engineer Review

No test changes.

@chzblych
[TRTLLMINF-250][infra] Migrate BoltProfileGen JNLP image
0a73506 
Signed-off-by: Yanchao Lu <yanchaol@nvidia.com>
@chzblych
chzblych requested a review from a team as a code owner September 7, 2026 03:06
@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: c07a31f4-6308-46b4-bce5-b4d2e71e1246

📥 Commits

Reviewing files that changed from the base of the PR and between 6e8fe90 and 0a73506.

📒 Files selected for processing (1)
  • jenkins/BoltProfileGen.groovy

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.

Walkthrough

The Kubernetes JNLP image in createKubernetesPodConfig now uses artifactory.pdx.nvidia.com instead of urm.nvidia.com. Other pod configuration remains unchanged.

Changes

Kubernetes pod configuration

Layer / File(s) Summary
Update JNLP image source
jenkins/BoltProfileGen.groovy		 createKubernetesPodConfig now uses the JNLP image from artifactory.pdx.nvidia.com.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 0a735

BoltProfileGen now pulls its JNLP agent image from the standard Artifactory endpoint while retaining the existing image variant and amd64 scheduling behavior. No current merge-blocking risk is identified.

Suggested reviewers: bowenfu

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the tracking ticket, change type, affected component, and primary migration.
Description check ✅ Passed The description explains the change, preserves relevant image details, lists validation steps, and includes the required checklist section.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@chzblych
chzblych merged commit 3a3871d into NVIDIA:main Sep 7, 2026
11 checks passed
@chzblych
chzblych deleted the fix/trtllminf-250-bolt-jnlp-image branch September 7, 2026 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ZhanruiSunCh ZhanruiSunCh Awaiting requested review from ZhanruiSunCh ZhanruiSunCh is a code owner automatically assigned from NVIDIA/trt-llm-infra-devs
@tburt-nv tburt-nv Awaiting requested review from tburt-nv tburt-nv is a code owner automatically assigned from NVIDIA/trt-llm-infra-devs

Assignees

@chzblych chzblych

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@chzblych