Skip to content

Publishing and NSPECT Pipeline Changes#53

Merged
JunAr7112 merged 1 commit into
NVIDIA:mainfrom
JunAr7112:ngc_pipeline_changes
Apr 15, 2026
Merged

Publishing and NSPECT Pipeline Changes#53
JunAr7112 merged 1 commit into
NVIDIA:mainfrom
JunAr7112:ngc_pipeline_changes

Conversation

@JunAr7112
Copy link
Copy Markdown
Contributor

@JunAr7112 JunAr7112 commented Nov 20, 2025

This MR will add a Publishing/NSPECT jobs to the nvidia yaml for publishing. We will get rid of the current release job, and instead use the images that are pushed to staging for publishing.

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Nov 20, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Copy link
Copy Markdown
Contributor

@tariq1890 tariq1890 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tariq1890
Copy link
Copy Markdown
Contributor

Thanks @JunAr7112! Let's merge this after either one of @cdesiniotis and @guptaNswati 's approvals

Copy link
Copy Markdown
Member

@elezar elezar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a comment. Does it make sense to switch to a single .nvidia-ci.yml file? (This does not have to block this PR thoug)

@JunAr7112
Copy link
Copy Markdown
Contributor Author

As a comment. Does it make sense to switch to a single .nvidia-ci.yml file? (This does not have to block this PR thoug)

I was thinking to keep the nvidia-ci.yml and common-ci.yml separate just to maintain the same format as the other projects.

Copy link
Copy Markdown

@rahulait rahulait left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see some issues with CI jobs as discussed internally. Lets get them addressed before merging this.

@JunAr7112 JunAr7112 force-pushed the ngc_pipeline_changes branch 2 times, most recently from 105479b to d571e42 Compare April 10, 2026 20:31
@JunAr7112
Copy link
Copy Markdown
Contributor Author

I see some issues with CI jobs as discussed internally. Lets get them addressed before merging this.

I updated this MR to resolve these issues.

@JunAr7112 JunAr7112 force-pushed the ngc_pipeline_changes branch from d571e42 to 0a07c2b Compare April 10, 2026 20:46
Copy link
Copy Markdown

@rahulait rahulait left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Signed-off-by: Arjun Gadiyar <agadiyar@nvidia.com>
@JunAr7112 JunAr7112 force-pushed the ngc_pipeline_changes branch from 0a07c2b to e57cf67 Compare April 15, 2026 18:38
@JunAr7112 JunAr7112 merged commit b80c8fa into NVIDIA:main Apr 15, 2026
2 checks passed
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.

5 participants