Skip to content

feat: apply grove patches last and added ability to add settings to openedx-lms-production-settings and openedx-cms-production-settings patch

Kaustav Banerjee requested to merge kaustav/lms_cms_production_settings into main

Description

This pull request does two things:

  1. Adds ability to configure setting through the openedx-lms-production-settings and openedx-cms-production-settings patches
  2. Ensures that the patches from this plugin are applied after all other plugins, so that don't get overriden.

Testing instructions

Steps to test the changes:

  1. Verify the plugin versions installed in the two instances kaustav-sandbox and keith-sandbox
  2. Also verify the PROFILE_MICROFRONTEND_URL is correctly set for both the instances kaustav-sandbox and keith-sandbox
  3. Check the order in which the PROFILE_MICROFRONTEND_URL is set in the two instances.
./kubectl exec -it -n kaustav-sandbox $(./kubectl get pods -n kaustav-sandbox -l=app.kubernetes.io/name=lms -o custom-columns=":metadata.name" |  tr -d '\r') -- grep PROFILE_MICROFRONTEND_URL lms/envs/tutor/production.py
./kubectl exec -it -n keith-sandbox $(./kubectl get pods -n keith-sandbox -l=app.kubernetes.io/name=lms -o custom-columns=":metadata.name" |  tr -d '\r') -- grep PROFILE_MICROFRONTEND_URL lms/envs/tutor/production.py

Additional context

OpenCraft internal ticket : BB-7303

Edited by Kaustav Banerjee

Merge request reports