Skip to content

Resolve "Clearing the model cache deletes all redis keys"

Nigel Gott requested to merge 884-clearing-the-model-cache-deletes-all-redis-keys into develop

Testing Notes

  1. Start up a develop Baserow, load a table.
  2. In a terminal run docker exec -it baserow_redis_1 redis-cli --pass baserow --scan to see all the keys, you should see something like
Warning: Using a password with '-a' or '-u' option on the command line interface may not be safe.
_kombu.binding.celery
_kombu.binding.celery.pidbox
asgi:group:users
baserow-generated-models-cache:None:full_table_model_195_1.9.1
redbeat::statics
redbeat:celery.backend_cleanup
redbeat:baserow.core.trash.tasks.mark_old_trash_for_permanent_deletion()
redbeat:baserow_premium.license.tasks.license_check()
redbeat:baserow.contrib.database.export.tasks.clean_up_old_jobs()
_kombu.binding.celeryev
asgi:group:table-192
redbeat::schedule
baserow-generated-models-cache:None:full_table_model_193_1.9.1
baserow-generated-models-cache:None:full_table_model_194_1.9.1
baserow-generated-models-cache:None:full_table_model_192_1.9.1
redbeat::lock
redbeat:baserow.core.trash.tasks.permanently_delete_marked_trash()
_kombu.binding.export
  1. In the backend lint tab run ./baserow migrate to trigger a generated model cache clear.
  2. Re-run docker exec -it baserow_redis_1 redis-cli --pass baserow --scan and observe all keys have been deleted by this bug.
  3. Stop Baserow
  4. Checkout this branch
  5. Startup Baserow, load some tables
  6. Do steps 2,3 and 4 again but this time observe only the entries starting with baserow-generated-models-cache:None:full_table_model have been deleted.

Merge Request Checklist

  • changelog.md has been updated if required
  • New/updated Premium features are separated correctly in the premium folder
  • The latest Chrome and Firefox have been used to test any new frontend features
  • Documentation has been updated
  • Quality Standards are met
  • Performance: tables are still fast with 100k+ rows, 100+ field tables
  • The redoc API pages have been updated for any REST API changes
  • Our custom API docs are updated for changes to endpoints accessed via api tokens
  • The UI/UX has been updated following UI Style Guide

Closes #884 (closed)

Edited by Nigel Gott

Merge request reports