michaelmcandrew / civicrm-buildkit-docker

This is a read only copy. Please make PRs here: https://lab.civicrm.org/michaelmcandrew/civicrm-buildkit-docker
https://lab.civicrm.org/michaelmcandrew/civicrm-buildkit-docker
GNU Affero General Public License v3.0
40 stars 31 forks source link

Apache logs from the civicrm container do not propogate to the host #12

Closed michaelmcandrew closed 6 years ago

michaelmcandrew commented 6 years ago

amp defines a custom log file for apache that prevents Docker from handling the logs. See https://github.com/amp-cli/amp/issues/52 for more details.

As soon as https://github.com/civicrm/civicrm-buildkit/pull/396 is merged, we can create a new Docker image based on a newer version of buildkit that uses the custom vhost and config added in this commit: https://github.com/michaelmcandrew/civicrm-buildkit-docker/commit/65a0971e4cdb2beb0a5f88880540fe2879c896f0.

Until then, a workaround is to manually edit the vhost files in /buildkit/.amp/apache.d and remove the lines that define the custom logs.

totten commented 6 years ago

The dependency https://github.com/civicrm/civicrm-buildkit/pull/396 is now merged.

michaelmcandrew commented 6 years ago

thanks @totten - closing this issue now.