Opened 3 years ago

Last modified 2 years ago

#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.

Change History (4)

comment:1 Changed 3 years ago 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 3 years ago by innerself

  • Blocking 1624 added

comment:3 Changed 2 years ago by matze

  • Blocked By 2882 added

comment:4 Changed 2 years ago by trev

  • Blocking 1327 removed
Note: See TracTickets for help on using tickets.