[Engine-patches] Change in ovirt-engine[master]: webadmin: disks main-tab - disable prefix localization

2012-11-16 Thread ecohen
Einav Cohen has submitted this change and it was merged. Change subject: webadmin: disks main-tab - disable prefix localization .. webadmin: disks main-tab - disable prefix localization Extract disks search prefix strings to NO

[Engine-patches] Change in ovirt-engine[master]: webadmin: disks main-tab - disable prefix localization

2012-11-16 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: webadmin: disks main-tab - disable prefix localization .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9296 To unsub

[Engine-patches] Change in ovirt-engine[master]: utils: ssh: move to member '_' convention

2012-11-16 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: utils: ssh: move to member '_' convention .. Patch Set 4: I agree with Yair on convention. At the same time I am open to new ideas. We should not follow multip

[Engine-patches] Change in ovirt-engine[master]: webadmin: disks main-tab - disable prefix localization

2012-11-16 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: disks main-tab - disable prefix localization .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/9296 To unsubscribe, visit http://gerrit.

[Engine-patches] Change in ovirt-engine[master]: webadmin: disks main-tab - disable prefix localization

2012-11-16 Thread derez
Daniel Erez has uploaded a new change for review. Change subject: webadmin: disks main-tab - disable prefix localization .. webadmin: disks main-tab - disable prefix localization Extract disks search prefix strings to NON-NLS co

[Engine-patches] Change in ovirt-engine[master]: utils: tar: throw an exception if base does not exist

2012-11-16 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: utils: tar: throw an exception if base does not exist .. Patch Set 4: Looks good to me, but someone else must approve (1 inline comment) one minor formatting co

[Engine-patches] Change in ovirt-engine[master]: registration: fix null pointer exception

2012-11-16 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: registration: fix null pointer exception .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/9158 To unsubs

[Engine-patches] Change in ovirt-engine[master]: core: Event log - a method to bypass flow

2012-11-16 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: core: Event log - a method to bypass flow .. Patch Set 7: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/9148 To unsub

[Engine-patches] Change in ovirt-engine[master]: core: Fix log message in VdsInstaller

2012-11-16 Thread juan . hernandez
Juan Hernandez has submitted this change and it was merged. Change subject: core: Fix log message in VdsInstaller .. core: Fix log message in VdsInstaller The current code uses {1} in a log message with only one parameter, it s

[Engine-patches] Change in ovirt-engine[master]: core: Fix log message in VdsInstaller

2012-11-16 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Fix log message in VdsInstaller .. Patch Set 1: Looks good to me, approved This is so minor that I am merging it without further review. -- To view, visi

[Engine-patches] Change in ovirt-engine[master]: core: Event log - a method to bypass flow

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: core: Event log - a method to bypass flow .. Patch Set 7: Verified; Looks good to me, but someone else must approve Rebase, sorry. -- To view, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: core: Fix log message in VdsInstaller

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: core: Fix log message in VdsInstaller .. Patch Set 1: Looks good to me, but someone else must approve Thanks! -- To view, visit http://gerrit.ovirt.org/9291 To u

[Engine-patches] Change in ovirt-engine[master]: gluster: replace VdsInstallerSSH: with the SSHDialog

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: gluster: replace VdsInstallerSSH: with the SSHDialog .. Patch Set 7: No score -- To view, visit http://gerrit.ovirt.org/9160 To unsubscribe, visit http://gerrit.o

[Engine-patches] Change in ovirt-engine[master]: utils: tar: split tar unit test from VdsInstallerSSH tests.

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: utils: tar: split tar unit test from VdsInstallerSSH tests. .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/9292 To unsubscribe, visit http://g

[Engine-patches] Change in ovirt-engine[master]: utils: ssh: cleanup: rename server with host

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: utils: ssh: cleanup: rename server with host .. Patch Set 3: Verified Fix error in the to be removed VdsInstallerSSH.java. -- To view, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: bootstrap: host add: differentiate between authentication er...

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: bootstrap: host add: differentiate between authentication error and other errors .. Patch Set 7: Verified Rebase. -- To view, visit http://gerrit.ovirt.org/9178

[Engine-patches] Change in ovirt-engine[master]: bootstrap: handle bootstrap messages using string member

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: bootstrap: handle bootstrap messages using string member .. Patch Set 6: Verified Rebase. -- To view, visit http://gerrit.ovirt.org/9173 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: bootstrap: split the logic of ovirt-node upgrade out

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: bootstrap: split the logic of ovirt-node upgrade out .. Patch Set 6: Verified Rebase. -- To view, visit http://gerrit.ovirt.org/9172 To unsubscribe, visit http:/

[Engine-patches] Change in ovirt-engine[master]: gluster: replace VdsInstallerSSH: with the SSHDialog

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: gluster: replace VdsInstallerSSH: with the SSHDialog .. Patch Set 7: Verified Mock the EngeineSSHDialog as requested. This is how one pays for compromising in de

[Engine-patches] Change in ovirt-engine[master]: bootstrap: allow distinguish error from non error messages

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: bootstrap: allow distinguish error from non error messages .. Patch Set 6: Verified Fix per Douglas comment. -- To view, visit http://gerrit.ovirt.org/9161 To un

[Engine-patches] Change in ovirt-engine[master]: utils: tar: split tar unit test from VdsInstallerSSH tests.

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: utils: tar: split tar unit test from VdsInstallerSSH tests. .. utils: tar: split tar unit test from VdsInstallerSSH tests. VdsInstallerSSH.java is about to retir

[Engine-patches] Change in ovirt-engine[master]: core: Fix log message in VdsInstaller

2012-11-16 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Fix log message in VdsInstaller .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/9291 To unsubscribe, visit http://gerrit.ovirt.org/sett

[Engine-patches] Change in ovirt-engine[master]: core: Fix log message in VdsInstaller

2012-11-16 Thread juan . hernandez
Juan Hernandez has uploaded a new change for review. Change subject: core: Fix log message in VdsInstaller .. core: Fix log message in VdsInstaller The current code uses {1} in a log message with only one parameter, it should us

[Engine-patches] Change in ovirt-iso-uploader[master]: tools: add the nossl option for the engine connection

2012-11-16 Thread fsimonce
Federico Simoncelli has uploaded a new change for review. Change subject: tools: add the nossl option for the engine connection .. tools: add the nossl option for the engine connection Change-Id: I84dcfe472006add3e0a7ba42661ea9d

[Engine-patches] Change in ovirt-engine[master]: pki: accept subject name for certificate

2012-11-16 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: pki: accept subject name for certificate .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/9

[Engine-patches] Change in ovirt-engine[master]: bootstrap: allow distinguish error from non error messages

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: bootstrap: allow distinguish error from non error messages .. Patch Set 5: (1 inline comment) Please don't abuse the -1.

[Engine-patches] Change in ovirt-engine[master]: bootstrap: allow distinguish error from non error messages

2012-11-16 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: bootstrap: allow distinguish error from non error messages .. Patch Set 5: I would prefer that you didn't submit this (1 inline comment) -1 since I

[Engine-patches] Change in ovirt-engine[master]: bootstrap: handle bootstrap messages using string member

2012-11-16 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: bootstrap: handle bootstrap messages using string member .. Patch Set 5: Looks good to me, but someone else must approve -- To view, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: utils: tests: do not use default key alias

2012-11-16 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: utils: tests: do not use default key alias .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org

[Engine-patches] Change in ovirt-engine[master]: utils: tar: throw an exception if base does not exist

2012-11-16 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: utils: tar: throw an exception if base does not exist .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: utils: tar: support long file names within tar

2012-11-16 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: utils: tar: support long file names within tar .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt

[Engine-patches] Change in ovirt-engine[master]: registration: fix null pointer exception

2012-11-16 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: registration: fix null pointer exception .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/9

[Engine-patches] Change in ovirt-engine[master]: restapi: API should expose hypervisor version (#829625)

2012-11-16 Thread rnori
Ravi Nori has posted comments on this change. Change subject: restapi: API should expose hypervisor version (#829625) .. Patch Set 3: (1 inline comment) File backend/manager

[Engine-patches] Change in ovirt-engine[master]: tools: engine-config need to remove all ver 2.2 values

2012-11-16 Thread rnori
Ravi Nori has posted comments on this change. Change subject: tools: engine-config need to remove all ver 2.2 values .. Patch Set 6: After moving fn_db_delete_version_map function to action_version_map_sp.sql, it was no longer

[Engine-patches] Change in ovirt-engine[master]: bootstrap: do not check connectivity nor unique id when VdsI...

2012-11-16 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: bootstrap: do not check connectivity nor unique id when VdsInstall=false .. Patch Set 2: Looks good to me, but someone else must approve -- To view

[Engine-patches] Change in ovirt-engine[master]: bootstrap: fix ovirt-node parameters and locations

2012-11-16 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: bootstrap: fix ovirt-node parameters and locations .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.o

[Engine-patches] Change in ovirt-engine[master]: core: Event log - a method to bypass flow

2012-11-16 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: core: Event log - a method to bypass flow .. Patch Set 6: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[master]: bootstrap: host add: differentiate between authentication er...

2012-11-16 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: bootstrap: host add: differentiate between authentication error and other errors .. Patch Set 6: Looks good to me, but someone else must approve --

[Engine-patches] Change in ovirt-engine[master]: utils: move Tar, CachedTar classes to own package

2012-11-16 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: utils: move Tar, CachedTar classes to own package .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ov

[Engine-patches] Change in ovirt-engine[master]: utils: ssh: cleanup: rename server with host

2012-11-16 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: utils: ssh: cleanup: rename server with host .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.o

[Engine-patches] Change in ovirt-engine[master]: core: introduce new libosinfo query

2012-11-16 Thread ewoud
Ewoud Kohl van Wijngaarden has posted comments on this change. Change subject: core: introduce new libosinfo query .. Patch Set 7: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/9047 T

[Engine-patches] [oVirt Jenkins] ovirt-engine_create_rpms - Build # 2191 - Fixed!

2012-11-16 Thread Jenkins oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt-engine_create_rpms/ Build: http://jenkins.ovirt.org/job/ovirt-engine_create_rpms/2191/ Build Number: 2191 Build Status: Fixed Triggered By: Started by user Eyal Edri - Changes Since Last Success:

[Engine-patches] Change in ovirt-engine[master]: UI: libosinfo - validate VM minimum RAM with libosinfo

2012-11-16 Thread ewoud
Ewoud Kohl van Wijngaarden has posted comments on this change. Change subject: UI: libosinfo - validate VM minimum RAM with libosinfo .. Patch Set 2: I would prefer that you didn't submit this (5 inline comments) Relative mino

[Engine-patches] Change in ovirt-engine[master]: webadmin: Aligning labels in Volume dialog for Japanese loca...

2012-11-16 Thread kmayilsa
Kanagaraj M has uploaded a new change for review. Change subject: webadmin: Aligning labels in Volume dialog for Japanese locale .. webadmin: Aligning labels in Volume dialog for Japanese locale Changed the width of the labels a

[Engine-patches] Change in ovirt-engine[master]: packagin: libosinfo - add libosinfo RPM dependency

2012-11-16 Thread ewoud
Ewoud Kohl van Wijngaarden has posted comments on this change. Change subject: packagin: libosinfo - add libosinfo RPM dependency .. Patch Set 5: I would prefer that you didn't submit this (1 inline comment) ..

[Engine-patches] Change in ovirt-engine[master]: core: libosinfo delete unused queries

2012-11-16 Thread ewoud
Ewoud Kohl van Wijngaarden has posted comments on this change. Change subject: core: libosinfo delete unused queries .. Patch Set 8: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/9055

[Engine-patches] [oVirt Jenkins] ovirt-engine_create_rpms - Build # 2190 - Failure!

2012-11-16 Thread Jenkins oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt-engine_create_rpms/ Build: http://jenkins.ovirt.org/job/ovirt-engine_create_rpms/2190/ Build Number: 2190 Build Status: Failure Triggered By: Started by an SCM change - Changes Since Last Success: ---

[Engine-patches] Change in ovirt-engine[master]: core:Do not force fencing proxy to be ...(#876235)

2012-11-16 Thread emesika
Eli Mesika has submitted this change and it was merged. Change subject: core:Do not force fencing proxy to be ...(#876235) .. core:Do not force fencing proxy to be ...(#876235) Do not force fencing proxy to be in UP status Cur

[Engine-patches] Change in ovirt-engine[master]: core:Do not force fencing proxy to be ...(#876235)

2012-11-16 Thread emesika
Eli Mesika has posted comments on this change. Change subject: core:Do not force fencing proxy to be ...(#876235) .. Patch Set 5: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9251 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: utils: ssh: move to member '_' convention

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: utils: ssh: move to member '_' convention .. Patch Set 4: Again, look at the discussion I had with the gluster people. I am sorry to say, but java people are pri

[Engine-patches] Change in ovirt-engine[master]: core: Event log - a method to bypass flow

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: core: Event log - a method to bypass flow .. Patch Set 6: Verified; Looks good to me, but someone else must approve All the patches needs to be merged now. You ca

[Engine-patches] Change in ovirt-engine[master]: utils: ssh: optimize progress streams

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: utils: ssh: optimize progress streams .. Patch Set 4: Please merge this as-is, when needed we move. -- To view, visit http://gerrit.ovirt.org/9170 To unsubscribe

[Engine-patches] Change in ovirt-engine[master]: pki: cleanup the ca interface

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: pki: cleanup the ca interface .. Patch Set 4: Better to use java-7 try-close. -- To view, visit http://gerrit.ovirt.org/9162 To unsubscribe, visit http://gerrit.

[Engine-patches] Change in ovirt-engine[master]: utils: tar: throw an exception if base does not exist

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: utils: tar: throw an exception if base does not exist .. Patch Set 4: A patch should do not move and change in same command. Please flag. -- To view, visit http

[Engine-patches] Change in ovirt-engine[master]: utils: tests: tar: cleanup main() entry point

2012-11-16 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: utils: tests: tar: cleanup main() entry point .. Patch Set 4: Totally unrelated. There is unit test: http://gerrit.ovirt.org/gitweb?p=ovirt-engine.git;a=blob;f=

[Engine-patches] Change in ovirt-engine[master]: core: Event log - a method to bypass flow

2012-11-16 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Event log - a method to bypass flow .. Patch Set 6: Suggestion - why not break dependency in patches and get this merged already? there is a seperate bug