Given that Aegir Deploy:

  • significantly improves the UX on the Add Platform form,
  • adds copious amounts of in-line documentation,
  • allows Composer-based sites to be provisioned directly from Packagist,
  • allows Git-based sites to be immutable thereby disallowing dangerous operations, and
  • clearly illustrates the differences between the various deployment strategies

...it should be enabled by default. But we can consider that later. At the very least, let's add the module to Golden Contrib so that it ships with Aegir.

Prerequisites

These other issues should be resolved before we can finalize this one.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

colan created an issue. See original summary.

colan’s picture

Status: Active » Needs review
FileSize
809 bytes

Do we need to add the dependencies as well, or do these get pulled in automatically?

helmo’s picture

I just created #3038831: deployfrompackegist error and #3038823: Text overlaps while testing this ... but +1 in general ...

the patch here then also needs to add it's dependencies ... features and hosting_objects

colan’s picture

Issue summary: View changes
Status: Needs review » Postponed
FileSize
1.08 KB

Patch updated, but we need to wait for those other issues. I updated the issue description with these new requirements.

colan’s picture

Issue summary: View changes
llamech’s picture

Note that #3038831: deployfrompackegist error is now RTBC. Can we now add this to golden-contrib?

llamech’s picture

Status: Postponed » Needs review
colan’s picture

Category: Feature request » Plan
Status: Needs review » Postponed

We should be good to go once that get in, and we cut a new release for it. So postponing on that.

colan’s picture

Category: Plan » Feature request
Status: Postponed » Needs review
Related issues: +#3045001: Create new release
FileSize
637 bytes
1.08 KB

Added releases from #3045001: Create new release.

ergonlogic’s picture

Looks good.

ergonlogic’s picture

Status: Needs review » Reviewed & tested by the community
helmo’s picture

Status: Reviewed & tested by the community » Fixed

Thanks, merged.

helmo’s picture

  • helmo committed 7ae58dc on 7.x-3.x authored by colan
    Issue #3037070 by colan: Add Aegir Deploy to Golden Contrib
    

Status: Fixed » Closed (fixed)

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