search

geerlingguy / ansible-role-drupal

Ansible Role - Drupal
https://galaxy.ansible.com/geerlingguy/drupal/
MIT License
92 stars 51 forks source link

Add handler to clear opcache #38

Closed mglaman closed 7 years ago

mglaman commented 7 years ago

This clears opcache via drush eval instead of restarting the webserver or the php-fpm service.

My go at a solution for #35

mglaman commented 7 years ago

@oxyc fixed. I actually updated it on #37, removing --root

mglaman commented 7 years ago

@geerlingguy I've tried, and tried. I don't know why drush si is failing :( But it is on both PRs.

I tried setting sendmail to mailhog, tee mail.out > /dev/null, etc. Nothing. I am assuming it's due to failure to email?

geerlingguy commented 7 years ago

@mglaman - That's really weird. I know BLT started seeing this issue in Travis CI builds recently: https://github.com/acquia/blt/issues/1336

It seems like it's related to either using Drupal 8.3.0+, or maybe Drush 8.1.10?

mglaman commented 7 years ago

@geerlingguy I wonder if installing sendmail to the Docker image? The error is sh: /usr/sbin/sendmail: No such file or directory. But my normal tricks seemed to fail to work, like setting sendmail path to which true

geerlingguy commented 7 years ago

@mglaman - Yeah; see my latest commit: https://github.com/geerlingguy/ansible-role-drupal/commit/3611e536fb501b789a98a22e265c1f8f091fa4d1

Tests are running right now, we'll see if that fixes things: https://travis-ci.org/geerlingguy/ansible-role-drupal/builds/223991723

geerlingguy commented 7 years ago

@mglaman - Can you rebase this PR? It seems like the fix worked.

mglaman commented 7 years ago

@geerlingguy 👍 rebased