-
Notifications
You must be signed in to change notification settings - Fork 5
Add support for AtoM 2.9 and PHP 8.x #128
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Add 2.9-search.yml (copy of 2.8) - Bump AtoM npm version
d03d436 to
4ed5ba6
Compare
|
@mamedin this PR is based off of your |
replaceafill
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
Running some tests:
|
|
Rocky9 upgrade from AtoM 2.8 failed: It was a file: It comes from php-cli package: Checking /var/log/dnf.log, it was installed at: And checking /var/log/messages it was done with ansible: More lines in dnf.log: When removing php-cli: After removing the package and running again the role, this time failed starting php-fpm service, because it has 2 fpm services installed and the old is running: |
|
@anvit I added a new commit to fix the 2 issues I saw in RedHat upgrade. |
It makes easier to maintain the php tasks for RedHat. We already had a task file only for php7.4 and RedHat. It is the same php-rh-7.4.yml file but adding a task to remove old php7 (required for 2.9 upgrades from 2.7/2.8). To avoid installing recommended packages, the following options have been added to yum task: * allowerasing: true * install_weak_deps: false
|
New error when testing rolling the role in AtoM 2.8 (Ubuntu20) and AtoM2.8->2.9 Upgrade in Rocky9: |
|
Reading the error log: So, it requires node>=20.5.0. But the problem is that it is not supported in Ubuntu18 or CentOS7, so it will break the role for old versions. See supported versions on deb and rpms here: https://github.com/nodesource/distributions Additional changes must be done to install the nodejs package with the correct package repo, but need to take into account the old supported AtoM and OS versions.
We need to decide what to do with We can define the This way, it will use the old version (9.8.1) for Ubuntu<=20.04 and RedHat <=7. RedHat is using a different
Ubuntu requires the changes in deps.yml:
|
* Added conditional for default npm version based on OS * Use nodesource 22.x repo for Ubuntu >= 22.04 * Use nodesource 22.x repo for RedHat >= 8 and php >= 8.1 * Fix nodesource 14.x repo for Ubuntu 20.04
Update README.md for AtoM 2.9 and new supported Operative Systems.
|
Test finished. Vagrant tests done: https://github.com/artefactual/deploy-pub/tree/dev/add-atom29-playbooks |
|
Thanks @mamedin ! I took a look at the new commits and they look good to me! |
This should add support for AtoM 2.9 (Ubuntu 22.04, Ubuntu 24.04, Rocky9) and PHP 8.x. Also bumps npm and node versions for AtoM