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

500 error when importing facility from different device #12299

Closed
marcellamaki opened this issue Jun 17, 2024 · 2 comments · Fixed by #12346
Closed

500 error when importing facility from different device #12299

marcellamaki opened this issue Jun 17, 2024 · 2 comments · Fixed by #12346
Assignees
Labels
P0 - critical Priority: Release blocker or regression TODO: needs QA re-review For stale issues that need to be revisited

Comments

@marcellamaki
Copy link
Member

Observed behavior

For @AlexVelezLl (0.17.0-alpha0 bug bash) during the setup flow, importing from an existing device was not working as expected.
image

Errors and logs

ERROR 2024-06-17 10:15:25,356 django.request Internal Server Error: /api/discovery/networklocation_facilities/59ba08844e8484c01d12b5fb54fa4123/
Traceback (most recent call last):
  File "/home/alexvelezll/.pex/installed_wheels/a319987fd510881ad7ed07d04a07b218a83ba205d6621d21a727bbf3d414153d/kolibri-0.17.0a0-py2.py3-none-any.whl/kolibri/dist/django/core/handlers/exception.py", line 47, in inner
    response = get_response(request)
  File "/home/alexvelezll/.pex/installed_wheels/a319987fd510881ad7ed07d04a07b218a83ba205d6621d21a727bbf3d414153d/kolibri-0.17.0a0-py2.py3-none-any.whl/kolibri/dist/django/core/handlers/base.py", line 181, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/home/alexvelezll/.pex/installed_wheels/a319987fd510881ad7ed07d04a07b218a83ba205d6621d21a727bbf3d414153d/kolibri-0.17.0a0-py2.py3-none-any.whl/kolibri/dist/django/views/decorators/csrf.py", line 54, in wrapped_view
    return view_func(*args, **kwargs)
  File "/home/alexvelezll/.pex/installed_wheels/a319987fd510881ad7ed07d04a07b218a83ba205d6621d21a727bbf3d414153d/kolibri-0.17.0a0-py2.py3-none-any.whl/kolibri/dist/rest_framework/viewsets.py", line 104, in view
    self = cls(**initkwargs)
  File "/home/alexvelezll/.pex/installed_wheels/a319987fd510881ad7ed07d04a07b218a83ba205d6621d21a727bbf3d414153d/kolibri-0.17.0a0-py2.py3-none-any.whl/kolibri/core/api.py", line 149, in __init__
    raise TypeError("values must be defined as a tuple")
TypeError: values must be defined as a tuple

Expected behavior

User should be able to import from other devices on the network, or be notified properly that there are no devices available

Steps to reproduce/Context

Clean setup on Kolibri

  • Ubuntu 22.04.4 LTS | Chrome 125.0.6422.141
@marcellamaki marcellamaki added P0 - critical Priority: Release blocker or regression TODO: needs QA re-review For stale issues that need to be revisited labels Jun 17, 2024
@marcellamaki
Copy link
Member Author

@radinamatic @pcenov - can you try to reproduce this across a few other OSes/installers and see if there is any additional details we can add into the issue?

@pcenov
Copy link
Member

pcenov commented Jun 18, 2024

Hi @marcellamaki here is a video of what I am observing:

2024-06-18_14-58-47.mp4

Logs:
Ubuntu2ndDevice.zip
UbuntuServer.zip

The import is also not working at Device > Facilities.

A note that I reported this issue while testing the following PR: #12165 but I thought that it was caused by the Python changes made there.

Also I am able to replicate the issue on 2 Windows devices as well so it's not OS specific.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 - critical Priority: Release blocker or regression TODO: needs QA re-review For stale issues that need to be revisited
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants