Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update NGC versions post-22.12 release #5190

Merged
merged 4 commits into from
Dec 21, 2022
Merged

Update NGC versions post-22.12 release #5190

merged 4 commits into from
Dec 21, 2022

Conversation

mc-nv
Copy link
Contributor

@mc-nv mc-nv commented Dec 21, 2022

No description provided.

@@ -89,7 +89,7 @@ properties:
modelRepositoryPath:
type: string
title: Bucket where models are stored. Please make sure the user/service account to create the GKE app has permission to this GCS bucket. Read Triton documentation on configs and formatting details, supporting TensorRT, TensorFlow, Pytorch, Onnx ... etc.
default: gs://triton_sample_models/22_11
default: gs://triton_sample_models/22_12
Copy link
Member

Choose a reason for hiding this comment

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

Does this bucket exist on GCS?

Copy link
Contributor Author

@mc-nv mc-nv Dec 21, 2022

Choose a reason for hiding this comment

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

Most likely not, but this value is part of the schema validation, and user always can use values.yaml to override values accordingly.
Reason of changing it that if customer keep reuse same directory to store models samples he will start seeing incompatible state if that volume is shared among cluster.

@mc-nv mc-nv requested a review from Tabrizian December 21, 2022 20:50
@mc-nv mc-nv merged commit 17a7e0d into main Dec 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants