The latest provision is causing upgrades to DevShop to fail, and I FINALLY figured out why!
if (d()->profile != 'hostmaster') {
}
Since some of us extend Aegir by using our own drupal profile, we cannot check against d()->profile.
I dealt with a similar issue in #2704291: Verify of a site with function "hosting_get_features" overwrites the `~/.drush/drushrc.php` file..
Instead, let's check that the root and URI match those for @hostmaster alias. That was we can be absolutely sure we dealing with the "@hostmaster" site.
if (!(d()->root == d('@hostmaster')->root && d()->uri == d('@hostmaster')->uri)) {
}
Comments
Comment #2
jon pughComment #3
jon pughComment #5
jon pughComment #6
helmo commentedCheck in https://github.com/aegir-project/provision/pull/10 is green and it makes sense.
So +1 ... but we could improve it a bit by adding a utility function e.g. 'provision_is_hostmaster_site()'
Comment #8
jon pughGreat idea, done!
Comment #10
jon pugh