Deploy updates not applied to multisites

acquia / blt

Acquia's toolset for automating Drupal 8 and 9 development, testing, and deployment.

https://docs.acquia.com/blt/

GNU General Public License v2.0

442 stars 394 forks source link

Deploy updates not applied to multisites #1141

Closed danepowell closed 7 years ago

danepowell commented 7 years ago

As part of #1092, I thought we had deploy:update running on multisite installs on ACE, but this turned out to be something of a red herring. Using the @sites alias was a good start, but because we also specify drush.uri (https://github.com/acquia/blt/blob/8.x/phing/build.yml#L73), updates only applied to the default site. We need to remove the drush.uri property for this to work.

bobbygryzynger commented 7 years ago

@danepowell - just throwing this out there. Would it suffice to override drush.uri during deploy:update?

<property name="drush.uri" value=""/>

danepowell commented 7 years ago

Yeah actually it might be that simple. I'll open a PR. Thanks!

grasmash commented 7 years ago

Fixed in #1146