I use Aegir to manage a bunch of OpenAtrium sites. I updated the 'views' module (because of security alert SA-CONTRIB-2011-052) by putting a new version of 'views' in sites/all/modules (rather than changing it in profiles/openatrium/modules) - which I think is best practice, because that way I can still update to a new version of OpenAtrium by replacing the profile directory.
Anyway, Aegir was building the packages for the profile incorrectly, by having the module in the profile override the module in sites/all/modules (ie. giving the wrong filename, version and schema_version on the hosting_package_instance table) rather than vice-versa.
The net result was that I couldn't Clone or Migrate to a later platform because it thought the schema_version would go backwards.
Attached is a patch that reverses this behavior. After applying the patch and doing a Verify on the platform, the platform comparison and Clone/Migrate start working correctly.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | hostmaster-packages.patch | 842 bytes | dsnopek |
| hostmaster-packages.patch | 649 bytes | dsnopek |
Comments
Comment #1
dsnopekRerolled the patch and hopefully moved to the appropriate issue queue. :-) I hadn't realized that development of the 'hosting' module was taking place in the 'hostmaster' project.
Comment #2
Robin Millette commentedI noticed the same discrepancy (installed modules vs modules in use) which leads to confusion. I tried the patch on Aegir 1.6 and it seems to be working. It's also consistent with drupal_system_listing(...), respecting precedence of modules.
Comment #3
steven jones commentedThanks for the bug report and the patch, this actually needed a little bit of work on provision too, so that we keeping the priority of the modules correctly.
Fixed in 6.x-1.x in hostmaster and provison
Comment #4
Robin Millette commentedThe provision commit is actually here: http://drupalcode.org/project/provision.git/commit/6530e8c6b4d0d1a724409...
Thanks!
Comment #5
steven jones commentedI was trying to be helpful and link to the commits, because d.o doesn't at the moment, sorry that I didn't get the linking right.