chore(preprod): Delete deprecated columns #106592
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
github-actions
bot
added
the
Scope: Backend
rbro112
force-pushed
the
ryan/mark_deprecated_columns_as_ready_for_deletion
branch
from
4ee6d6b to
60fbb6b
Compare
rbro112
force-pushed
the
ryan/delete_deprecated_columns
branch
from
cd6fb6d to
2ebc324
Compare
rbro112
changed the title
Contributor
|
This PR has a migration; here is the generated SQL for for --
-- Moved preprodartifact.build_version field to pending deletion state
--
-- (no-op)
--
-- Moved preprodartifact.build_number field to pending deletion state
--
-- (no-op)
--
-- Moved preprodartifact.app_name field to pending deletion state
--
-- (no-op)
--
-- Moved preprodartifact.app_icon_id field to pending deletion state
--
-- (no-op)for --
-- Remove field build_version from preprodartifact
--
ALTER TABLE "sentry_preprodartifact" DROP COLUMN "build_version" CASCADE;
--
-- Remove field build_number from preprodartifact
--
ALTER TABLE "sentry_preprodartifact" DROP COLUMN "build_number" CASCADE;
--
-- Remove field app_name from preprodartifact
--
ALTER TABLE "sentry_preprodartifact" DROP COLUMN "app_name" CASCADE;
--
-- Remove field app_icon_id from preprodartifact
--
ALTER TABLE "sentry_preprodartifact" DROP COLUMN "app_icon_id" CASCADE; |
rbro112
force-pushed
the
ryan/mark_deprecated_columns_as_ready_for_deletion
branch
from
60fbb6b to
c70438e
Compare
rbro112
force-pushed
the
ryan/delete_deprecated_columns
branch
from
2ebc324 to
d07081e
Compare
rbro112
force-pushed
the
ryan/delete_deprecated_columns
branch
from
d07081e to
564b46c
Compare
rbro112
force-pushed
the
ryan/mark_deprecated_columns_as_ready_for_deletion
branch
from
c70438e to
c4c7cbe
Compare
rbro112
force-pushed
the
ryan/delete_deprecated_columns
branch
from
564b46c to
c4c0dd8
Compare
rbro112
force-pushed
the
ryan/mark_deprecated_columns_as_ready_for_deletion
branch
from
c4c7cbe to
a556be0
Compare
rbro112
force-pushed
the
ryan/delete_deprecated_columns
branch
from
c4c0dd8 to
449e32c
Compare
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
src/sentry/preprod/migrations/0024_delete_deprecated_preprod_artifact_fields.py
Show resolved
Hide resolved
rbro112
force-pushed
the
ryan/delete_deprecated_columns
branch
from
449e32c to
062b5b3
Compare
rbro112
force-pushed
the
ryan/delete_deprecated_columns
branch
from
062b5b3 to
e1a26c4
Compare
Contributor
|
This PR has a migration; here is the generated SQL for for --
-- Moved preprodartifact.build_version field to pending deletion state
--
-- (no-op)
--
-- Moved preprodartifact.build_number field to pending deletion state
--
-- (no-op)
--
-- Moved preprodartifact.app_name field to pending deletion state
--
-- (no-op)
--
-- Moved preprodartifact.app_icon_id field to pending deletion state
--
-- (no-op) |
rbro112
added a commit
that referenced
this pull request
Jan 21, 2026
Following https://develop.sentry.dev/backend/application-domains/database-migrations/#deleting-columns, this now properly marks the deprecated columns as ready for deletion, which sets us up to delete in #106592
Base automatically changed from
ryan/mark_deprecated_columns_as_ready_for_deletion
to
master
January 21, 2026 18:17
rbro112
force-pushed
the
ryan/delete_deprecated_columns
branch
from
e1a26c4 to
0b4d9ab
Compare
Contributor
|
This PR has a migration; here is the generated SQL for for --
-- Remove field build_version from preprodartifact
--
ALTER TABLE "sentry_preprodartifact" DROP COLUMN "build_version" CASCADE;
--
-- Remove field build_number from preprodartifact
--
ALTER TABLE "sentry_preprodartifact" DROP COLUMN "build_number" CASCADE;
--
-- Remove field app_name from preprodartifact
--
ALTER TABLE "sentry_preprodartifact" DROP COLUMN "app_name" CASCADE;
--
-- Remove field app_icon_id from preprodartifact
--
ALTER TABLE "sentry_preprodartifact" DROP COLUMN "app_icon_id" CASCADE; |
trevor-e
approved these changes
Jan 21, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Following https://develop.sentry.dev/backend/application-domains/database-migrations/#deleting-columns, this fully deletes deprecated columns from PreprodArtifact.