January 2016 - Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: packaging: make the reference to mgmt. console clear
by sbonazzo＠redhat.com 19 Jan '16

19 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: make the reference to mgmt. console clear ...................................................................... packaging: make the reference to mgmt. console clear Modify the text so that it will be clear we're talking about a mgmt. console VM. Change-Id: I589cdb9da976c81689ee78ba8d83c51d94f0fb44 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1241430 (cherry picked from commit 2dc0d189b6bdb38387bacc871dde7d6e031a8117) --- M src/plugins/ovirt-hosted-engine-setup/core/misc.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52425 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I589cdb9da976c81689ee78ba8d83c51d94f0fb44 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 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 ovirt-hosted-engine-setup[master]: packaging: make the reference to mgmt. console clear
by sbonazzo＠redhat.com 19 Jan '16

19 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: make the reference to mgmt. console clear ...................................................................... packaging: make the reference to mgmt. console clear Modify the text so that it will be clear we're talking about a mgmt. console VM. Change-Id: I589cdb9da976c81689ee78ba8d83c51d94f0fb44 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1241430 --- M src/plugins/ovirt-hosted-engine-setup/core/misc.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/51928 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I589cdb9da976c81689ee78ba8d83c51d94f0fb44 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: packaging: setup: honor $TMPDIR as temporary directory, if p...
by sbonazzo＠redhat.com 19 Jan '16

19 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: honor $TMPDIR as temporary directory, if provided ...................................................................... packaging: setup: honor $TMPDIR as temporary directory, if provided Bug-Url: https://bugzilla.redhat.com/1251965 Change-Id: I15b2887b9e2f1b466f4aaa83ce69a4932b54755b Signed-off-by: Rafael Martins <rmartins(a)redhat.com> (cherry picked from commit 87ac12562efb21d0227250c8550be5115d92136e) --- M src/plugins/ovirt-hosted-engine-setup/vm/boot_disk.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52424 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15b2887b9e2f1b466f4aaa83ce69a4932b54755b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 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 ovirt-reports[master]: build: after build reports for 3.6.2.4
by sradco＠redhat.com 19 Jan '16

19 Jan '16

Shirly Radco has submitted this change and it was merged. Change subject: build: after build reports for 3.6.2.4 ...................................................................... build: after build reports for 3.6.2.4 Change-Id: I953978c10093d330d5966a1b3686a70f460ac6af Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M version.mak 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52420 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I953978c10093d330d5966a1b3686a70f460ac6af Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-reports[master]: build: build reports for 3.6.2.4
by sradco＠redhat.com 19 Jan '16

19 Jan '16

Shirly Radco has submitted this change and it was merged. Change subject: build: build reports for 3.6.2.4 ...................................................................... build: build reports for 3.6.2.4 Change-Id: Icc353f4fc2391976e82f7614af77848cf56fbc9f Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M ovirt-engine-reports.spec.in M version.mak 2 files changed, 7 insertions(+), 3 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52419 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc353f4fc2391976e82f7614af77848cf56fbc9f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: packaging: rename: fixes for vmconsole proxy helper
by tnisan＠redhat.com 19 Jan '16

19 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: rename: fixes for vmconsole proxy helper ...................................................................... packaging: rename: fixes for vmconsole proxy helper Change-Id: Ia6ed40d6b31dfe891650f6b3b7a648e638b58b3e Bug-Url: https://bugzilla.redhat.com/1290088 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit a4d12daf5b914f5a70591ad9d814a06c34fa6f9f) --- A packaging/setup/plugins/ovirt-engine-rename/vmconsole_proxy_helper/__init__.py A packaging/setup/plugins/ovirt-engine-rename/vmconsole_proxy_helper/config.py 2 files changed, 130 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/52416 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia6ed40d6b31dfe891650f6b3b7a648e638b58b3e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: honor $TMPDIR as temporary directory, if p...
by sbonazzo＠redhat.com 19 Jan '16

19 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: honor $TMPDIR as temporary directory, if provided ...................................................................... packaging: setup: honor $TMPDIR as temporary directory, if provided Bug-Url: https://bugzilla.redhat.com/1251965 Change-Id: I15b2887b9e2f1b466f4aaa83ce69a4932b54755b Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/vm/boot_disk.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Rafael Martins: Verified -- To view, visit https://gerrit.ovirt.org/52401 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15b2887b9e2f1b466f4aaa83ce69a4932b54755b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Rafael Martins <rmartins(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: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: vdsm-jsonrpc-java:Added standard ci jobs
by dcaro＠redhat.com 19 Jan '16

19 Jan '16

David Caro has submitted this change and it was merged. Change subject: vdsm-jsonrpc-java:Added standard ci jobs ...................................................................... vdsm-jsonrpc-java:Added standard ci jobs Added also the 3.6 publisher changes Change-Id: I06dc4ef16bce73ec6f08241c67a09aa36b86ec62 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/projects/ovirt/publish-rpms-nightly-3.6.yaml A jobs/confs/projects/vdsm-jsonrpc-java/vdsm-jsonrpc-java_standard.yaml 2 files changed, 27 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for review Jenkins CI: Passed CI tests David Caro: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/51884 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I06dc4ef16bce73ec6f08241c67a09aa36b86ec62 Gerrit-PatchSet: 7 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: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sagi Shnaidman <sshnaidm(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: AddVmCommand variable initializers
by amureini＠redhat.com 19 Jan '16

19 Jan '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: AddVmCommand variable initializers ...................................................................... core: AddVmCommand variable initializers Removed redundant intializers that only hinder the ability to understand the code. Change-Id: Ice48e5e9cd95d9ba7b23b0631a1c86464692b65f Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java 1 file changed, 2 insertions(+), 3 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52242 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ice48e5e9cd95d9ba7b23b0631a1c86464692b65f Gerrit-PatchSet: 8 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: Arik Hadas <ahadas(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: Enable FindBugs naming convention checking
by amureini＠redhat.com 19 Jan '16

19 Jan '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: Enable FindBugs naming convention checking ...................................................................... core: Enable FindBugs naming convention checking Commit aba908b96ed47ece9c429d97cb53597fda3b9acf solved the last naming convention issues in the project. This parch re-enables FindBugs checking for that subject, as an extra safety measure. Change-Id: I4046692a8c18c9d9912b46517b610d19d506bab2 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M frontend/webadmin/modules/frontend/exclude-filters.xml 1 file changed, 0 insertions(+), 10 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Daniel Erez: Looks good to me, but someone else must approve Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52237 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4046692a8c18c9d9912b46517b610d19d506bab2 Gerrit-PatchSet: 7 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: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0