Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.On Travis-CI, our updrade tests are failing because panopoly_demo won't install:
https://travis-ci.org/lsolesen/panopoly/jobs/17908866
I'm not 100% why, but I suspect it's because a new version of panopoly_demo has hit the Apps server and it depends on something that isn't present in the older version of Panopoly.
I don't really know much about Apps, but is it possible to say that an older version of panopoly_demo will get used if panopoly_core is at an older version?
This will also probably affect users of older versions of Panopoly that haven't upgraded yet. If they go to install the Panopoly Demo app now, it will probably fail (although, I haven't tested this).
Comments
Comment #1
mrfelton commentedYes, I'm honestly not too sure about this one either. I updated the app server at the same time as making the release so that the updated versions of the 3 external apps (news, faq, demo) are listed by the app server.
I don't think so.
In terms of the app server, there are 2 urls:
http://apps.getpantheon.com/panopoly-dev for dev, and http://apps.getpantheon.com/panopoly for live. As you can see, the live version now references the 1.1 version.
I tested a new installation locally right after making the release just to be sure that it is working and everything went through fine so I'm not sure what the issues is when its run through travis.
Essentially I would expect the upgrade path testing to be broken since the latest version of panopoly_demo will likely not install on an older version of panopoly. But testing a new install on the latest version should work.
Comment #2
dsnopekI can make the Travis-CI tests force an install of Panopoly Demo with the same version as the release it's upgrading from. So, we can totally solve the tests that way! :-)
However, anyone with a site that still runs Panopoly 1.0-rc5 (or lower) will have to upgrade to Panopoly 1.1 in order to install the Panopoly Demo app, which is unfortunately, but probably acceptable. I wish there was a solution via Apps to target specific versions - that'd be cool!
I may still try installing Panopoly 1.0-rc5 and see what the actual error is.
Comment #3
dsnopekActually, this has me thinking that maybe Panopoly Demo shouldn't be an App at all?
It's really more of an integral part of a Panopoly release. We really can NEVER expect one version of Panopoly Demo to work with a different version of a Panopoly!
Whereas you could strive to keep Panopoly News or some other imaginary App working across several version of Panopoly...
Comment #4
dsnopekOk, this was made way more complicated and hackity because we didn't make a panopoly_demo-1.0-rc4 or panopoly_demo-1.0-rc5 release, so I couldn't just pull the same version as the upgrade. In the future, I think we should always make a panopoly_demo release with the main release (which again, makes it seem un-App like - but I'll leave that issue for now).