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.

Sign up for GitHub

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

Jump to bottom

Remove old migrations #4079

Open
tim-allen-ck opened this issue Aug 29, 2024 · 0 comments · May be fixed by #4168
Open

Remove old migrations #4079

tim-allen-ck opened this issue Aug 29, 2024 · 0 comments · May be fixed by #4168
Labels
story Stories are the smallest unit of work to be done for a project.

Comments

@tim-allen-ck
Copy link
Collaborator

tim-allen-ck commented Aug 29, 2024 •
edited
Loading

Eliminate deprecated migrations from both the migrate.sh script and the API migrate endpoint to enhance deployment speed.
@tim-allen-ck tim-allen-ck added the story Stories are the smallest unit of work to be done for a project. label Aug 29, 2024
@marrobi marrobi moved this to Up Next in Azure TRE - Engineering Aug 29, 2024
marrobi added a commit that referenced this issue Dec 5, 2024
@marrobi
Remove old API migrations
348be18 
Fixes #4079

Remove deprecated migration steps from the `migrate_database` function in `api_app/api/routes/migrations.py`.

* **Migration Steps Removal**
  - Remove multiple deprecated migration steps from the `migrate_database` function.
  - Update the `migrate_database` function to only include necessary migration steps.
* **Version Update**
  - Update the version in `api_app/_version.py` from "0.19.3" to "0.20.0".
* **Test Updates**
  - Remove tests related to deprecated migration steps in `api_app/tests_ma/test_api/test_routes/test_migrations.py`.
  - Update tests to only include necessary migration steps.
* **File Deletions**
  - Delete `api_app/db/migrations/airlock.py`.
  - Delete `api_app/db/migrations/resources.py`.
  - Delete `api_app/db/migrations/shared_services.py`.
  - Delete `api_app/db/migrations/workspaces.py`.
  - Delete `api_app/tests_ma/test_db/test_migrations/test_workspace_migration.py`.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/microsoft/AzureTRE/issues/4079?shareId=XXXX-XXXX-XXXX-XXXX).
@marrobi marrobi linked a pull request Dec 5, 2024 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
story Stories are the smallest unit of work to be done for a project.
Projects
Azure TRE - Engineering
Status: Up Next
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove old API migrations microsoft/AzureTRE
1 participant
@tim-allen-ck