Skip to content

Conversation

@jcpunk
Copy link
Contributor

@jcpunk jcpunk commented Dec 16, 2025

This sets up a skeleton for how multiple "kernel" packages might be checked on hosts that have alternate kernels installed (kernel-rt or kernel-ml) and tries to track updates of the alt kernel rather than the stock kernel.

This passed my minimal smoke test, but I wouldn't call it release ready...

I got lucky that each package manager's search result object has a .name attribute :)

I do wonder if there is any value switching those to @staticmethod since they just pop off an object attribute?

On the kernel package name front, I wonder if those should just land in the applications.xml file as a new type of thing rather than hard coded here?

I still have no idea how to handle "debian" style kernel package names...

Signed-off-by: Pat Riehecky <riehecky@fnal.gov>
@FrostyX
Copy link
Owner

FrostyX commented Jan 13, 2026

Thank you for the PR @jcpunk,

I got lucky that each package manager's search result object has a .name attribute :)

Hehe, I haven't seen the code for a long long time, but I think it wasn't luck, and possibly they are all instances of

class Package:

If yes, we probably won't need the package_name_only abstraction.

But otherwise the change looks good to me.

I still have no idea how to handle "debian" style kernel package names...

I don't either, but I wouldn't consider it a blocker. We can merge it if the feature works well on Fedora/RHEL and anyone can contribute the support for Debian later.

Also, @jcpunk, since you are the most active contributor these days, would you like a commit access to the repository?

@jcpunk
Copy link
Contributor Author

jcpunk commented Jan 13, 2026

It seems to work alright for my test hosts, but I wouldn't mind more testing.

I'm fine with using a github fork and sending PRs as I get time/inspiration.

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.

2 participants