Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Nov 21, 2024
Merged

Conversation

aramprice
Copy link
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, jpalermo and a-hassanin 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 <[email protected]>
@jpalermo jpalermo merged commit b7c2721 into main Nov 21, 2024
20 checks passed
@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