Change in vdsm[master]: spec: vdsm-hook-sriov on el6 should require libvirt

2013-08-01 Thread danken
Dan Kenigsberg has uploaded a new change for review. Change subject: spec: vdsm-hook-sriov on el6 should require libvirt .. spec: vdsm-hook-sriov on el6 should require libvirt As reported by a dead horse, el6 does not carry

Change in vdsm[master]: spec: vdsm-hook-sriov on el6 should require libvirt

2013-08-01 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: spec: vdsm-hook-sriov on el6 should require libvirt .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3662/ :

Change in vdsm[master]: spec: vdsm-hook-sriov on el6 should require libvirt

2013-08-01 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: spec: vdsm-hook-sriov on el6 should require libvirt .. Patch Set 1: Shouldn't vdsm-hook-* depend on vdsm so the libvirt dependency would be implicit?

Change in vdsm[master]: QoS host networks: caps now reports QoS for networks.

2013-08-01 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: QoS host networks: caps now reports QoS for networks. .. Patch Set 5: No score Thanks for the feedback Mark -- To view, visit

Change in vdsm[master]: QoS host networks: caps now reports QoS for networks.

2013-08-01 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: QoS host networks: caps now reports QoS for networks. .. Patch Set 6: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3663/ :

Change in vdsm[master]: QoS host networks: caps now reports QoS for networks.

2013-08-01 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: QoS host networks: caps now reports QoS for networks. .. Patch Set 6: Verified Some refactorings, verfied in the usual way. -- To view, visit

Change in vdsm[master]: VDSM now writes its version number in VDSM-generated ifcfg f...

2013-08-01 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: VDSM now writes its version number in VDSM-generated ifcfg files .. Patch Set 1: Build Successful

Change in vdsm[master]: infra: Add logging in case PoolHandler crashes

2013-08-01 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: infra: Add logging in case PoolHandler crashes .. Patch Set 7: (1 inline comment) File

Change in vdsm[master]: Add support for direct setting of cpu_shares when creating a...

2013-08-01 Thread msivak
Martin Sivák has posted comments on this change. Change subject: Add support for direct setting of cpu_shares when creating a VM .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/17528 To unsubscribe, visit

Change in vdsm[master]: VDSM now writes its version number in VDSM-generated ifcfg f...

2013-08-01 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: VDSM now writes its version number in VDSM-generated ifcfg files .. Patch Set 1: Looks good to me, approved Thanks! -- To view, visit

Change in vdsm[master]: VDSM now writes its version number in VDSM-generated ifcfg f...

2013-08-01 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: VDSM now writes its version number in VDSM-generated ifcfg files .. VDSM now writes its version number in VDSM-generated ifcfg files Change-Id:

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-01 Thread amuller
Assaf Muller has uploaded a new change for review. Change subject: Management network is now kept in main routing table .. Management network is now kept in main routing table Only the management network should add its default

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-01 Thread amuller
Assaf Muller has posted comments on this change. Change subject: Management network is now kept in main routing table .. Patch Set 1: Verified Verification: Added and removed bonds, vlans, bridges over nics and the ovirtmgmt

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-01 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: Management network is now kept in main routing table .. Patch Set 3: I would prefer that you didn't submit this -- To view, visit

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-01 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Management network is now kept in main routing table .. Patch Set 3: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3667/ :

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-01 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: Management network is now kept in main routing table .. Patch Set 3: (1 inline comment) File

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-01 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Management network is now kept in main routing table .. Patch Set 3: I would prefer that you didn't submit this -- To view, visit

Change in vdsm[master]: QoS host networks: caps now reports QoS for networks.

2013-08-01 Thread wudxw
Mark Wu has posted comments on this change. Change subject: QoS host networks: caps now reports QoS for networks. .. Patch Set 6: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/17218

Change in vdsm[master]: QoS host networks: caps now reports QoS for networks.

2013-08-01 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: QoS host networks: caps now reports QoS for networks. .. Patch Set 6: Fails (1 inline comment) File

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-01 Thread amuller
Assaf Muller has posted comments on this change. Change subject: Management network is now kept in main routing table .. Patch Set 4: Verified -- To view, visit http://gerrit.ovirt.org/17575 To unsubscribe, visit

Change in vdsm[master]: QoS host networks: caps now reports QoS for networks.

2013-08-01 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: QoS host networks: caps now reports QoS for networks. .. Patch Set 6: No score (1 inline comment)

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-01 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Management network is now kept in main routing table .. Patch Set 4: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3668/ :

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-01 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: Management network is now kept in main routing table .. Patch Set 4: I would prefer that you didn't submit this (1 inline comment) -1 for visibility.

Change in vdsm[master]: spec: vdsm-hook-sriov on el6 should require libvirt

2013-08-01 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: spec: vdsm-hook-sriov on el6 should require libvirt .. Patch Set 1: I hate to clear DHC's verification, but Toni's comment it correct. -- To view, visit

Change in vdsm[master]: spec: vdsm-hook-sriov on el6 should not require non-existing...

2013-08-01 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: spec: vdsm-hook-sriov on el6 should not require non-existing rpm .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit

Change in vdsm[master]: QoS host networks: caps now reports QoS for networks.

2013-08-01 Thread gvallare
Giuseppe Vallarelli has posted comments on this change. Change subject: QoS host networks: caps now reports QoS for networks. .. Patch Set 7: Verified Added a fix to the functional network test. -- To view, visit

Change in vdsm[master]: spec: vdsm-hook-sriov on el6 should not require non-existing...

2013-08-01 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: spec: vdsm-hook-sriov on el6 should not require non-existing rpm .. Patch Set 2: Build Successful

Change in vdsm[master]: QoS host networks: caps now reports QoS for networks.

2013-08-01 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: QoS host networks: caps now reports QoS for networks. .. Patch Set 7: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3670/ :

Change in vdsm[master]: spec: vdsm-hook-sriov on el6 should not require non-existing...

2013-08-01 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: spec: vdsm-hook-sriov on el6 should not require non-existing rpm .. spec: vdsm-hook-sriov on el6 should not require non-existing rpm As reported by a

Change in vdsm[master]: spec: vdsm-hook-sriov on el6 should not require non-existing...

2013-08-01 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: spec: vdsm-hook-sriov on el6 should not require non-existing rpm .. Patch Set 2: Looks good to me, approved Thanks, guys. -- To view, visit

Change in vdsm[ovirt-3.3]: spec: vdsm-hook-sriov on el6 should not require non-existing...

2013-08-01 Thread danken
Hello Antoni Segura Puimedon, Dead Horse, I'd like you to do a code review. Please visit http://gerrit.ovirt.org/17591 to review the following change. Change subject: spec: vdsm-hook-sriov on el6 should not require non-existing rpm

Change in vdsm[master]: Add the ability to use bus and device IDs with the hostusb h...

2013-08-01 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: Add the ability to use bus and device IDs with the hostusb hook. .. Patch Set 6: Fails Build Failed

Change in vdsm[ovirt-3.3]: spec: vdsm-hook-sriov on el6 should not require non-existing...

2013-08-01 Thread asegurap
Antoni Segura Puimedon has posted comments on this change. Change subject: spec: vdsm-hook-sriov on el6 should not require non-existing rpm .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit

Change in vdsm[ovirt-3.3]: spec: vdsm-hook-sriov on el6 should not require non-existing...

2013-08-01 Thread danken
Dan Kenigsberg has submitted this change and it was merged. Change subject: spec: vdsm-hook-sriov on el6 should not require non-existing rpm .. spec: vdsm-hook-sriov on el6 should not require non-existing rpm As reported by a

Change in vdsm[ovirt-3.3]: spec: vdsm-hook-sriov on el6 should not require non-existing...

2013-08-01 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: spec: vdsm-hook-sriov on el6 should not require non-existing rpm .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17591 To

Change in vdsm[master]: Management network is now kept in main routing table

2013-08-01 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: Management network is now kept in main routing table .. Patch Set 4: I would prefer that you didn't submit this (1 inline comment)