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

When syncing job fails it's not scheduled again #11469

Closed
jredrejo opened this issue Oct 27, 2023 · 1 comment · Fixed by #11508
Closed

When syncing job fails it's not scheduled again #11469

jredrejo opened this issue Oct 27, 2023 · 1 comment · Fixed by #11508
Assignees
Labels
P1 - important Priority: High impact on UX

Comments

@jredrejo
Copy link
Member

Observed behavior

A LOD tried to sync and the process didn't finish because the sqlite db was locked by another process (in this case it was sqlitebrowser). That's fine with sqlite.

However when the job failed, kolibri didn't schedule it again, so this LOD stopped syncing until it was restarted.
After restarting it the kolibri.core.auth.tasks.soud_sync_processing job was scheduled again and everything was fine.

Errors and logs

It can be seen beginning on line 550 of this log file
kolibri.txt

Expected behavior

In case a syncing job fails, it should be scheduled again

User-facing consequences

Syncing stop until the kolibri instance of the user is restarted

Steps to reproduce

It happened when the networklocation.sqlite3 file was open in sqlitebrowser but it'll likely happen if db.sqlite3 or job_storage.sqlite3 are open too or kolibri is doing the vacuum task

Context

kolibri 0.16 installed on this PR code #11390

@jredrejo jredrejo added the P1 - important Priority: High impact on UX label Oct 27, 2023
@rtibbles
Copy link
Member

rtibbles commented Nov 2, 2023

Looking at this again - I don't think the database locked error is the issue here, as it is being caught and reported. I think this is actually another instance of #11485, as I am seeing elsewhere in the logs a 404 from the transfersession endpoint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 - important Priority: High impact on UX
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants