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

installonly_limit value not respected (removes running kernel) #436

Closed
rlmenge opened this issue Jun 28, 2023 · 3 comments · Fixed by #439
Closed

installonly_limit value not respected (removes running kernel) #436

rlmenge opened this issue Jun 28, 2023 · 3 comments · Fixed by #439
Labels

Comments

@rlmenge
Copy link

rlmenge commented Jun 28, 2023

Describe the bug

The installonly_limit value in tdnf.conf does not change the behavior of tdnf. Only 1 kernel and its modules are ever present. An upgrade/downgrade will remove all other kernels except the new kernel being installed. rpm -ihv and dnf install -y behave as expected,

ie. installonly_limit=3 should result in up to 3 kernels. However, all other kernels are always removed for the new kernel being installed.

This option is mentioned in the documentation in the example tdnf.conf and config code

Reproduction steps

# confirm installonly_limit=3
cat /etc/tdnf/tdnf.conf 
tdnf install -y kernel-<different version>
# check that running kernel vmlinuz has been removed
 ls /boot
# check that running kernel's modules have been removed (folder still present)
ls /lib/modules/$(uname -r)

Expected behavior

# confirm installonly_limit=3
cat /etc/tdnf/tdnf.conf 
tdnf install -y kernel-<different version>
# check that running kernel vmlinuz and new vmlinuz present
 ls /boot

Additional context

No response

@rlmenge rlmenge added the bug label Jun 28, 2023
@oliverkurth
Copy link
Contributor

Thank you for reporting.

Looks like, sadly, the option is not implemented at all.

@rlmenge
Copy link
Author

rlmenge commented Jul 1, 2023

Thanks for checking into this! Are there plans to implement installonly_limit in the future?

@oliverkurth
Copy link
Contributor

Feel free to checkout out PR #439 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants