[Engine-patches] Change in ovirt-engine[master]: core: Change class name from action_version_map to ActionVer...

2012-12-25 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: Change class name from action_version_map to ActionVersionMap. .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://ger

[Engine-patches] Change in ovirt-engine[master]: core: Renamed vm_template_image_map_id.java to VmTemplateIma...

2012-12-25 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: Renamed vm_template_image_map_id.java to VmTemplateImageMapId.java .. Patch Set 1: No score well it looks like this is used only in vm_template_image_map,

[Engine-patches] Change in ovirt-engine[master]: core: Renamed vm_template_image_map.java to VmTemplateImageM...

2012-12-25 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: Renamed vm_template_image_map.java to VmTemplateImageMap.java .. Patch Set 1: looks like this class can be just removed completely, no? -- To view, visit

[Engine-patches] Change in ovirt-engine[master]: core: Renamed vm_template_image_map_id.java to VmTemplateIma...

2012-12-25 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: Renamed vm_template_image_map_id.java to VmTemplateImageMapId.java .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/100

[Engine-patches] Change in ovirt-engine[master]: core Renamed vm_pool_map.java to VmPoolMap.java

2012-12-25 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core Renamed vm_pool_map.java to VmPoolMap.java .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10085 To unsubscribe, visit h

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

2012-12-25 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core: RemoveVmCommand canDoAction cleanup .. Patch Set 4: (1 inline comment) File backend/manager/modules/bl

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

2012-12-25 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core : BLL changes for Multi-Tier fencing .. Patch Set 5: (1 inline comment) File backend/manager/modules/bl

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

2012-12-25 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core : BLL changes for Multi-Tier fencing .. Patch Set 5: (1 inline comment) File backend/manager/modules/bl

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

2012-12-25 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core : BLL changes for Multi-Tier fencing .. Patch Set 5: (1 inline comment) File backend/manager/modules/bl

[Engine-patches] Change in ovirt-engine[master]: restapi: Activate/Deactivate vnic should use edit

2012-12-25 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: Activate/Deactivate vnic should use edit .. Patch Set 2: I would prefer that you didn't submit this well done Alona!, 1. please add api tests for t

[Engine-patches] Change in ovirt-engine[master]: restapi: fix NPE when adding vm with no cpu mode

2012-12-25 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: fix NPE when adding vm with no cpu mode .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10372 To unsubscribe, v

[Engine-patches] Change in ovirt-engine[master]: engine: Validation added in Import Cluster

2012-12-25 Thread dgopal
Dhandapani Gopal has posted comments on this change. Change subject: engine: Validation added in Import Cluster .. Patch Set 4: (2 inline comments) Incorporated review comments.

[Engine-patches] Change in ovirt-engine[master]: webadmin: Allow remove of a VM without removing its disks

2012-12-25 Thread tnisan
Tal Nisan has uploaded a new change for review. Change subject: webadmin: Allow remove of a VM without removing its disks .. webadmin: Allow remove of a VM without removing its disks Change-Id: Iba531f7a87295564e20bd99763d185f78

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

2012-12-25 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: RemoveVmCommand canDoAction cleanup .. Patch Set 4: (4 inline comments) File backend/manager/modules/bll/

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

2012-12-25 Thread liron . aravot
liron aravot has posted comments on this change. Change subject: core: RemoveVmCommand canDoAction cleanup .. Patch Set 4: (4 inline comments) File backend/manager/modules/b

[Engine-patches] Change in ovirt-engine[master]: core: Renamed event_map.java to EventMap.java

2012-12-25 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: core: Renamed event_map.java to EventMap.java .. Patch Set 3: Verified rebased. no code changes -- To view, visit http://gerrit.ovirt.org/10053 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: Renamed event_notification_methods.java to EventNotifi...

2012-12-25 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: core: Renamed event_notification_methods.java to EventNotificationMethod.java .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/10068 To unsubs

[Engine-patches] Change in ovirt-engine[master]: core: Renamed event_map_id.java EventMapId.java

2012-12-25 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: core: Renamed event_map_id.java EventMapId.java .. Patch Set 2: Verified rebased, no code changes -- To view, visit http://gerrit.ovirt.org/10052 To unsubscribe

[Engine-patches] Change in ovirt-engine[master]: core: Renamed event_audit_log_subscriber.java to EventAuditL...

2012-12-25 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: core: Renamed event_audit_log_subscriber.java to EventAuditLogSubscriber.java .. Patch Set 2: Verified rebased, no code change. -- To view, visit http://gerrit

[Engine-patches] Change in ovirt-engine[master]: core: Renamed bookmarks.java to Bookmark.java

2012-12-25 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: core: Renamed bookmarks.java to Bookmark.java .. Patch Set 5: Verified rebased, no code change -- To view, visit http://gerrit.ovirt.org/10049 To unsubscribe, v

[Engine-patches] Change in ovirt-engine[master]: core: Fixing compensation of RemoveStoragePool

2012-12-25 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Fixing compensation of RemoveStoragePool .. core: Fixing compensation of RemoveStoragePool The following patch takes advantage of the on delete c

[Engine-patches] Change in ovirt-engine[master]: core: Fixing compensation of RemoveStoragePool

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Fixing compensation of RemoveStoragePool .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10333 To unsubscri

[Engine-patches] Change in ovirt-engine[master]: core: Fixing compensation of RemoveStoragePool

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Fixing compensation of RemoveStoragePool .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10333 T

[Engine-patches] Change in ovirt-engine[master]: core:[RFE]Improve power management proxy ...

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core:[RFE]Improve power management proxy ... .. Patch Set 8: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9673 To unsubscribe, visit htt

[Engine-patches] Change in ovirt-engine[master]: core: Removed unused class.

2012-12-25 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: core: Removed unused class. .. Patch Set 2: Can I get help to test the upgrade script? -- To view, visit http://gerrit.ovirt.org/10071 To unsubscribe, visit htt

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

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core : BLL changes for Multi-Tier fencing .. Patch Set 5: 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: RemoveVmCommand canDoAction cleanup

2012-12-25 Thread abonas
Alissa Bonas has posted comments on this change. Change subject: core: RemoveVmCommand canDoAction cleanup .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10373 To unsub

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

2012-12-25 Thread tnisan
Tal Nisan has uploaded a new change for review. Change subject: core: RemoveVmCommand canDoAction cleanup .. core: RemoveVmCommand canDoAction cleanup Change-Id: Iad7c7f44437bed6280a81bcd99aa2d556818e6be Signed-off-by: Tal Nisan

[Engine-patches] Change in ovirt-engine[master]: engine: Fix storage quota distribution bug

2012-12-25 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: engine: Fix storage quota distribution bug .. Patch Set 1: I would prefer that you didn't submit this (2 inline comments) ..

[Engine-patches] Change in ovirt-engine[master]: restapi: fix NPE when adding vm with no cpu mode

2012-12-25 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: restapi: fix NPE when adding vm with no cpu mode .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/10372 To unsubscribe, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: restapi: fix NPE when adding vm with no cpu mode

2012-12-25 Thread ahadas
Arik Hadas has uploaded a new change for review. Change subject: restapi: fix NPE when adding vm with no cpu mode .. restapi: fix NPE when adding vm with no cpu mode A check whether the lately added attribute 'cpu mode' is defin

[Engine-patches] Change in ovirt-engine[master]: restapi: Guest Agent Network Devices of Nic (WIP)

2012-12-25 Thread msalem
Muli Salem has uploaded a new change for review. Change subject: restapi: Guest Agent Network Devices of Nic (WIP) .. restapi: Guest Agent Network Devices of Nic (WIP) This patch adds the Guest Agent information to a Nic, accord

[Engine-patches] Change in ovirt-engine[master]: API: [RFE] Add PM Proxy Preferences

2012-12-25 Thread emesika
Eli Mesika has posted comments on this change. Change subject: API: [RFE] Add PM Proxy Preferences .. Patch Set 8: (4 inline comments) File backend/manager/modules/restapi/i

[Engine-patches] Change in ovirt-engine[master]: webadmin: remove "all" filter from Network Host sub tab

2012-12-25 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: webadmin: remove "all" filter from Network Host sub tab .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10368 To unsubscribe

[Engine-patches] Change in ovirt-engine[master]: webadmin: Add filters to NetworkVm sub tab- running, not run...

2012-12-25 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: webadmin: Add filters to NetworkVm sub tab- running, not running. .. Patch Set 1: Looks good to me, approved (1 inline comment) ack, tiny comment/question ...

[Engine-patches] Change in ovirt-engine[master]: webadmin: Show additional columns for events

2012-12-25 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: webadmin: Show additional columns for events .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10315 To unsubscribe, visit htt

[Engine-patches] Change in ovirt-engine[master]: engine: Add validation to cpu pinning

2012-12-25 Thread laravot
Liron Aravot has posted comments on this change. Change subject: engine: Add validation to cpu pinning .. Patch Set 2: (2 inline comments) File backend/manager/modules/bll/s

[Engine-patches] Change in ovirt-engine[master]: restapi: fix usb mapping on update vm call

2012-12-25 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: restapi: fix usb mapping on update vm call .. Patch Set 2: (4 inline comments) File backend/manager/modules/re

[Engine-patches] Change in ovirt-engine[master]: restapi: fix usb mapping on update vm call

2012-12-25 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: restapi: fix usb mapping on update vm call .. Patch Set 4: Verified -- To view, visit http://gerrit.ovirt.org/9792 To unsubscribe, visit http://gerrit.ovirt.org/set

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

2012-12-25 Thread emesika
Eli Mesika has posted comments on this change. Change subject: core : API changes Multi-Tier fencing .. Patch Set 5: (4 inline comments) File backend/manager/modules/restapi

[Engine-patches] Change in ovirt-engine[master]: engine: Add validation to cpu pinning

2012-12-25 Thread laravot
Liron Aravot has posted comments on this change. Change subject: engine: Add validation to cpu pinning .. Patch Set 2: (8 inline comments) File backend/manager/modules/bll/s

[Engine-patches] Change in ovirt-engine[master]: restapi: fix usb mapping on update vm call

2012-12-25 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: restapi: fix usb mapping on update vm call .. Patch Set 4: No score -- To view, visit http://gerrit.ovirt.org/9792 To unsubscribe, visit http://gerrit.ovirt.org/set

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

2012-12-25 Thread emesika
Eli Mesika has posted comments on this change. Change subject: core : BLL changes for Multi-Tier fencing .. Patch Set 4: (2 inline comments) File backend/manager/modules/bll

[Engine-patches] Change in ovirt-engine[master]: restapi: fix usb mapping on update vm call

2012-12-25 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: restapi: fix usb mapping on update vm call .. Patch Set 4: Verified -- To view, visit http://gerrit.ovirt.org/9792 To unsubscribe, visit http://gerrit.ovirt.org/set

[Engine-patches] Change in ovirt-engine[master]: engine: Fix storage quota distribution bug

2012-12-25 Thread omasad
ofri masad has uploaded a new change for review. Change subject: engine: Fix storage quota distribution bug .. engine: Fix storage quota distribution bug In user portal the storage quota distribution was not presented because th

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

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core : adding GetAuditLogByIdQuery .. Patch Set 7: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10092 To unsubscribe, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: core: Adding external events BE support and testing

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Adding external events BE support and testing .. Patch Set 7: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10091 To unsubscribe, v

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

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: DB Changes .. Patch Set 7: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10090 To unsubscribe, visit http://gerrit.ovirt.org/settin

[Engine-patches] Change in ovirt-engine[master]: core: RFE: Allow plugins to add events into the...

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: RFE: Allow plugins to add events into the... .. Patch Set 7: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10089 To unsubscribe, vi

[Engine-patches] Change in ovirt-guest-agent[master]: updated change-log for version 1.0.7

2012-12-25 Thread bazulay
Barak Azulay has submitted this change and it was merged. Change subject: updated change-log for version 1.0.7 .. updated change-log for version 1.0.7 Change-Id: I8439b85ca0b479455dc7b03ff25ceccd04594f60 Signed-off-by: Gal Hamm

[Engine-patches] Change in ovirt-guest-agent[master]: updated change-log for version 1.0.7

2012-12-25 Thread bazulay
Barak Azulay has posted comments on this change. Change subject: updated change-log for version 1.0.7 .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10369 To unsubscribe, visit http://gerri

[Engine-patches] Change in ovirt-guest-agent[master]: updated change-log for version 1.0.7

2012-12-25 Thread ghammer
Gal Hammer has posted comments on this change. Change subject: updated change-log for version 1.0.7 .. Patch Set 1: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10369 To un

[Engine-patches] Change in ovirt-guest-agent[master]: updated change-log for version 1.0.7

2012-12-25 Thread ghammer
Gal Hammer has uploaded a new change for review. Change subject: updated change-log for version 1.0.7 .. updated change-log for version 1.0.7 Change-Id: I8439b85ca0b479455dc7b03ff25ceccd04594f60 Signed-off-by: Gal Hammer --- M

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

2012-12-25 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: cleanup in VdsDeploy .. core: cleanup in VdsDeploy Remove unused variable and code in VdsDeploy Change-Id: I09d8322c88bd4982c232fa8217532a541e24

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

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: cleanup in VdsDeploy .. Patch Set 6: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10290 To unsubscribe, visit http://ger

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

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: cleanup in VdsDeploy .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10290 To unsubscribe, visit http://gerrit.ovirt.

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

2012-12-25 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core,ui: support multiple concurrent disks migration .. Patch Set 4: same comment for LiveMigrateDisksParameters. This classes will be hard to maintain when the h

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

2012-12-25 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core,ui: support multiple concurrent disks migration .. Patch Set 4: Consider to change the class name LiveMigrateDisksCommand, it could be confusing with LiveMi

[Engine-patches] Change in ovirt-engine[master]: webadmin: remove "all" filter from Network Host sub tab

2012-12-25 Thread alkaplan
Alona Kaplan has uploaded a new change for review. Change subject: webadmin: remove "all" filter from Network Host sub tab .. webadmin: remove "all" filter from Network Host sub tab The all radio button in the Network Host sub t

[Engine-patches] Change in ovirt-engine[master]: webadmin: Add filters to NetworkVm sub tab- running, not run...

2012-12-25 Thread alkaplan
Alona Kaplan has uploaded a new change for review. Change subject: webadmin: Add filters to NetworkVm sub tab- running, not running. .. webadmin: Add filters to NetworkVm sub tab- running, not running. In this patch a radio but

[Engine-patches] Change in ovirt-engine[master]: frontend: [RFE] Display of both the console user and the gue...

2012-12-25 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: frontend: [RFE] Display of both the console user and the guest user .. Patch Set 4: Verified -- To view, visit http://gerrit.ovirt.org/10274 To unsubscribe, visi

[Engine-patches] Change in ovirt-engine[master]: core: [RFE] Display of both the console user and the guest u...

2012-12-25 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: [RFE] Display of both the console user and the guest user .. Patch Set 8: Verified -- To view, visit http://gerrit.ovirt.org/10219 To unsubscribe, visit htt

[Engine-patches] Change in ovirt-engine[master]: webadmin: Show additional columns for events

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: webadmin: Show additional columns for events .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10315 To

[Engine-patches] Change in ovirt-engine[master]: core: Search Engine changes

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Search Engine changes .. Patch Set 7: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10095 To unsubscribe, visit http://gerrit.ovirt

[Engine-patches] Change in ovirt-engine[master]: core : implementing AddExternalEventCommand ...

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core : implementing AddExternalEventCommand ... .. Patch Set 7: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10094 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Set Object Type & Action Group for external events

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Set Object Type & Action Group for external events .. Patch Set 7: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10093 To unsubscri

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

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core : BLL changes for Multi-Tier fencing .. Patch Set 4: Please fix according to Liron's comment -- To view, visit http://gerrit.ovirt.org/10260 To unsubscrib

[Engine-patches] Change in ovirt-engine-cli[master]: cli: "remove permit" command doesnt work by-id #887805

2012-12-25 Thread mpastern
Michael Pasternak has uploaded a new change for review. Change subject: cli: "remove permit" command doesnt work by-id #887805 .. cli: "remove permit" command doesnt work by-id #887805 https://bugzilla.redhat.com/show_bug.cgi?id

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

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

[Engine-patches] Change in ovirt-engine-cli[master]: cli: wrong response for suspend VM #886944

2012-12-25 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: cli: wrong response for suspend VM #886944 .. Patch Set 1: (1 inline comment) File src/ovirtcli/command/actio

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

2012-12-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: Adding host bios fields to vds_dynamic table .. Patch Set 7: (1 inline comment) Yaniv, please close about bios prefixes with Alon Bar Lev. As you see - I asked

[Engine-patches] Change in ovirt-engine-cli[master]: cli: wrong response for suspend VM #886944

2012-12-25 Thread mpastern
Michael Pasternak has uploaded a new change for review. Change subject: cli: wrong response for suspend VM #886944 .. cli: wrong response for suspend VM #886944 https://bugzilla.redhat.com/show_bug.cgi?id=886944 Change-Id: I345

[Engine-patches] Change in ovirt-engine[master]: webadmin: allow a single user lookup at a time

2012-12-25 Thread alkaplan
Alona Kaplan has posted comments on this change. Change subject: webadmin: allow a single user lookup at a time .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10241 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: webadmin: correlate models to move multiple disks API

2012-12-25 Thread derez
Daniel Erez has abandoned this change. Change subject: webadmin: correlate models to move multiple disks API .. Patch Set 3: Abandoned Squashed into http://gerrit.ovirt.org/#/c/10154/ -- To view, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[master]: engine: Add validation to cpu pinning

2012-12-25 Thread nslomian
Noam Slomianko has posted comments on this change. Change subject: engine: Add validation to cpu pinning .. Patch Set 2: Verified Still waiting for answer about pCPU validation -- To view, visit http://gerrit.ovirt.org/10364 T

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

2012-12-25 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core: OvfDataUpdater - removal of update/remove OVF vdsm calls .. Patch Set 29: vered, your comments will be addressed in a further patch -- To view, visit http:

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

2012-12-25 Thread gchaplik
Gilad Chaplik has submitted this change and it was merged. Change subject: webamin: hide cpuPin section according to config values .. webamin: hide cpuPin section according to config values According to cpuPinningEnabled flag,

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

2012-12-25 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: webamin: hide cpuPin section according to config values .. Patch Set 8: Verified; Looks good to me, approved rebased & verified. -- To view, visit http://gerrit

[Engine-patches] Change in ovirt-engine[master]: engine: Add validation to cpu pinning

2012-12-25 Thread nslomian
Noam Slomianko has uploaded a new change for review. Change subject: engine: Add validation to cpu pinning .. engine: Add validation to cpu pinning Validation the following conditions: - syntax is correct - all given vcpus exist

[Engine-patches] Change in ovirt-engine[master]: perform connectStorageServer to hosts after during reconstru...

2012-12-25 Thread derez
Daniel Erez has posted comments on this change. Change subject: perform connectStorageServer to hosts after during reconstruct/recovery(#880180) .. Patch Set 7: Looks good to me, approved -- To view, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: core: ReconstructMasterDomain -connect/disconnect hosts simu...

2012-12-25 Thread derez
Daniel Erez has posted comments on this change. Change subject: core: ReconstructMasterDomain -connect/disconnect hosts simultaneously .. Patch Set 4: Looks good to me, approved please fix comment on rebase -- To view, visit

[Engine-patches] Change in ovirt-engine[master]: core: add executor for execution of runnables with CountDown...

2012-12-25 Thread derez
Daniel Erez has posted comments on this change. Change subject: core: add executor for execution of runnables with CountDownLatch .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.

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

2012-12-25 Thread laravot
Liron Aravot has posted comments on this change. Change subject: Adding bios information to vds object .. Patch Set 19: (1 inline comment) File backend/manager/modules/vdsbr

[Engine-patches] Change in ovirt-host-deploy[master]: vdsm: pki: pending key fixups

2012-12-25 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: vdsm: pki: pending key fixups .. vdsm: pki: pending key fixups pending key existence should be deferred to misc stage as we do not have the actual vdsm k

[Engine-patches] Change in ovirt-host-deploy[master]: vdsm: pki: pending key fixups

2012-12-25 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: vdsm: pki: pending key fixups .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10362 To unsubscribe, visit http://ge

[Engine-patches] Change in ovirt-host-deploy[master]: vdsm: pki: fix accept enrollment

2012-12-25 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: vdsm: pki: fix accept enrollment .. vdsm: pki: fix accept enrollment Due to python indent typo Change-Id: I2cb780e61a6071616187e0e21aae768866aba976 Sign

[Engine-patches] Change in ovirt-host-deploy[master]: vdsm: pki: fix accept enrollment

2012-12-25 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: vdsm: pki: fix accept enrollment .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10361 To unsubscribe, visit http:/

[Engine-patches] Change in ovirt-engine[master]: core: ReconstructMasterDomain -connect/disconnect hosts simu...

2012-12-25 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core: ReconstructMasterDomain -connect/disconnect hosts simultaneously .. Patch Set 3: (7 inline comments) vered, sorry - forgot to publish the comments as well

[Engine-patches] Change in ovirt-host-deploy[master]: vdsm: pki: pending key fixups

2012-12-25 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: vdsm: pki: pending key fixups .. vdsm: pki: pending key fixups pending key existence should be deferred to misc stage as we do not have the actual vdsm key store

[Engine-patches] Change in ovirt-host-deploy[master]: vdsm: pki: fix accept enrollment

2012-12-25 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: vdsm: pki: fix accept enrollment .. vdsm: pki: fix accept enrollment Due to python indent typo Change-Id: I2cb780e61a6071616187e0e21aae768866aba976 Signed-off-b

[Engine-patches] Change in otopi[master]: core: misc: dump environment before validation

2012-12-25 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: core: misc: dump environment before validation .. core: misc: dump environment before validation Useful for debug. Change-Id: I026da7e83bd7cf9b17b73156c

[Engine-patches] Change in otopi[master]: core: misc: dump environment before validation

2012-12-25 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: core: misc: dump environment before validation .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10360 To unsubscribe

[Engine-patches] Change in otopi[master]: core: config: add enforce environment configuration

2012-12-25 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: core: config: add enforce environment configuration .. core: config: add enforce environment configuration Change-Id: I50129b16d59a7ad8e1337fad9fc9b2cb65

[Engine-patches] Change in otopi[master]: core: config: add enforce environment configuration

2012-12-25 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: core: config: add enforce environment configuration .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10359 To unsubs

[Engine-patches] Change in otopi[master]: core: config: add enforce environment configuration

2012-12-25 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: core: config: add enforce environment configuration .. core: config: add enforce environment configuration Change-Id: I50129b16d59a7ad8e1337fad9fc9b2cb650e8e0a S

[Engine-patches] Change in otopi[master]: core: misc: dump environment before validation

2012-12-25 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: core: misc: dump environment before validation .. core: misc: dump environment before validation Useful for debug. Change-Id: I026da7e83bd7cf9b17b73156c7e090649

[Engine-patches] Change in ovirt-engine[master]: frontend: [RFE] Display of both the console user and the gue...

2012-12-25 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: frontend: [RFE] Display of both the console user and the guest user .. Patch Set 1: (1 inline comment) File

[Engine-patches] Change in ovirt-engine[master]: API: [RFE] Add PM Proxy Preferences

2012-12-25 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: API: [RFE] Add PM Proxy Preferences .. Patch Set 8: I would prefer that you didn't submit this (5 inline comments) please implement mapping tests for your u

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

2012-12-25 Thread mpastern
Michael Pasternak 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 (1 inline comment) ...

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

2012-12-25 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: core : API changes Multi-Tier fencing .. Patch Set 5: I would prefer that you didn't submit this (4 inline comments) please added mapping test for this use-

  1   2   >