mib1185 commented on a change in pull request #5110:
URL: https://github.com/apache/cloudstack/pull/5110#discussion_r651628126
##########
File path: packaging/centos8/cloud.spec
##########
@@ -96,21 +97,22 @@ The Apache CloudStack files shared between agent and
management server
%package agent
Summary: CloudStack Agent for KVM hypervisors
-Requires: openssh-clients
+Requires: openssh
Requires: java-11-openjdk
Requires: %{name}-common = %{_ver}
Requires: libvirt
Requires: ebtables
Requires: iptables
+Requires: selinux-tools
Requires: ethtool
Requires: net-tools
+Requires: net-tools-deprecated
Review comment:
This package is needed for `python/lib/cloudutils/networkConfig.py` ,
since there are quite old tools `ifconfig` and `route` used, which were already
replaced by modern `ip` command since many years, those IMHO honestly would
recommend to adjust `networkConfig.py` to use `ip` command and not to install a
deprecated called package 🙂
Will create a PR with needed changes of `networkConfig.py` on your
fork/branch.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
