Change in vdsm[master]: storage.monitor: Streamline names
danken at redhat.com
danken at redhat.com
Mon Feb 2 18:36:54 UTC 2015
Dan Kenigsberg has submitted this change and it was merged.
Change subject: storage.monitor: Streamline names
......................................................................
storage.monitor: Streamline names
Rename internal classes using shorter names, considering the package
namespace. For example:
storage.monitor.DomainMonitorStatus -> storage.monitor.Status
In this case the name is more correct, as this is not the status of the
single DomainMonitor object but status of one of the monitor threads.
Since DomainMonitorThread was renamed, the logger was renamed too. We do
not need muliple loggers for the DomainMonitor and the MonitorThread, so
both of them use now the Storage.Monitor logger.
Change-Id: I8eae2f8548ebd1b336df10aea7dde47a4aeabf70
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/32507
Reviewed-by: Federico Simoncelli <fsimonce at redhat.com>
---
M tests/storageMonitorTests.py
M vdsm/storage/monitor.py
2 files changed, 10 insertions(+), 10 deletions(-)
Approvals:
Nir Soffer: Verified
Federico Simoncelli: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/32507
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8eae2f8548ebd1b336df10aea7dde47a4aeabf70
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yoav Kleinberger <ykleinbe at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the vdsm-patches
mailing list