[patch dracut v2] fix teamd exec in case config does not include devname

[patch dracut v2] fix teamd exec in case config does not include devname

[Jiri Pirko]

In case that configuration file did not include name of team device,
teamd would not start. Fix this by adding "-t" parameter.

Signed-off-by: Jiri Pirko <jiri-rHqAuBHg3fBzbRFIqnYvSA@public.gmane.org>
---

v2->v1:
-fixed bad order of args
	
modules.d/40network/ifup.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules.d/40network/ifup.sh b/modules.d/40network/ifup.sh
index 216b1e3..6c3133e 100755
--- a/modules.d/40network/ifup.sh
+++ b/modules.d/40network/ifup.sh
@@ -187,7 +187,7 @@ if [ -e /tmp/team.info ]; then
             fi
         done
         # Do not add slaves now
-        teamd -d -U -n -f /etc/teamd/$teammaster.conf
+        teamd -d -U -n -t $teammaster -f /etc/teamd/$teammaster.conf
         for slave in $working_slaves; do
             # team requires the slaves to be down before joining team
             ip link set $slave down
-- 
1.8.1

Re: [patch dracut v2] fix teamd exec in case config does not include devname

[Cong Wang]

On Thu, 2013-01-10 at 15:02 +0100, Jiri Pirko wrote:
> In case that configuration file did not include name of team device,
> teamd would not start. Fix this by adding "-t" parameter.
> 
> Signed-off-by: Jiri Pirko <jiri-rHqAuBHg3fBzbRFIqnYvSA@public.gmane.org>
> ---
> 
> v2->v1:
> -fixed bad order of args

Looks good! I didn't test the ifcfg case.

Acked-by: Cong Wang <amwang-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

Thanks!

Re: [patch dracut v2] fix teamd exec in case config does not include devname

[Harald Hoyer]

Am 10.01.2013 15:02, schrieb Jiri Pirko:
> In case that configuration file did not include name of team device,
> teamd would not start. Fix this by adding "-t" parameter.
> 
> Signed-off-by: Jiri Pirko <jiri-rHqAuBHg3fBzbRFIqnYvSA@public.gmane.org>
> ---
> 
> v2->v1:
> -fixed bad order of args
> 	
> modules.d/40network/ifup.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/modules.d/40network/ifup.sh b/modules.d/40network/ifup.sh
> index 216b1e3..6c3133e 100755
> --- a/modules.d/40network/ifup.sh
> +++ b/modules.d/40network/ifup.sh
> @@ -187,7 +187,7 @@ if [ -e /tmp/team.info ]; then
>              fi
>          done
>          # Do not add slaves now
> -        teamd -d -U -n -f /etc/teamd/$teammaster.conf
> +        teamd -d -U -n -t $teammaster -f /etc/teamd/$teammaster.conf
>          for slave in $working_slaves; do
>              # team requires the slaves to be down before joining team
>              ip link set $slave down
> 

merged