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

Removed some more unnecessary firmware packages #303

Merged
merged 1 commit into from Dec 16, 2014
Merged

Removed some more unnecessary firmware packages #303

merged 1 commit into from Dec 16, 2014

Conversation

ghost
Copy link

@ghost ghost commented Dec 15, 2014

I've used package-cleanup --leaves --all and repoquery -l to identify some more unused firmware packages.

If size matters, the following packages could be deleted as well. It depends on your definition of „JeOS“.
I prefer to remove as much as possible and provision the base box with Chef afterwards.
On the one hand this approach leads to a least common multiple for all operating systems - on the other hand it requires provisioning real low-level stuff.

-parted
-postfix
-NetworkManager-tui
-firewalld
-btrfs-progs
-microcode_ctl
-tuned
-selinux-policy-targeted
-rsyslog
-lvm2

@juliandunn
Copy link
Contributor

I'm ok with removing the packages you mentioned. The only one we need to check is selinux-policy-targeted because I'd like to keep building the boxes with SELinux enabled but in Permissive mode. As long as it doesn't blow that out of the water, we can remove the Targeted policy.

@ghost
Copy link
Author

ghost commented Dec 16, 2014

I'll test the impact on typical use cases when removing those packages and open another pull request as soon as it's done.

@juliandunn
Copy link
Contributor

Excellent; I'll merge this for now.

juliandunn added a commit that referenced this pull request Dec 16, 2014
Removed some more unnecessary firmware packages
@juliandunn juliandunn merged commit a24ef53 into chef:master Dec 16, 2014
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