<div dir="ltr">exactly - just posted a patch again that sings all fails as broken . <div>we&#39;ll get the report soon and I&#39;ll publish it as well. hope the run will take less time now</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 16, 2015 at 6:19 PM, Nir Soffer <span dir="ltr">&lt;<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Nice, but we cannot enable this until all the tests pass or disabled.<br>
<br>
There is no point in broken or flaky functional tests.<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Dec 16, 2015 at 5:34 PM, Yaniv Bronheim &lt;<a href="mailto:ybronhei@redhat.com">ybronhei@redhat.com</a>&gt; wrote:<br>
&gt; So its not stable. It won&#39;t block merges and at least give us report after<br>
&gt; each merge. It takes really long time to run it (because of the tests<br>
&gt; themselves. Lago things takes maximum 15minutes, but the run last for more<br>
&gt; than 2hrs right now and I suspect functional/storageTests.py gets stuck)<br>
&gt;<br>
&gt; Bellow you can see where we stand (before I added python-rtslib package).<br>
&gt;<br>
&gt; Now, I still want to merge the patch <a href="https://gerrit.ovirt.org/#/c/48268/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/48268/</a> -<br>
&gt; which enables this run after merges, and I still want you to consider the<br>
&gt; addition of Automation CI flag to our gerrit so that developer will be able<br>
&gt; to use it as a trigger for the check-merged.sh script run, just to see if<br>
&gt; their patch fixes\brakes something realted to the functional tests<br>
&gt;<br>
&gt; <a href="http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc23-x86_64/1480/" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc23-x86_64/1480/</a> - is<br>
&gt; an example of how the run looks like. I still work to improve the output<br>
&gt;<br>
&gt;<br>
&gt; Please reply and let me know if the idea around the automation flag is<br>
&gt; acceptable by you.. and please review the patch for comments and acks.<br>
&gt; We can ask dcaro to add the flag until Friday, otherwise we&#39;ll need to delay<br>
&gt; this effort after the holiday..<br>
&gt;<br>
&gt;<br>
&gt; functional.sosPluginTests.SosPluginTest<br>
&gt;     testSosPlugin                                               OK<br>
&gt; functional.vmRecoveryTests.RecoveryTests<br>
&gt;     test_vm_recovery                                            FAIL<br>
&gt; functional.vmQoSTests.VMQosTests<br>
&gt;     testSmallVMBallooning                                       FAIL<br>
&gt; functional.virtTests.VirtTest<br>
&gt;     testComplexVm                                               FAIL<br>
&gt;     testHeadlessVm                                              OK<br>
&gt;     testSimpleVm                                                FAIL<br>
&gt;     testVmDefinitionGraphics(&#39;spice&#39;)                           FAIL<br>
&gt;     testVmDefinitionGraphics(&#39;vnc&#39;)                             OK<br>
&gt;     testVmDefinitionLegacyGraphics(&#39;qxl&#39;)                       FAIL<br>
&gt;     testVmDefinitionLegacyGraphics(&#39;vnc&#39;)                       OK<br>
&gt;     testVmDefinitionMultipleGraphics(&#39;spice&#39;, &#39;vnc&#39;)            FAIL<br>
&gt;     testVmDefinitionMultipleGraphics(&#39;vnc&#39;, &#39;spice&#39;)            FAIL<br>
&gt;     testVmWithCdrom(&#39;self&#39;)                                     FAIL<br>
&gt;     testVmWithCdrom(&#39;specParams&#39;)                               FAIL<br>
&gt;     testVmWithCdrom(&#39;vmPayload&#39;)                                FAIL<br>
&gt;     testVmWithDevice(&#39;hotplugDisk&#39;)                             FAIL<br>
&gt;     testVmWithDevice(&#39;hotplugNic&#39;)                              FAIL<br>
&gt;     testVmWithDevice(&#39;smartcard&#39;)                               FAIL<br>
&gt;     testVmWithDevice(&#39;virtioNic&#39;)                               FAIL<br>
&gt;     testVmWithDevice(&#39;virtioRng&#39;)                               FAIL<br>
&gt;     testVmWithSla                                               FAIL<br>
&gt;     testVmWithStorage(&#39;iscsi&#39;)                                  SKIP:<br>
&gt; python-rtslib is not installed.<br>
&gt;     testVmWithStorage(&#39;localfs&#39;)                                FAIL<br>
&gt;     testVmWithStorage(&#39;nfs&#39;)                                    FAIL<br>
&gt; functional.storageTests.StorageTest<br>
&gt;     testCreatePoolErrors                                        OK<br>
&gt;     testStorage(&#39;glusterfs&#39;, 0)                                 ERROR<br>
&gt;     testStorage(&#39;glusterfs&#39;, 3)                                 ERROR<br>
&gt;     testStorage(&#39;iscsi&#39;, 0)                                     SKIP:<br>
&gt; python-rtslib is not installed.<br>
&gt;     testStorage(&#39;iscsi&#39;, 3)                                     SKIP:<br>
&gt; python-rtslib is not installed.<br>
&gt;     testStorage(&#39;localfs&#39;, 0)                                   FAIL<br>
&gt;     testStorage(&#39;localfs&#39;, 3)                                   FAIL<br>
&gt;     testStorage(&#39;nfs&#39;, 0)                                       FAIL<br>
&gt;     testStorage(&#39;nfs&#39;, 3)                                       FAIL<br>
&gt; functional.networkTests.NetworkTest<br>
&gt;     testAddVlanedBridgeless                                     ERROR<br>
&gt;     testAddVlanedBridgeless_oneCommand                          ERROR<br>
&gt;     testAfterNetworkSetupHook                                   ERROR<br>
&gt;     testBeforeNetworkSetupHook                                  ERROR<br>
&gt;     testBondHwAddress(False)                                    ERROR<br>
&gt;     testBondHwAddress(True)                                     ERROR<br>
&gt;     testBrokenNetworkReplacement(False)                         ERROR<br>
&gt;     testBrokenNetworkReplacement(True)                          ERROR<br>
&gt;     testDelNetworkBondAccumulation                              ERROR<br>
&gt;     testDelNetworkWithMTU(False)                                ERROR<br>
&gt;     testDelNetworkWithMTU(True)                                 ERROR<br>
&gt;     testDelWithoutAdd                                           ERROR<br>
&gt;     testDhclientLeases(4, &#39;default&#39;)                            ERROR<br>
&gt;     testDhclientLeases(4, &#39;local&#39;)                              ERROR<br>
&gt;     testDhclientLeases(6, None)                                 ERROR<br>
&gt;     testDhcpReplaceNicWithBridge                                ERROR<br>
&gt;     testFailWithInvalidBondingName(False)                       ERROR<br>
&gt;     testFailWithInvalidBondingName(True)                        ERROR<br>
&gt;     testFailWithInvalidBridgeName                               ERROR<br>
&gt;     testFailWithInvalidIpConfig                                 ERROR<br>
&gt;     testFailWithInvalidNic(False)                               ERROR<br>
&gt;     testFailWithInvalidNic(True)                                ERROR<br>
&gt;     testFailWithInvalidParams(False)                            ERROR<br>
&gt;     testFailWithInvalidParams(True)                             ERROR<br>
&gt;     testGetRouteDeviceTo                                        ERROR<br>
&gt;     testHonorBlockingDhcp                                       SKIP: Slow<br>
&gt; tests are disabled<br>
&gt;     testIpLinkWrapper                                           ERROR<br>
&gt;     testLowerMtuDoesNotOverride                                 FAIL<br>
&gt;     testNoBridgeLeftovers                                       FAIL<br>
&gt;     testReconfigureBrNetWithVanishedPort                        ERROR<br>
&gt;     testRedefineBondedNetworkIPs                                ERROR<br>
&gt;     testRemovingBridgeDoesNotLeaveBridge                        ERROR<br>
&gt;     testReorderBondingOptions(False)                            ERROR<br>
&gt;     testReorderBondingOptions(True)                             ERROR<br>
&gt;     testRestoreNetworksOnlyRestoreUnchangedDevices              ERROR<br>
&gt;     testRestoreToBlockingDHCP                                   ERROR<br>
&gt;     testSafeNetworkConfig(False)                                ERROR<br>
&gt;     testSafeNetworkConfig(True)                                 ERROR<br>
&gt;     testSelectiveRestoreDuringUpgrade                           ERROR<br>
&gt;     testSelectiveRestoreIgnoresVdsmRegParams                    FAIL<br>
&gt;     testSetupNetworkOutboundQos(False)                          ERROR<br>
&gt;     testSetupNetworkOutboundQos(True)                           FAIL<br>
&gt;     testSetupNetworksActiveSlave                                ERROR<br>
&gt;     testSetupNetworksAddBadParams(False)                        ERROR<br>
&gt;     testSetupNetworksAddBadParams(True)                         FAIL<br>
&gt;     testSetupNetworksAddBondWithManyVlans(False)                ERROR<br>
&gt;     testSetupNetworksAddBondWithManyVlans(True)                 FAIL<br>
&gt;     testSetupNetworksAddDelBondedNetwork(False)                 ERROR<br>
&gt;     testSetupNetworksAddDelBondedNetwork(True)                  FAIL<br>
&gt;     testSetupNetworksAddDelDhcp(False, (4, 6))                  ERROR<br>
&gt;     testSetupNetworksAddDelDhcp(False, (4,))                    ERROR<br>
&gt;     testSetupNetworksAddDelDhcp(False, (6,))                    ERROR<br>
&gt;     testSetupNetworksAddDelDhcp(True, (4, 6))                   FAIL<br>
&gt;     testSetupNetworksAddDelDhcp(True, (4,))                     FAIL<br>
&gt;     testSetupNetworksAddDelDhcp(True, (6,))                     FAIL<br>
&gt;     testSetupNetworksAddManyVlans(False)                        ERROR<br>
&gt;     testSetupNetworksAddManyVlans(True)                         FAIL<br>
&gt;     testSetupNetworksAddNetworkToNicAfterBondBreaking(False)    ERROR<br>
&gt;     testSetupNetworksAddNetworkToNicAfterBondBreaking(True)     FAIL<br>
&gt;     testSetupNetworksAddNetworkToNicAfterBondResizing(False)    ERROR<br>
&gt;     testSetupNetworksAddNetworkToNicAfterBondResizing(True)     ERROR<br>
&gt;     testSetupNetworksAddOverExistingBond(False)                 ERROR<br>
&gt;     testSetupNetworksAddOverExistingBond(True)                  ERROR<br>
&gt;     testSetupNetworksAddVlan(False)                             ERROR<br>
&gt;     testSetupNetworksAddVlan(True)                              FAIL<br>
&gt;     testSetupNetworksConnectivityCheck                          FAIL<br>
&gt;     testSetupNetworksConvertVlanNetBridgeness                   ERROR<br>
&gt;     testSetupNetworksDelOneOfBondNets                           ERROR<br>
&gt;     testSetupNetworksDeletesTheBridgeOnlyWhenItIsReconfigured   ERROR<br>
&gt;     testSetupNetworksEmergencyDevicesCleanupBondOverwrite(False)ERROR<br>
&gt;     testSetupNetworksEmergencyDevicesCleanupBondOverwrite(True) SKIP: Slow<br>
&gt; tests are disabled<br>
&gt;     testSetupNetworksEmergencyDevicesCleanupVlanOverwrite(False)ERROR<br>
&gt;     testSetupNetworksEmergencyDevicesCleanupVlanOverwrite(True) SKIP: Slow<br>
&gt; tests are disabled<br>
&gt;     testSetupNetworksKeepNetworkOnBondAfterBondResizing(False)  ERROR<br>
&gt;     testSetupNetworksKeepNetworkOnBondAfterBondResizing(True)   FAIL<br>
&gt;     testSetupNetworksMtus(False)                                ERROR<br>
&gt;     testSetupNetworksMtus(True)                                 SKIP: This<br>
&gt; test fails because the 2 different networks are getting configured with the<br>
&gt; same MTU. The test should assert that the reported MTUs are equal to the<br>
&gt; requested ones.<br>
&gt;     testSetupNetworksMultiMTUsOverBond(False)                   ERROR<br>
&gt;     testSetupNetworksMultiMTUsOverBond(True)                    FAIL<br>
&gt;     testSetupNetworksMultiMTUsOverNic(False)                    ERROR<br>
&gt;     testSetupNetworksMultiMTUsOverNic(True)                     FAIL<br>
&gt;     testSetupNetworksNetCompatibilityMultipleNetsSameNic(False) ERROR<br>
&gt;     testSetupNetworksNetCompatibilityMultipleNetsSameNic(True)  FAIL<br>
&gt;     testSetupNetworksNicless                                    FAIL<br>
&gt;     testSetupNetworksNiclessBridgeless                          ERROR<br>
&gt;     testSetupNetworksOverDhcpIface                              ERROR<br>
&gt;     testSetupNetworksOverExistingBridge                         FAIL<br>
&gt;     testSetupNetworksReconfigureBridge                          FAIL<br>
&gt;     testSetupNetworksRemoveBondWithKilledEnslavedNics           ERROR<br>
&gt;     testSetupNetworksRemoveSlavelessBond                        ERROR<br>
&gt;     testSetupNetworksResizeBond(False)                          ERROR<br>
&gt;     testSetupNetworksResizeBond(True)                           ERROR<br>
&gt;     testSetupNetworksStableBond(False)                          ERROR<br>
&gt;     testSetupNetworksStableBond(True)                           ERROR<br>
&gt;     testStaticNetworkConfig((4, 6))                             FAIL<br>
&gt;     testStaticNetworkConfig((4,))                               FAIL<br>
&gt;     testStaticNetworkConfig((6,))                               FAIL<br>
&gt;     testStaticSourceRouting(False)                              ERROR<br>
&gt;     testStaticSourceRouting(True)                               FAIL<br>
&gt;     testTwiceAdd(False)                                         ERROR<br>
&gt;     testTwiceAdd(True)                                          ERROR<br>
&gt;     testVolatileConfig(False)                                   ERROR<br>
&gt;     testVolatileConfig(True)                                    ERROR<br>
&gt;     test_drop_initial_bond_slaves_ip_config                     FAIL<br>
&gt;     test_getVdsStats                                            ERROR<br>
&gt;     test_remove_bond_under_network                              FAIL<br>
&gt;     test_remove_initial_network_nic_ip_config                   FAIL<br>
&gt;     test_rollback                                               FAIL<br>
&gt;     test_setupNetworks_bond_with_custom_option                  ERROR<br>
&gt;     test_setupNetworks_on_external_bond                         ERROR<br>
&gt;     test_setupNetworks_on_external_vlaned_bond                  ERROR<br>
&gt; functional.networkTestsOVS.OVSNetworkTest<br>
&gt;     testHonorBlockingDhcp                                       SKIP: Slow<br>
&gt; tests are disabled<br>
&gt;     testLowerMtuDoesNotOverride                                 OK<br>
&gt;     testNoBridgeLeftovers                                       FAIL<br>
&gt;     testSelectiveRestoreIgnoresVdsmRegParams                    FAIL<br>
&gt;     testSetupNetworkOutboundQos(True)                           OK<br>
&gt;     testSetupNetworksAddBadParams(True)                         OK<br>
&gt;     testSetupNetworksAddBondWithManyVlans(True)                 FAIL<br>
&gt;     testSetupNetworksAddDelBondedNetwork(True)                  FAIL<br>
&gt;     testSetupNetworksAddDelDhcp(True, (4, 6))                   FAIL<br>
&gt;     testSetupNetworksAddDelDhcp(True, (4,))                     FAIL<br>
&gt;     testSetupNetworksAddDelDhcp(True, (6,))                     FAIL<br>
&gt;     testSetupNetworksAddManyVlans(True)                         OK<br>
&gt;     testSetupNetworksAddNetworkToNicAfterBondBreaking(True)     FAIL<br>
&gt;     testSetupNetworksAddVlan(True)                              FAIL<br>
&gt;     testSetupNetworksConnectivityCheck                          OK<br>
&gt;     testSetupNetworksEmergencyDevicesCleanupBondOverwrite(True) SKIP: Slow<br>
&gt; tests are disabled<br>
&gt;     testSetupNetworksEmergencyDevicesCleanupVlanOverwrite(True) SKIP: Slow<br>
&gt; tests are disabled<br>
&gt;     testSetupNetworksKeepNetworkOnBondAfterBondResizing(True)   FAIL<br>
&gt;     testSetupNetworksMtus(True)                                 SKIP: This<br>
&gt; test fails because the 2 different networks are getting configured with the<br>
&gt; same MTU. The test should assert that the reported MTUs are equal to the<br>
&gt; requested ones.<br>
&gt;     testSetupNetworksMultiMTUsOverBond(True)                    FAIL<br>
&gt;     testSetupNetworksMultiMTUsOverNic(True)                     FAIL<br>
&gt;     testSetupNetworksNetCompatibilityMultipleNetsSameNic(True)  FAIL<br>
&gt;     testSetupNetworksNicless                                    OK<br>
&gt;     testSetupNetworksOverExistingBridge                         OK<br>
&gt;     testSetupNetworksReconfigureBridge                          FAIL<br>
&gt;     testStaticNetworkConfig((4, 6))                             FAIL<br>
&gt;     testStaticNetworkConfig((4,))                               FAIL<br>
&gt;     testStaticNetworkConfig((6,))                               FAIL<br>
&gt;     testStaticSourceRouting(True)                               FAIL<br>
&gt;     test_drop_initial_bond_slaves_ip_config                     FAIL<br>
&gt;     test_ovirtmgmtm_to_ovs                                      FAIL<br>
&gt;     test_remove_bond_under_network                              SKIP: OVS<br>
&gt; does not support bridgeless networks<br>
&gt;     test_remove_initial_network_nic_ip_config                   OK<br>
&gt;     test_rollback                                               FAIL<br>
&gt;<br>
&gt;<br>
&gt; On Sun, Dec 13, 2015 at 10:07 PM, Piotr Kliczewski &lt;<a href="mailto:pkliczew@redhat.com">pkliczew@redhat.com</a>&gt;<br>
&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; I like the idea but I have the same feelings as Francesco. I think that we<br>
&gt;&gt; need to make sure that functional tests for each vertical are stable before<br>
&gt;&gt; enabling this process.<br>
&gt;&gt;<br>
&gt;&gt; On Sun, Dec 13, 2015 at 8:34 AM, Eyal Edri &lt;<a href="mailto:eedri@redhat.com">eedri@redhat.com</a>&gt; wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; adding also infra team for visibility on the change in CI.<br>
&gt;&gt;&gt; also inline.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; On Fri, Dec 11, 2015 at 4:19 PM, Francesco Romani &lt;<a href="mailto:fromani@redhat.com">fromani@redhat.com</a>&gt;<br>
&gt;&gt;&gt; wrote:<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; ----- Original Message -----<br>
&gt;&gt;&gt;&gt; &gt; From: &quot;Yaniv Bronheim&quot; &lt;<a href="mailto:ybronhei@redhat.com">ybronhei@redhat.com</a>&gt;<br>
&gt;&gt;&gt;&gt; &gt; To: <a href="mailto:devel@ovirt.org">devel@ovirt.org</a>, &quot;Francesco Romani&quot; &lt;<a href="mailto:fromani@redhat.com">fromani@redhat.com</a>&gt;, &quot;Nir<br>
&gt;&gt;&gt;&gt; &gt; Soffer&quot; &lt;<a href="mailto:nsoffer@redhat.com">nsoffer@redhat.com</a>&gt;, &quot;Piotr Kliczewski&quot;<br>
&gt;&gt;&gt;&gt; &gt; &lt;<a href="mailto:pkliczew@redhat.com">pkliczew@redhat.com</a>&gt;<br>
&gt;&gt;&gt;&gt; &gt; Cc: &quot;danken&quot; &lt;<a href="mailto:danken@redhat.com">danken@redhat.com</a>&gt;, &quot;David Caro&quot; &lt;<a href="mailto:dcaroest@redhat.com">dcaroest@redhat.com</a>&gt;,<br>
&gt;&gt;&gt;&gt; &gt; &quot;Eyal Edri&quot; &lt;<a href="mailto:eedri@redhat.com">eedri@redhat.com</a>&gt;<br>
&gt;&gt;&gt;&gt; &gt; Sent: Thursday, December 10, 2015 6:46:37 PM<br>
&gt;&gt;&gt;&gt; &gt; Subject: Automation CI for vdsm<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; [...]<br>
&gt;&gt;&gt;&gt; &gt; We want to allow developers to trigger the script once reviews and<br>
&gt;&gt;&gt;&gt; &gt; verification are ready (last step before merge). To do so we agreed to<br>
&gt;&gt;&gt;&gt; &gt; add<br>
&gt;&gt;&gt;&gt; &gt; Continues Integration flag for each vdsm patch.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; This flag will be called &#39;Workflow&#39; or we can name it otherwise, we just<br>
&gt;&gt;&gt; need to choose what makes sense.<br>
&gt;&gt;&gt; David/Yaniv - Please correct me if I&#39;m wrong.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Once this flag will be<br>
&gt;&gt;&gt;&gt; &gt; signed with +1 it will trigger Jenkins CI to run the check-merged<br>
&gt;&gt;&gt;&gt; &gt; script<br>
&gt;&gt;&gt;&gt; &gt; (adding new button to gerrit is not an option - you can image that<br>
&gt;&gt;&gt;&gt; &gt; flag as<br>
&gt;&gt;&gt;&gt; &gt; a trigger button), on success Jenkins CI flag will turn to +2. on fail<br>
&gt;&gt;&gt;&gt; &gt; we&#39;ll get -1 and once new patchset is ready the developer will remove<br>
&gt;&gt;&gt;&gt; &gt; the<br>
&gt;&gt;&gt;&gt; &gt; +1 and add it back to the Continues Integration flag to re-trigger the<br>
&gt;&gt;&gt;&gt; &gt; job.<br>
&gt;&gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt;&gt; &gt; Please ack the process before we move on with that<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Sounds good, even though I&#39;m a little scared (just gut feeling, no<br>
&gt;&gt;&gt;&gt; evidence<br>
&gt;&gt;&gt;&gt; whatsoever) that this could add even more complexity and fragility to<br>
&gt;&gt;&gt;&gt; the jenkins<br>
&gt;&gt;&gt;&gt; fleet.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; In the long run, when this is reliable, it will help greatly.<br>
&gt;&gt;&gt;&gt; In the short term, I&#39;m scared because this can lead to false positives<br>
&gt;&gt;&gt;&gt; and bogus<br>
&gt;&gt;&gt;&gt; failures.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Let me stress I don&#39;t have concrete item to share or specific flaws.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; As action item on me, I will find some time next week to check virt<br>
&gt;&gt;&gt;&gt; functional tests,<br>
&gt;&gt;&gt;&gt; to see if they need some fixes, work reliably and so forth<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; &gt; The patch for those scripts still under review and testing -<br>
&gt;&gt;&gt;&gt; &gt; <a href="https://gerrit.ovirt.org/#/c/48268" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/48268</a><br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Will review asap.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; --<br>
&gt;&gt;&gt;&gt; Francesco Romani<br>
&gt;&gt;&gt;&gt; RedHat Engineering Virtualization R &amp; D<br>
&gt;&gt;&gt;&gt; Phone: 8261328<br>
&gt;&gt;&gt;&gt; IRC: fromani<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; --<br>
&gt;&gt;&gt; Eyal Edri<br>
&gt;&gt;&gt; Supervisor, RHEV CI<br>
&gt;&gt;&gt; EMEA ENG Virtualization R&amp;D<br>
&gt;&gt;&gt; Red Hat Israel<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; phone: <a href="tel:%2B972-9-7692018" value="+97297692018">+972-9-7692018</a><br>
&gt;&gt;&gt; irc: eedri (on #tlv #rhev-dev #rhev-integ)<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; --<br>
&gt; Yaniv Bronhaim.<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><span style="font-size:12.8px"><b>Yaniv Bronhaim.</b></span><br></div></div></div></div></div>
</div>