Possibility of unhandled exception during diskcache usage

[rtibbles]

Observed behavior

A very odd behaviour whereby diskcache seems to lose track of its transaction id.

Errors and logs

ERROR 2024-01-23 15:39:39,832 django.request Internal Server Error: /coach/api/notifications/
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/kolibri/dist/diskcache/core.py", line 762, in _transact
    yield sql, filenames.append
  File "/usr/lib/python3/dist-packages/kolibri/dist/diskcache/core.py", line 827, in set
    rows = sql(
sqlite3.OperationalError: disk I/O error

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/kolibri/dist/django/core/handlers/exception.py", line 41, in inner
    response = get_response(request)
  File "/usr/lib/python3/dist-packages/kolibri/dist/django/core/handlers/base.py", line 172, in _get_response
    resolver_match = resolver.resolve(request.path_info)
  File "/usr/lib/python3/dist-packages/kolibri/dist/django/urls/resolvers.py", line 366, in resolve
    sub_match = pattern.resolve(new_path)
  File "/usr/lib/python3/dist-packages/kolibri/dist/django/urls/resolvers.py", line 366, in resolve
    sub_match = pattern.resolve(new_path)
  File "/usr/lib/python3/dist-packages/kolibri/dist/django/urls/resolvers.py", line 366, in resolve
    sub_match = pattern.resolve(new_path)
  [Previous line repeated 1 more time]
  File "/usr/lib/python3/dist-packages/kolibri/dist/django/urls/resolvers.py", line 361, in resolve
    match = self.regex.search(path)
  File "/usr/lib/python3/dist-packages/kolibri/core/device/translation.py", line 158, in regex
    device_language = get_device_language() or get_settings_language()
  File "/usr/lib/python3/dist-packages/kolibri/core/device/translation.py", line 26, in get_device_language
    if not device_provisioned():
  File "/usr/lib/python3/dist-packages/kolibri/core/device/utils.py", line 68, in device_provisioned
    return get_device_setting("is_provisioned")
  File "/usr/lib/python3/dist-packages/kolibri/core/device/utils.py", line 54, in get_device_setting
    device_settings = DeviceSettings.objects.get()
  File "/usr/lib/python3/dist-packages/kolibri/core/device/models.py", line 73, in get
    cache.set(DEVICE_SETTINGS_CACHE_KEY, model, 600)
  File "/usr/lib/python3/dist-packages/kolibri/deployment/default/custom_django_cache.py", line 76, in set
    return super(CustomDjangoCache, self).set(
  File "/usr/lib/python3/dist-packages/kolibri/dist/diskcache/djangocache.py", line 147, in set
    return self._cache.set(key, value, timeout, read, tag, retry)
  File "/usr/lib/python3/dist-packages/kolibri/dist/diskcache/fanout.py", line 97, in set
    return shard.set(key, value, expire, read, tag, retry)
  File "/usr/lib/python3/dist-packages/kolibri/dist/diskcache/core.py", line 842, in set
    return True
  File "/usr/lib/python3.8/contextlib.py", line 131, in __exit__
    self.gen.throw(type, value, traceback)
  File "/usr/lib/python3/dist-packages/kolibri/dist/diskcache/core.py", line 765, in _transact
    assert self._txn_id == tid
AssertionError

Expected behavior

Our CustomDjangoCache object should also handle AssertionErrors:

kolibri/kolibri/deployment/default/custom_django_cache.py

Line 7 in fc94f27

class CustomDjangoCache(DjangoCache):

Might be simplest to do a small refactor to create a class level handled errors tuple, and then use that in each try/except block to handle.

User-facing consequences

Very limited, has only been observed once in testing.

Context

First reported here: #11765 (comment)

[thesujai]

May i work on this?

[AlexVelezLl]

Hi @thesujai! Sure! I will assign this to you. Thanks! 👐

[rtibbles]

Fixed in #11808