Skip to content
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

Update PHP, Drupal, Drush, Composer for Composer 2 support #193

Merged
merged 3 commits into from
Nov 5, 2020

Conversation

alxp
Copy link
Contributor

@alxp alxp commented Oct 30, 2020

GitHub Issue: (link)

Composer issue when running "vagrant up" #1663

What does this Pull Request do?

Vagrant / Ansible installation was failing due to composer being updated to version 2.0 in the Ubuntu base install.

Bumping version of composer packages revealed incompatibilities that needed to also be updated, including PHP to 7.4, Drush to 10.x, and Drupal Core to 8.9, which I did by changing the Drupal composer project to the officially-supported drupal/recommended-project.

What's new?

  • Drupal Core updated to 8.9.1

  • Drush updated to 10.3.1

  • PHP updated to 7.4.11

  • Composer to 2.0.3

  • Removed drupal/console , composer 2 compatibility is an open issue in the project. If needed console can be installed directly by the user until this is resolved. drupal-console-extend-plugin: Compatibility with Composer 2

  • Does this change require documentation to be updated?

No

  • Does this change add any new dependencies?

Just updates

  • Does this change require any other modifications to be made to the repository

No

  • Could this change impact execution of existing code?

None of these changes should affect Islandora behaviour, however, I observed the Ansible build step of installing packages via composer is significantly faster.

How should this be tested?

*** Crayfish PR has been merged so edited out no-longer-needed testing steps ***

  1. Initiate a build of the playbook via vagrant up. (Thus, you must set the environment variable ISLANDORA_DISTRO=ubuntu/bionic64)
  2. Observe the automated test results
  3. Go through the process of creating nodes and media and ensure that everything works as expected.

Interested parties

@Islandora-Devops/committers

Copy link
Member

@seth-shaw-unlv seth-shaw-unlv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested in conjunction with islandora-deprecated/ansible-role-fits#8 and works as advertised. 👍

@seth-shaw-unlv seth-shaw-unlv merged commit 7cd01da into Islandora-Devops:dev Nov 5, 2020
@ruebot
Copy link
Contributor

ruebot commented Nov 10, 2020

Should this also be updated to remi-php74?

ruebot added a commit to yorkulibraries/yudl-playbook that referenced this pull request Nov 10, 2020
- Pull in changes from Islandora-Devops/islandora-playbook#193
- Add multitail
- Pull in changes from Islandora-Devops/islandora-playbook#190
- Switch back to upstream FITS role
- Switch back to upstream Crayfish role
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants