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: bind-utils #120

Closed
sinnykumari opened this issue Jan 17, 2019 · 12 comments
Closed

Keep/Remove Python dependent package: bind-utils #120

sinnykumari opened this issue Jan 17, 2019 · 12 comments
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 bind-utils in FCOS base system? If yes, how can we provide bind-utils or equivalent functionalities without shipping Python.

@sinnykumari
Copy link
Contributor Author

RFE to split python utilities to their own binary package - https://bugzilla.redhat.com/show_bug.cgi?id=1649398

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

Latest bind-utils-9.11.5-8.P1.fc30 available in Fedora 30 doesn't contain Python dependent utilities.

With this change, all dnssec related utilities including Python utilites has been moved to a new sub-package bind-dnssec-utils .

As a result, bind-utils-9.11.5-8.P1.fc30 (python independent ) contains now following utilities:

/etc/trusted-key.key (config)
/usr/bin/arpaname
/usr/bin/delv
/usr/bin/dig
/usr/bin/host
/usr/bin/nslookup
/usr/bin/nsupdate

And bind-dnssec-utils-9.11.5-8.P1.fc30.x86_64.rpm (Python dependent)

/usr/sbin/ddns-confgen
/usr/sbin/dnssec-checkds (Python utility)
/usr/sbin/dnssec-coverage ((Python utility))
/usr/sbin/dnssec-dsfromkey
/usr/sbin/dnssec-importkey
/usr/sbin/dnssec-keyfromlabel
/usr/sbin/dnssec-keygen
/usr/sbin/dnssec-keymgr (Python utility)
/usr/sbin/dnssec-revoke
/usr/sbin/dnssec-settime
/usr/sbin/dnssec-signzone
/usr/sbin/dnssec-verify
/usr/sbin/genrandom
/usr/sbin/isc-hmac-fixup
/usr/sbin/named-checkzone
/usr/sbin/named-compilezone
/usr/sbin/nsec3hash
/usr/sbin/tsig-keygen

Does utilities available in bind-utils-9.11.5-8.P1.fc30` looks sufficient for FCOS?

@sinnykumari
Copy link
Contributor Author

Maybe we can get this discussed in today's FCOS community meeting.

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

Discussed in the meeting today.

We agreed on the following statement:

AGREED: we will try to ship just bind-utils without 
bind-dnssec-utils (python deps) and see if we hit any roadblocks

@dustymabe
Copy link
Member

@sinnykumari - what are the remaining items to do here?

@sinnykumari
Copy link
Contributor Author

@sinnykumari - what are the remaining items to do here?

Considering we all agreed to go with just bind-utils package and see how it goes, there is no remaining action item. bind-utils is already added in https://github.com/coreos/fedora-coreos-config/blob/master/fedora-coreos-base.yaml#L202. When we will start building FCOS based on F30 content, python dependencies from bind-utils should go away.

@sinnykumari
Copy link
Contributor Author

If there is no objection, we can close this ticket.

@lucab
Copy link
Contributor

lucab commented Feb 14, 2019

@sinnykumari for my own knowledge, do we have to specify something in the treefile to opt-out from Suggests: bind-dnssec-utils or is that happening by default?

@sinnykumari
Copy link
Contributor Author

I don't think we need to specify anything additionally. Packages specified as suggest in rpm spec are not installed by default as package dependencies but considered as a hint providing additional functionalities. https://docs.fedoraproject.org/en-US/packaging-guidelines/WeakDependencies/ explains more .

@dustymabe
Copy link
Member

related: coreos/rpm-ostree@40be3fb

we may need to add recommends: false to our treefile

@dustymabe
Copy link
Member

actually i always get confused between recommends and suggests so maybe we don't need to do anything.

@sinnykumari
Copy link
Contributor Author

Closing this as fixed.

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

No branches or pull requests

4 participants