Skip to content

Conversation

@mikemckiernan
Copy link
Member

@mikemckiernan mikemckiernan commented Sep 30, 2024

Review HTML: https://nvidia.github.io/cloud-native-docs/review/pr-108/gpu-operator/latest/overview.html#licenses-and-contributing

I noticed that both the NGC pages for the driver container and the GDS container include "License for Customer Use of NVIDIA Software". I left it in, but I'm willing to remove it if that's preferred.

@github-actions
Copy link

Documentation preview

https://nvidia.github.io/cloud-native-docs/review/pr-108

@mikemckiernan mikemckiernan self-assigned this Sep 30, 2024
@mikemckiernan mikemckiernan added the documentation Issue/PR focused on fixing/editing/adding documentation bits label Sep 30, 2024
@mikemckiernan
Copy link
Member Author

@francisguillier, PTAL and LMK if this is what you had in mind.

@francisguillier
Copy link
Contributor

@mikemckiernan

Request from Anders was the following:

"Can you please change the license for all components listed on https://docs.nvidia.com/datacenter/cloud-native/gpu-operator/latest/index.html currently under the Deep Learning Container license to the AI Product license? Here are the URLs: https://www.nvidia.com/en-us/agreements/enterprise-software/nvidia-software-license-agreement/ and https://www.nvidia.com/en-us/agreements/enterprise-software/product-specific-terms-for-ai-products/."

So my understanding is we just need to update the current table and whenever it is mentioned "Deep Learning Container license", we replace it with "AI Product license"

Clicking on the "AI Product license" should redirect to this link:
https://www.nvidia.com/en-us/agreements/enterprise-software/product-specific-terms-for-ai-products/

Signed-off-by: Mike McKiernan <mmckiernan@nvidia.com>
Signed-off-by: Mike McKiernan <mmckiernan@nvidia.com>
@mikemckiernan mikemckiernan merged commit afac8af into NVIDIA:main Oct 4, 2024
@mikemckiernan mikemckiernan deleted the licenses branch October 4, 2024 17:52
@francisguillier
Copy link
Contributor

@mikemckiernan did you update the page?
Let me know if you need help

@mikemckiernan
Copy link
Member Author

I updated the source, but didn't republish--thinking that we could catch a handful of updates before republishing.

If #111 looks good, then I'll merge and republish. Thanks!

@francisguillier
Copy link
Contributor

yes #111 LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Issue/PR focused on fixing/editing/adding documentation bits

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants