From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: resourceManager: Use log.debug for unimportant messages
......................................................................
resourceManager: Use log.debug for unimportant messages
Registering an existing namespace is neither an interesting event nor
a state change that should be logged in info log level. This is
unfortunate debug message that we see each time a domain is refreshed,
because of the horrible way domain objects are managed.
This change removes these useless logs from the default log level:
Resource namespace 01_img_93404146-0d02-4d98-8310-f356733d7176 already registered
(sd:658)
Resource namespace 02_vol_93404146-0d02-4d98-8310-f356733d7176 already registered
(sd:667)
Resource namespace 03_lvm_93404146-0d02-4d98-8310-f356733d7176 already registered
(blockSD:874)
We need to check later if it is possible to avoid the registration in this
case, or maybe fix domain life cycle handling so we can register the namespace
after the previous instance unregistered it.
Change-Id: I785b411320735a2c24352da470f63a8b2dab554e
Signed-off-by: Nir Soffer <nsoffer(a)redhat.com>
---
M vdsm/storage/blockSD.py
M vdsm/storage/sd.py
2 files changed, 6 insertions(+), 6 deletions(-)
Approvals:
Nir Soffer: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
Freddy Rolland: Looks good to me, but someone else must approve
Maor Lipchuk: Looks good to me, but someone else must approve
Ala Hino: Looks good to me, but someone else must approve
--
To view, visit
https://gerrit.ovirt.org/66948
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I785b411320735a2c24352da470f63a8b2dab554e
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>