[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in several bll classes

2013-12-29 Thread abonas
Alissa Bonas has abandoned this change. Change subject: core: remove unused methods in several bll classes .. Abandoned -- To view, visit http://gerrit.ovirt.org/21944 To unsubscribe, visit http://gerrit.ovirt.org/settings Ge

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

2013-12-12 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 5: Build Unstable http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/12844/ : SUCCE

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

2013-12-12 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 5: Build Successful http://jenkins.ovirt.org/job/ovirt_engine_3.3_update_db_to_master_gerrit/3599/ : SUCCES

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

2013-12-12 Thread ofrenkel
Omer Frenkel has submitted this change and it was merged. Change subject: core: remove unused methods .. core: remove unused methods Remove unused methods in RunVmCommand Change-Id: Iabe99ebc6ce6cca069f9333ac3d0b4117f3dfd3b Si

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

2013-12-12 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 5: Verified+1 Code-Review+2 -- To view, visit http://gerrit.ovirt.org/22071 To unsubscribe, visit http://gerrit.ovirt.or

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

2013-12-12 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/22071 To unsubscribe, visit http://gerrit.ovirt.org/settings

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

2013-12-12 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.ovirt.org/22071 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerr

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

2013-12-11 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 4: Build Successful http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/12816/ : SUC

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

2013-12-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 3: Code-Review+1 Omer/Roy, any reason not to merge this? -- To view, visit http://gerrit.ovirt.org/22071 To unsubscri

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

2013-12-10 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 3: Build Successful http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/12645/ : SUC

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

2013-12-10 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.ovirt.org/22071 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerr

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in several bll classes

2013-12-05 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: remove unused methods in several bll classes .. Patch Set 1: (1 comment) File backend/manager/modul

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in several bll classes

2013-12-05 Thread rnori
Ravi Nori has posted comments on this change. Change subject: core: remove unused methods in several bll classes .. Patch Set 1: (2 comments) File backend/manager/modules/b

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

2013-12-05 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.ovirt.org/22071 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerr

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

2013-12-05 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/22071 To unsubscribe, visit http://gerrit.ovirt.org/setting

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

2013-12-05 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/ovirt_engine_master_find_bugs_gerrit/1006/ : SUCCESS http

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

2013-12-05 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 1: Verified-1 Build Failed http://jenkins.ovirt.org/job/ovirt_engine_master_find_bugs_gerrit/993/ : SUCCESS

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

2013-12-05 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: core: remove unused methods .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/22071 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerr

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

2013-12-05 Thread ahadas
Arik Hadas has uploaded a new change for review. Change subject: core: remove unused methods .. core: remove unused methods Remove unused methods in RunVmCommand Change-Id: Iabe99ebc6ce6cca069f9333ac3d0b4117f3dfd3b Signed-off-b

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in several bll classes

2013-12-04 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: remove unused methods in several bll classes .. Patch Set 1: Code-Review-1 (1 comment) Seems Omer is correct. @Alissa - please verify. Also, I'd conside

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in several bll classes

2013-12-03 Thread iheim
Itamar Heim has posted comments on this change. Change subject: core: remove unused methods in several bll classes .. Patch Set 1: can you find where/how they are used? -- To view, visit http://gerrit.ovirt.org/21944 To unsub

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in several bll classes

2013-12-03 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: remove unused methods in several bll classes .. Patch Set 1: (1 comment) not sure about the commandBase methods, iirc, these are part of the async tasks i

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in several bll classes

2013-12-03 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: remove unused methods in several bll classes .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/ovirt_engine_master_find_bugs_gerrit

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in several bll classes

2013-12-03 Thread abonas
Alissa Bonas has uploaded a new change for review. Change subject: core: remove unused methods in several bll classes .. core: remove unused methods in several bll classes Change-Id: Ied79a5d374b0834b335bfe6b5acfea984c7ee222 Sig

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in IPAddress

2013-11-07 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: remove unused methods in IPAddress .. Patch Set 3: Build Successful http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in IPAddress

2013-11-07 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: remove unused methods in IPAddress .. Patch Set 3: Build Successful http://jenkins.ovirt.org/job/ovirt_engine_3.3_update_db_to_master_gerrit/40

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in IPAddress

2013-11-07 Thread masayag
Moti Asayag has submitted this change and it was merged. Change subject: core: remove unused methods in IPAddress .. core: remove unused methods in IPAddress Change-Id: I12ee6018dd651a396e884f79bfba9230119b4ac2 Signed-off-by: A

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in IPAddress

2013-11-07 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: remove unused methods in IPAddress .. Patch Set 3: Verified+1 Code-Review+2 -- To view, visit http://gerrit.ovirt.org/21020 To unsubscribe, visit http://ger

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in IPAddress

2013-11-07 Thread abonas
Alissa Bonas has posted comments on this change. Change subject: core: remove unused methods in IPAddress .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.ovirt.org/21020 To unsubscribe, visit http://gerrit.ovirt.org

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in IPAddress

2013-11-07 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: remove unused methods in IPAddress .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/21020 To unsubscribe, visit http://gerrit.ovirt.o

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in IPAddress

2013-11-07 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: remove unused methods in IPAddress .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/

[Engine-patches] Change in ovirt-engine[master]: core: remove unused methods in IPAddress

2013-11-07 Thread abonas
Alissa Bonas has posted comments on this change. Change subject: core: remove unused methods in IPAddress .. Patch Set 2: addressed comments, instead of renaming, just removed the methods. -- To view, visit http://gerrit.ovir

[Engine-patches] Change in ovirt-engine[master]: core: Remove unused methods from UserCommandBase

2013-09-09 Thread amureini
Allon Mureinik has submitted this change and it was merged. Change subject: core: Remove unused methods from UserCommandBase .. core: Remove unused methods from UserCommandBase Removed ProcessAdElementDefaultRole(Guid, String)

[Engine-patches] Change in ovirt-engine[master]: core: Remove unused methods from UserCommandBase

2013-09-09 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Remove unused methods from UserCommandBase .. Patch Set 4: Verified+1 Code-Review+2 Rebased, no code changes. -- To view, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[master]: core: Remove unused methods from UserCommandBase

2013-09-09 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Remove unused methods from UserCommandBase .. Patch Set 3: Verified+1 Code-Review+2 Rebased, no changes. -- To view, visit http://gerrit.ovirt.org/18870

[Engine-patches] Change in ovirt-engine[master]: core: Remove unused methods from UserCommandBase

2013-09-08 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: Remove unused methods from UserCommandBase .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/18870 To unsubscribe, visit http://gerrit.o

[Engine-patches] Change in ovirt-engine[master]: core: Remove unused methods from UserCommandBase

2013-09-08 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: Remove unused methods from UserCommandBase .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/18870 To unsubscribe, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: core: Remove unused methods from UserCommandBase

2013-09-04 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Remove unused methods from UserCommandBase .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.ovirt.org/18870 To unsubscribe, visit http://gerrit

[Engine-patches] Change in ovirt-engine[master]: core: Remove unused methods from UserCommandBase

2013-09-04 Thread amureini
Allon Mureinik has uploaded a new change for review. Change subject: core: Remove unused methods from UserCommandBase .. core: Remove unused methods from UserCommandBase Removed ProcessAdElementDefaultRole(Guid, String) and CanA

[Engine-patches] Change in ovirt-engine[master]: core: Remove Unused methods from VdsFenceOptions

2013-08-11 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: Remove Unused methods from VdsFenceOptions .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/ovirt_engine_sanity_compile_checkstyle

[Engine-patches] Change in ovirt-engine[master]: core: Remove Unused methods from VdsFenceOptions

2013-08-11 Thread amureini
Allon Mureinik has submitted this change and it was merged. Change subject: core: Remove Unused methods from VdsFenceOptions .. core: Remove Unused methods from VdsFenceOptions Removed the unused methods getFencingAgentInstance

[Engine-patches] Change in ovirt-engine[master]: core: Remove Unused methods from VdsFenceOptions

2013-08-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Remove Unused methods from VdsFenceOptions .. Patch Set 2: Verified+1 Code-Review+2 Rebased, no changes. -- To view, visit http://gerrit.ovirt.org/16818

[Engine-patches] Change in ovirt-engine[master]: core: Remove Unused methods from VdsFenceOptions

2013-07-30 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Remove Unused methods from VdsFenceOptions .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16818 To unsubscribe, vis

[Engine-patches] Change in ovirt-engine[master]: core: Remove Unused methods from VdsFenceOptions

2013-07-30 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: Remove Unused methods from VdsFenceOptions .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16818 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Remove Unused methods from VdsFenceOptions

2013-07-14 Thread amureini
Allon Mureinik has uploaded a new change for review. Change subject: core: Remove Unused methods from VdsFenceOptions .. core: Remove Unused methods from VdsFenceOptions Removed the unused methods getFencingAgentInstanceOptionsM

[Engine-patches] Change in ovirt-engine[master]: core: Remove Unused methods from VdsFenceOptions

2013-07-14 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change. Change subject: core: Remove Unused methods from VdsFenceOptions .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/ovirt_engine_sanity_compile_checkstyle

[Engine-patches] Change in ovirt-engine[master]: core: Remove Unused methods from VdsFenceOptions

2013-07-14 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Remove Unused methods from VdsFenceOptions .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/16818 To unsubscribe, visit http://gerrit.o