Change in ovirt-engine[master]: core: VdsDeployVdsmUnit LinqUtils
by oourfali@redhat.com
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>
9 years, 1 month
Change in jenkins[master]: Added possibility to filter jobs to deploy
by dcaro@redhat.com
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>
9 years, 1 month
Change in jenkins[master]: Using arrays for the deploy job options
by dcaro@redhat.com
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>
9 years, 1 month
Change in jenkins[master]: Added the FLUSH_CACHE option to deploy yamls
by dcaro@redhat.com
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>
9 years, 1 month
Change in ovirt-engine[master]: core: AddVdsSpmIdCommand DAOs
by amureini@redhat.com
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>
9 years, 1 month
Change in ovirt-engine[master]: core: ImportVmCommand streaming API
by amureini@redhat.com
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>
9 years, 1 month
Change in ovirt-engine[master]: core: GetImageByIdQuery streaming API
by amureini@redhat.com
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>
9 years, 1 month
Change in ovirt-engine[master]: core: ExportVmCommand streaming API
by amureini@redhat.com
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>
9 years, 1 month
Change in ovirt-engine[master]: core: AddVdsSpmIdCommand streaming API
by amureini@redhat.com
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>
9 years, 1 month
Change in ovirt-engine[master]: core: AddVdsSpmIdCommand comments
by amureini@redhat.com
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>
9 years, 1 month