release-22.2: upgrades: repair broken 22.1 migration so 22.2 migration works #91308
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.
Backport 1/1 commits from #91304.
/cc @cockroachdb/release
This also fixes the logic which allowed the 22.1 migration to proceed in the
first place. This will need to be backported to 22.2. Any users which upgraded
from 21.2->22.1->22.2.did_not_contain_this_patch will not have the correct
indexes on their system.statement_diagnostics_requests table.
Fixes #91300
Release justification: Fixes a feature-impacting migration.
Release note (bug fix): Fixed a bug which caused a migration in 22.1 to fail
to drop an index on system.statement_diagnostics_requests and, in turn, caused
upgrades from 22.1->22.2 which had used the previous, faulty upgrade migration
to now fail to create a new index with the same name as the index which had
been thought to have been dropped.