Dan Kenigsberg has submitted this change and it was merged.

Change subject: init: fix network service status checking
......................................................................


init: fix network service status checking

In init file, we check if network service is running by its status
return code. Problem is, that this code is "always" 0, therefore we never
start network service when needed (for example, it's not autostarted
on CentOS 6).

`service network status` returns lists of configured and active devices,
if the list of active devices (on the last row) does not contains all
VDSM's ONBOOT=yes devices, it means, that network service did not start
our devices and we should call service network start.

Change-Id: Idfe796744e44af21f08974f2ccb4bbb503cc6d67
Signed-off-by: Petr Horáček <phora...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/38173
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
---
M init/sysvinit/vdsmd.init.in
1 file changed, 39 insertions(+), 2 deletions(-)

Approvals:
  Petr Horáček: Verified
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/38173
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Idfe796744e44af21f08974f2ccb4bbb503cc6d67
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phora...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Petr Horáček <phora...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to