Skip to content

[FAL-2031] Add Redis support for Ocim provisioned instances

Boros Gábor requested to merge gabor/add-redis-support into master

This Pull Request adds Redis cache support to Ocim. The changes won't take effect on already provisioned servers, but new ones or those that explicitly set to use Redis over RabbitMQ. This way we can slowly roll out changes for every server while we are not generating more work when new instances are created.

Dependencies Related PRs:

Screenshots:

Admin site integration of RedisServer

Screenshot 2021-07-01 at 16 10 00

Instance admin configuration

Screenshot 2021-07-01 at 14 49 00

Redis cluster showing the Ocim created user and its ACL

Screenshot 2021-07-01 at 15 35 27

Sandbox URL: Upon request, I can create an ngrok URL for my local Ocim

Merge deadline: None

Testing instructions:

  1. Checkout the changes on Ocim stage
  2. Create a Redis user and ACL for Ocim if no user was created (or use the default user) and double-check the connection to the Redis server
  3. Set the necessary environment variables in the .env file (ie. DEFAULT_INSTANCE_REDIS_URL )
  4. Restart Ocim
  5. Provision a new app server
  6. Login with edX user
  7. Go to instructor/download data tab
  8. Download a grades report (as that uses celery, that would use Redis now)
  9. Validate the instructor task is finished successfully

A note about staging env: The instance creation nowadays is hectic -- it seems for some instances the LB URL registration did not work.

Author notes and concerns:

N/A

Reviewers

Merge request reports