Ensure only valid columns are listed in sharding_key_spec.rb
What does this MR do and why?
This MR cleans up the schema_spec.rb file by removing references to columns that no longer exist in the database schema. It also adds a test to ensure that no invalid values are accepted in the future. The changes ensure that the spec only validates foreign keys for columns that are actually present in the tables.
Changes made:
- Added validation to ensure ignored columns actually exist in the table schema before attempting to validate foreign keys
-
Removed obsolete column references from the
removed_fks_mapandignored_fks_map:alert_management_alerts.prometheus_alert_id-
analytics_cycle_analytics_aggregations.last_full_run_issues_idandlast_full_run_merge_requests_id broadcast_messages.namespace_idcompliance_management_frameworks.group_idnotes.confirmed_by_idprojects.ci_idbackup_vulnerabilities.vulnerability_idvulnerability_reads.namespace_idwebauthn_registrations.u2f_registration_idzoekt_repositories.namespace_id
References
Related to #547650 (closed)
MR acceptance checklist
Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Edited by Pedro Pombeiro