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

Revert to Jessie + init fixes #58

Merged
merged 1 commit into from
Apr 8, 2016
Merged

Revert to Jessie + init fixes #58

merged 1 commit into from
Apr 8, 2016

Conversation

djdefi
Copy link
Contributor

@djdefi djdefi commented Apr 8, 2016

Addresses a few outstanding issues.

  • Fixes PHP-APC is not included to docker container #57 - Reverted to a Debian Jessie base, using official php and APCu packages
  • Closes Include startup actions like "migrate" #36 - On first run, key is generated, database is initialized and /.cachet-installed touch file is added, so that we don't run the php artisan key:generate and php artisan app:install on the second run. Caveat to this is if the Cachet container is deleted (such as on an upgrade) we lose the touchfile. Per the warning in the Cachet install guide: Never change the APP_KEY after installation on production environment. This will result in all of your encrypted/hashed data being lost. -- May need to clarify what that data is that is lost. Another option (probably the right option) is to query for an appropriate value in the database.
  • May also address Error 500 after restart the container #34 - I am able to restart the container consistently.
  • Adds a check for DB reachability to prevent race conditions.

@djdefi djdefi merged commit a5445f5 into master Apr 8, 2016
@djdefi djdefi deleted the apc branch April 8, 2016 23:55
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