Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve "Backfill Enrollment Data" managmeent command #435

Open
johnbaldwin opened this issue Mar 2, 2022 · 0 comments
Open

Improve "Backfill Enrollment Data" managmeent command #435

johnbaldwin opened this issue Mar 2, 2022 · 0 comments

Comments

@johnbaldwin
Copy link
Contributor

johnbaldwin commented Mar 2, 2022

Improve "Backfill Enrollment Data" management command

I'm working on a PR to update the management comment backfill_figures_enrollment_data to use the new workflow as described here: #428

To get this implemented quickly but still allow some flexibility, I'm implementing two command line keyword options, --sites and --courses. Sites takes one or more site domains. Course takes one or more string course ids. If --courses is used, it ignores the --sites argument.

We currently require specifying sites or course instead of defaulting to process all sites / all courses. While this has usability issues, we do it because this is potentially a quite expensive command, particularly on large multi-site Open edX deployments. This is where it could use UX improvement.

One potential small improvement might be to allow use of both params. This could take the form of processing both all sites listed in the --sites argument and all the courses in the --courses argument, skipping over any courses in the --courses for courses that exist in any of the sites in the --sites parameter. Another form would be to only process course ids if they exist in sites provided in the --sites argument. We should think over this. Maybe we don't do either.

Another potential improvement is to provide a JSON or YAML file parameter that would list the site domains and/or course ids to process. What would be helpful here would be to report on sites and/or courses that need updating.

Another potential improvement is to do arbitrary limiting the number of enrollments to be processed. For example: "process 100,000 out of date enrollments"

Since A. I don't have time to implement these options with appropriate test coverage and B. we might not want these or may come up with some other option, I'm just describing the options here for discussion and linking this issue into the docstring for the backfill management command.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant