Oved Ourfali has uploaded a new change for review. Change subject: core: set current user in import glance image as template ......................................................................
core: set current user in import glance image as template Importing a glance image may take a while. Thus, when calling AddVmTemplate the session may no longer be valid, so getCurrentUser() will return null. This patch sets the user in the parameters class. CommandBase makes sure to take that in case getCurrentUser() is null Change-Id: I69dd535db983471ba31b8d427b4e0a373f1f7dc4 Bug-Url: https://bugzilla.redhat.com/1085712 Signed-off-by: Oved Ourfali <oourf...@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCopyTaskHandler.java 1 file changed, 3 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/01/26601/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCopyTaskHandler.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCopyTaskHandler.java index e15a98f..1b59c3a 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCopyTaskHandler.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCopyTaskHandler.java @@ -120,6 +120,9 @@ String vmTemplateName = "GlanceTemplate-" + Guid.newGuid().toString().substring(0, 7); AddVmTemplateParameters parameters = new AddVmTemplateParameters(masterVm, vmTemplateName, templateDiskImage.getDiskDescription()); + // Setting the user from the parent command, as the session might already be invalid + parameters.setParametersCurrentUser(getEnclosingCommand().getParameters().getParametersCurrentUser()); + // Setting the cluster ID, and other related properties derived from it if (getEnclosingCommand().getParameters().getClusterId() != null) { masterVm.setVdsGroupId(getEnclosingCommand().getParameters().getClusterId()); -- To view, visit http://gerrit.ovirt.org/26601 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I69dd535db983471ba31b8d427b4e0a373f1f7dc4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <oourf...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
