Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hefox’s picture

hefox’s picture

Updated as panopoly_core updated features

mpotter’s picture

Issue tags: +oa patches
mpotter’s picture

Updating Features to latest 2.6-rc1 release.

dsnopek’s picture

Title: Patches for features from OpenAtrium » Update Features to version 2.6-rc1
Issue tags: +sprint

Retitling and adding to sprint board.

dsnopek’s picture

Category: Feature request » Task

Here's a test run of this patch on Travis:

https://travis-ci.org/panopoly/panopoly/builds/67034379

dsnopek’s picture

Status: Needs review » Reviewed & tested by the community

The tests have passed. @mpotter told me over IRC that this is stable enough for production! I'll merge this soon.

dsnopek’s picture

Status: Reviewed & tested by the community » Fixed

Committed!

  • dsnopek committed 1d12cfd on 7.x-1.x
    Update Panopoly Core for Issue #2477391 by hefox, mpotter: Update...
micbar’s picture

I had to hotfix my custom install profile and revert to features 7.x-2.5. The drush features revert command triggers a PHP Fatal Error by exceeding the memory limit.

micbar’s picture

Status: Fixed » Needs work
micbar’s picture

dsnopek’s picture

Status: Needs work » Fixed

Thanks for the report! It should be totally safe to revert to 2.5 - we didn't change any of our code for Features 2.6-rc1.

I'm marking this back to Fixed, since we did successfully update to Features 2.6-rc1. Depending on what happens on #2508501: Memory Size exceeded while reverting all features we might decide to revert to 2.5, in which case it makes sense to re-open this issue. But if we end up including a patch to fix or upgrade to 2.6-rc2 or something, then that would be a new issue!

mpotter’s picture

Status: Fixed » Needs review
FileSize
405 bytes

Reopening this. Just released Features 2.6 that has the patch regarding the recursive issue that was causing memory size and timeout errors.

Should update panopoly_core to Features 2.6.

It would be nice *not* to revert back to 2.5 because of all the improvements to Features for removing un-needed overrides in distros like Open Atrium that build on Panopoly.

dsnopek’s picture

@mpotter: Can you make a new issue for that? We really only re-open issues if (a) we need to revert or (b) the original goal of the issue wasn't actually accomplished, even though we thought it was. We successfully updated to Features 2.6-rc1, updating to 2.6 should be a new issue!

dsnopek’s picture

Title: Update Features to version 2.6-rc1 » Update Features to version 2.6

Out of laziness, I'm just going to rename this issue.

Here's the test run on Travis-CI: https://travis-ci.org/panopoly/panopoly/builds/69169030

I'll merge if the tests pass!

dsnopek’s picture

Status: Needs review » Fixed

Tests passed! Commited.

  • dsnopek committed d22ce17 on
    Update Panopoly Core for Issue #2477391 by hefox, mpotter: Update...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

mpotter’s picture

Title: Update Features to version 2.6 » Update Features to version 2.8
Status: Closed (fixed) » Needs review
FileSize
397 bytes

New version of Features is out, so need to update Panopoly.

dsnopek’s picture

Can you please open a new issue for this? We really only re-open issues if we're going to revert the change (usually because the committed didn't actually do as promised or some problem was found with it).

dsnopek’s picture

Title: Update Features to version 2.8 » Update Features to version 2.6
Status: Needs review » Closed (fixed)
Related issues: +#2706259: Update to Features 2.9 for SA-CONTRIB-2016-020