As pointed out in #2532336: migrate-upgrade fails with dependency errors, the legacy-db-url option does not have a way to specify a database prefix - other drush commands requiring database credentials have a separate db-prefix option. We should have one too (for consistency, let's call it legacy-db-prefix).

CommentFileSizeAuthor
#1 add_prefix_option_to-2533190-1.patch1.29 KBmikeryan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan at Acquia commented
Status: Active » Needs review
FileSize
add_prefix_option_to-2533190-1.patch1.29 KB

Looks simple enough, needs someone with a prefixed database to test it.

  • mikeryan committed b37b663 on 8.x-1.x 
    Issue #2533190 by mikeryan: Add legacy-db-prefix option to migrate-...
mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan at Acquia commented
Status: Needs review » Fixed

Committed, if any problems are found with this please open a fresh issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan at Acquia commented
Status: Closed (fixed) » Active

D'oh! Looks like I accidentally reverted this in #2535228: Make use of proposed builder interface, need to revert the revert...

  • mikeryan committed 2cd3a61 on 8.x-1.x 
    Issue #2533190 by mikeryan: Reapply patch
mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan at Acquia commented
Status: Active » Fixed

Redone.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.