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

Keep/Remove Python dependent package: setools-console #122

Closed
sinnykumari opened this issue Jan 17, 2019 · 11 comments · Fixed by coreos/fedora-coreos-config#88
Closed

Keep/Remove Python dependent package: setools-console #122

sinnykumari opened this issue Jan 17, 2019 · 11 comments · Fixed by coreos/fedora-coreos-config#88
Labels
-python card related to removing a python dependency

Comments

@sinnykumari
Copy link
Contributor

sinnykumari commented Jan 17, 2019

FCOS community members are trying not to ship Python in base system. In ticket #92, we have identified list of Python dependent packages which are currently installed in FCOS.

This ticket is to investigate that do we really need setools-console in FCOS base system? If yes, how can we provide setools-console or equivalent functionalities without shipping Python.

@dustymabe dustymabe added the -python card related to removing a python dependency label Jan 17, 2019
@sinnykumari
Copy link
Contributor Author

sinnykumari commented Jan 18, 2019

setools-console rpm package provides three utilities to analyze selinux policies:
/usr/bin/sediff
/usr/bin/seinfo
/usr/bin/sesearch

All three utilities requires Python.
Do we really need these utilities on FCOS base system? If possible maybe user can use fedora-toolbox to get selinux policies related information of Host machine?

@dustymabe
Copy link
Member

hmm. They are not utilities I regularly use, but I'm only a sample size of 1. I think we can make it without them.

@jlebon @cgwalters @bgilbert - WDYT?

@bgilbert
Copy link
Contributor

I'm okay omitting them.

@ajeddeloh
Copy link
Contributor

They are all tools for querying info about the SELinux policy, not changing it. So while they might be useful for debugging they aren't needed to actually run the system or even make changes. Interestingly it looks like setools didn't use to require python but now does. We ship setools on CL, its just old enough to not need python.

@jlebon
Copy link
Member

jlebon commented Jan 18, 2019

SGTM. I've only used sesearch of those, which is indeed nice for debugging. Though most SELinux issues don't require going that far.

@ajeddeloh
Copy link
Contributor

ajeddeloh commented Jan 18, 2019

I don't know so, but I'm just going to assume SELinux tools don't containerize well.

@sinnykumari
Copy link
Contributor Author

Let's finalize in today's FCOS community meeting if we are ok with not having setools-console package in FCOS

@sinnykumari sinnykumari added the meeting topics for meetings label Feb 13, 2019
@dustymabe
Copy link
Member

@lucab brought up that some packages might be depending on this information for rpm scriptlets. While not a comprehensive answer at least on Fedora 29 Atomic Host it doesn't appear any scriptlets excute those utilities:

[dustymabe@dhcp137-98 annex]$ rpm -qa --scripts | grep sesearch
[dustymabe@dhcp137-98 annex]$ rpm -qa --scripts | grep sediff
[dustymabe@dhcp137-98 annex]$ rpm -qa --scripts | grep seinfo

@bgilbert bgilbert removed the meeting topics for meetings label Feb 13, 2019
@dustymabe
Copy link
Member

Discussed in the meeting today.

We are going to try to exclude setools-console package and see where we land.

One requested followup by @lucab is to try to run those packages inside a container with appropriate bind mounts and verify they can be used containerized. @sinnykumari can you try that?

@sinnykumari
Copy link
Contributor Author

Thanks Dusty! will try running setools-console tools in container and will update here.

@dustymabe
Copy link
Member

Thanks Dusty! will try running setools-console tools in container and will update here.

might as well try semanage as well for #126

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-python card related to removing a python dependency
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants