Follow-up to #2694391: Separate cache bin for migrations

Now that this is in, my db settings that are defined as 'drupal6' instead of 'migrate' no long are ready. Not sure if this is a problem where migrate_tools or migrate_plus isn't inserting its alter into the mix earlier enough. But on a couple machines now in my office we've seen this error and the only way around it is to rename our entry in settings.local.php from using 'drupal6' => 'migrate'.

It also leads to other weird errors, like messages that migrations we *know* we know exist, "Migration d6_node__foo' does not exist'.

It also seems to be exacerbated by using migrate-rollback.

Comments

heddn created an issue. See original summary.

heddn’s picture

Issue summary: View changes
vasi’s picture

I've seen something similar to this. Haven't bothered to debug yet, but if I see it again I'll make sure to do so.

Version: 8.1.x-dev » 8.2.x-dev

Drupal 8.1.9 was released on September 7 and is the final bugfix release for the Drupal 8.1.x series. Drupal 8.1.x will not receive any further development aside from security fixes. Drupal 8.2.0-rc1 is now available and sites should prepare to upgrade to 8.2.0.

Bug reports should be targeted against the 8.2.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.3.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

heddn’s picture

I think this is because of

  public function getDatabase() {
    if (!isset($this->database)) {
      // See if the database info is in state - if not, fallback to
      // configuration.
      if (isset($this->configuration['database_state_key'])) {
        $this->database = $this->setUpDatabase($this->state->get($this->configuration['database_state_key']));
      }
      elseif (($fallback_state_key = $this->state->get('migrate.fallback_state_key'))) {
        $this->database = $this->setUpDatabase($this->state->get($fallback_state_key));
      }

And is related to #2658282: Unable to run migrate drush commands: Argument 1 passed to Drupal\migrate\Plugin\migrate\source\SqlBase::setUpDatabase() must be of the type array, null given.

heddn’s picture

I think this is because of

  public function getDatabase() {
    if (!isset($this->database)) {
      // See if the database info is in state - if not, fallback to
      // configuration.
      if (isset($this->configuration['database_state_key'])) {
        $this->database = $this->setUpDatabase($this->state->get($this->configuration['database_state_key']));
      }
      elseif (($fallback_state_key = $this->state->get('migrate.fallback_state_key'))) {
        $this->database = $this->setUpDatabase($this->state->get($fallback_state_key));
      }

And is related to #2658282: Unable to run migrate drush commands: Argument 1 passed to Drupal\migrate\Plugin\migrate\source\SqlBase::setUpDatabase() must be of the type array, null given.