The database dumps for the D6 > D8 migration tests were recently updated, and now the D7 > D8 dumps need to be updated accordingly.

Comments

hosef’s picture

StatusFileSize
new99.28 KB

Here is a dump file I made from a D7 site to start.

benjy’s picture

I think we need a few extra things:

* A new script for exporting tables, just copy the d6 script for now, we can combine them later.
* The tables exporting into a patch.
* The D7 codebase pushing to a repo, I think we could probably put that in another branch next to the D6 code: https://www.drupal.org/sandbox/benjy/2405029

hosef’s picture

StatusFileSize
new1.14 MB

Ok, I made a new script, as you asked, and made the classes.

hosef’s picture

StatusFileSize
new1.14 MB

New one to remove my name/email from it.

benjy’s picture

benjy’s picture

Project: IMP » Drupal core
Version: » 8.0.x-dev
Component: Code » migration system
benjy’s picture

Status: Active » Needs review

OK, this issue is a hard blocker to start D7 which we're gearing up to start. The patch applies, albeit with white space errors in the dump. @hosef, can you push the codebase to the sandbox. You should have access: https://www.drupal.org/sandbox/benjy/2405029

Once the code is in the sandbox and I can confirm it works with the d7.gz database this is ready to be committed.

benjy’s picture

StatusFileSize
new1.19 MB

Binary formatted the dump in the patch.

andypost’s picture

Is there a steps how to check that dumps are match?
Doing a fresh D7 install is not enough

andypost’s picture

benjy’s picture

Is there a steps how to check that dumps are match?
Doing a fresh D7 install is not enough

Not without the codebase that was used to produce the dumps. We really need hosef to push the code to the sandbox otherwise someone will need to create the d7 dumps from scratch.

hosef’s picture

Hi, I just pushed up my D7 site to the sandbox. Hopefully I did it right. :)

andypost’s picture

Please update summary allowing other people to get involved here

benjy’s picture

Someone needs to clone the codebase from: https://www.drupal.org/sandbox/benjy/2405029 and then ensure that importing the database in the patch from #8 works with that codebase.

Then we're ready to commit this.

benjy’s picture

benjy’s picture

pjezek’s picture

Hi Benjy, Hi Hosef

I am at drupal dev days in Montpellier and asked Ryan if I could help on migration. He told me to get in contact with you to help out with this issue.

pjezek’s picture

I updated https://www.drupal.org/sandbox/benjy/2405029 until the last point.
./cores/scripts/migrate-dump.sh is probably:

php ./core/scripts/dump-database-d7.sh

But I got some exceptions and stopped working on it. Hope the updated doc will help someone.

phenaproxima’s picture

The patch at #30 in #2469623: Process for creating migration source DBs for automated tests adds Drupal 7 support for dumping and restoring tables. Once that lands, we can probably get this issue done in a hurry :)

phenaproxima’s picture

Status: Needs review » Postponed
phenaproxima’s picture

phenaproxima’s picture

Status: Active » Needs review
StatusFileSize
new1.38 MB

Re-generated the D7 dump files, from the gzip file in patch #8.

ultimike’s picture

Status: Needs review » Reviewed & tested by the community

Sweet - let's open the floodgates to D7->D8 migrations!

-mike

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Awesome! Let the good times roll!!

Committed and pushed to 8.0.x. Thanks!

  • webchick committed dc38797 on 8.0.x
    Issue #2410625 by hosef, benjy, phenaproxima: Update D7 dumps to match...

Status: Fixed » Closed (fixed)

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