PR#11: hugetlbfs group should be added as a system group - rpms/openvswitch

rpms / openvswitch

#11 hugetlbfs group should be added as a system group

Merged 3 years ago by aconole. Opened 4 years ago by imsedgar.

rpms/ imsedgar/openvswitch master into master

hugetlbfs group should be added as a system group

Edgar Hoch • 4 years ago

c702d1f

openvswitch.spec

file modified

+1 -1

		`@@ -408,7 +408,7 @@`
		`getent passwd openvswitch >/dev/null \|\| \`
		`useradd -r -g openvswitch -d / -s /sbin/nologin \`
		`-c "Open vSwitch Daemons" openvswitch`
		`- getent group hugetlbfs >/dev/null \|\| groupadd hugetlbfs`
		`+ getent group hugetlbfs >/dev/null \|\| groupadd -r hugetlbfs`
		`usermod -a -G hugetlbfs openvswitch`
		`exit 0`

imsedgar commented 4 years ago

Please fix https://bugzilla.redhat.com/show_bug.cgi?id=1646774 and create new packages.

I got the following mail from Aaron Conole on 2018-11-06, but Fedora packages still contains the bug. I hope this fix will make it into the Fedora releases 31 and later. Thanks!

Reported-by: Edgar Hoch edgar.hoch@ims.uni-stuttgart.de
Signed-off-by: Aaron Conole aconole@redhat.com

rhel/openvswitch-fedora.spec.in | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in
index ba55fb55c..7a3fcf93a 100644
--- a/rhel/openvswitch-fedora.spec.in
+++ b/rhel/openvswitch-fedora.spec.in
@@ -410,7 +410,7 @@ getent passwd openvswitch >/dev/null || \
-c "Open vSwitch Daemons" openvswitch

%if %{with dpdk}
- getent group hugetlbfs >/dev/null || groupadd hugetlbfs
+ getent group hugetlbfs >/dev/null || groupadd -r hugetlbfs
usermod -a -G hugetlbfs openvswitch
%endif
exit 0
--
2.19.1