[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code from CommandBase.

2014-06-18 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: engine: Remove unused code from CommandBase. .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/ovirt-engine_master_create-rpms-quick_merg

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code from CommandBase.

2014-06-18 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: engine: Remove unused code from CommandBase. .. Patch Set 2: * Update tracker::IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found. -- To

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code from CommandBase.

2014-06-18 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Remove unused code from CommandBase. .. Patch Set 2: Code-Review+2 Verified+1 -- To view, visit http://gerrit.ovirt.org/28630 To unsubscribe, visit http:/

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code from CommandBase.

2014-06-18 Thread masayag
Moti Asayag has submitted this change and it was merged. Change subject: engine: Remove unused code from CommandBase. .. engine: Remove unused code from CommandBase. Private methods getAsyncTask and internalGetTaskType aren't i

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code from CommandBase.

2014-06-18 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: engine: Remove unused code from CommandBase. .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_ger

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code from CommandBase.

2014-06-18 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: engine: Remove unused code from CommandBase. .. Patch Set 2: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code from CommandBase.

2014-06-18 Thread yzaspits
Yevgeny Zaspitsky has posted comments on this change. Change subject: engine: Remove unused code from CommandBase. .. Patch Set 1: Yair/Moti, please merge the change -- To view, visit http://gerrit.ovirt.org/28630 To unsubscr

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code from CommandBase.

2014-06-12 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Remove unused code from CommandBase. .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/28630 To unsubscribe, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code from CommandBase.

2014-06-12 Thread yzaspits
Yevgeny Zaspitsky has posted comments on this change. Change subject: engine: Remove unused code from CommandBase. .. Patch Set 1: Yes, it passed the build locally and on Jenkins -- To view, visit http://gerrit.ovirt.org/2863

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code from CommandBase.

2014-06-12 Thread yzaspits
Yevgeny Zaspitsky has posted comments on this change. Change subject: engine: Remove unused code from CommandBase. .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/28630 To unsubscribe, visit http://gerrit.

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code from CommandBase.

2014-06-12 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine: Remove unused code from CommandBase. .. Patch Set 1: I assume you verified by compilation at least, no? :) -- To view, visit http://gerrit.ovirt.org/2

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code from CommandBase.

2014-06-12 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: engine: Remove unused code from CommandBase. .. Patch Set 1: * Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code from CommandBase.

2014-06-12 Thread yzaspits
Yevgeny Zaspitsky has uploaded a new change for review. Change subject: engine: Remove unused code from CommandBase. .. engine: Remove unused code from CommandBase. Private methods getAsyncTask and internalGetTaskType aren't in

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code

2013-08-27 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: engine: Remove unused code .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/18544 To unsubscribe, visit http://gerrit.ovirt.org/settings

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code

2013-08-27 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: engine: Remove unused code .. engine: Remove unused code The removed method and constant are being used. Change-Id: Ic926cf9f2944bef43fc9e80d23ba80154b

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code

2013-08-27 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: engine: Remove unused code .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/ovirt_engine_sanity_compile_checkstyle_gerrit/7273/ : SUCCE

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code

2013-08-27 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Remove unused code .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.ovirt.org/18544 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerr

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code

2013-08-27 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Remove unused code .. Patch Set 1: (4 comments) File backend/manager/modules/common/src/main/java/or

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code

2013-08-26 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: engine: Remove unused code .. Patch Set 1: (4 comments) Nice cleanup, it can be a bit more extensive while you're at it.. .

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code

2013-08-26 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Remove unused code .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/18544 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerr

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code

2013-08-26 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: engine: Remove unused code .. Patch Set 1: Build Successful http://jenkins.ovirt.org/job/ovirt_engine_sanity_compile_checkstyle_gerrit/7149/ : SUCCE

[Engine-patches] Change in ovirt-engine[master]: engine: Remove unused code

2013-08-26 Thread masayag
Moti Asayag has uploaded a new change for review. Change subject: engine: Remove unused code .. engine: Remove unused code The removed method and constant are being used. Change-Id: Ic926cf9f2944bef43fc9e80d23ba80154b9f185b Sig