-
Notifications
You must be signed in to change notification settings - Fork 8
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
add CUDA container for logdetective #56
Conversation
Signed-off-by: Tomas Tomecek <ttomecek@redhat.com>
I wonder, could we somehow use RHEL AI? Isn't there a Fedora AI variant? |
&& echo "[cuda-fedora39-x86_64]" >> /etc/yum.repos.d/cuda.repo \ | ||
&& echo "name=cuda-fedora39-x86_64" >> /etc/yum.repos.d/cuda.repo \ | ||
&& echo "baseurl=https://developer.download.nvidia.com/compute/cuda/repos/fedora39/x86_64" >> /etc/yum.repos.d/cuda.repo \ | ||
&& echo "enabled=1" >> /etc/yum.repos.d/cuda.repo \ | ||
&& echo "gpgcheck=1" >> /etc/yum.repos.d/cuda.repo \ | ||
&& echo "gpgkey=https://developer.download.nvidia.com/compute/cuda/repos/fedora39/x86_64/D42D0685.pub" >> /etc/yum.repos.d/cuda.repo \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't it be better to just have .repo file next to the Containerfile and COPY it into the container. It's seems simpler.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes it would, but your comment would become even more relevant :) I agree I don't want to flood the root dir of this repo with "random" files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really like that now we can see the content of the repo file and i.e. once F40 variant is available, we can s/39/40/g
in one file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
since we discussed this on our sync meeting, I'll reply here as well there is no official release right now of those: once something's out, I totally want to explore :) |
this enables running logdetective in a container while accelerated by CUDA
details: https://blog.tomecek.net/post/running-logdetective-service-on-cuda-ec2/