Juan Hernandez has posted comments on this change. Change subject: restapi: pass the data needed for version check ......................................................................
Patch Set 5: (1 comment) https://gerrit.ovirt.org/#/c/41948/5/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmGraphicsConsoleResource.java File backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmGraphicsConsoleResource.java: Line 62: Line 63: ConsoleOptions consoleOptions = new ConsoleOptions(graphicsType); Line 64: consoleOptions.setVmId(vmGuid); Line 65: ConfigureConsoleOptionsParams queryParams = new ConfigureConsoleOptionsParams(consoleOptions, true); Line 66: queryParams.setConsoleClientResourcesUrl(BrandingManager.getInstance().getMessage("obrand.common.console_client_resources_url")); > but the BrandingManager is not available in the bll module. Even if added t You need to add "org.ovirt.engine.core.branding" to ear/src/main/application/META-INF/jboss-deployment-structure.xml. Line 67: Line 68: ConsoleOptions configuredOptions = runQuery(VdcQueryType.ConfigureConsoleOptions, Line 69: queryParams).getReturnValue(); Line 70: -- To view, visit https://gerrit.ovirt.org/41948 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0b3fe2dfa3e254968dc7d14e82497dec7005bd15 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjeli...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com> Gerrit-Reviewer: Ori Liel <ol...@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
