The Distribution Update Status Manager module has been written that enhances the Update (core) manager module specifically for distributions. We plan to include this in the Panopoly distribution but we wrote it generically enough for any distribution to use.

We tested it locally with the Commerce Kickstart distro by removing the update hooks currently in use and then enabling the distro_update module. The features of the distro_update module are listed on the project page.

I'm hoping to get more distributions using the same module so we're not all reinventing the wheel.

If commerce kickstart is interested in using it, I can create the patch.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bojanz’s picture

That looks awesome. Definitely interested.

caschbre’s picture

Status: Active » Needs review
FileSize
4.17 KB

Attached is a patch that removes the projects/status alter hooks and any update admin page form/settings alterations. The distro_update module was added to the .make file as well.

lsolesen’s picture

Status: Needs review » Needs work

Should probably also be enabled?

caschbre’s picture

Good catch.

Here's an updated patch that:
* Changes distro_update to beta3
* Adds distro_update to .info file
* Adds update hook to enable the module

caschbre’s picture

Assigned: Unassigned » caschbre
Status: Needs work » Needs review
lsolesen’s picture

I applied it on the most current code (though based on a Drupal 7.30) and I get the message that I should upgrade my Drupal version. Maybe, there could be some sort of explanation that it is good to wait for the distro update?

caschbre’s picture

Yep, I can add a message to the top of that page as well as the normal report page.

In that screenie... is drupal core out of sync with the distro .make file?

Edit: out of the box, any module that is out of date but the version matches the .make file is hidden on that update page.

lsolesen’s picture

@caschbre Yeah on that screenshot they are out of sync.

caschbre’s picture

@lsolesen... I haven't tagged a new release yet since I want to wait and see if there are other issues with the module but I did incorporate the warning/help text on the top of the various pages and inline in some cases.

#2328175: Add warning text to top of update report/update pages

caschbre’s picture

FYI... I'll be creating beta4 soon. Mostly minor enhancements to handle edge case scenarios. I'll update this patch when I get that completed.

caschbre’s picture

Attached is an updated patch that uses distro_update 1.0-beta4. This includes mostly edge case scenarios.

lsolesen’s picture

Status: Needs review » Reviewed & tested by the community

This looks good to me. I've created a pull request to see what the automatic tests says.

https://github.com/commerceguys/commerce_kickstart/pull/78

candelas’s picture

Hello

Does it works with Commerce Kickstart 2.22? Thanks

candelas’s picture

I add this errors that I get in a fresh 2.22 install on features.

mglaman’s picture

Assigned: caschbre » Unassigned
Status: Reviewed & tested by the community » Needs work
Issue tags: +sprint

PR is failing tests: https://github.com/commerceguys/commerce_kickstart/pull/78

Tagging for kanban board, would like to get this in.

mglaman’s picture

Patch no longer applies to 7.x-2.x HEAD

mglaman’s picture

Status: Needs work » Needs review
FileSize
5 KB

  • mglaman committed 1da6850 on 7.x-2.x
    Merge pull request #144 from mglaman/2325349-distro-update-manager...
  • mglaman committed 95b381e on 7.x-2.x
    Issue #2325349 by caschbre: Use the new Distribution Update Status...
mglaman’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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