On the heels of #2569469: [Policy, no patch] No need to provide an upgrade path for Migrations, we will not be able to get certain tests to pass until migrations are not in the fixtures we use for update path testing. drupal-8.filled.standard.php, at least, contains migrations which need to be removed.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

phenaproxima created an issue. See original summary.

phenaproxima’s picture

Status: Active » Needs review
FileSize
1.11 MB

Removed all migrate.migration.* entities from the config table.

Status: Needs review » Needs work

The last submitted patch, 2: 2569605-2.patch, failed testing.

The last submitted patch, 2: 2569605-2.patch, failed testing.

phenaproxima’s picture

Status: Needs work » Needs review
FileSize
1.12 MB

Please gawd...

neclimdul’s picture

We should probably empty and leave the update method with a comment.

phenaproxima’s picture

neclimdul’s picture

Status: Needs review » Reviewed & tested by the community

non-db changes look straightforward.

phenaproxima’s picture

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Great, with the creation of that issue I think we can proceed here. This will certainly help us during RC to not have to figure out strange Migrate upgrade path issues (doesn't come up often, but when it does happen it's tricky to work out).

Committed and pushed to 8.0.x. Thanks!

  • webchick committed 5355f8c on 8.0.x
    Issue #2569605 by phenaproxima, neclimdul, catch: Remove migrations from...

Status: Fixed » Closed (fixed)

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