Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sync a facility - Hard to reproduce issue where syncing a facility fails #8227

Closed
pcenov opened this issue Jul 27, 2021 · 6 comments · Fixed by #8231
Closed

Sync a facility - Hard to reproduce issue where syncing a facility fails #8227

pcenov opened this issue Jul 27, 2021 · 6 comments · Fixed by #8231

Comments

@pcenov
Copy link
Member

pcenov commented Jul 27, 2021

Observed behavior

When as an administrator I attempt to sync a single facility the radio button with the facility's URL is disabled, there's a loader spinning and I am unable to sync the facility unless I manually add a new address.

2021-07-27_16-45-31

Note that when I changed the URL I was able to start the syncing process:

2021-07-27_17-04-41

Also note that after a while the initially disabled URL was successfully changed to the correct one. Not sure whether I disabled and then enabled the WiFi to achieve that:

2021-07-27_17-13-00

Steps to reproduce (Note: I don't have straightforward steps to replicate the issue at this moment, will follow up)

  1. As an administrator create a single facility, set up all classes and user accounts on that facility and import the desired channels.
  2. Install Kolibri on a individual learner device, import the facility from the central server by entering the super admin credentials and import the learner data and relevant channels in the learner device.
  3. Simulate various real world scenarios with both admin and learner users on both devices and occasionally attempt to sync the facilities. This includes logging in and logging out, changing the WiFi networks, disabling the WiFi and so on.

Errors and logs

I didn't observe any related errors in the browser console, will attempt to do so tomorrow.

Expected behavior

An admin user should be able to sync successfully facilities when Kolibri is running on the devices and they are within the same network.

Context

Tell us about your environment, including:

  • Kolibri version: 0.14.7
  • Operating system: Windows 10
  • Browser: Chrome
@radinamatic
Copy link
Member

radinamatic commented Jul 27, 2021

cc @rtibbles @jamalex @bjester

Some findings while investigating the zeroconf issue.

@pcenov
Copy link
Member Author

pcenov commented Jul 28, 2021

Adding the log files:
Admin logs.zip
Learner Logs.zip

@pcenov
Copy link
Member Author

pcenov commented Jul 28, 2021

Update:
I was able to successfully reproduce the issue. Here are the steps:

  1. Stop both Kolibri instances on both devices.

  2. As a Learner:

  • Connect to a different WiFi network (I created a hotspot using my mobile phone)
  • Launch Kolibri and start doing some exercises
  1. As an Admin on the other laptop:
  1. As a Learner:
  • Connect to the same WiFi network as the admin’s
  1. As an Admin:
  • Observe the Select network address modal window - the learner’s device is displayed now but with incorrect URL and the radio button is disabled

2021-07-28_16-44-23

The sync issue gets fixed if I restart both Kolibri instances while being connected to the same network.

The logs from today:
Admin logs.zip
Learner logs.zip

@bjester
Copy link
Member

bjester commented Jul 28, 2021

Thanks for the great reproduce detail @pcenov! This is the issue that I hypothesized and also reproduced @jamalex

@rtibbles
Copy link
Member

This will hopefully be fixed by #8231 - we have merged, but would be good to confirm the fix still on release-v0.15.x

@bjester
Copy link
Member

bjester commented Aug 31, 2021

I have observed this as fixed

@bjester bjester closed this as completed Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants