Thanks Davide for the initial review. I think I addressed the remaining feedback.
Anything else we want to test before merging?
On Fri, Sep 15, 2023 at 7:52 PM Ilan Rabinovitch ilan@linuxfests.org wrote:
Hi,
I took a stab at updating the cookbooks we use for the website to use centos8. Motivation was getting a new version of php so we can use some drupal plugins that dont support php5.4, but i think this was on the backlog already for a while.
https://github.com/socallinuxexpo/scale-chef/pull/283
To test this out, I spun up scale-web2 with a clone of the production scale-drupal database, and then pointed my host file at it for www.socallinuxexpo.org:
I then tested the following:
- chef runs complete end to end
- Backing up / restoring static assets (see backup scripts in
/usr/local/bin or in the scale-drupal cookbook)
- browsing legacy static website (eg
https://socallinuxexpo.org/past/2002/ https://socallinuxexpo.org/past/2003/)
- Registering as a speaker, confirming that I got the email and that
my account worked.
- publishing the submitted talk
- creating sponsors/exhibitors/blog post/events/etc
Pending things:
- Centos8 seems to default to using php-fpm instead of mod_php. Not
opposed but it's different. Seems to work fine, but I've not load tested it or anything.
- drush (https://www.drush.org/) packages no longer seemed to be
present in centos or epel. remi seems to have it but the drush docs suggest we should be installing this in our drupal code base via composer instead of as a package. I dont have strong opinions, but we should figure out the right path before merging.
Cheers, Ilan