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

add action to build latest kernel #4

Merged
merged 1 commit into from
Oct 12, 2021
Merged

add action to build latest kernel #4

merged 1 commit into from
Oct 12, 2021

Conversation

some-natalie
Copy link
Owner

No description provided.

@some-natalie some-natalie merged commit 53ac82f into main Oct 12, 2021
@some-natalie some-natalie deleted the docker-action branch October 12, 2021 04:09
Copy link
Contributor

@stefanleh stefanleh left a comment

Choose a reason for hiding this comment

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

was wrong about the kernel listing, changed the comment

@some-natalie
Copy link
Owner Author

@stefanleh , sorry it took me a while to get back to you here! This one is a bit weird because (to the best of my knowledge) the Fedora docker image will always only have the latest kernel ... so you're correct like 99% of the time and this is the one place where my lazy regex is appropriate enough 😆

What I was really trying to do here was shove the build of this into GitHub Actions but I didn't read the docs first. The linux runners hosted by GitHub have 14 GB disk space and that's nowhere near enough to compile the kernel. I'm still trying to think if I want to keep this code and eventually host something a bit more substantial myself or just revert it. Am super open to feedback here ... and if you'd like to help review any future PRs, I'd ❤️ all the help I can get!

@stefanleh
Copy link
Contributor

stefanleh commented Oct 19, 2021

Hi @some-natalie , you are totally correct, for a moment i forgot that we are in the latest fedora image when building the kernel. As i use an intel NIC in SRIOV mode i need the ACS patch myself as the virtual NICs dont get own IOMMU groups. Therefore i build my own docker image: https://github.com/stefanleh/fedora_acs_kernel_build, which needs the version as input parameter. I thought about setting up CI in my own hardware for building the kernels but then how to share them? So maybe it would be wise to have a look at the COPR infrastructure (https://copr.fedorainfracloud.org/)? Then it would be very easy for the folks to use the kernels. Best regards, Stefan

@some-natalie
Copy link
Owner Author

@stefanleh , you're totally right on using COPR instead of GitHub here. I definitely don't have the bandwidth to learn a new CI system at the moment though and was hoping this could be good enough. Let me make investigating COPR into an issue and hopefully I could get to it soon. If you beat me to it, mind sending any links or other things that helped you out my way?

Issue created here - #5

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.

Investigate using COPR to build and host RPMs
2 participants