diff mbox series

[ovs-dev] redhat: change variable used for non-root user support

Message ID eb048cb178c827fb7a9b0427f26b311f945927e8.1542740680.git.tredaelli@redhat.com
State Accepted
Delegated to: Ian Stokes
Headers show
Series [ovs-dev] redhat: change variable used for non-root user support | expand

Commit Message

Timothy Redaelli Nov. 20, 2018, 7:04 p.m. UTC
In DPDK 18.08 the runtime config file has changed and $XDG_RUNTIME_DIR
is used instead of $HOME.

This commit changes the variable to be used for DPDK semi-persistent RTE
configuration from $HOME to $XDG_RUNTIME_DIR.

Signed-off-by: Timothy Redaelli <tredaelli@redhat.com>
---
 rhel/usr_lib_systemd_system_ovs-vswitchd.service.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Aaron Conole Nov. 20, 2018, 8:44 p.m. UTC | #1
Timothy Redaelli <tredaelli@redhat.com> writes:

> In DPDK 18.08 the runtime config file has changed and $XDG_RUNTIME_DIR
> is used instead of $HOME.
>
> This commit changes the variable to be used for DPDK semi-persistent RTE
> configuration from $HOME to $XDG_RUNTIME_DIR.
>
> Signed-off-by: Timothy Redaelli <tredaelli@redhat.com>
> ---

Acked-by: Aaron Conole <aconole@redhat.com>

Thanks, Timothy!
Timothy Redaelli Nov. 21, 2018, 2:37 p.m. UTC | #2
On Tue, 20 Nov 2018 20:04:40 +0100
Timothy Redaelli <tredaelli@redhat.com> wrote:

> In DPDK 18.08 the runtime config file has changed and $XDG_RUNTIME_DIR
> is used instead of $HOME.
> 
> This commit changes the variable to be used for DPDK semi-persistent RTE
> configuration from $HOME to $XDG_RUNTIME_DIR.
> 
> Signed-off-by: Timothy Redaelli <tredaelli@redhat.com>

This patch is for dpdk-latest (DPDK 18.08 or 18.11) since it's not
compatible with DPDK 17.11.

Thank you
Stokes, Ian Nov. 22, 2018, 3:28 p.m. UTC | #3
> Timothy Redaelli <tredaelli@redhat.com> writes:
> 
> > In DPDK 18.08 the runtime config file has changed and $XDG_RUNTIME_DIR
> > is used instead of $HOME.
> >
> > This commit changes the variable to be used for DPDK semi-persistent
> > RTE configuration from $HOME to $XDG_RUNTIME_DIR.
> >
> > Signed-off-by: Timothy Redaelli <tredaelli@redhat.com>
> > ---
> 
> Acked-by: Aaron Conole <aconole@redhat.com>
> 
> Thanks, Timothy!

Thanks Timothy & Aaron, I've pushed this to dpdk-latest and dpdk-hwol.

Ian
diff mbox series

Patch

diff --git a/rhel/usr_lib_systemd_system_ovs-vswitchd.service.in b/rhel/usr_lib_systemd_system_ovs-vswitchd.service.in
index 11b34c686..525deae0b 100644
--- a/rhel/usr_lib_systemd_system_ovs-vswitchd.service.in
+++ b/rhel/usr_lib_systemd_system_ovs-vswitchd.service.in
@@ -10,7 +10,7 @@  PartOf=openvswitch.service
 [Service]
 Type=forking
 Restart=on-failure
-Environment=HOME=/var/run/openvswitch
+Environment=XDG_RUNTIME_DIR=/var/run/openvswitch
 EnvironmentFile=/etc/openvswitch/default.conf
 EnvironmentFile=-/etc/sysconfig/openvswitch
 EnvironmentFile=-/run/openvswitch/useropts