July 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: cleanup in VmAnalyzer
by ahadas＠redhat.com 14 Jul '16

14 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: cleanup in VmAnalyzer ...................................................................... core: cleanup in VmAnalyzer 1. rename updateVmStatistics to updateStatistics as it update different kind of statistics and not only vm-statistics. 2. extract the vm-statistics handling into a separate method called updateVmStatistics. Change-Id: Ieb40461fe6b064ce9b62d238a6f293cbbe223a3b Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java 1 file changed, 9 insertions(+), 5 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/60708 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb40461fe6b064ce9b62d238a6f293cbbe223a3b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: use stream to detect unchangeable fields in vm-dynamic
by ahadas＠redhat.com 14 Jul '16

14 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: use stream to detect unchangeable fields in vm-dynamic ...................................................................... core: use stream to detect unchangeable fields in vm-dynamic Use stream API in order to detect the fields within VmDynamic that are unchangeable by VDSM. Change-Id: I74954aa02b17fa373df80fc17c70aa27eb480b00 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java 1 file changed, 5 insertions(+), 6 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/60707 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74954aa02b17fa373df80fc17c70aa27eb480b00 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: scripts: use sudo for installing autoconf and automake
by sbonazzo＠redhat.com 14 Jul '16

14 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: scripts: use sudo for installing autoconf and automake ...................................................................... scripts: use sudo for installing autoconf and automake Change-Id: I5ade1f65be8b3ad3efae935da034cbf5c152ccbb Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/shell-scripts/build_local_srcrpm.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/60718 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5ade1f65be8b3ad3efae935da034cbf5c152ccbb Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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]: build_local_srcrpm: ensure autoconf and automake
by sbonazzo＠redhat.com 14 Jul '16

14 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build_local_srcrpm: ensure autoconf and automake ...................................................................... build_local_srcrpm: ensure autoconf and automake Ensure that autoconf and automake are available before calling autogen.sh script. Change-Id: I76372f6417e5aae2257ac40d460537b30b26870f Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/shell-scripts/build_local_srcrpm.sh 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/60717 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I76372f6417e5aae2257ac40d460537b30b26870f Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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]: ovirt-imageio: move master to fc24
by sbonazzo＠redhat.com 14 Jul '16

14 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-imageio: move master to fc24 ...................................................................... ovirt-imageio: move master to fc24 - move master to fc24 Change-Id: I9bc67e65573ec755212562056efc19c221bce29d Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-imageio/ovirt-imageio.yaml 1 file changed, 8 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60716 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9bc67e65573ec755212562056efc19c221bce29d Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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]: ovirt-hosted-engine-ha: move from fc22 to fc24
by sbonazzo＠redhat.com 14 Jul '16

14 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-hosted-engine-ha: move from fc22 to fc24 ...................................................................... ovirt-hosted-engine-ha: move from fc22 to fc24 - dropped fc22 gone EOL - moved master to fc24 - added extra build packages since they're not in the mock chroot by default Change-Id: I5e57b67c24503ede0c6b6749a8b84081d81d0d69 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-hosted-engine-ha/ovirt-hosted-engine-ha_create-rpms.yaml 1 file changed, 23 insertions(+), 21 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60714 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5e57b67c24503ede0c6b6749a8b84081d81d0d69 Gerrit-PatchSet: 5 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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]: ovirt-iso-uploader: move from fc22 to fc24
by sbonazzo＠redhat.com 14 Jul '16

14 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-iso-uploader: move from fc22 to fc24 ...................................................................... ovirt-iso-uploader: move from fc22 to fc24 - Drop fc22 on 3.6 since fc22 gone EOL - Move master to fc24 - move 4.0 to its own branch Change-Id: Ie77ba3aa87038ed2ef3fb9963a65bd6a0963b636 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-iso-uploader/ovirt-iso-uploader_create-rpms.yaml M jobs/confs/projects/ovirt-iso-uploader/ovirt-iso-uploader_standard.yaml 2 files changed, 11 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60669 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie77ba3aa87038ed2ef3fb9963a65bd6a0963b636 Gerrit-PatchSet: 5 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(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: Use Java 8 streaming API in StorageConnectionValidator...
by tnisan＠redhat.com 14 Jul '16

14 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Use Java 8 streaming API in StorageConnectionValidatorTest ...................................................................... core: Use Java 8 streaming API in StorageConnectionValidatorTest Use Java 8 streaming API to create connections in StorageConnectionValidatorTest instead of the loop Change-Id: I2820953f2d0e43051ed851f6cff5489db276e3c1 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/validator/storage/StorageConnectionValidatorTest.java 1 file changed, 6 insertions(+), 8 deletions(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Ala Hino: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/60618 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2820953f2d0e43051ed851f6cff5489db276e3c1 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(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: Create connections in StorageConnectionValidatorTest u...
by tnisan＠redhat.com 14 Jul '16

14 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Create connections in StorageConnectionValidatorTest using a loop ...................................................................... core: Create connections in StorageConnectionValidatorTest using a loop Create the existing connections in StorageConnectionValidatorTest with a loop instead of boilerplate code Change-Id: Ia6afc199309692985973571aeefa3265cebc8074 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/validator/storage/StorageConnectionValidatorTest.java 1 file changed, 6 insertions(+), 9 deletions(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Ala Hino: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/60617 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia6afc199309692985973571aeefa3265cebc8074 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(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: Create random IDs in StorageConnectionValidatorTest
by tnisan＠redhat.com 14 Jul '16

14 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Create random IDs in StorageConnectionValidatorTest ...................................................................... core: Create random IDs in StorageConnectionValidatorTest The GUIDs in StorageConnectionValidatorTest were created statically from arbitrary GUIDs, this is a bad practice as the behavior should not use specific GUIDs as they are meaningless but rather use random generated ones Change-Id: I0d02c24f9dc0800149b3b0d3d58a9f1d99c92fed Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/validator/storage/StorageConnectionValidatorTest.java 1 file changed, 9 insertions(+), 6 deletions(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Ala Hino: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/60616 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0d02c24f9dc0800149b3b0d3d58a9f1d99c92fed Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(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