Change in vdsm[master]: daemon: avoid contradicting logs about affinity

danken at redhat.com danken at redhat.com
Fri Nov 27 20:00:20 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: daemon: avoid contradicting logs about affinity
......................................................................


daemon: avoid contradicting logs about affinity

Vdsm can autodisable the affinity if it detects just one cpu.
In this case the logs read like:

MainThread::INFO::2015-11-26
17:31:35,932::vdsm::229::vds::(__set_cpu_affinity) VDSM will run with
cpu affinity: frozenset([1])
MainThread::DEBUG::2015-11-26
17:31:35,933::vdsm::233::vds::(__set_cpu_affinity) Only one cpu
detected: affinity disabled

which feels like VDSM is contradicting itslef.
Better to log the affinity used only if we actually use it.

Change-Id: I26c2658e1ea1c623dda5638e01171555e22e5bc0
Signed-off-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/49271
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Continuous-Integration: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Continuous-Integration: Jenkins CI
---
M vdsm/vdsm
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Piotr Kliczewski: Looks good to me, but someone else must approve
  Nir Soffer: Looks good to me, approved; Passed CI tests
  Jenkins CI: Passed CI tests
  Francesco Romani: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I26c2658e1ea1c623dda5638e01171555e22e5bc0
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list