Skip to content

Consolidate all migrations prior to 2021-09-02#2583

Merged
jpalermo merged 1 commit intomainfrom
migrations-roll-up
Nov 21, 2024
Merged

Consolidate all migrations prior to 2021-09-02#2583
jpalermo merged 1 commit intomainfrom
migrations-roll-up

Conversation

@aramprice
Copy link
Copy Markdown
Member

These are now integrated into 20110209010747_initial.rb

  • relocate migrations and specs up one directory, removing director/ since there is only one Sequel DB in use on the bosh director
  • Update DBMigrator to pass allow_missing_migration_files: true since the rolled-up migration files have been removed from disk

This change is required because newer versions of sequel were not compatible with the older migration files in the Bosh director.

@aramprice aramprice requested review from a team, a-hassanin and jpalermo and removed request for a team November 11, 2024 22:54
@aramprice aramprice changed the title Consolidate all migrations prior to 20210902232124 Consolidate all migrations prior to 2021-09-02 Nov 11, 2024
@ystros ystros force-pushed the migrations-roll-up branch from 8b966e2 to db38bdf Compare November 11, 2024 23:11
@jpalermo jpalermo requested a review from a team November 14, 2024 16:04
These are now integrated into 20110209010747_initial.rb

- relocate migrations and specs up one directory, removing `director/`
  since there is only one Sequel DB in use on the bosh director
- Update DBMigrator to pass `allow_missing_migration_files: true` since
  the rolled-up migration files have been removed from disk
- collapse migration_digests.json into verify_migration_digest_spec.rb
- simplify DBSpecHelper and use existing MIGRATIONS_DIR constant from
  DBMigrator

Signed-off-by: Brian Upton <brian.upton@broadcom.com>
@jpalermo jpalermo merged commit b7c2721 into main Nov 21, 2024
@jpalermo jpalermo deleted the migrations-roll-up branch November 21, 2024 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants