August 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: ui: Grammar fix console connection denied: 'to not' -> 'not to'
by ofrenkel＠redhat.com 31 Aug '15

31 Aug '15

Omer Frenkel has submitted this change and it was merged. Change subject: ui: Grammar fix console connection denied: 'to not' -> 'not to' ...................................................................... ui: Grammar fix console connection denied: 'to not' -> 'not to' Bug-Url: https://bugzilla.redhat.com/1080383 Change-Id: I0617118ca41d07c9e7964bdb40d4c84eea5f7918 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 4 files changed, 4 insertions(+), 3 deletions(-) Approvals: Tomas Jelinek: Looks good to me, but someone else must approve Shahar Havivi: Verified Jenkins CI: Passed CI tests Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/45482 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0617118ca41d07c9e7964bdb40d4c84eea5f7918 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: packaging: setup: create admin@internal with proper firstname
by sbonazzo＠redhat.com 31 Aug '15

31 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: create admin@internal with proper firstname ...................................................................... packaging: setup: create admin@internal with proper firstname Change-Id: I9d4c2ec9d03ec850fd465ecc640fb74b27cc8a0a Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1258373 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/aaajdbc.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Looks good to me, approved Martin Peřina: Verified; Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45451 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9d4c2ec9d03ec850fd465ecc640fb74b27cc8a0a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: pki: pki-enroll-openssh-cert.sh: do not allow interactive pa...
by sbonazzo＠redhat.com 31 Aug '15

31 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: pki: pki-enroll-openssh-cert.sh: do not allow interactive passphrase prompt ...................................................................... pki: pki-enroll-openssh-cert.sh: do not allow interactive passphrase prompt passphrase is done in older ssh-keygen implementation in any error condition. for example, if ca key cannot be read a passphrase prompt will be issued. feeding empty passphrase will workaround this for now. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1258365 Change-Id: I6da2a357b428778fa897392b48d06849ca8727e7 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/bin/pki-enroll-openssh-cert.sh 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45475 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6da2a357b428778fa897392b48d06849ca8727e7 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: aaa: AuthRecord.VALID_TO should be Z timezone
by tnisan＠redhat.com 31 Aug '15

31 Aug '15

Tal Nisan has submitted this change and it was merged. Change subject: aaa: AuthRecord.VALID_TO should be Z timezone ...................................................................... aaa: AuthRecord.VALID_TO should be Z timezone Fix also extension documentation to match java format expectation to avoid future mistakes. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1258275 Change-Id: I282fdd4a1a2a197da9a4616c55066d200de62755 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/LoginBaseCommand.java M backend/manager/modules/extensions-api-root/extensions-api/src/main/java/org/ovirt/engine/api/extensions/aaa/Authn.java 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/45506 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I282fdd4a1a2a197da9a4616c55066d200de62755 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin, userportal: translations update (de-DE only)
by tnisan＠redhat.com 31 Aug '15

31 Aug '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin, userportal: translations update (de-DE only) ...................................................................... webadmin, userportal: translations update (de-DE only) pulled translation from Zanata oVirt project, master version. [ovirt-3.6, translation cycle 3 "leftovers", de-DE only] [at this point, there is still no 'ovirt-3.6' version in Zanata; translations pulled here are of source strings pushed before ovirt-engine-3.6 was forked out of master, so strings here should be valid for both master and ovirt-engine-3.6] de-DE translations are updated here, separately from the other locales, to allow backport to ovirt-3.5, required by BZ#1218706. Change-Id: I6af1c01d422d2606b797a139d7e272c24e03c7b7 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1128426 Signed-off-by: Einav Cohen <ecohen(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationConstants_de_DE.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums_de_DE.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIConstants_de_DE.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIMessages_de_DE.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_de_DE.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors_de_DE.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationConstants_de_DE.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationMessages_de_DE.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_de_DE.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors_de_DE.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants_de_DE.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationMessages_de_DE.properties M packaging/branding/ovirt.brand/messages_de_DE.properties 13 files changed, 1,090 insertions(+), 306 deletions(-) Approvals: Jenkins CI: Passed CI tests Einav Cohen: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/45497 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6af1c01d422d2606b797a139d7e272c24e03c7b7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: aaa: AuthRecord.VALID_TO should be Z timezone
by tnisan＠redhat.com 31 Aug '15

31 Aug '15

Tal Nisan has submitted this change and it was merged. Change subject: aaa: AuthRecord.VALID_TO should be Z timezone ...................................................................... aaa: AuthRecord.VALID_TO should be Z timezone Fix also extension documentation to match java format expectation to avoid future mistakes. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1258275 Change-Id: I282fdd4a1a2a197da9a4616c55066d200de62755 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/LoginBaseCommand.java M backend/manager/modules/extensions-api-root/extensions-api/src/main/java/org/ovirt/engine/api/extensions/aaa/Authn.java 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/45505 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I282fdd4a1a2a197da9a4616c55066d200de62755 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: allow create template from snapshot for UP vms
by tnisan＠redhat.com 31 Aug '15

31 Aug '15

Tal Nisan has submitted this change and it was merged. Change subject: core: allow create template from snapshot for UP vms ...................................................................... core: allow create template from snapshot for UP vms Change-Id: I39bab3ad2942f903aeec1cc154f1a456cc004ed6 Bug-Url: https://bugzilla.redhat.com/1252030 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateFromSnapshotCommand.java 2 files changed, 18 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Verified Jenkins CI: Passed CI tests Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/45510 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I39bab3ad2942f903aeec1cc154f1a456cc004ed6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: Revert "setup: aaa: Preserve external_id of existing admin u...
by sbonazzo＠redhat.com 31 Aug '15

31 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Revert "setup: aaa: Preserve external_id of existing admin user" ...................................................................... Revert "setup: aaa: Preserve external_id of existing admin user" Wrong solution / different issue. This reverts commit d1aeee28287a1b8ad6248aeb8a79ebce77130e7c. Change-Id: I2882f4e5dfd02d3ccb4a844273d5c1066e26aed3 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1257077 --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/aaainternal.py 1 file changed, 1 insertion(+), 34 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45508 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2882f4e5dfd02d3ccb4a844273d5c1066e26aed3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: fix VM dialog for German translation
by tjelinek＠redhat.com 31 Aug '15

31 Aug '15

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: fix VM dialog for German translation ...................................................................... webadmin: fix VM dialog for German translation - Changed the the SerialNumberPolicy to use the standard way of showing the question mark - made the width of the dialog bigger Change-Id: I311377b5befc4ca4bbe89381bb9ce4472f19de2d Bug-Url: https://bugzilla.redhat.com/1255678 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/AbstractVmPopupView.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/SerialNumberPolicyWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/SerialNumberPolicyWidget.ui.xml 3 files changed, 21 insertions(+), 45 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Martin Betak: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/45280 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I311377b5befc4ca4bbe89381bb9ce4472f19de2d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: allow create template from snapshot for UP vms
by tjelinek＠redhat.com 31 Aug '15

31 Aug '15

Tomas Jelinek has submitted this change and it was merged. Change subject: core: allow create template from snapshot for UP vms ...................................................................... core: allow create template from snapshot for UP vms Change-Id: I39bab3ad2942f903aeec1cc154f1a456cc004ed6 Bug-Url: https://bugzilla.redhat.com/1252030 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateFromSnapshotCommand.java 2 files changed, 18 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Verified Jenkins CI: Passed CI tests Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/45427 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I39bab3ad2942f903aeec1cc154f1a456cc004ed6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0