Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.We've had a long standing issue with Drush-based installs failing due to weird interactions between Apps and Migrate when installing the panopoly_demo, which is installed by default:
#2202885: Install fails on BOA/Drush/Aegir - No migration found with machine name PanopolyDemoNode
More recently, we had a change in Apps break this too:
We really need a test for this on Travis-CI, so that we can never break it again!
I think the simplest thing to do, would be to change the upgrade test from the most recently released version to use Apps to download panopoly_demo, rather than doing it manually (like we're doing currently to work around the problem). We could just add another environment variable to that job that prevents that step.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | panopoly-apps-panopoly-demo-2430925-2.patch | 1.3 KB | dsnopek |
Comments
Comment #1
dsnopekHere's a patch that uses Apps to install panopoly_demo for the current -dev.
I tried doing it for the last release (like I described in the issue summary) but that failed because of #2430921: Failing to download 'panopoly_demo' when installing via 'drush si': "Downloading updates failed:Failed to download from ". Which actually got me thinking that testing the LAST version, while something we should do, doesn't really help with introducing regressions because we won't find out it fails until after release!
So, we really do need to test this with the current -dev like in the patch.
I'm waiting for some Travis-CI builds to finish, but assuming everything looks as it should, I'll commit this. Here's the build I'm waiting on:
https://travis-ci.org/dsnopek/panopoly/builds/51637859
EDIT: I pasted the wrong Travis link... It's right now!
Comment #2
dsnopekWow, ok, that wasn't a patch. :-) Here's the real patch!
Comment #3
dsnopekLooking good on Travis-CI! Committed.