We'd need to add an option to provision-delete first, then pass that option into the task.

Comments

Jon Pugh’s picture

Jon Pugh
he/him
Primary language English
Location Newburgh, NY
CreditAttribution: Jon Pugh at OpenDevShop Inc (CLOSED) commented
Assigned: Unassigned » Jon Pugh
Status: Active » Needs review

Seemed like low hanging fruit so I picked it.

Jon Pugh’s picture

Jon Pugh
he/him
Primary language English
Location Newburgh, NY
CreditAttribution: Jon Pugh at OpenDevShop Inc (CLOSED) commented

http://cgit.drupalcode.org/hosting/commit/?id=57042f3

http://cgit.drupalcode.org/provision/commit/?id=58f00ee

Branch is 2473679-pre-delete-backup-optional in both repos.

helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions for Aegir Cooperative commented
Status: Needs review » Needs work

Not sure why but after merging 2473679-pre-delete-backup-optional branch I get these errors:

Calling hook drush_provision_drupal_pre_provision_delete	
-
Skipping backup.	
-
Returned from hook drush_provision_drupal_pre_provision_delete	
-
Calling hook drush_db_provision_delete	
-
Dropping database 	
-
Failed to drop database 	
-
Unable to drop database.	
-
Returned from hook drush_db_provision_delete