learningequality / kolibri-installer-windows

Repository for Kolibri windows installer
MIT License
5 stars 12 forks source link

Taskbar notification without IP:port on Windows 7 #119

Open radinamatic opened 5 years ago

radinamatic commented 5 years ago

Testing the built asset from https://github.com/learningequality/kolibri/pull/5762 in an upgrade scenario from 0.9.2 on Windows 7. The upgrade process apparently finishes without issues, and the server is starting correctly, only the taskbar notification in not picking up the server IP:port value.

no-port

At first I thought that it was only the fluke after the upgrade, but the issue is there on subsequent reboots, and even after changing the port in options.ini, the result is the same: server runs, but the taskbar notification does not report the IP:port.

cc @cpauya @mrpau-richard

radinamatic commented 5 years ago

It seems to be a Win7 issue only: I repeated the exact upgrade scenario on Windows 10, and the IP:port is correctly reported in the taskbar notification.

mrpau-richard commented 5 years ago

@radinamatic I can't reproduce this issue on Windows 7 after I upgrade from 0.9.2. Did the issue still exist on the fresh installation?

Screen Shot 2019-07-22 at 3 19 01 PM

radinamatic commented 5 years ago

@mrpau-dev As we discussed this morning on our call, I repeated the test case on a fresh (non-upgraded) install, and the IP:port was correctly displayed.

The error is inconsistent, as when I fired up the same VM this afternoon, taskbar notification was correct. However, if the user changes the port in the option.ini file, or removes the file completely from the .kolibri folder, the error reappears. I cloned, zipped and uploaded the whole VM HD at the snapshot with this issue, if you want to investigate.

mrpau-richard commented 5 years ago

@radinamatic the VM that you uploaded was corrupted when I run it. I think it's better if you upload the .ova file version.

radinamatic commented 5 years ago

Ok, exported the VM as appliance (.ova)! @mrpau-richard

mrpau-richard commented 5 years ago

@radinamatic the VM worked on me now. But when I test Kolibri GUI notification it display the right port url after the Kolibri server started.

Screen Shot 2019-07-30 at 4 14 16 PM

radinamatic commented 5 years ago

VM worked on me now.

Great! :tada:

Now stop the server, put the options.ini file you see on the Desktop inside the Users/IEUser/.kolibri folder, restart the server. and observe closely the notification message. In my case the first restart after the port change always had this issue appearing.

mrpau-richard commented 5 years ago

Same output when I used the option.ini file from the desktop. Screen Shot 2019-07-30 at 5 10 45 PM

radinamatic commented 5 years ago

Even when you move the options.ini away, and restart the server again?

mrpau-richard commented 5 years ago

Even when you move the options.ini away, and restart the server again?

Yup, it will still display the port at the Kolibri notification.

cpauya commented 5 years ago

After trying a few times between the following:

  1. Stop Kolibri, copy options.ini from Desktop in the c:/Users/IEUser/.kolibri, launch Kolibri
  2. Stop Kolibri, delete c:/Users/IEUser/.kolibri/options.ini, launch Kolibri
  3. Copy options.ini from Desktop in the c:/Users/IEUser/.kolibri, Stop Kolibri, launch Kolibri after a few seconds.
  4. Delete c:/Users/IEUser/.kolibri/options.ini, Stop Kolibri, launch Kolibri after a few seconds.

I saw at least two instances of this issue, refer to attached screenshot.

Screenshot 2019-07-31 21 49 57

@radinamatic is right, so we should keep this open until we figure out how to reproduce this.

radinamatic commented 5 years ago

Glad I was not imagining this issue, phew... 😌