November 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: VdsDeployVdsmUnit LinqUtils
by oourfali＠redhat.com 26 Nov '15

26 Nov '15

Oved Ourfali has submitted this change and it was merged. Change subject: core: VdsDeployVdsmUnit LinqUtils ...................................................................... core: VdsDeployVdsmUnit LinqUtils Replace the old-fashioned LinqUtils usage with the modern, built-in streaming API introduced in Java 8. Change-Id: I55735082e5e1b1c2f1623ef6ed03a9f9124bc54a Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/VdsDeployVdsmUnit.java 1 file changed, 3 insertions(+), 10 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Verified Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/49194 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55735082e5e1b1c2f1623ef6ed03a9f9124bc54a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Added possibility to filter jobs to deploy
by dcaro＠redhat.com 25 Nov '15

25 Nov '15

David Caro has submitted this change and it was merged. Change subject: Added possibility to filter jobs to deploy ...................................................................... Added possibility to filter jobs to deploy Now you can use a comma separated list of globs to control which jobs will get deployed Change-Id: I389cb3783df5ce2fe2ba940797bd5353472037d2 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/projects/jenkins/jenkins_deploy_yamls.yaml M jobs/confs/shell-scripts/jenkins_deploy_yamls.sh 2 files changed, 14 insertions(+), 1 deletion(-) Approvals: David Caro: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/48387 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I389cb3783df5ce2fe2ba940797bd5353472037d2 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Using arrays for the deploy job options
by dcaro＠redhat.com 25 Nov '15

25 Nov '15

David Caro has submitted this change and it was merged. Change subject: Using arrays for the deploy job options ...................................................................... Using arrays for the deploy job options Change-Id: Ic2383f35bd1a7601a5f499c1112e679dd1172819 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/shell-scripts/jenkins_deploy_yamls.sh 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: David Caro: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/48386 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic2383f35bd1a7601a5f499c1112e679dd1172819 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Added the FLUSH_CACHE option to deploy yamls
by dcaro＠redhat.com 25 Nov '15

25 Nov '15

David Caro has submitted this change and it was merged. Change subject: Added the FLUSH_CACHE option to deploy yamls ...................................................................... Added the FLUSH_CACHE option to deploy yamls So we can force to redeploy all the jobs Change-Id: Iad43ae2562f52495c09edca7c3a03d9aa96e9e11 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/projects/jenkins/jenkins_deploy_yamls.yaml M jobs/confs/shell-scripts/jenkins_deploy_yamls.sh 2 files changed, 14 insertions(+), 0 deletions(-) Approvals: David Caro: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/48385 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad43ae2562f52495c09edca7c3a03d9aa96e9e11 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: AddVdsSpmIdCommand DAOs
by amureini＠redhat.com 25 Nov '15

25 Nov '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: AddVdsSpmIdCommand DAOs ...................................................................... core: AddVdsSpmIdCommand DAOs Remove useless DbFacade calls to make the code both easier to read and to test. Change-Id: Ic3aef7cd582ab2f9f27d15437dfa6054f6a70172 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsSpmIdCommand.java 1 file changed, 2 insertions(+), 4 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests Ala Hino: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/49039 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic3aef7cd582ab2f9f27d15437dfa6054f6a70172 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: ImportVmCommand streaming API
by amureini＠redhat.com 25 Nov '15

25 Nov '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: ImportVmCommand streaming API ...................................................................... core: ImportVmCommand streaming API Replaced the cumbersome LinqUtils call with a more modern and elegant usage of Java 8's streaming API. Change-Id: I11c49d50040c33d337d1152cfe502096fb0c00c6 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java 1 file changed, 2 insertions(+), 8 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests Ala Hino: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/49045 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I11c49d50040c33d337d1152cfe502096fb0c00c6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: GetImageByIdQuery streaming API
by amureini＠redhat.com 25 Nov '15

25 Nov '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: GetImageByIdQuery streaming API ...................................................................... core: GetImageByIdQuery streaming API Replaced the cumbersome LinqUtils call with a more modern and elegant usage of Java 8's streaming API. Change-Id: Icd403220c28cf1d7403537cc2da4b100e4fbf11a Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetImageByIdQuery.java 1 file changed, 2 insertions(+), 8 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests Ala Hino: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/49044 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icd403220c28cf1d7403537cc2da4b100e4fbf11a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: ExportVmCommand streaming API
by amureini＠redhat.com 25 Nov '15

25 Nov '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: ExportVmCommand streaming API ...................................................................... core: ExportVmCommand streaming API Replaced the cumbersome LinqUtils call with a more modern and elegant usage of Java 8's streaming API. Change-Id: I7fde7655ec75efb489cd9e9f347944c2efcd5daf Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ExportVmCommand.java 1 file changed, 1 insertion(+), 10 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests Ala Hino: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/49043 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7fde7655ec75efb489cd9e9f347944c2efcd5daf Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: AddVdsSpmIdCommand streaming API
by amureini＠redhat.com 25 Nov '15

25 Nov '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: AddVdsSpmIdCommand streaming API ...................................................................... core: AddVdsSpmIdCommand streaming API Replaced the outdated LinqUtils call with a more modern use of Java 8's stream APIs. Besides cleaning up the code, it offers a slight performance improvement by iterating the list only once instead of twice - one for extracting the spm_vds_ids and another one for sorting it. Change-Id: I23080c2241ee8470b5eeae44e70097ce555d23f1 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsSpmIdCommand.java 1 file changed, 4 insertions(+), 9 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests Ala Hino: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/49042 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I23080c2241ee8470b5eeae44e70097ce555d23f1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: AddVdsSpmIdCommand comments
by amureini＠redhat.com 25 Nov '15

25 Nov '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: AddVdsSpmIdCommand comments ...................................................................... core: AddVdsSpmIdCommand comments Remove useless comment. Change-Id: I6594610b991b3a3daa3c48dd28b579e37733ef2d Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsSpmIdCommand.java 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Allon Mureinik: Verified; Passed CI tests Ala Hino: Looks good to me, but someone else must approve Liron Aravot: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/49041 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6594610b991b3a3daa3c48dd28b579e37733ef2d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0