Does this have do with Drupal 8 settings.php settings not allowing for a $db_url ?

db/Provision/Service/db/pdo.php

- has no $creds['pass'] available Undefined index: pass

This seems not to be fatal while it should IMHO

Checking for setters show we use urldecode.

~/.drush/provision$ grep -ri '\->creds =' .
./db/Provision/Service/db.php:    $this->creds = array_map('urldecode', parse_url($this->server->master_db));

platform/drupal/install_8.inc

- has no $db_spec['db_prefix'] = $GLOBALS['db_prefix']; <Undefined index: db_prefix install_8.inc:12x
- now and then error: if ($db_spec['driver'] == 'mysqli') { <Undefined index: driver install_8.inc:12x

Array
(
    [driver] => mysql
    [database] => drupal8dummyclem
    [username] => drupal8dummyclem
    [password] => MD8D54LHGs
    [host] => localhost
    [port] => 3306
    [prefix] => Array
        (
            [default] =>
        )

)

provision_drushrc_site.tpl.php

This file may need changes for Drupal 8

./Provision/Config/Drushrc/provision_drushrc_site.tpl.php

provision_drupal_settings.tpl.php

This file needs changing and/or splitting out into version specific templates.

Run failing task manually

Fill in the correct number for failing task (230 below)

drush @hostmaster hosting-task --force --debug --verbose 230
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

clemens.tolboom’s picture

Status: Active » Needs review
FileSize
659 bytes

Attached patch makes the call to get the current config Drush 7 specific as Drupal 8 requires.

helmo’s picture

Issue summary: View changes
Deciphered’s picture

Issue summary: View changes
Deciphered’s picture

Status: Needs review » Needs work
FileSize
692 bytes

Updated version of patch from #1, keeps the legacy methods (for old versions of drush) and ensures the file containing the function is included.

Additional changes related to this install are coming.

helmo’s picture

I doubt that Drush 6 will support Drupal 8 ;)

Deciphered’s picture

@helmo, The legacy support isn't so Drush 6 can support Drupal 8, it's so if someone running Drush 6 can still create Drupal 7 sites if using the latest version of Provision. But it maybe that that isn't something we need to deal with.

Deciphered’s picture

FileSize
8.16 KB

Brainfart, only for D8.

Updated with that as well as a few other to get a working site, now time to clean it up.

helmo’s picture

first install attempt via the ui failed... sites/default/settings.php ...

run via drush and it worked.. site install task green.

Now when i visit the new site I get "If you have just changed code (for example deployed a new module or moved an existing one".

With "PHP Fatal error: Call to a member function get() on a non-object in /var/aegir/platforms/drupal-8/core/lib/Drupal.php on line 632" in the apache logs.

Deciphered’s picture

Status: Needs work » Needs review
FileSize
9.39 KB

Updated patch with proper generation of hash salt.

Not certain if that other issue is directly related, digging.

Deciphered’s picture

FileSize
9.31 KB

Left a debug function in there. OOps.

  • helmo committed 54048fd on 7.x-3.x authored by Deciphered
    Issue #2311005 by Deciphered, clemens.tolboom: Fixed D8 Site install...

  • helmo committed 54048fd on 2358795-provision-save-on-verify authored by Deciphered
    Issue #2311005 by Deciphered, clemens.tolboom: Fixed D8 Site install...
helmo’s picture

Status: Needs review » Fixed

has been committed a while ago

Status: Fixed » Closed (fixed)

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