Opened on 11/26/2014 at 04:22:21 PM

Last modified on 10/28/2015 at 10:35:45 AM

#1614 new defect

Refactor discourse puppet module to enable more general use

Reported by: kzar Assignee:
Priority: P3 Milestone:
Module: Infrastructure Keywords: discourse forum puppet
Cc: Blocked By: #2882
Blocking: #1624 Platform: Unknown
Ready: no Confidential: no
Tester: Verified working: no
Review URL(s):

Description (last modified by kzar)

Background

The discourse puppet module in the infrastructure repository was designed to provision the intraforum server. We now want to run at least one other Discourse server and therefore need to generalise the module to allow that.

What to change

Anything that prevents the module from being used for a different forum running on a different server. Also the use of the module will likely need to change in the intraforum manifest, to pass in any new parameters that will now be required.

The init-discourse Python script in the module might need to be replaced / refactored to facilitate this. Ideally the logic would be completely replaced with Puppet but maybe that's not possible / practical.

Notes

The aim of this ticket is to allow us to continue testing of the phpBB to Discourse forum migration in ticket #1327. It would be wise to look at the changes already made to the infrastructure repository for this issue https://github.com/kzar/adblockplus-infrastructure/tree/1327-forum-migration when making the changes required.

It might even be desirable to fork that branch directly when working on the changes, as some have been started IIRC. A code review patchset could then be made against master.

Attachments (0)

Change History (4)

comment:1 Changed on 11/26/2014 at 04:24:07 PM by kzar

  • Description modified (diff)
  • Summary changed from Refactor Discourse puppet module to enable more general use to Refactor discourse puppet module to enable more general use

comment:2 Changed on 11/27/2014 at 11:45:32 AM by innerself

  • Blocking 1624 added

comment:3 Changed on 08/11/2015 at 05:47:03 AM by matze

  • Blocked By 2882 added

comment:4 Changed on 10/28/2015 at 10:35:45 AM by trev

  • Blocking 1327 removed

Add Comment

Modify Ticket

Change Properties
Action
as new .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from (none).
Next status will be 'reviewing'.
 
Note: See TracTickets for help on using tickets.