| Message ID | 20190614031659.3313-1-liuhangbin@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | initscripts: need add the port before update config | expand |
On Sat, Jun 15, 2019 at 5:41 AM Hangbin Liu <liuhangbin@gmail.com> wrote: > > After c8b356a3cd36 ("teamd: config: update local prio to kernel"), > we update team port prio not only on local, but also on kernel side. > > So we need to add the port to team first before update TEAM_PORT_CONFIG. > Or the teamd_config_port_update() would be failed because we could not find > the port in teamd_get_port_by_ifname() during teamd_config_port_set(). > > Reported-by: LiLiang <liali@redhat.com> > Fixes: c8b356a3cd36 ("teamd: config: update local prio to kernel") > Signed-off-by: Hangbin Liu <liuhangbin@gmail.com> > --- > teamd/redhat/initscripts/network-scripts/ifup-TeamPort | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/teamd/redhat/initscripts/network-scripts/ifup-TeamPort b/teamd/redhat/initscripts/network-scripts/ifup-TeamPort > index 8c3cc3a..954b47a 100755 > --- a/teamd/redhat/initscripts/network-scripts/ifup-TeamPort > +++ b/teamd/redhat/initscripts/network-scripts/ifup-TeamPort > @@ -51,8 +51,8 @@ if [ -n "${TEAM_MASTER}" ]; then > exit 0 > fi > /sbin/ip link set dev ${DEVICE} down > + /usr/bin/teamdctl ${TEAM_MASTER} port add ${DEVICE} || exit 1 > if [ -n "${TEAM_PORT_CONFIG}" ]; then > /usr/bin/teamdctl ${TEAM_MASTER} port config update ${DEVICE} "${TEAM_PORT_CONFIG}" || exit 1 > fi > - /usr/bin/teamdctl ${TEAM_MASTER} port add ${DEVICE} || exit 1 > fi Ohh, it seems that this fixes the same issue as the patch I just posted, [PATCH] teamd: return 0 if tdport doesn't exist in teamd_config_port_set This issue actually also breaks NM-team, which means if you insist on your patch, you've got to ask NM to do the same, it will kinda drop the "feature" (port conf can go first). More importantly, it will break old use. > -- > 2.19.2 > _______________________________________________ > libteam mailing list -- libteam@lists.fedorahosted.org > To unsubscribe send an email to libteam-leave@lists.fedorahosted.org > Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html > List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines > List Archives: https://lists.fedorahosted.org/archives/list/libteam@lists.fedorahosted.org
Mon, Jul 01, 2019 at 09:57:18AM CEST, lucien.xin@gmail.com wrote: >On Sat, Jun 15, 2019 at 5:41 AM Hangbin Liu <liuhangbin@gmail.com> wrote: >> >> After c8b356a3cd36 ("teamd: config: update local prio to kernel"), >> we update team port prio not only on local, but also on kernel side. >> >> So we need to add the port to team first before update TEAM_PORT_CONFIG. >> Or the teamd_config_port_update() would be failed because we could not find >> the port in teamd_get_port_by_ifname() during teamd_config_port_set(). >> >> Reported-by: LiLiang <liali@redhat.com> >> Fixes: c8b356a3cd36 ("teamd: config: update local prio to kernel") >> Signed-off-by: Hangbin Liu <liuhangbin@gmail.com> >> --- >> teamd/redhat/initscripts/network-scripts/ifup-TeamPort | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/teamd/redhat/initscripts/network-scripts/ifup-TeamPort b/teamd/redhat/initscripts/network-scripts/ifup-TeamPort >> index 8c3cc3a..954b47a 100755 >> --- a/teamd/redhat/initscripts/network-scripts/ifup-TeamPort >> +++ b/teamd/redhat/initscripts/network-scripts/ifup-TeamPort >> @@ -51,8 +51,8 @@ if [ -n "${TEAM_MASTER}" ]; then >> exit 0 >> fi >> /sbin/ip link set dev ${DEVICE} down >> + /usr/bin/teamdctl ${TEAM_MASTER} port add ${DEVICE} || exit 1 >> if [ -n "${TEAM_PORT_CONFIG}" ]; then >> /usr/bin/teamdctl ${TEAM_MASTER} port config update ${DEVICE} "${TEAM_PORT_CONFIG}" || exit 1 >> fi >> - /usr/bin/teamdctl ${TEAM_MASTER} port add ${DEVICE} || exit 1 >> fi >Ohh, it seems that this fixes the same issue as the patch I just posted, > > [PATCH] teamd: return 0 if tdport doesn't exist in teamd_config_port_set > >This issue actually also breaks NM-team, which means if you insist >on your patch, you've got to ask NM to do the same, it will kinda >drop the "feature" (port conf can go first). More importantly, it >will break old use. I'll rather like to fix this in teamd, as Xin is suggesting. Dropping this patch. > >> -- >> 2.19.2 >> _______________________________________________ >> libteam mailing list -- libteam@lists.fedorahosted.org >> To unsubscribe send an email to libteam-leave@lists.fedorahosted.org >> Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html >> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines >> List Archives: https://lists.fedorahosted.org/archives/list/libteam@lists.fedorahosted.org
diff --git a/teamd/redhat/initscripts/network-scripts/ifup-TeamPort b/teamd/redhat/initscripts/network-scripts/ifup-TeamPort index 8c3cc3a..954b47a 100755 --- a/teamd/redhat/initscripts/network-scripts/ifup-TeamPort +++ b/teamd/redhat/initscripts/network-scripts/ifup-TeamPort @@ -51,8 +51,8 @@ if [ -n "${TEAM_MASTER}" ]; then exit 0 fi /sbin/ip link set dev ${DEVICE} down + /usr/bin/teamdctl ${TEAM_MASTER} port add ${DEVICE} || exit 1 if [ -n "${TEAM_PORT_CONFIG}" ]; then /usr/bin/teamdctl ${TEAM_MASTER} port config update ${DEVICE} "${TEAM_PORT_CONFIG}" || exit 1 fi - /usr/bin/teamdctl ${TEAM_MASTER} port add ${DEVICE} || exit 1 fi
After c8b356a3cd36 ("teamd: config: update local prio to kernel"), we update team port prio not only on local, but also on kernel side. So we need to add the port to team first before update TEAM_PORT_CONFIG. Or the teamd_config_port_update() would be failed because we could not find the port in teamd_get_port_by_ifname() during teamd_config_port_set(). Reported-by: LiLiang <liali@redhat.com> Fixes: c8b356a3cd36 ("teamd: config: update local prio to kernel") Signed-off-by: Hangbin Liu <liuhangbin@gmail.com> --- teamd/redhat/initscripts/network-scripts/ifup-TeamPort | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)