- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0]: engine: RHEV to RHV in error messages
by oourfali＠redhat.com 02 Aug '16

02 Aug '16

Oved Ourfali has submitted this change and it was merged. Change subject: engine: RHEV to RHV in error messages ...................................................................... engine: RHEV to RHV in error messages Change-Id: I242bbb7b25cdf59eb89e5d33edd006b94b1f0a03 Bug-Url: https://bugzilla.redhat.com/1360920 Signed-off-by: Scott J Dickerson <sdickers(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties 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 5 files changed, 9 insertions(+), 9 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Scott Dickerson: Verified -- To view, visit https://gerrit.ovirt.org/61788 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I242bbb7b25cdf59eb89e5d33edd006b94b1f0a03 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Scott Dickerson <sdickers(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Scott Dickerson <sdickers(a)redhat.com> 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

Change in jenkins[master]: publisher: python-windows_master moved to fc24
by sbonazzo＠redhat.com 02 Aug '16

02 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: publisher: python-windows_master moved to fc24 ...................................................................... publisher: python-windows_master moved to fc24 Change-Id: I168a47aa65abee7eddfedc3f98155d8ae5ac14ca Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt/publish-rpms-nightly-master.yaml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61798 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I168a47aa65abee7eddfedc3f98155d8ae5ac14ca 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[master]: engine: RHEV to RHV in error messages
by awels＠redhat.com 01 Aug '16

01 Aug '16

Alexander Wels has submitted this change and it was merged. Change subject: engine: RHEV to RHV in error messages ...................................................................... engine: RHEV to RHV in error messages Change-Id: I242bbb7b25cdf59eb89e5d33edd006b94b1f0a03 Bug-Url: https://bugzilla.redhat.com/1360920 Signed-off-by: Scott J Dickerson <sdickers(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties 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 5 files changed, 9 insertions(+), 9 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Scott Dickerson: Verified -- To view, visit https://gerrit.ovirt.org/61587 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I242bbb7b25cdf59eb89e5d33edd006b94b1f0a03 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Scott Dickerson <sdickers(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Scott Dickerson <sdickers(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Yaniv Kaul <ykaul(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0.2]: webadmin: Allow UI plugins to run certain actions while loading
by oourfali＠redhat.com 01 Aug '16

01 Aug '16

Oved Ourfali has submitted this change and it was merged. Change subject: webadmin: Allow UI plugins to run certain actions while loading ...................................................................... webadmin: Allow UI plugins to run certain actions while loading UI plugin API performs requested actions only if the given plugin is either initializing (within UiInit callback) or in use (within other callbacks). This means all API actions are no-op while the plugin is loading, e.g. before the plugin calls the ready() function that triggers the UiInit callback. This limits the usability of plugin API, for example: var api = window.top.pluginApi('my-plugin'); api.register(objectWithCallbacks); // problem: cannot use e.g. `api.currentLocale` yet! api.ready(); // triggers UiInit callback execution This patch fixes the above issue by letting following API functions to be executed also while the plugin is loading (before api.ready): - loginUserName - loginUserId - ssoToken - engineBaseUrl - currentLocale Change-Id: I8996f4e7bb69696fb13a99068cf1a0c267b1481a Bug-Url: https://bugzilla.redhat.com/1361255 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/plugin/PluginManager.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/plugin/PluginState.java 2 files changed, 26 insertions(+), 18 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Vojtech Szocs: Verified -- To view, visit https://gerrit.ovirt.org/61779 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8996f4e7bb69696fb13a99068cf1a0c267b1481a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.2 Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> 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

Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Allow UI plugins to run certain actions while loading
by oourfali＠redhat.com 01 Aug '16

01 Aug '16

Oved Ourfali has submitted this change and it was merged. Change subject: webadmin: Allow UI plugins to run certain actions while loading ...................................................................... webadmin: Allow UI plugins to run certain actions while loading UI plugin API performs requested actions only if the given plugin is either initializing (within UiInit callback) or in use (within other callbacks). This means all API actions are no-op while the plugin is loading, e.g. before the plugin calls the ready() function that triggers the UiInit callback. This limits the usability of plugin API, for example: var api = window.top.pluginApi('my-plugin'); api.register(objectWithCallbacks); // problem: cannot use e.g. `api.currentLocale` yet! api.ready(); // triggers UiInit callback execution This patch fixes the above issue by letting following API functions to be executed also while the plugin is loading (before api.ready): - loginUserName - loginUserId - ssoToken - engineBaseUrl - currentLocale Change-Id: I8996f4e7bb69696fb13a99068cf1a0c267b1481a Bug-Url: https://bugzilla.redhat.com/1361255 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/plugin/PluginManager.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/plugin/PluginState.java 2 files changed, 26 insertions(+), 18 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Vojtech Szocs: Verified -- To view, visit https://gerrit.ovirt.org/61778 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8996f4e7bb69696fb13a99068cf1a0c267b1481a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> 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

Change in ovirt-engine[master]: spec: Require ovirt-imageio-proxy
by sbonazzo＠redhat.com 01 Aug '16

01 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: spec: Require ovirt-imageio-proxy ...................................................................... spec: Require ovirt-imageio-proxy For easy setup of ovirt-imageio-proxy, it should be required in ovirt's spec. otherwise, for uploading images in ovirt, the user will need to manual install ovirt-imageio-proxy. Change-Id: I1227bd8789fca98334b13afbb5d6d390d114da01 Bug-Url: https://bugzilla.redhat.com/1361845 Signed-off-by: Amit Aviram <aaviram(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Amit Aviram: Verified Allon Mureinik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/61726 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1227bd8789fca98334b13afbb5d6d390d114da01 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(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[master]: webadmin: Allow UI plugins to run certain actions while loading
by vszocs＠redhat.com 01 Aug '16

01 Aug '16

Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin: Allow UI plugins to run certain actions while loading ...................................................................... webadmin: Allow UI plugins to run certain actions while loading UI plugin API performs requested actions only if the given plugin is either initializing (within UiInit callback) or in use (within other callbacks). This means all API actions are no-op while the plugin is loading, e.g. before the plugin calls the ready() function that triggers the UiInit callback. This limits the usability of plugin API, for example: var api = window.top.pluginApi('my-plugin'); api.register(objectWithCallbacks); // problem: cannot use e.g. `api.currentLocale` yet! api.ready(); // triggers UiInit callback execution This patch fixes the above issue by letting following API functions to be executed also while the plugin is loading (before api.ready): - loginUserName - loginUserId - ssoToken - engineBaseUrl - currentLocale Change-Id: I8996f4e7bb69696fb13a99068cf1a0c267b1481a Bug-Url: https://bugzilla.redhat.com/1361255 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/plugin/PluginManager.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/plugin/PluginState.java 2 files changed, 26 insertions(+), 16 deletions(-) Approvals: Jenkins CI: Passed CI tests Martin Betak: Looks good to me, but someone else must approve Greg Sheremeta: Looks good to me, approved Vojtech Szocs: Verified -- To view, visit https://gerrit.ovirt.org/61664 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8996f4e7bb69696fb13a99068cf1a0c267b1481a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: vdsm: remove el7 from check-merged jobs
by sbonazzo＠redhat.com 01 Aug '16

01 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: vdsm: remove el7 from check-merged jobs ...................................................................... vdsm: remove el7 from check-merged jobs el7 check-merged jobs are failing because currently there's only lago setup for fc24. A setup for el7 will be added in the future. Change-Id: I8a33f636abc9dc90aed081b310a2e554b5935498 Signed-off-by: Irit Goihman <igoihman(a)redhat.com> --- M jobs/confs/projects/vdsm/vdsm_standard.yaml 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61766 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a33f636abc9dc90aed081b310a2e554b5935498 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: ImportVmFromConfigurationCommand type inference
by amureini＠redhat.com 01 Aug '16

01 Aug '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: ImportVmFromConfigurationCommand type inference ...................................................................... core: ImportVmFromConfigurationCommand type inference Change-Id: I3de5a404f97e563090e0edd0afd03cabf770fe9d Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmFromConfigurationCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Allon Mureinik: Verified; Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/61724 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3de5a404f97e563090e0edd0afd03cabf770fe9d 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: Daniel Erez <derez(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: ImportVmTemplateCommand local variables
by amureini＠redhat.com 01 Aug '16

01 Aug '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: ImportVmTemplateCommand local variables ...................................................................... core: ImportVmTemplateCommand local variables Remove redundant useless variables to improve the code's readability. Change-Id: I80540de17374ef4aaa8ea09982a337656f9bb381 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmTemplateCommand.java 1 file changed, 14 insertions(+), 16 deletions(-) Approvals: Allon Mureinik: Verified; Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/61746 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I80540de17374ef4aaa8ea09982a337656f9bb381 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: Daniel Erez <derez(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