[Engine-patches] Change in ovirt-engine[master]: core: OvfDataUpdater - removal of update/remove OVF vdsm cal...

2012-12-23 Thread vvolansk
Vered Volansky has posted comments on this change. Change subject: core: OvfDataUpdater - removal of update/remove OVF vdsm calls .. Patch Set 28: (5 inline comments) Sorry, some minor comments

[Engine-patches] Change in ovirt-engine[master]: engine: Added VM NIC validator

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: engine: Added VM NIC validator .. engine: Added VM NIC validator Validator will be used to check if the NIC is valid for unlinking or for setting no net

[Engine-patches] Change in ovirt-engine[master]: engine: Added variable replacements to ValidationResult

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: engine: Added variable replacements to ValidationResult .. engine: Added variable replacements to ValidationResult Added a possibility to include variab

[Engine-patches] Change in ovirt-engine[master]: core: HotPlugEnabled shouldn't be reloadable.

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: HotPlugEnabled shouldn't be reloadable. .. core: HotPlugEnabled shouldn't be reloadable. We shouldn't allow modifying this property. Change-Id: I

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking to AddVmInterfaceCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: add NetworkLinking to AddVmInterfaceCommand .. core: add NetworkLinking to AddVmInterfaceCommand - canDoAction- If "UnlinkingSupported" is false f

[Engine-patches] Change in ovirt-engine[master]: core: Add NetworkLinking support to UpdateVmCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Add NetworkLinking support to UpdateVmCommand .. core: Add NetworkLinking support to UpdateVmCommand - Avoid NPE in case there is a vnic with null

[Engine-patches] Change in ovirt-engine[master]: core: adding AbstractVmInterfaceCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: adding AbstractVmInterfaceCommand .. core: adding AbstractVmInterfaceCommand This command will contain common logic of VmIntrefaceCommands. For n

[Engine-patches] Change in ovirt-engine[master]: engine: Rename HotPlugUnplgNicVDSParameters

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: engine: Rename HotPlugUnplgNicVDSParameters .. engine: Rename HotPlugUnplgNicVDSParameters Renamed the parameters to VmNicDeviceVDSParameters so that th

[Engine-patches] Change in ovirt-engine[master]: core: adding VdsmError for failure on VmUpdateDevice on Vnic...

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: adding VdsmError for failure on VmUpdateDevice on Vnic. .. core: adding VdsmError for failure on VmUpdateDevice on Vnic. If the vdsm fails to upda

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking to AddVmTemplateInterfaceCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: add NetworkLinking to AddVmTemplateInterfaceCommand .. core: add NetworkLinking to AddVmTemplateInterfaceCommand - canDoAction- If "UnlinkingSuppo

[Engine-patches] Change in ovirt-engine[master]: core: allow editing VmInterface while the VM is running.

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: allow editing VmInterface while the VM is running. .. core: allow editing VmInterface while the VM is running. NetworkLinking feature allows editi

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking to UpdateVmInterfaceCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: add NetworkLinking to UpdateVmInterfaceCommand .. core: add NetworkLinking to UpdateVmInterfaceCommand - messages- add bll errors related to "link

[Engine-patches] Change in ovirt-engine[master]: core, ui: add/update linking bll-error messages.

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core,ui: add/update linking bll-error messages. .. core,ui: add/update linking bll-error messages. UNLINKING_IS_NOT_SUPPORTED NULL_NETWORK_IS_NOT_SUPPOR

[Engine-patches] Change in ovirt-engine[master]: core: adding "NetworkLinkingSupported" to vdc_options.

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: adding "NetworkLinkingSupported" to vdc_options. .. core: adding "NetworkLinkingSupported" to vdc_options. Changing LinkState is supported only fo

[Engine-patches] Change in ovirt-engine[master]: core: Add hashCode, equals & toString to ValidationResult

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Add hashCode, equals & toString to ValidationResult .. core: Add hashCode, equals & toString to ValidationResult Added these methods to allow test

[Engine-patches] Change in ovirt-engine[master]: core: 'No Network' on a Vnic should be marked as null and no...

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: 'No Network' on a Vnic should be marked as null and not empty String. .. core: 'No Network' on a Vnic should be marked as null and not empty Strin

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking to UpdateVmTemplateInterfaceCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: add NetworkLinking to UpdateVmTemplateInterfaceCommand .. core: add NetworkLinking to UpdateVmTemplateInterfaceCommand - canDoAction- If "Unlinkin

[Engine-patches] Change in ovirt-engine[master]: engine: DetachNetworkToVdsGroup for NetworkLinking

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: engine: DetachNetworkToVdsGroup for NetworkLinking .. engine: DetachNetworkToVdsGroup for NetworkLinking This patch makes sure NPEs are avoided due to t

[Engine-patches] Change in ovirt-engine[master]: core: HotPlugNicVDSCommand should send "linkActive" to the v...

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: HotPlugNicVDSCommand should send "linkActive" to the vdsm. .. core: HotPlugNicVDSCommand should send "linkActive" to the vdsm. - If "linked" prope

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking support to RunVm command.

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: add NetworkLinking support to RunVm command. .. core: add NetworkLinking support to RunVm command. - If "linked" property on VmNetworkInterface is

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking to AddVmTemplateCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: add NetworkLinking to AddVmTemplateCommand .. core: add NetworkLinking to AddVmTemplateCommand Linked property on VmNetworkInterface should also b

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking to ChangeVMClusterCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: add NetworkLinking to ChangeVMClusterCommand .. core: add NetworkLinking to ChangeVMClusterCommand When changing the VM Cluster- if the new Cluste

[Engine-patches] Change in ovirt-engine[master]: core : add NetwookLinking support to VmInterfaceManager

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core : add NetwookLinking support to VmInterfaceManager .. core : add NetwookLinking support to VmInterfaceManager - Support the case when vm has vnic/s

[Engine-patches] Change in ovirt-engine[master]: core: create UpdateVmInterfaceVDSCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: create UpdateVmInterfaceVDSCommand .. core: create UpdateVmInterfaceVDSCommand The command will be called from UpdateVmInterfaceCommand. It will b

[Engine-patches] Change in ovirt-engine[master]: core: Add NetworkLinking support to OvfVm.

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Add NetworkLinking support to OvfVm. .. core: Add NetworkLinking support to OvfVm. - Support null networkName of a vnic. - New "Linked" property o

[Engine-patches] Change in ovirt-engine[master]: webadmin: copy "linked" property on VmNetworkInterface clone...

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: webadmin: copy "linked" property on VmNetworkInterface cloner. .. webadmin: copy "linked" property on VmNetworkInterface cloner. "linked" boolean is a n

[Engine-patches] Change in ovirt-engine[master]: core: Add NetworkLinking support to OvfTemplate.

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Add NetworkLinking support to OvfTemplate. .. core: Add NetworkLinking support to OvfTemplate. - Support null networkName of a vnic. - New "Linked

[Engine-patches] Change in ovirt-engine[master]: core: adding "NetworkLinkingSupported" to vdc_options.

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: adding "NetworkLinkingSupported" to vdc_options. .. Patch Set 10: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit htt

[Engine-patches] Change in ovirt-engine[master]: engine: Add NetworkLinking support to ImportVmTemplateComman...

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: engine: Add NetworkLinking support to ImportVmTemplateCommand. .. engine: Add NetworkLinking support to ImportVmTemplateCommand. - Add linking to the co

[Engine-patches] Change in ovirt-engine[master]: core: HotPlugEnabled shouldn't be reloadable.

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: HotPlugEnabled shouldn't be reloadable. .. Patch Set 15: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: core: Add hashCode, equals & toString to ValidationResult

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Add hashCode, equals & toString to ValidationResult .. Patch Set 3: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit h

[Engine-patches] Change in ovirt-engine[master]: core, ui: add/update linking bll-error messages.

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core,ui: add/update linking bll-error messages. .. Patch Set 25: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http://ger

[Engine-patches] Change in ovirt-engine[master]: engine: Added VM NIC validator

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: engine: Added VM NIC validator .. Patch Set 3: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http://gerrit.ovirt.org/1026

[Engine-patches] Change in ovirt-engine[master]: engine: Added variable replacements to ValidationResult

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: engine: Added variable replacements to ValidationResult .. Patch Set 3: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit htt

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking to AddVmInterfaceCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: add NetworkLinking to AddVmInterfaceCommand .. Patch Set 9: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking support to RunVm command.

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: add NetworkLinking support to RunVm command. .. Patch Set 22: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http://

[Engine-patches] Change in ovirt-engine[master]: core: 'No Network' on a Vnic should be marked as null and no...

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: 'No Network' on a Vnic should be marked as null and not empty String. .. Patch Set 20: Verified; Looks good to me, approved Rebased, no changes since ack

[Engine-patches] Change in ovirt-engine[master]: core: HotPlugNicVDSCommand should send "linkActive" to the v...

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: HotPlugNicVDSCommand should send "linkActive" to the vdsm. .. Patch Set 22: Verified; Looks good to me, approved Rebased, no changes since ack -- To view,

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking to ChangeVMClusterCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: add NetworkLinking to ChangeVMClusterCommand .. Patch Set 20: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http://

[Engine-patches] Change in ovirt-engine[master]: engine: DetachNetworkToVdsGroup for NetworkLinking

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: engine: DetachNetworkToVdsGroup for NetworkLinking .. Patch Set 10: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http://

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking to UpdateVmTemplateInterfaceCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: add NetworkLinking to UpdateVmTemplateInterfaceCommand .. Patch Set 7: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visi

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking to AddVmTemplateInterfaceCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: add NetworkLinking to AddVmTemplateInterfaceCommand .. Patch Set 24: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking to AddVmTemplateCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: add NetworkLinking to AddVmTemplateCommand .. Patch Set 20: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: core: adding AbstractVmInterfaceCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: adding AbstractVmInterfaceCommand .. Patch Set 17: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http://gerrit.ovir

[Engine-patches] Change in ovirt-engine[master]: core: allow editing VmInterface while the VM is running.

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: allow editing VmInterface while the VM is running. .. Patch Set 19: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit h

[Engine-patches] Change in ovirt-engine[master]: engine: Rename HotPlugUnplgNicVDSParameters

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: engine: Rename HotPlugUnplgNicVDSParameters .. Patch Set 3: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http://gerrit.o

[Engine-patches] Change in ovirt-engine[master]: core: adding VdsmError for failure on VmUpdateDevice on Vnic...

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: adding VdsmError for failure on VmUpdateDevice on Vnic. .. Patch Set 17: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, vi

[Engine-patches] Change in ovirt-engine[master]: core: create UpdateVmInterfaceVDSCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: create UpdateVmInterfaceVDSCommand .. Patch Set 18: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: core: add NetworkLinking to UpdateVmInterfaceCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: add NetworkLinking to UpdateVmInterfaceCommand .. Patch Set 21: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http:

[Engine-patches] Change in ovirt-engine[master]: core : add NetwookLinking support to VmInterfaceManager

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core : add NetwookLinking support to VmInterfaceManager .. Patch Set 15: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit ht

[Engine-patches] Change in ovirt-engine[master]: core: Add NetworkLinking support to UpdateVmCommand

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Add NetworkLinking support to UpdateVmCommand .. Patch Set 12: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http:/

[Engine-patches] Change in ovirt-engine[master]: engine: Add NetworkLinking support to ImportVmTemplateComman...

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: engine: Add NetworkLinking support to ImportVmTemplateCommand. .. Patch Set 12: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, v

[Engine-patches] Change in ovirt-engine[master]: core: Add NetworkLinking support to OvfVm.

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Add NetworkLinking support to OvfVm. .. Patch Set 12: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http://gerrit.o

[Engine-patches] Change in ovirt-engine[master]: core: Add NetworkLinking support to OvfTemplate.

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Add NetworkLinking support to OvfTemplate. .. Patch Set 12: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: webadmin: copy "linked" property on VmNetworkInterface clone...

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: webadmin: copy "linked" property on VmNetworkInterface cloner. .. Patch Set 12: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, v

[Engine-patches] Change in ovirt-engine[master]: core: adding "linked" property to VmNetworkInterface

2012-12-23 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: adding "linked" property to VmNetworkInterface .. core: adding "linked" property to VmNetworkInterface adding linked property to- - VmNetworkInter

[Engine-patches] Change in ovirt-engine[master]: core: adding "linked" property to VmNetworkInterface

2012-12-23 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: adding "linked" property to VmNetworkInterface .. Patch Set 19: Verified; Looks good to me, approved Rebased, no changes since ack -- To view, visit http:

[Engine-patches] Change in ovirt-engine[master]: core: adding GetAllDisksByStorageDomainIdQuery

2012-12-23 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: adding GetAllDisksByStorageDomainIdQuery .. Patch Set 2: Michael Pasternak - are you asking or saying it will serve also API? Regarding API queries, we sh

[Engine-patches] Change in ovirt-engine[master]: core: adding GetAllDisksByStorageDomainIdQuery

2012-12-23 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: adding GetAllDisksByStorageDomainIdQuery .. Patch Set 2: Please add unit-test for the query as well. -- To view, visit http://gerrit.ovirt.org/10321 To u

[Engine-patches] Change in ovirt-engine[master]: webadmin: Filter Non-Operational DCs when adding new domain

2012-12-23 Thread asaf
Asaf Shakarchi has posted comments on this change. Change subject: webadmin: Filter Non-Operational DCs when adding new domain .. Patch Set 1: (1 inline comment) File fronte

[Engine-patches] Change in ovirt-engine[master]: Adding HostSubTab to UI for BIOS information

2012-12-23 Thread derez
Daniel Erez has posted comments on this change. Change subject: Adding HostSubTab to UI for BIOS information .. Patch Set 6: (1 inline comment) File frontend/webadmin/module

[Engine-patches] Change in ovirt-engine[master]: Adding HostSubTab to UI for BIOS information

2012-12-23 Thread derez
Daniel Erez has posted comments on this change. Change subject: Adding HostSubTab to UI for BIOS information .. Patch Set 6: (1 inline comment) File frontend/webadmin/module

[Engine-patches] Change in ovirt-engine[master]: packaging: add configurable prefix to rpm list

2012-12-23 Thread mgoldboi
Moran Goldboim has posted comments on this change. Change subject: packaging: add configurable prefix to rpm list .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10307 T

[Engine-patches] Change in ovirt-engine[master]: core: support multiple concurrent disks migration

2012-12-23 Thread derez
Daniel Erez has posted comments on this change. Change subject: core: support multiple concurrent disks migration .. Patch Set 3: (1 inline comment) File frontend/webadmin/m

[Engine-patches] Change in ovirt-engine[master]: core: remove obsolete StorageDomainTemplateQuery

2012-12-23 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: core: remove obsolete StorageDomainTemplateQuery .. core: remove obsolete StorageDomainTemplateQuery Cleanup - removing obsolete query: StorageDomainTempl

[Engine-patches] Change in ovirt-engine[master]: core: remove obsolete StorageDomainTemplateQuery

2012-12-23 Thread derez
Daniel Erez has posted comments on this change. Change subject: core: remove obsolete StorageDomainTemplateQuery .. Patch Set 2: Verified; Looks good to me, approved rebased -- To view, visit http://gerrit.ovirt.org/10320 To u

[Engine-patches] Change in ovirt-engine[master]: Adding HostSubTab to UI for BIOS information

2012-12-23 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Adding HostSubTab to UI for BIOS information .. Patch Set 6: Verified -- To view, visit http://gerrit.ovirt.org/10115 To unsubscribe, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: Adding host bios fields to vdsStatic table

2012-12-23 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Adding host bios fields to vdsStatic table .. Patch Set 4: Verified -- To view, visit http://gerrit.ovirt.org/10114 To unsubscribe, visit http://gerrit.ovirt.or

[Engine-patches] Change in ovirt-engine[master]: Adding bios information to vds object

2012-12-23 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Adding bios information to vds object .. Patch Set 14: Verified -- To view, visit http://gerrit.ovirt.org/9337 To unsubscribe, visit http://gerrit.ovirt.org/set

[Engine-patches] Change in ovirt-engine[master]: Adding HostSubTab to UI for BIOS information

2012-12-23 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: Adding HostSubTab to UI for BIOS information .. Patch Set 6: (2 inline comments) minor comments File front

[Engine-patches] Change in ovirt-engine[master]: frontend: remove Config dependency from frontend

2012-12-23 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: frontend: remove Config dependency from frontend .. Patch Set 1: But if you don't complete this patch and fetch from server, it will be harder to inject the cons

[Engine-patches] Change in ovirt-engine[master]: frontend: remove Config dependency from frontend

2012-12-23 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: frontend: remove Config dependency from frontend .. Patch Set 1: I agree and disagree; I agree that this is only the first half of the solution, still need to f

[Engine-patches] Change in ovirt-engine[master]: core: log message on evaluateCorrelationId failure

2012-12-23 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: core: log message on evaluateCorrelationId failure .. core: log message on evaluateCorrelationId failure Added a log message warning for evaluateCorrelati

[Engine-patches] Change in ovirt-engine[master]: core: log message on evaluateCorrelationId failure

2012-12-23 Thread derez
Daniel Erez has posted comments on this change. Change subject: core: log message on evaluateCorrelationId failure .. Patch Set 2: Verified; Looks good to me, approved rebased -- To view, visit http://gerrit.ovirt.org/10318 To

[Engine-patches] Change in ovirt-engine[master]: core: log message on evaluateCorrelationId failure

2012-12-23 Thread derez
Daniel Erez has posted comments on this change. Change subject: core: log message on evaluateCorrelationId failure .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/10318 To unsubscribe, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: webadmin: local storage - missing correlation ID

2012-12-23 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: webadmin: local storage - missing correlation ID .. webadmin: local storage - missing correlation ID Configure local storage: added correlation ID for Add

[Engine-patches] Change in ovirt-engine[master]: webadmin: local storage - missing correlation ID

2012-12-23 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: local storage - missing correlation ID .. Patch Set 2: Verified; Looks good to me, approved rebased -- To view, visit http://gerrit.ovirt.org/10317 To u

[Engine-patches] Change in ovirt-engine[master]: webadmin: local storage - missing correlation ID

2012-12-23 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: local storage - missing correlation ID .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/10317 To unsubscribe, visit http://gerrit.ovirt

[Engine-patches] Change in ovirt-engine[master]: core: cleanup in VdsDeploy

2012-12-23 Thread abonas
Alissa Bonas has posted comments on this change. Change subject: core: cleanup in VdsDeploy .. Patch Set 3: Verified Yair - please do merge :) -- To view, visit http://gerrit.ovirt.org/10290 To unsubscribe, visit http://gerrit

[Engine-patches] Change in ovirt-engine[master]: webadmin: VmInterfaceListModelTable style modifications

2012-12-23 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: VmInterfaceListModelTable style modifications .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/10324 To unsubscribe, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: webadmin: VmInterfaceListModelTable style modifications

2012-12-23 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: webadmin: VmInterfaceListModelTable style modifications .. webadmin: VmInterfaceListModelTable style modifications Modified inner panels' margins in VmInt

[Engine-patches] Change in ovirt-engine[master]: webadmin: VmSnapshotListModelTable style modifications

2012-12-23 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: webadmin: VmSnapshotListModelTable style modifications .. webadmin: VmSnapshotListModelTable style modifications Modified inner panels' margins in VmSnaps

[Engine-patches] Change in ovirt-engine[master]: webadmin: VmSnapshotListModelTable style modifications

2012-12-23 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: VmSnapshotListModelTable style modifications .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10323 To unsu

[Engine-patches] Change in ovirt-engine[master]: frontend: remove Config dependency from frontend

2012-12-23 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: frontend: remove Config dependency from frontend .. Patch Set 1: Oh... I thought we fetch it from server. There is no advantage of this over the previous state,

[Engine-patches] Change in ovirt-engine[master]: frontend: remove Config dependency from frontend

2012-12-23 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: frontend: remove Config dependency from frontend .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/10330 To unsubscribe, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: frontend: remove Config dependency from frontend

2012-12-23 Thread gchaplik
Gilad Chaplik has uploaded a new change for review. Change subject: frontend: remove Config dependency from frontend .. frontend: remove Config dependency from frontend The only use of class org.ovirt.engine.core.common.config.C

[Engine-patches] Change in ovirt-engine[master]: webadmin: Network tab, minor sub tab fixes

2012-12-23 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: webadmin: Network tab, minor sub tab fixes .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10329 To unsubscribe, visit http:

[Engine-patches] Change in ovirt-engine[master]: webadmin: Network tab, minor sub tab fixes

2012-12-23 Thread alkaplan
Alona Kaplan has uploaded a new change for review. Change subject: webadmin: Network tab, minor sub tab fixes .. webadmin: Network tab, minor sub tab fixes network->host. - default should be attached. - s/Nic/Network Device (on

[Engine-patches] Change in ovirt-engine[master]: core: cleanup and in VdsDeploy

2012-12-23 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: cleanup and in VdsDeploy .. Patch Set 2: Alissa, Please follow Vered's comments, and I'll merge. -- To view, visit http://gerrit.ovirt.org/10290 To unsub

[Engine-patches] Change in ovirt-engine[master]: packaging: sync miniyum implementation with recent otopi

2012-12-23 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: sync miniyum implementation with recent otopi .. Patch Set 1: Alex, I really want to push this series *BEFORE* the rebase. -- To view, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: core: cleanup and in VdsDeploy

2012-12-23 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: core: cleanup and in VdsDeploy .. Patch Set 2: Just a question... how so many people reache to this patch? -- To view, visit http://gerrit.ovirt.org/10290 To uns

[Engine-patches] Change in ovirt-engine[master]: core: cleanup and in VdsDeploy

2012-12-23 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core: cleanup and in VdsDeploy .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10290 To unsubscribe, vis

[Engine-patches] Change in ovirt-engine[master]: userportal: Change VerticalPanel to FlowPanel

2012-12-23 Thread omasad
ofri masad has uploaded a new change for review. Change subject: userportal: Change VerticalPanel to FlowPanel .. userportal: Change VerticalPanel to FlowPanel Replace VerticalPanel with FlowPanel in few places in the SideTabExt

[Engine-patches] Change in ovirt-host-deploy[master]: vdsm: bridge: remove unused true always enabled

2012-12-23 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: vdsm: bridge: remove unused true always enabled .. vdsm: bridge: remove unused true always enabled Change-Id: I0784d0fdcc89f4774fdc883512d7ee2dafe0b49c S

[Engine-patches] Change in ovirt-host-deploy[master]: vdsm: bridge: remove unused true always enabled

2012-12-23 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: vdsm: bridge: remove unused true always enabled .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10327 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: webamin: hide cpuPin section according to config values

2012-12-23 Thread dfediuck
Doron Fediuck has posted comments on this change. Change subject: webamin: hide cpuPin section according to config values .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9664 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: adding "linked" property to VmNetworkInterface

2012-12-23 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: adding "linked" property to VmNetworkInterface .. Patch Set 18: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9440 To unsubscribe, vis

[Engine-patches] Change in ovirt-engine[master]: core: cleanup and in VdsDeploy

2012-12-23 Thread abonas
Alissa Bonas has posted comments on this change. Change subject: core: cleanup and in VdsDeploy .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/10290 To unsubscribe, visit http://gerrit.ovirt.org/settings Ge

[Engine-patches] Change in ovirt-engine[master]: core: remove unused permissions classes

2012-12-23 Thread abonas
Alissa Bonas has posted comments on this change. Change subject: core: remove unused permissions classes .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/10284 To unsubscribe, visit http://gerrit.ovirt.org/set

[Engine-patches] Change in ovirt-engine[master]: core: ClassCastException when attempting to get storage doma...

2012-12-23 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: ClassCastException when attempting to get storage domain of disk .. Patch Set 1: (1 inline comment) Fi

  1   2   3   >