In #694018: Split alias loading into conditional includes, Adrian noted that the sitealias database settings lookup failed under Aegir due to the fact that the database settings are not available until bootstrap_drupal_configuration.

This patch will fetch the database settings via backend_invoke('sql-conf'), insuring that a full database bootstrap is done before the db settings are accessed.

CommentFileSizeAuthor
#1 drush_alias_aegir.diff1.83 KBadrian
sitealias-database.patch1.43 KBgreg.1.anderson

Comments

adrian’s picture

adrian commented
Assigned: adrian » greg.1.anderson
Priority: Normal » Critical
StatusFileSize
new drush_alias_aegir.diff1.83 KB

I've had to modify this on my end to not ever use the db url from the settings file, since your new code was not being run at all.

The one weird thing i have is that without the full parameter i am getting : 

Moya:test.dev adrian$ drush sa test.dev testclone.dev

0
1

which i don't think is my issue.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Fixed

committed #1 with approval from greg.

the 0,1 bug is a known issue. will be fixed separately.

Status: Fixed » Closed (fixed)

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