Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-17 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 1: (1 comment) File vdsm/storage/sp.py Line 1548:

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-17 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 4: Hi Eduardo. I find a problem in patch. Maybe we need to fix it. -- To view, visit

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-17 Thread zhshzhou
Zhou Zheng Sheng has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 1: (1 comment) File vdsm/storage/sp.py Line 1033:

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-14 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: Adding [start|stop]MonitoringDomain(). .. Adding [start|stop]MonitoringDomain(). startMonitoringDomain() is added for monitoring a storage domain

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-14 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/19762 To unsubscribe, visit

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-12 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 3: Thanks Sandro for fixing this patch. I still cannot take it since it depends on a faulty patch. -- To

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-11 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 2: Verified+1 Verified with both: - http://gerrit.ovirt.org/20099 - http://gerrit.ovirt.org/19867 See

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-11 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 3: Verified+1 Addressed my comments as per Dan Kenigsberg request. Verified with both: -

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-11 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 3: Verified-1 Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4935/ : FAILURE

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-10 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/19762 To unsubscribe, visit

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-10 Thread laravot
Liron Ar has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.ovirt.org/19762 To unsubscribe, visit

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-10 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 2: Code-Review-1 (1 comment) File client/vdsClient.py

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-07 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/4750/ : SUCCESS

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-07 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 1: (1 comment) File vdsm/storage/sp.py Line 1545:

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-04 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 1: (2 comments) File client/vdsClient.py Line 1694:

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-04 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 1: Verified-1 Fails starting the VM for hosted engine setup after disconnecting the pool and calling

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-04 Thread ewarszaw
Eduardo has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 1: (1 comment) File client/vdsClient.py Line 1694: Line

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-04 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 1: (1 comment) File client/vdsClient.py Line 1694:

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-02 Thread abaron
Ayal Baron has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 1: (1 comment) File vdsm/storage/sp.py Line 1545:

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-02 Thread sgotliv
Sergey Gotliv has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 1: (1 comment) File vdsm_api/vdsmapi-schema.json Line

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-02 Thread ewarszaw
Eduardo has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 1: (1 comment) File vdsm/storage/sp.py Line 1545: #

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-01 Thread ewarszaw
Eduardo has uploaded a new change for review. Change subject: Adding [start|stop]MonitoringDomain(). .. Adding [start|stop]MonitoringDomain(). startMonitoringDomain() is added for monitoring a storage domain without being

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-01 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/3848/ : SUCCESS

Change in vdsm[master]: Adding [start|stop]MonitoringDomain().

2013-10-01 Thread ewarszaw
Eduardo has posted comments on this change. Change subject: Adding [start|stop]MonitoringDomain(). .. Patch Set 1: Verified+1 More verification (with engine) is required! [root@localhost ~]# vdsClient -s 0