August 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: AddVmTemplateFromSnapshotCommand type inference
by tnisan＠redhat.com 30 Aug '16

30 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: core: AddVmTemplateFromSnapshotCommand type inference ...................................................................... core: AddVmTemplateFromSnapshotCommand type inference Change-Id: Ie898b141622d165ee039b85bc03cea62a34c5f63 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateFromSnapshotCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/62908 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie898b141622d165ee039b85bc03cea62a34c5f63 Gerrit-PatchSet: 3 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]: webadmin: Show warning if cpu topology will be lost
by dfediuck＠redhat.com 30 Aug '16

30 Aug '16

Doron Fediuck has submitted this change and it was merged. Change subject: webadmin: Show warning if cpu topology will be lost ...................................................................... webadmin: Show warning if cpu topology will be lost Shows a warning dialog if the cpu pinning topology of a currently edited VM will be lost when the VM is saved. Change-Id: I82a686629619fc215b55e92dc86c35c7f2b608c5 Bug-Url: https://bugzilla.redhat.com/1135976 Signed-off-by: Andrej Krejcir <akrejcir(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIConstants.properties 4 files changed, 45 insertions(+), 1 deletion(-) Approvals: Tomas Jelinek: Looks good to me, approved Martin Sivák: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Jenny Tokar: Looks good to me, but someone else must approve Andrej Krejcir: Verified -- To view, visit https://gerrit.ovirt.org/60919 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I82a686629619fc215b55e92dc86c35c7f2b608c5 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0.4]: packaging: engine-backup: clarify random password
by tnisan＠redhat.com 30 Aug '16

30 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: engine-backup: clarify random password ...................................................................... packaging: engine-backup: clarify random password Change the text output to the user when creating an extra postgresql user with a random password. Change-Id: I93ee9bd5808f7f713465bc2db5373c5d1181e55b Bug-Url: https://bugzilla.redhat.com/1369757 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit ca35ab6569ec186304d017836b3c0351925943a4) --- M packaging/bin/engine-backup.sh.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Passed CI tests Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62991 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I93ee9bd5808f7f713465bc2db5373c5d1181e55b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.4 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: packaging: engine-backup: clarify random password
by tnisan＠redhat.com 30 Aug '16

30 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: engine-backup: clarify random password ...................................................................... packaging: engine-backup: clarify random password Change the text output to the user when creating an extra postgresql user with a random password. Change-Id: I93ee9bd5808f7f713465bc2db5373c5d1181e55b Bug-Url: https://bugzilla.redhat.com/1369757 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit ca35ab6569ec186304d017836b3c0351925943a4) --- M packaging/bin/engine-backup.sh.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/62915 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I93ee9bd5808f7f713465bc2db5373c5d1181e55b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 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-log-collector[ovirt-log-collector-4.0]: build: post ovirt-log-collector-4.0.1
by sbonazzo＠redhat.com 30 Aug '16

30 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-log-collector-4.0.1 ...................................................................... build: post ovirt-log-collector-4.0.1 Change-Id: If63dbbcebb9fca86f74db121cf75e7e2fd4b75a1 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.sh M configure.ac M ovirt-log-collector.spec.in 3 files changed, 7 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62988 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If63dbbcebb9fca86f74db121cf75e7e2fd4b75a1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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-log-collector[ovirt-log-collector-4.0]: build: ovirt-log-collector-4.0.1
by sbonazzo＠redhat.com 30 Aug '16

30 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-log-collector-4.0.1 ...................................................................... build: ovirt-log-collector-4.0.1 Change-Id: I70b2d2ad5a2732a0152e16b2e3ac852b3b94b07f Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.sh M configure.ac M ovirt-log-collector.spec.in 3 files changed, 10 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62987 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I70b2d2ad5a2732a0152e16b2e3ac852b3b94b07f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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 jenkins[master]: ovirt-engine: move from 4.0.3 to 4.0.4
by sbonazzo＠redhat.com 30 Aug '16

30 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine: move from 4.0.3 to 4.0.4 ...................................................................... ovirt-engine: move from 4.0.3 to 4.0.4 Change-Id: I69839a9911cf7e135316c18b16f45fe02d477efc Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml 1 file changed, 8 insertions(+), 8 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62990 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I69839a9911cf7e135316c18b16f45fe02d477efc 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 ovirt-engine[ovirt-engine-4.0]: build: post ovirt-engine-4.0.4 branching
by tnisan＠redhat.com 30 Aug '16

30 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: build: post ovirt-engine-4.0.4 branching ...................................................................... build: post ovirt-engine-4.0.4 branching Change-Id: If2d1e96c44b62543ba1e5481311eeeefbf9cbb95 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M backend/manager/dependencies/common/pom.xml M backend/manager/dependencies/pom.xml M backend/manager/dependencies/tools/pom.xml M backend/manager/extensions-tool/pom.xml M backend/manager/modules/aaa/pom.xml M backend/manager/modules/auth-plugin/pom.xml M backend/manager/modules/bll/pom.xml M backend/manager/modules/branding/pom.xml M backend/manager/modules/builtin-extensions/pom.xml M backend/manager/modules/common/pom.xml M backend/manager/modules/compat/pom.xml M backend/manager/modules/dal/pom.xml M backend/manager/modules/docs/pom.xml M backend/manager/modules/enginesso/pom.xml M backend/manager/modules/extensions-manager/pom.xml M backend/manager/modules/logger/pom.xml M backend/manager/modules/pom.xml M backend/manager/modules/restapi/interface/common/jaxrs/pom.xml M backend/manager/modules/restapi/interface/common/pom.xml M backend/manager/modules/restapi/interface/definition/pom.xml M backend/manager/modules/restapi/interface/pom.xml M backend/manager/modules/restapi/jaxrs/pom.xml M backend/manager/modules/restapi/pom.xml M backend/manager/modules/restapi/types/pom.xml M backend/manager/modules/restapi/webapp/pom.xml M backend/manager/modules/root/pom.xml M backend/manager/modules/scheduler/pom.xml M backend/manager/modules/searchbackend/pom.xml M backend/manager/modules/services/pom.xml M backend/manager/modules/utils/pom.xml M backend/manager/modules/uutils/pom.xml M backend/manager/modules/vdsbroker/pom.xml M backend/manager/modules/welcome/pom.xml M backend/manager/pom.xml M backend/manager/tools/pom.xml M backend/pom.xml M build-tools-root/checkstyles/pom.xml M build-tools-root/ovirt-checkstyle-extension/pom.xml M build-tools-root/ovirt-findbugs-filters/pom.xml M build-tools-root/pom.xml M build/validations/pom.xml M ear/pom.xml M frontend/pom.xml M frontend/webadmin/modules/frontend-symbols/pom.xml M frontend/webadmin/modules/frontend/pom.xml M frontend/webadmin/modules/gwt-common/pom.xml M frontend/webadmin/modules/gwt-extension/pom.xml M frontend/webadmin/modules/pom.xml M frontend/webadmin/modules/uicommonweb/pom.xml M frontend/webadmin/modules/uicompat/pom.xml M frontend/webadmin/modules/userportal-gwtp/pom.xml M frontend/webadmin/modules/webadmin/pom.xml M frontend/webadmin/pom.xml M mavenmake/pom.xml M pom.xml M version.mak 56 files changed, 57 insertions(+), 57 deletions(-) Approvals: Tal Nisan: Passed CI tests Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62949 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If2d1e96c44b62543ba1e5481311eeeefbf9cbb95 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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>

1 0

Change in ovirt-engine-cli[master]: cli: Ignore "id" and "name" options for the "show" command
by juan.hernandez＠redhat.com 30 Aug '16

30 Aug '16

Juan Hernandez has submitted this change and it was merged. Change subject: cli: Ignore "id" and "name" options for the "show" command ...................................................................... cli: Ignore "id" and "name" options for the "show" command Currently auto-completion for the "show" command includes the "id" and "name" options. This is wrong, as these options aren't used to specify the object to be displayed. The reason they are included in auto-completion is that the underlying Python SDK method is "get", and that method usually has both the "id" and "name" parameters. We simply and blindly extract the parameters from the method and assume that they are valid options. This patch changes the CLI so that it ignores these parameters, thus they won't appear in the auto-completion list. Change-Id: I068686a6c600d0aac582adc239439c5f7dba0f13 Bug-Url: https://bugzilla.redhat.com/1358227 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M src/ovirtcli/shell/showcmdshell.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/61095 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I068686a6c600d0aac582adc239439c5f7dba0f13 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-log-collector[ovirt-log-collector-4.0]: Removed the 'all' keyword fron the config file. reason: it's...
by sbonazzo＠redhat.com 30 Aug '16

30 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Removed the 'all' keyword fron the config file. reason: it's not working since 3.2.0 . ...................................................................... Removed the 'all' keyword fron the config file. reason: it's not working since 3.2.0 . Change-Id: I655ee352db2bdc74a076bfadc017884678569dc8 Signed-off-by: Ido Rosenzwig <irosenzw(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1355966 (cherry picked from commit ebb5b14b3e39665af7b2f856ab91e3ea85e8379c) --- M src/logcollector.conf 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62986 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I655ee352db2bdc74a076bfadc017884678569dc8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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