Skip to content

LMS setting sync for LEARNING_ASSISTANT_PROMPT_TEMPLATE #1040

Open
@robrap

Description

@robrap

Ensure that LEARNING_ASSISTANT_PROMPT_TEMPLATE is valid and matches in ec2 and k8s.

Note: This only affects a single setting. However, it seems quite possible that its value could be updated, and would get out of sync without anyone knowing without this fix.

Acceptance Criteria:

Notes/Questions:

  • This ticket is blocked on Enable setting sync for lms stage #1025, and was broken out of this ticket.
  • LEARNING_ASSISTANT_PROMPT_TEMPLATE should be removed from EXCLUDED_PATHS, and possibly replaced with special compare logic that handles the differences that were introduced in https://github.com/edx/edx-internal/pull/12679/files.
  • [question] Does the LEARNING_ASSISTANT_PROMPT_TEMPLATE setting also need a separate more permanent check to ensure it is formatted properly and won't break in the future?
  • [question] Alternatively, why do we use jinja on the yaml file, and is there a better design to accomplish the same?

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    Status

    Backlog

    Status

    Todo

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions