[ovs-dev] rhel: hugetlbfs group should be added as a system group.

Message ID 20181106172815.14403-1-aconole@redhat.com
State Accepted
Headers show
Series
  • [ovs-dev] rhel: hugetlbfs group should be added as a system group.
Related show

Commit Message

Aaron Conole Nov. 6, 2018, 5:28 p.m.
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(-)

Comments

Ben Pfaff Nov. 9, 2018, 9:37 p.m. | #1
On Tue, Nov 06, 2018 at 12:28:15PM -0500, Aaron Conole wrote:
> 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

Thanks, I applied this to master and backported as far as branch-2.9.

Patch

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