Skip to content

BB-1542: Remove old K/V format support

Giovanni Cimolin da Silva requested to merge giovanni/BB-1542-remove-consul-locks into master

This PR removes code related to the old K/V format and operations that used Consul transactions.

Testing instructions: Since we don't usually have Consul set up locally, this testing should be performed on stage.

  1. Deploy this branch on Ocim Stage.
  2. Go into Ocim shell and pick one active instance and run (don't forget to change some-instance to an instance with active appservers):
x = OpenEdXInstance.objects.get(internal_lms_domain__contains='some-instance')
  1. Check the consul key related to that instance on Consul web interface (username: consul, password: here).
  2. On ocim shell, run x.purge_consul_metadata() and check that the key was deleted on Consul.
  3. Then run x.update_consul_metadata() and check that the instance's Consul entry is correctly created using the new format.

Reviewer:

  • @viadanna

Merge request reports