[Engine-patches] Change in ovirt-engine[master]: api: Support libvirt version for Host in API (#672148)

2012-12-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: api: Support libvirt version for Host in API (#672148) .. Patch Set 1: I would prefer that you didn't submit this For some reason api.xsd got polluted by white

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

2012-12-24 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: Adding host bios fields to vds_dynamic table .. Patch Set 7: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10114 To un

[Engine-patches] Change in ovirt-engine[master]: core : DB FK validation

2012-12-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core : DB FK validation .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10248 To unsubscribe, visit http://gerrit.ovirt.org

[Engine-patches] Change in ovirt-engine[master]: core: fix javadoc and remove unused method

2012-12-24 Thread vvolansk
Vered Volansky has posted comments on this change. Change subject: core: fix javadoc and remove unused method .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10351 To un

[Engine-patches] Change in ovirt-engine[master]: core: fix javadoc and remove unused method

2012-12-24 Thread abaron
Ayal Baron has posted comments on this change. Change subject: core: fix javadoc and remove unused method .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10351 To unsubs

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

2012-12-24 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Adding bios information to vds object .. Patch Set 17: (2 inline comments) File backend/manager/modules/vd

[Engine-patches] [oVirt Jenkins] ovirt_engine_find_bugs - Build # 3058 - Fixed!

2012-12-24 Thread Jenkins oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt_engine_find_bugs/ Build: http://jenkins.ovirt.org/job/ovirt_engine_find_bugs/3058/ Build Number: 3058 Build Status: Fixed Triggered By: Started by upstream project "ovirt_engine" build number 3,716 - Changes Since La

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

2012-12-24 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: core: adding GetAllDisksByStorageDomainIdQuery .. core: adding GetAllDisksByStorageDomainIdQuery Adding a new query for getting DiskImages by StorageDomai

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

2012-12-24 Thread derez
Daniel Erez has posted comments on this change. Change subject: core: adding GetAllDisksByStorageDomainIdQuery .. Patch Set 5: Verified; Looks good to me, approved rebased -- To view, visit http://gerrit.ovirt.org/10321 To uns

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

2012-12-24 Thread derez
Daniel Erez has posted comments on this change. Change subject: core: adding GetAllDisksByStorageDomainIdQuery .. Patch Set 4: Verified; Looks good to me, approved rebased -- To view, visit http://gerrit.ovirt.org/10321 To uns

[Engine-patches] Change in ovirt-engine[master]: core : GWT changes for Multi-Tier fencing

2012-12-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core : GWT changes for Multi-Tier fencing .. Patch Set 4: I agree with Gilad -- To view, visit http://gerrit.ovirt.org/10261 To unsubscribe, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: core : BLL changes for Multi-Tier fencing

2012-12-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core : BLL changes for Multi-Tier fencing .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10260 To unsubscribe, visit http:

[Engine-patches] Change in ovirt-engine[master]: core : BE & DAO changes for Multi-Tier fencing

2012-12-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core : BE & DAO changes for Multi-Tier fencing .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10259 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: DB Changes for Multi-Tier fencing

2012-12-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: DB Changes for Multi-Tier fencing .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10258 To unsubscribe, visit http://

[Engine-patches] Change in ovirt-engine[master]: core: RFE: Multi-Tier fencing support

2012-12-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: RFE: Multi-Tier fencing support .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10257 To unsubscribe, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: webadmin: Avoid duplicate server queries

2012-12-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: webadmin: Avoid duplicate server queries .. Patch Set 5: (1 inline comment) File frontend/webadmin/modules

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

2012-12-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: Adding host bios fields to vds_dynamic table .. Patch Set 7: Indeed, you do need to set the config value per each version. -- To view, visit http://gerrit.ovir

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

2012-12-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: Adding bios information to vds object .. Patch Set 17: (1 inline comment) File backend/manager/modules/vds

[Engine-patches] [oVirt Jenkins] ovirt_engine_find_bugs - Build # 3057 - Failure!

2012-12-24 Thread Jenkins oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt_engine_find_bugs/ Build: http://jenkins.ovirt.org/job/ovirt_engine_find_bugs/3057/ Build Number: 3057 Build Status: Failure Triggered By: Started by upstream project "ovirt_engine" build number 3,715 - Changes Since

[Engine-patches] Change in ovirt-engine[master]: core: fix javadoc and remove unused method

2012-12-24 Thread abonas
Alissa Bonas has posted comments on this change. Change subject: core: fix javadoc and remove unused method .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/10351 To unsubscribe, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[master]: core: fix javadoc and remove unused method

2012-12-24 Thread abonas
Alissa Bonas has uploaded a new change for review. Change subject: core: fix javadoc and remove unused method .. core: fix javadoc and remove unused method fix javadoc parameters and remove unused method in VmDeviceUtils Change

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

2012-12-24 Thread vvolansk
Vered Volansky has uploaded a new change for review. Change subject: core: BackendVmDisksResource cleanup .. core: BackendVmDisksResource cleanup Avoided if-else clause in attachDiskToVm(). Change-Id: Iea65c24b84d99654dea6bbfea

[Engine-patches] Change in ovirt-engine[master]: core: add comments to addDiskCommand

2012-12-24 Thread mlipchuk
Maor Lipchuk has submitted this change and it was merged. Change subject: core: add comments to addDiskCommand .. core: add comments to addDiskCommand In case of async task execution, we need to set the found storage domain id

[Engine-patches] Change in ovirt-engine[master]: core: add comments to addDiskCommand

2012-12-24 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: add comments to addDiskCommand .. Patch Set 5: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10344 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: core: cleanup AttachDettachVmDiskParameters c'tor

2012-12-24 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core: cleanup AttachDettachVmDiskParameters c'tor .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10349

[Engine-patches] [oVirt Jenkins] ovirt_engine_find_bugs - Build # 3055 - Fixed!

2012-12-24 Thread Jenkins oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt_engine_find_bugs/ Build: http://jenkins.ovirt.org/job/ovirt_engine_find_bugs/3055/ Build Number: 3055 Build Status: Fixed Triggered By: Started by upstream project "ovirt_engine" build number 3,711, Started by upstream project "ovirt_engine" build numb

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

2012-12-24 Thread laravot
Liron Aravot has posted comments on this change. Change subject: Adding bios information to vds object .. Patch Set 17: (2 inline comments) File backend/manager/modules/vdsb

[Engine-patches] Change in ovirt-engine[master]: core: cleanup AttachDettachVmDiskParameters c'tor

2012-12-24 Thread vvolansk
Vered Volansky has uploaded a new change for review. Change subject: core: cleanup AttachDettachVmDiskParameters c'tor .. core: cleanup AttachDettachVmDiskParameters c'tor AttachDettachVmDiskParameters has two c'tors. Implementa

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

2012-12-24 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Adding host bios fields to vds_dynamic table .. Patch Set 7: Verified seems like i need field for each version.. wait for eli response. -- To view, visit http:

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

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

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

2012-12-24 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Adding bios information to vds object .. Patch Set 17: Verified You right, It gets updated afterwards and I don't need to perform this update each time. I omit

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

2012-12-24 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Adding bios information to vds object .. Patch Set 16: (1 inline comment) File backend/manager/modules/vds

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

2012-12-24 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Adding host bios fields to vds_dynamic table .. Patch Set 6: Verified (1 inline comment) File backend/mana

[Engine-patches] Change in ovirt-engine[master]: engine: Added option to filter by running

2012-12-24 Thread masayag
Moti Asayag has submitted this change and it was merged. Change subject: engine: Added option to filter by running .. engine: Added option to filter by running The GetVmsAndNetworkInterfacesByNetworkIdQuery can now optionally f

[Engine-patches] Change in ovirt-engine[master]: engine: Added option to filter by running

2012-12-24 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Added option to filter by running .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10348 To unsubscribe, visit http://g

[Engine-patches] Change in ovirt-engine[master]: engine: Added option to filter by running

2012-12-24 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: engine: Added option to filter by running .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/10348 To unsubscribe, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[master]: engine, webadmin: Add specific query parameters

2012-12-24 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: engine, webadmin: Add specific query parameters .. engine, webadmin: Add specific query parameters Added specific parameters for GetVmsAndNetworkInterfa

[Engine-patches] Change in ovirt-engine[master]: engine, webadmin: Add specific query parameters

2012-12-24 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: engine, webadmin: Add specific query parameters .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/10347 To unsubscribe, visit http://gerrit.ovir

[Engine-patches] Change in ovirt-engine[master]: engine: Added option to filter by running

2012-12-24 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: engine: Added option to filter by running .. Patch Set 1: (1 inline comment) File backend/manager/modules/b

[Engine-patches] Change in ovirt-engine[master]: engine: Added option to filter by running

2012-12-24 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Added option to filter by running .. Patch Set 1: (1 inline comment) File backend/manager/modules/bll

[Engine-patches] Change in ovirt-guest-agent[master]: guest-agent: Unification of line endings to unix

2012-12-24 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: guest-agent: Unification of line endings to unix .. Patch Set 6: Verified; Looks good to me, but someone else must approve Rebased -- To view, visit http://g

[Engine-patches] Change in ovirt-engine[master]: webadmin: Disks sub-tab under Storage main-tab

2012-12-24 Thread laravot
Liron Aravot has posted comments on this change. Change subject: webadmin: Disks sub-tab under Storage main-tab .. Patch Set 2: (4 inline comments) File frontend/webadmin/mo

[Engine-patches] [oVirt Jenkins] ovirt_engine_find_bugs - Build # 3054 - Failure!

2012-12-24 Thread Jenkins oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt_engine_find_bugs/ Build: http://jenkins.ovirt.org/job/ovirt_engine_find_bugs/3054/ Build Number: 3054 Build Status: Failure Triggered By: Started by upstream project "ovirt_engine" build number 3,710 - Changes Since

[Engine-patches] Change in ovirt-guest-agent[master]: Let udev reset user rights on virtio-channel during package ...

2012-12-24 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: Let udev reset user rights on virtio-channel during package removal .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/10286 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: engine, webadmin: Add specific query parameters

2012-12-24 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine, webadmin: Add specific query parameters .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10347 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: webadmin: Avoid duplicate server queries

2012-12-24 Thread laravot
Liron Aravot has posted comments on this change. Change subject: webadmin: Avoid duplicate server queries .. Patch Set 5: (1 inline comment) File frontend/webadmin/modules/f

[Engine-patches] Change in ovirt-engine[master]: engine, webadmin: Add specific query parameters

2012-12-24 Thread mkolesni
Mike Kolesnik has uploaded a new change for review. Change subject: engine, webadmin: Add specific query parameters .. engine, webadmin: Add specific query parameters Added specific parameters for GetVmsAndNetworkInterfacesByNet

[Engine-patches] Change in ovirt-engine[master]: engine: Added option to filter by running

2012-12-24 Thread mkolesni
Mike Kolesnik has uploaded a new change for review. Change subject: engine: Added option to filter by running .. engine: Added option to filter by running The GetVmsAndNetworkInterfacesByNetworkIdQuery can now optionally filter

[Engine-patches] Change in ovirt-engine[master]: webadmin: Disks sub-tab under Storage main-tab

2012-12-24 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: webadmin: Disks sub-tab under Storage main-tab .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10322 To

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

2012-12-24 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: remove unused permissions classes .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10284 To unsubscribe, visit http://ge

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

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

[Engine-patches] Change in ovirt-engine[master]: core: add comments to addDiskCommand

2012-12-24 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: add comments to addDiskCommand .. Patch Set 3: (3 inline comments) File backend/manager/modules/bll/sr

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

2012-12-24 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: webadmin: Filter Non-Operational DCs when adding new domain .. Patch Set 1: Looks good to me, but someone else must approve (1 inline comment) ..

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

2012-12-24 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: Adding bios information to vds object .. Patch Set 16: No score (1 inline comment) File backend/manager/mod

[Engine-patches] Change in ovirt-guest-agent[master]: Fixed support for reporting devices with only ipv6.

2012-12-24 Thread bazulay
Barak Azulay has submitted this change and it was merged. Change subject: Fixed support for reporting devices with only ipv6. .. Fixed support for reporting devices with only ipv6. ethtool.get_active_devices does not consider a

[Engine-patches] Change in ovirt-guest-agent[master]: Fixed support for reporting devices with only ipv6.

2012-12-24 Thread bazulay
Barak Azulay has posted comments on this change. Change subject: Fixed support for reporting devices with only ipv6. .. Patch Set 5: Verified; Looks good to me, approved Verified by: Antoni Approved by: Dan & Vinzenz -- To vie

[Engine-patches] Change in ovirt-guest-agent[master]: Fixed support for reporting devices with only ipv6.

2012-12-24 Thread bazulay
Barak Azulay has posted comments on this change. Change subject: Fixed support for reporting devices with only ipv6. .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10223 To unsubscribe, vis

[Engine-patches] Change in ovirt-guest-agent[master]: Fix pep8 errors in the linux guest agent.

2012-12-24 Thread bazulay
Barak Azulay has posted comments on this change. Change subject: Fix pep8 errors in the linux guest agent. .. Patch Set 3: Verified; Looks good to me, approved Verified by: Antoni Approved by: Vinzenz -- To view, visit http://

[Engine-patches] Change in ovirt-guest-agent[master]: Fix pep8 errors in the linux guest agent.

2012-12-24 Thread bazulay
Barak Azulay has submitted this change and it was merged. Change subject: Fix pep8 errors in the linux guest agent. .. Fix pep8 errors in the linux guest agent. Cleanup of the code to bring it up to standards of the rest of ovi

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

2012-12-24 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: Adding host bios fields to vds_dynamic table .. Patch Set 6: (1 inline comment) File backend/manager/dbscript

[Engine-patches] Change in ovirt-guest-agent[master]: Fix pep8 errors in the linux guest agent.

2012-12-24 Thread bazulay
Barak Azulay has posted comments on this change. Change subject: Fix pep8 errors in the linux guest agent. .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10218 To unsubscribe, visit http://

[Engine-patches] Change in ovirt-guest-agent[master]: guest-agent: Unification of line endings to unix

2012-12-24 Thread bazulay
Barak Azulay has posted comments on this change. Change subject: guest-agent: Unification of line endings to unix .. Patch Set 5: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9463 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: core: cleanup RunVmCommand#statelessVmTreatment

2012-12-24 Thread rgolan
Roy Golan has submitted this change and it was merged. Change subject: core: cleanup RunVmCommand#statelessVmTreatment .. core: cleanup RunVmCommand#statelessVmTreatment - Remove unnecessary local field 'p' - Rename 'tmpVar' fi

[Engine-patches] Change in ovirt-engine[master]: core: cleanup RunVmCommand#statelessVmTreatment

2012-12-24 Thread rgolan
Roy Golan has posted comments on this change. Change subject: core: cleanup RunVmCommand#statelessVmTreatment .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10331 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: introduce InternalMigrateVmParameters

2012-12-24 Thread rgolan
Roy Golan has submitted this change and it was merged. Change subject: core: introduce InternalMigrateVmParameters .. core: introduce InternalMigrateVmParameters Introduce InternalMigrateVmParameters class that should be used w

[Engine-patches] Change in ovirt-engine[master]: core: introduce InternalMigrateVmParameters

2012-12-24 Thread rgolan
Roy Golan has posted comments on this change. Change subject: core: introduce InternalMigrateVmParameters .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10319 To unsubscribe, visit http://g

[Engine-patches] Change in ovirt-engine[master]: core: introduce InternalMigrateVmParameters

2012-12-24 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: core: introduce InternalMigrateVmParameters .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/10319 To unsubscribe, visit http://gerrit.ovirt.org/s

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

2012-12-24 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: Adding HostSubTab to UI for BIOS information .. Patch Set 8: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10115 To unsubscribe, visit htt

[Engine-patches] Change in ovirt-engine[master]: core: Refactor InternalMigrateVmCommand#canMigrateVm

2012-12-24 Thread rgolan
Roy Golan has submitted this change and it was merged. Change subject: core: Refactor InternalMigrateVmCommand#canMigrateVm .. core: Refactor InternalMigrateVmCommand#canMigrateVm Change InternalMigrateVmCommand#canMigrateVm me

[Engine-patches] Change in ovirt-engine[master]: core: Refactor InternalMigrateVmCommand#canMigrateVm

2012-12-24 Thread rgolan
Roy Golan has posted comments on this change. Change subject: core: Refactor InternalMigrateVmCommand#canMigrateVm .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10107 To unsubscr

[Engine-patches] Change in ovirt-engine[master]: core: Refactor InternalMigrateVmCommand#canMigrateVm

2012-12-24 Thread rgolan
Roy Golan has posted comments on this change. Change subject: core: Refactor InternalMigrateVmCommand#canMigrateVm .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10107 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: cleanup RunVmCommand#statelessVmTreatment

2012-12-24 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: core: cleanup RunVmCommand#statelessVmTreatment .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/10331 To unsubscribe, visit http://gerrit.ovirt.o

[Engine-patches] Change in ovirt-engine[master]: restapi: Guest Agent Ifaces to VM

2012-12-24 Thread msalem
Muli Salem has uploaded a new change for review. Change subject: restapi: Guest Agent Ifaces to VM .. restapi: Guest Agent Ifaces to VM This patch adds the Vm Guest Agent information to a VM, according to the feature design in:

[Engine-patches] Change in ovirt-engine[master]: webadmin: Avoid duplicate server queries

2012-12-24 Thread alkaplan
Alona Kaplan has posted comments on this change. Change subject: webadmin: Avoid duplicate server queries .. Patch Set 5: I would prefer that you didn't submit this (2 inline comments) .

[Engine-patches] Change in ovirt-engine[master]: api: Support libvirt version for Host in API (#672148)

2012-12-24 Thread achub
Alexey Chub has uploaded a new change for review. Change subject: api: Support libvirt version for Host in API (#672148) .. api: Support libvirt version for Host in API (#672148) Change-Id: I24b955e979be57c487e2c21fffee771d1ef74

[Engine-patches] Change in ovirt-engine[master]: engine: Add VmDao.getAllForDataCenter

2012-12-24 Thread masayag
Moti Asayag has abandoned this change. Change subject: engine: Add VmDao.getAllForDataCenter .. Patch Set 2: Abandoned the queries don't match the requirements.for the network main tab. -- To view, visit http://gerrit.ovirt.or

[Engine-patches] Change in ovirt-engine[master]: engine: Add GetVmsByDataCenterIdQuery

2012-12-24 Thread masayag
Moti Asayag has abandoned this change. Change subject: engine: Add GetVmsByDataCenterIdQuery .. Patch Set 2: Abandoned the queries don't match the requirements.for the network main tab. -- To view, visit http://gerrit.ovirt.or

[Engine-patches] Change in ovirt-engine[master]: engine: Add VmDao.getAllWithoutNetwork

2012-12-24 Thread masayag
Moti Asayag has abandoned this change. Change subject: engine: Add VmDao.getAllWithoutNetwork .. Patch Set 2: Abandoned the queries don't match the requirements.for the network main tab. -- To view, visit http://gerrit.ovirt.o

[Engine-patches] Change in ovirt-engine[master]: engine: Add GetVmsWithoutNetworkQuery

2012-12-24 Thread masayag
Moti Asayag has abandoned this change. Change subject: engine: Add GetVmsWithoutNetworkQuery .. Patch Set 3: Abandoned the queries don't match the requirements.for the network main tab. -- To view, visit http://gerrit.ovirt.or

[Engine-patches] Change in ovirt-guest-agent[master]: guest-agent: Unification of line endings to unix

2012-12-24 Thread ghammer
Gal Hammer has posted comments on this change. Change subject: guest-agent: Unification of line endings to unix .. Patch Set 5: No score -- To view, visit http://gerrit.ovirt.org/9463 To unsubscribe, visit http://gerrit.ovirt.o

[Engine-patches] Change in ovirt-guest-agent[master]: agent: apply virtual-guest tuning profile on installation.

2012-12-24 Thread ghammer
Gal Hammer has abandoned this change. Change subject: agent: apply virtual-guest tuning profile on installation. .. Patch Set 1: Abandoned -- To view, visit http://gerrit.ovirt.org/9361 To unsubscribe, visit http://gerrit.ovirt

[Engine-patches] Change in ovirt-guest-agent[master]: Let udev reset user rights on virtio-channel during package ...

2012-12-24 Thread ghammer
Gal Hammer has posted comments on this change. Change subject: Let udev reset user rights on virtio-channel during package removal .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit

[Engine-patches] Change in ovirt-engine[master]: restapi: Add "All-Content" Header.

2012-12-24 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: Add "All-Content" Header. .. Patch Set 3: I would prefer that you didn't submit this Ori, good job, but the thing is this patch breaks backward com

[Engine-patches] Change in ovirt-engine[master]: core: introduce InternalMigrateVmParameters

2012-12-24 Thread rgolan
Roy Golan has posted comments on this change. Change subject: core: introduce InternalMigrateVmParameters .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10319 To unsubscribe, visit http://g

[Engine-patches] Change in ovirt-engine[master]: core: cleanup RunVmCommand#statelessVmTreatment

2012-12-24 Thread rgolan
Roy Golan has posted comments on this change. Change subject: core: cleanup RunVmCommand#statelessVmTreatment .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10331 To unsubscribe, visit http

[Engine-patches] Change in ovirt-engine[master]: restapi: #839205 Display Host Hooks

2012-12-24 Thread oliel
Ori Liel has submitted this change and it was merged. Change subject: restapi: #839205 Display Host Hooks .. restapi: #839205 Display Host Hooks https://bugzilla.redhat.com/839205 Change-Id: I38a3d38cc52d5ec77f55fe7db4d5e47b09

[Engine-patches] Change in ovirt-engine[master]: restapi: #839205 Display Host Hooks

2012-12-24 Thread oliel
Ori Liel has posted comments on this change. Change subject: restapi: #839205 Display Host Hooks .. Patch Set 5: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9478 To unsubscribe, visit http://g

[Engine-patches] Change in ovirt-engine[master]: restapi: #839205 Display Host Hooks

2012-12-24 Thread oliel
Ori Liel has posted comments on this change. Change subject: restapi: #839205 Display Host Hooks .. Patch Set 4: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9478 To unsubscribe, visit http://g

[Engine-patches] Change in ovirt-engine[master]: engine: Add GetVmsByDataCenterIdQuery

2012-12-24 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: engine: Add GetVmsByDataCenterIdQuery .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10340 To unsubscribe, visit http://ger

[Engine-patches] Change in ovirt-engine[master]: restapi: add /disks sub-collection to /storagedomain resourc...

2012-12-24 Thread oliel
Ori Liel has posted comments on this change. Change subject: restapi: add /disks sub-collection to /storagedomain resource .. Patch Set 2: (2 inline comments) looks great, only some missing tests ..

[Engine-patches] Change in ovirt-engine[master]: core: add comments to addDiskCommand

2012-12-24 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core: add comments to addDiskCommand .. Patch Set 3: Verified -- To view, visit http://gerrit.ovirt.org/10344 To unsubscribe, visit http://gerrit.ovirt.org/settin

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

2012-12-24 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: engine: remove Config dependency from frontend .. Patch Set 2: Someone should review this, no? -- To view, visit http://gerrit.ovirt.org/10330 To unsubscribe, vi

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

2012-12-24 Thread omasad
ofri masad has posted comments on this change. Change subject: userportal: Change VerticalPanel to FlowPanel .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/10328 To unsubscribe, visit http://gerrit.ovirt.org

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

2012-12-24 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: engine: remove Config dependency from frontend .. Patch Set 2: do you want me to merge it? -- To view, visit http://gerrit.ovirt.org/10330 To unsubscribe, visit

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

2012-12-24 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: engine: remove Config dependency from frontend .. Patch Set 2: I don't understand any of this.. :) But I do see that it fetch it from server now using some magic

[Engine-patches] Change in ovirt-engine[master]: core: add comments to addDiskCommand

2012-12-24 Thread laravot
Liron Aravot has uploaded a new change for review. Change subject: core: add comments to addDiskCommand .. core: add comments to addDiskCommand In case of async task execution, we need to set the found storage domain id in the p

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

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

[Engine-patches] Change in ovirt-engine[master]: core: error when adding disks without specifying storage dom...

2012-12-24 Thread mlipchuk
Maor Lipchuk has submitted this change and it was merged. Change subject: core: error when adding disks without specifying storage domain .. core: error when adding disks without specifying storage domain When adding a disk and

[Engine-patches] Change in ovirt-engine[master]: core: error when adding disks without specifying storage dom...

2012-12-24 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: error when adding disks without specifying storage domain .. Patch Set 6: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10289 To unsu

[Engine-patches] Change in ovirt-engine[master]: core: error when adding disks without specifying storage dom...

2012-12-24 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core: error when adding disks without specifying storage domain .. Patch Set 6: Verified; Looks good to me, but someone else must approve +1 from Vered -- To vie

  1   2   >