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

Remove old migration system. #15373

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Remove old migration system. #15373

wants to merge 2 commits into from

Conversation

samwho
Copy link
Collaborator

@samwho samwho commented Jan 15, 2025

Description

This PR removes the old global migration system.

As far as I was able to tell, this is not used anymore. We have a newer migration system that works per-app, and confusingly the endpoint to check the status of these new migrations was intermingled with endpoints that operated on the old migration system.

Launchcontrol

  • Removes an old migration system. The following internal endpoints will no longer be present:
    • POST /api/migrations/run
    • GET /api/migrations/status
    • POST /api/system/migrations/run
    • GET /api/system/migrations/definitions

Copy link

qa-wolf bot commented Jan 15, 2025

QA Wolf here! As you write new code it's important that your test coverage is keeping up.
Click here to request test coverage for this PR!

@github-actions github-actions bot added firestorm Data/Infra/Revenue Team size/xl labels Jan 15, 2025
@samwho samwho added the feature-branch-enterprise Release this PR code into a feature branch with a Enterprise license label Jan 15, 2025
@samwho samwho marked this pull request as ready for review January 16, 2025 09:12
@samwho samwho requested a review from a team as a code owner January 16, 2025 09:12
@samwho samwho requested review from mike12345567 and removed request for a team January 16, 2025 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-branch-enterprise Release this PR code into a feature branch with a Enterprise license firestorm Data/Infra/Revenue Team size/xl
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant