Skip to content

Add image pull secrets to nemodatastore CR sample and remove authSecret from NeMoDatastore Spec#362

Merged
shivamerla merged 2 commits intoNVIDIA:mainfrom
shengnuo:datastore-sample-ips
Mar 13, 2025
Merged

Add image pull secrets to nemodatastore CR sample and remove authSecret from NeMoDatastore Spec#362
shivamerla merged 2 commits intoNVIDIA:mainfrom
shengnuo:datastore-sample-ips

Conversation

@shengnuo
Copy link
Copy Markdown
Collaborator

@shengnuo shengnuo commented Mar 7, 2025

No description provided.

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Mar 7, 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.

@shengnuo shengnuo force-pushed the datastore-sample-ips branch from 2da486f to 4e06af5 Compare March 7, 2025 19:13
Comment thread config/samples/apps_v1alpha1_nemodatastore.yaml Outdated
@shengnuo shengnuo force-pushed the datastore-sample-ips branch 2 times, most recently from dca71d7 to 09cedcd Compare March 7, 2025 20:56
@shengnuo shengnuo changed the title Add image pull secrets to nemodatastore CR sample Add image pull secrets to nemodatastore CR sample and remove authSecret from NeMoDatastore Spec Mar 7, 2025
Copy link
Copy Markdown
Collaborator

@shivamerla shivamerla 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: Sheng Lin <shelin@nvidia.com>
Signed-off-by: Sheng Lin <shelin@nvidia.com>
@shivamerla shivamerla force-pushed the datastore-sample-ips branch from 09cedcd to 9746ba9 Compare March 13, 2025 16:54
@shivamerla shivamerla enabled auto-merge (rebase) March 13, 2025 16:56
@shivamerla shivamerla merged commit 311fc56 into NVIDIA:main Mar 13, 2025
9 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.

2 participants