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 🤖