The migrate-rollback command will be implemented for D8 migrate. It will take an optional comma-separated list of migrations to run rollbacks on - alternatively, a group may be specified with --group, or --all may be used to run all "registered" migrations (pending [#2202476]). Options other than --group or --all will be implemented in follow-up issues.

CommentFileSizeAuthor
#13 implement-2403411-13.patch5.7 KBmikeryan
#13 implement-2403411-13.patch0 bytesmikeryan
#12 implement-2403411-9.patch5.73 KBmikeryan
#8 implement-2403411-8.patch2.29 KBzxaos
#7 implement-2403411-7.patch2.29 KBzxaos
#5 implement-2403411-5.patch2.23 KBdevin carlson
#3 implement-2403411-3.patch2.11 KBmikeryan

Comments

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Issue summary: View changes
mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Status: Active » Postponed
Related issues: +#2361093: Add a rollback functionality to migrate

Not much we can do here until the core framework supports it: #2361093: Add a rollback functionality to migrate

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
StatusFileSize
new implement-2403411-3.patch2.11 KB

Here's a patch implemented rollback - it requires the core patch at #2361093: Add a rollback functionality to migrate.

saltednut’s picture

saltednut
she/her
Primary language English
Location Chicago
commented

I believe this will need to be rerolled as well now that #2506583: Add migrate-manifest to migrate_plus has landed.

devin carlson’s picture

devin carlson commented
StatusFileSize
new implement-2403411-5.patch2.23 KB

A reroll of #3.

saltednut’s picture

saltednut
she/her
Primary language English
Location Chicago
commented

This needs another reroll to move it into migrate_tools per #2491643: How should migrate_plus be split up?

zxaos’s picture

zxaos commented
StatusFileSize
new implement-2403411-7.patch2.29 KB

#5 re-rolled to refer to migrate_tools Use fixed below

zxaos’s picture

zxaos commented
StatusFileSize
new implement-2403411-8.patch2.29 KB

Oops, ignore that last one, it had a missed change. #5 re-rolled to refer to migrate_tools

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Version: » 8.x-1.x-dev

New patch which works with latest core patch at https://www.drupal.org/node/2361093#comment-10249031 - now displays what it did.

tannerjfco’s picture

tannerjfco commented

I'm guessing the latest patch didn't get attached? #8 can not apply with the current dev release.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented

Damn! Ready to update the patch and I've lost the last version... /me sighs and gets to work...

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
StatusFileSize
new implement-2403411-9.patch5.73 KB

Ah, found the missing patch, preserving here before I see if it needs any more work...

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
StatusFileSize
new implement-2403411-13.patch0 bytes
new implement-2403411-13.patch5.7 KB

Updated patch, works with https://www.drupal.org/node/2361093#comment-10320825.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented

This should be rerolled for the latest core patch, which introduces a supportsRollback() API.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Status: Postponed » Needs review

Rollback is in core, will test the latest patch here and hopefully commit it shortly.

  • mikeryan committed e4b5015 on 8.x-1.x 
    Issue #2403411 by mikeryan, zxaos, Devin Carlson: Implement migrate-...
mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Status: Needs review » Fixed

Done!

Status: Fixed » Closed (fixed)

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