Ondřej Svoboda has posted comments on this change. Change subject: ifcfg: extract _ifup and call _ifup_nonblocking when 'async' flag was used ......................................................................
Patch Set 3: (1 comment) https://gerrit.ovirt.org/#/c/40478/3/vdsm/network/configurators/ifcfg.py File vdsm/network/configurators/ifcfg.py: Line 773: # contains the error reason. Line 774: raise ConfigNetworkError(ERR_FAILED_IFUP, out[-1] if out else '') Line 775: Line 776: Line 777: def _ifup_nonblocking(iface_name, async=False): > Could you explain (in the commit message) the motivation of this patch? You are right, it is only capable of being non-blocking, if asked to. Is _ifup_threaded a better name? (Again, potentially, but this is the function’s raison d’être.) Or even _ifup_async_capable? I’ll update the commitmsg when the name is agreed upon. Line 778: if async: Line 779: # wait for dhcp in another thread, so vdsm won't get stuck (BZ#498940) Line 780: t = threading.Thread(target=_ifup, name='ifup-waiting-on-dhcp', Line 781: args=(iface_name,)) -- To view, visit https://gerrit.ovirt.org/40478 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3c40cb09f05a0865b1942448a64fd37b735857f9 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ondřej Svoboda <osvob...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Ido Barkan <ibar...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondřej Svoboda <osvob...@redhat.com> Gerrit-Reviewer: Petr Horáček <phora...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches