Change in vdsm[master]: clientIF: stop VmStatsThread on shutdown

fromani at redhat.com fromani at redhat.com
Fri Mar 27 10:20:40 UTC 2015


Francesco Romani has posted comments on this change.

Change subject: clientIF: stop VmStatsThread on shutdown
......................................................................


Patch Set 2: Verified+1

verification:
- runned a patched VDSM (including all three patches in the series)
- booted 4 VMs
- killed libvirt using kill -ABRT $LIBVIRT_PID
- inspected vdsm logs:

found
MainThread::DEBUG::2015-03-27 11:18:43,317::vmchannels::192::vds::(stop) VM channels listener was stopped.
MainThread::DEBUG::2015-03-27 11:18:43,331::sampling::445::vm.Vm::(stop) vmId=`65365b4c-da8f-4fc3-82de-e0e43054bbe5`::Stop statistics collection
MainThread::DEBUG::2015-03-27 11:18:43,346::sampling::445::vm.Vm::(stop) vmId=`7b6fc7c8-b351-4b93-8972-03dbc7aaa5a2`::Stop statistics collection
Thread-821::DEBUG::2015-03-27 11:18:43,347::sampling::476::vm.Vm::(run) vmId=`65365b4c-da8f-4fc3-82de-e0e43054bbe5`::Stats thread finished
MainThread::DEBUG::2015-03-27 11:18:43,348::sampling::445::vm.Vm::(stop) vmId=`a7769b63-5cb9-4d48-8c3c-ac8395e2a202`::Stop statistics collection
Thread-811::DEBUG::2015-03-27 11:18:43,348::sampling::476::vm.Vm::(run) vmId=`7b6fc7c8-b351-4b93-8972-03dbc7aaa5a2`::Stats thread finished
MainThread::DEBUG::2015-03-27 11:18:43,349::sampling::445::vm.Vm::(stop) vmId=`96721047-f7cf-4b0a-b828-412a242afdbb`::Stop statistics collection
Thread-786::DEBUG::2015-03-27 11:18:43,349::sampling::476::vm.Vm::(run) vmId=`a7769b63-5cb9-4d48-8c3c-ac8395e2a202`::Stats thread finished
MainThread::DEBUG::2015-03-27 11:18:43,350::sampling::445::vm.Vm::(stop) vmId=`e6ebae82-e9d8-4791-81c6-d7eb1d943bb3`::Stop statistics collection
Thread-857::DEBUG::2015-03-27 11:18:43,350::sampling::476::vm.Vm::(run) vmId=`96721047-f7cf-4b0a-b828-412a242afdbb`::Stats thread finished
MainThread::INFO::2015-03-27 11:18:43,351::momIF::87::MOM::(stop) Shutting down MOM

VDSM was correctly restarted shortly after.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id1c01f7a3ea1c22ba0d64b39fb74393e9a52e236
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list