Skip to content

ci: fix ci pipeline #90

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

Merged
merged 12 commits into from
May 16, 2025
Merged

ci: fix ci pipeline #90

merged 12 commits into from
May 16, 2025

Conversation

DeJayDev
Copy link
Member

@DeJayDev DeJayDev commented May 1, 2025

Currently, this repo depends on Depot and the build doesn't even work.

In my need to test this repo to fix the failing builds, I stumbled across Blacksmith. Blacksmith is a better fit for RunPod's Containers due to their Layer Caching and pay per use model. These builds are fast and incremental building allows us to move faster than we could previously in improving these models in the future.

Currently, we only automatically build the base model (and we should figure out how to make this scale better!) - but every template in this repo now builds locally. I tried not to change the functionality of any images, just help them build.

It's my hope that the cleanup helps users without very much docker bake experience absorb how the repo functions better and in the future actually using our base image to optimize the workflow of images in this repo and outside of it such as https://github.com/runpod/tetra_rp, https://github.com/runpod-workers/worker-template, and users building their own Pod workflows on RunPod.

I've also taken the time to drop new builds for CUDA versions less than 12 to match the RunPod fleet, and Python 3.8 due to it no longer being supported by get-pip.py.

@max4c max4c merged commit feb700b into main May 16, 2025
3 checks passed
@max4c max4c deleted the dj/blacksmith branch May 16, 2025 22:13
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.

3 participants