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

Add some restart tests #166

Merged
merged 9 commits into from
Mar 14, 2017
Merged

Add some restart tests #166

merged 9 commits into from
Mar 14, 2017

Conversation

djdefi
Copy link
Contributor

@djdefi djdefi commented Mar 9, 2017

Add some tests for restart behavior. The goal is to have something better for restart detection than the .cachet-installed logic in entrypoint.sh

ref: #161 (comment)

@djdefi
Copy link
Contributor Author

djdefi commented Mar 9, 2017

First pass at a refactor of init detection. Have only tested so far with Postgres, and will need to setup better tests to cover MySQL a bit more.

djdefi and others added 5 commits March 8, 2017 23:41
…rt-tests

* 'restart-tests' of github.com:CachetHQ/Docker:
  MAX_CHILDREN changed to PHP_MAX_CHILDREN
  Added environment setup for pm.max_children, default:5
APP_ENV=${APP_ENV:-development}
APP_DEBUG=${APP_DEBUG:-true}
APP_URL=${APP_URL:-http://localhost}
APP_KEY=${APP_KEY:-SECRET}
APP_KEY=${APP_KEY:-base64:SGZXUdds0Qnbf55/7diaHMPPM2TXfOSxHtUAXz6POSw=}
Copy link
Contributor Author

@djdefi djdefi Mar 14, 2017

Choose a reason for hiding this comment

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

I realize a key is being shipped here, but it needed some real default value for now. This should be set via -e APP_KEY="base64:<Your Real Key>" in production anyway.

@djdefi djdefi self-assigned this Mar 14, 2017
@djdefi
Copy link
Contributor Author

djdefi commented Mar 14, 2017

cc: @csandanov based on #161 (comment)

@djdefi djdefi merged commit f687210 into master Mar 14, 2017
@djdefi djdefi deleted the restart-tests branch March 14, 2017 17:24
@djdefi djdefi added this to the 2.4 milestone Mar 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant