Skip to content

Remove storage_version column from snippets

What does this MR do?

In !22275 (merged), we added the columns repository_storage and storage_version to the snippet table. Nevertheless, after #39176 (comment 270821384) it seems we don't need storage_version since it's a column needed only for legacy storage and we're only using hashed storage in snippet repositories.

https://docs.gitlab.com/ee/development/what_requires_downtime.html#dropping-columns is the workflow to delete a column. I didn't check it and I got !23004 (merged) merged. We reverted that MR and we created !23166 (merged) to follow the process starting with ignore_column.

This is step 2, where we actually remove the column.

Does this MR meet the acceptance criteria?

Conformity

Edited by 🤖 GitLab Bot 🤖

Merge request reports