Skip to content

make db persistent for testing migration#6266

Closed
anna-parker wants to merge 2 commits intomainfrom
test_versionComment
Closed

make db persistent for testing migration#6266
anna-parker wants to merge 2 commits intomainfrom
test_versionComment

Conversation

@anna-parker
Copy link
Copy Markdown
Contributor

@anna-parker anna-parker commented Apr 11, 2026

preview for testing #6198

🚀 Preview: https://test-versioncomment.loculus.org

@anna-parker anna-parker added the preview Triggers a deployment to argocd label Apr 11, 2026
@claude claude bot added the deployment Code changes targetting the deployment infrastructure label Apr 11, 2026
…mn to metadata

Store revocation versionComment in originalData metadata instead of the
dedicated version_comment column, unifying how versionComment is handled
for both revisions and revocations.

- When creating a revocation, store versionComment in originalData.metadata
  and compress using the compression service
- Update the database view to automatically construct joint_metadata for
  revocations from originalData, making it pipeline-version-independent
- Remove versionCommentColumn from SequenceEntriesTable and SequenceEntriesView
- Remove versionComment field from RawProcessedData (now comes through
  processedData metadata via the view)
- Remove conditional versionComment addition in ReleasedDataModel
- Add migration V1.26 to move existing version_comment values into
  originalData metadata and drop the column

Closes #3135

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deployment Code changes targetting the deployment infrastructure preview Triggers a deployment to argocd

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants