[ckan-changes] [ckan/ckan] 26fb24: docker: Don't do configuration in the runfile

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/ckan/ckan
  Commit: 26fb24c055efe99aaf122599f8c619eb4dfc1d31
      https://github.com/ckan/ckan/commit/26fb24c055efe99aaf122599f8c619eb4dfc1d31
  Author: Nick Stenning <nick at whiteink.com>
  Date:   2014-07-28 (Mon, 28 Jul 2014)

  Changed paths:
    M Dockerfile
    A contrib/docker/my_init.d/50_configure
    A contrib/docker/my_init.d/70_initdb
    M contrib/docker/svc/ckan/run

  Log Message:
  -----------
  docker: Don't do configuration in the runfile

This commit extracts the config file generation and database
initialisation parts of the CKAN service into separate scripts which are
run at container "boot" rather than in the service file.

This is just a bit cleaner, and means that the scripts are guaranteed to
only run once on boot, rather than every time CKAN is started (e.g. if
CKAN crashes, runit will rerun the `run` file, but not the init scripts)

This also makes it easier to replace the configuration step with, for
example, a standalone Python script if this is deemed easier to
maintain.


  Commit: 4c90eb2a958985e2a109d653d51d9d9afde29a00
      https://github.com/ckan/ckan/commit/4c90eb2a958985e2a109d653d51d9d9afde29a00
  Author: Sean Hammond <seanh at users.noreply.github.com>
  Date:   2014-07-28 (Mon, 28 Jul 2014)

  Changed paths:
    M Dockerfile
    A contrib/docker/my_init.d/50_configure
    A contrib/docker/my_init.d/70_initdb
    M contrib/docker/svc/ckan/run

  Log Message:
  -----------
  Merge pull request #1860 from nickstenning/docker-split-run

docker: Don't do configuration in the runfile


Compare: https://github.com/ckan/ckan/compare/73909560b1d9...4c90eb2a9589