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

[DEPR]: paver update_db #32683

Closed
dianakhuang opened this issue Jul 7, 2023 · 3 comments
Closed

[DEPR]: paver update_db #32683

dianakhuang opened this issue Jul 7, 2023 · 3 comments
Labels
depr Proposal for deprecation & removal per OEP-21

Comments

@dianakhuang
Copy link
Contributor

dianakhuang commented Jul 7, 2023

Proposal Date

2023-07-07

Target Ticket Acceptance Date

2023-07-14

Earliest Open edX Named Release Without This Functionality

Redwood - 2024-04

Rationale

Currently, we are trying to deprecate paver overall and this method is currently not working in one of our environments, so we believe the best way forward is to delete this method.

Removal

https://github.com/openedx/edx-platform/blob/master/pavelib/servers.py#L243

Replacement

We will be replacing this call with a make migrate command in the Makefile in order to make it more consistent with other IDAs in the Open edX ecosystem.

Deprecation

No response

Migration

No response

Additional Info

No response

@github-actions github-actions bot added the depr Proposal for deprecation & removal per OEP-21 label Jul 7, 2023
@robrap
Copy link
Contributor

robrap commented Jul 7, 2023

Note: The newer make migrate just run migrations, and will no longer install requirements first like paver update_db used to.

@robrap
Copy link
Contributor

robrap commented Jul 7, 2023

Also, this DEPR will resolve the ticket #32252.

@kdmccormick
Copy link
Member

Burn it down 🔥 🔥

Potentially worth nothing, the replacement to this that Tutor provides is:

tutor [dev|local|k8s] run lms ./manage.py lms migrate
tutor [dev|local|k8s] run cms ./manage.py cms migrate

Alternatively, re-running tutor ... launch will run migrations, among other several other environment updates:

tutor [dev|local|k8s] launch

And for Devstack, it's:

make dev.up.lms+cms
make dev.migrate.lms dev.migrate.cms

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
depr Proposal for deprecation & removal per OEP-21
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants