diff mbox series

[4/6] multipathd.service: drop ExecStartPre for loading dm-multipath

Message ID 20220201234024.27675-5-mwilck@suse.com (mailing list archive)
State Not Applicable, archived
Delegated to: Mike Snitzer
Headers show
Series multipath-tools: kernel module loading improvements | expand

Commit Message

Martin Wilck Feb. 1, 2022, 11:40 p.m. UTC 
From: Martin Wilck <mwilck@suse.com>

As we install a modules-load.d file now, no need to call modprobe
in ExecStartPre any more.

Signed-off-by: Martin Wilck <mwilck@suse.com>
---
 multipathd/multipathd.service | 1 -
 1 file changed, 1 deletion(-)

Comments

Benjamin Marzinski Feb. 3, 2022, 12:40 a.m. UTC | #1 
On Wed, Feb 02, 2022 at 12:40:22AM +0100, mwilck@suse.com wrote:
> From: Martin Wilck <mwilck@suse.com>
> 
> As we install a modules-load.d file now, no need to call modprobe
> in ExecStartPre any more.
> 
> Signed-off-by: Martin Wilck <mwilck@suse.com>
Reviewed-by: Benjamin Marzinski <bmarzins@redhat.com>
> ---
>  multipathd/multipathd.service | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/multipathd/multipathd.service b/multipathd/multipathd.service
> index ae98034..aec62db 100644
> --- a/multipathd/multipathd.service
> +++ b/multipathd/multipathd.service
> @@ -16,7 +16,6 @@ ConditionVirtualization=!container
>  [Service]
>  Type=notify
>  NotifyAccess=main
> -ExecStartPre=-/sbin/modprobe dm-multipath
>  ExecStart=/sbin/multipathd -d -s
>  ExecReload=/sbin/multipathd reconfigure
>  TasksMax=infinity
> -- 
> 2.34.1

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel
diff mbox series

Patch

diff --git a/multipathd/multipathd.service b/multipathd/multipathd.service
index ae98034..aec62db 100644
--- a/multipathd/multipathd.service
+++ b/multipathd/multipathd.service
@@ -16,7 +16,6 @@  ConditionVirtualization=!container
 [Service]
 Type=notify
 NotifyAccess=main
-ExecStartPre=-/sbin/modprobe dm-multipath
 ExecStart=/sbin/multipathd -d -s
 ExecReload=/sbin/multipathd reconfigure
 TasksMax=infinity