-
Notifications
You must be signed in to change notification settings - Fork 56
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
Use nvcr.io private registry to stage CI internal containers #76
base: main
Are you sure you want to change the base?
Conversation
I don't like the idea of random end user needing to register to get the containers to replicate the CI. I'm good to keep container for shorter without registration. Like only 30 days. |
Shortening the keep window to a month seems really short; especially from the perspective of someone adding a new model. They may start developing their model but then after 1.5 months, they won't be able to reproduce their numbers because the base container is gone. If we have to make the window 1 month for ghcr, my vote would be to mirror the registry on nvcr.io and have a longer keep window (maybe 6 months). |
This is what I propose, have it on both for 1 months and keep the longer storage on nvcr if we need to lower the memory usage in ghrc. |
To clarify, I am only proposing moving containers in the private Here is an example of the current container storage solution:
Two candidate solutions to use
|
and I agree with @nouiz and @terrykong that solution 2 seems a better choice. |
This is the cost of private repo storage on
So if we accumulate 10 TB of containers after 6 months, the monthly bill will be 10000 GB x $0.008/GB/day x 31days/month = $2480/month. FYI pushing and pulling the containers in a CI job using |
I'm good with both of those options. 1 seem simpler. 2 a little better in that it will speed up a little bit the jobs. But I'm not sure it will make an impact in the end. |
/assistant summarize the entire discussion |
The discussion revolves around the idea of moving containers to the Nouiz expresses concern about the inconvenience for end-users to register to access the containers to replicate the CI. He proposes to keep the containers for 30 days. Terrykong agrees with Nouiz but raises a concern about a 30-day storage limit from a developer's perspective. He suggests mirroring the registry on Nouiz then suggests keeping it on both for 1 month and using nvcr.io for longer storage if memory usage needs to be reduced at Yhtang further explains his proposal by presenting two possible approaches: 1) Using In terms of storage costs, Yhtang cites the cost of private repo storage on In the end, Nouiz is fine with both options presented by Yhtang. He suggests Option 1 is simpler, while Option 2 might speeding up jobs slightly, though he doubts it will make a substantial impact. |
Pros:
Cons:
This PR is for preview purposes and may not represent the final work to be merged.
A good compromise may be to use ghcr.io for instantaneous access in jobs, but use a periodically scheduled Actions workflow to archive containers older than a certain time to nvcr.io.
UPDATE: clarification and examples in comment below.