Skip to content
This repository has been archived by the owner on Aug 25, 2019. It is now read-only.

Upgrade to PHP 5.6 with Zend OpCache and APCu #16

Closed
wants to merge 33 commits into from

Conversation

renatomassaro
Copy link

  • Upgrade to PHP 5.6
  • Add "new" Zend OpCache
  • Add "new" APCu cache (faster than APC)
  • Install ImageMagick bin (used by Phabricator for meme creation)

Rebuilt from scratch and all is working well

@hach-que
Copy link
Contributor

hach-que commented Feb 2, 2016

As per the comment in #13, this image needs to be migrated towards a derive and bake model before I can merge in more major changes.

@lamjack
Copy link

lamjack commented Mar 16, 2016

can i search this image in docker repository?

location /.well-known/ {
root /srv/letsencrypt-webroot;
}
#location /.well-known/ {
Copy link
Contributor

Choose a reason for hiding this comment

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

We can't remove this as it's required for SSL to work.

@hach-que
Copy link
Contributor

There are some issues with this change, and it needs to be rebased on the latest version in order to be merged.

@renatomassaro
Copy link
Author

Yep, this PR changes were supposed to go only until commit f02e2d1. Not sure how I change the PR HEAD here on Github.

Since you said this image must be rewritten, I decided to reshape my fork to my own needs.

@hach-que
Copy link
Contributor

Just an FYI, I'm working on bringing these changes upstream, along with a number of configuration improvements. In particular:

  • The base image is upgraded to OpenSUSE Leap 42.1
  • It'll have PHP 5.6 and APCu installed
  • The number of layers in the Docker image are being significantly reduced
  • You'll be able to configure it the conventional way with /config
  • You'll be able to configure it by setting environment variables
  • You'll be able to configure it by deriving your own image from this image, and running setup commands to bake the configuration in

I'll be updating the README with the new configuration options when this work is done.

@hach-que
Copy link
Contributor

hach-que commented May 2, 2016

@renatomassaro All of these changes are now available in this image, and it now supports configuration via environment variables as well as scripts.

Going to close this PR as there's nothing left to do here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants