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