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

Updates docker-compose to use updated cli container #2533

Merged
merged 5 commits into from
Jun 5, 2018
Merged

Conversation

acouch
Copy link
Contributor

@acouch acouch commented May 23, 2018

Our current cli container is extremely old (ubuntu 12.04). This updates to 16.04 and sets the cli memory to unlimited.

This also has a couple of small updates to keep xdebug working. Xdebug is off by default but ahoy docker xdebug start should start xdebug..

The new cli container lives here: https://github.com/GetDKAN/dkan-docker/tree/master/.docker/php5-cli and is available in docker hub as getdkan/dkan-docker:php5-cli

Acceptance criteria

  • Tests pass
  • Run ahoy docker up locally
  • Run ahoy cmd-proxy "php -i | grep memory" and see memory is unlimited
  • Run ahoy cmd-proxy "php -i | grep xdebug" and see xdebug is off by default
  • Run ahoy docker xdebug start and ahoy cmd-proxy "php -i | grep xdebug" and see xdebug is on
  • Run ahoy docker xdebug stop ahoy cmd-proxy "php -i | grep memory" and see xdebug is off again

@acouch acouch requested a review from dafeder May 30, 2018 14:39
@janette janette merged commit 7ea7f44 into 7.x-1.x Jun 5, 2018
@janette janette deleted the cli-container branch June 5, 2018 19:14
dafeder pushed a commit that referenced this pull request Apr 24, 2020
* Updates to our own container

* Updates enabling xdebug

* Fixes artifacts

* Updates config

* Update config.yml
dafeder pushed a commit that referenced this pull request Apr 24, 2020
* Updates to our own container

* Updates enabling xdebug

* Fixes artifacts

* Updates config

* Update config.yml
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.

2 participants