Mirantis OpenStack

noVNC doesn't work when offloaded to port 80 or 443

Bug #1410223 reported by Timur Nurlygayanov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Invalid
Medium
Aleksander Mogylchenko
Mirantis OpenStack 6.0-updates
6.0.x
Invalid
Medium
Aleksander Mogylchenko
Mirantis OpenStack 6.0-updates
6.1.x
Invalid
Medium
Aleksander Mogylchenko
Mirantis OpenStack 6.1

Bug Description

Customer found bug: https://bugs.launchpad.net/nova/+bug/1228649

When offloading nova-novnc to port 80 or 443 the javascript code does not load the websockets code properly, and the page simply shows "Loading" in black text.

The problem is due to the javascript using `window.location.port` which parses the browser's address bar. This is always an empty string when the protocol is http or https.

The noVNC project addressed this issue in the following patches.

https://github.com/kanaka/noVNC/pull/245
https://github.com/kanaka/noVNC/pull/252

Would like to request a newer nova-novnc be built, or patch the existing package with the PR above, and backport to grizzly's UEC ppas.

Timur Nurlygayanov (tnurlygayanov)
Changed in mos:
assignee: nobody → MOS Linux (mos-linux)
status: New → Confirmed
importance: Undecided → High
Revision history for this message
Aleksander Mogylchenko (amogylchenko) wrote :

We are using novnc from cloud archive:
http://ubuntu-cloud.archive.canonical.com/ubuntu/pool/main/n/novnc/

New version is available with those fixes:
http://ubuntu-cloud.archive.canonical.com/ubuntu/pool/main/n/novnc/novnc_0.4+dfsg+1+20131010+gitf68af8af3d-2~cloud0.dsc

Changed in mos:
assignee: MOS Linux (mos-linux) → Aleksander Mogylchenko (amogylchenko)
Aleksander Mogylchenko (amogylchenko)
Changed in mos:
milestone: none → 6.1
Revision history for this message
OSCI Robot (oscirobot) wrote :

RPM package novnc has been built for project packages/centos6/novnc
Package version == 0.4, package release == 99

Changeset: https://review.fuel-infra.org/2033
project: packages/centos6/novnc
branch: 6.1
author: Aleksandr Mogylchenko
committer: Aleksandr Mogylchenko
subject: Update novnc sources and bump version.
status: patchset-created

Files placed on repository:
novnc-0.4-99.mira1.noarch.rpm

NOTE: Changeset is not merged, created temporary package repository.
RPM repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-6.1-stable-2033/centos

Revision history for this message
OSCI Robot (oscirobot) wrote :

DEB package novnc has been built for project packages/precise/novnc
Package version == 1:0.4+dfsg+1+20131010+gitf68af8af3d, package release == ubuntu1

Changeset: https://review.fuel-infra.org/2029
project: packages/precise/novnc
branch: 6.1
author: Aleksandr Mogylchenko
committer: Aleksandr Mogylchenko
subject: Update noVNC version to latest 0.4 build
status: patchset-created

Files placed on repository:
novnc_0.4+dfsg+1+20131010+gitf68af8af3d-ubuntu1_all.deb
python-novnc_0.4+dfsg+1+20131010+gitf68af8af3d-ubuntu1_all.deb

NOTE: Changeset is not merged, created temporary package repository.
DEB repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-6.1-stable-2029/ubuntu

Revision history for this message
Aleksander Mogylchenko (amogylchenko) wrote :

Timur, could you please test those packages?

Revision history for this message
OSCI Robot (oscirobot) wrote :

DEB package novnc has been built for project packages/precise/novnc
Package version == 1:0.4+dfsg+1+20131010+gitf68af8af3d, package release == ubuntu1

Changeset: https://review.fuel-infra.org/2029
project: packages/precise/novnc
branch: 6.1
author: Aleksandr Mogylchenko
committer: Aleksandr Mogylchenko
subject: Update noVNC version to latest 0.4 build
status: patchset-created

Files placed on repository:
novnc_0.4+dfsg+1+20131010+gitf68af8af3d-ubuntu1_all.deb
python-novnc_0.4+dfsg+1+20131010+gitf68af8af3d-ubuntu1_all.deb

NOTE: Changeset is not merged, created temporary package repository.
DEB repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-6.1-stable-2029/ubuntu

Revision history for this message
Timur Nurlygayanov (tnurlygayanov) wrote :

Andrey Shestakov:

noVNC with these patches will works with https, but browser warns about some unsafe content on page.
Now in MOX we using noVNC-0.5.1 witch works ok.

So, let's just update noVNC package to fix the issue in MOS.

Revision history for this message
Timur Nurlygayanov (tnurlygayanov) wrote :

We have other bug description with the other suggestion to fix the issue:
https://bugs.launchpad.net/fuel/+bug/1418567

Aleksander Mogylchenko (amogylchenko)
Changed in mos:
status: In Progress → Won't Fix
Revision history for this message
Michael Semenov (msemenov) wrote :

Fixed by https://bugs.launchpad.net/fuel/+bug/1418567

Revision history for this message
Vitaly Sedelnik (vsedelnik) wrote :

As the issue is actually fixed in another bug the proper status would be Invalid, not Won't Fix

Changed in mos:
status: Won't Fix → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.