search

dokku / dokku-elasticsearch

an elasticsearch plugin for dokku
MIT License
94 stars 35 forks source link

fixes #75 - kernel panic #78

Closed dfurber closed 4 years ago

dfurber commented 4 years ago

There were two different problems and one cleanup item, all unrelated to the docker upgrade referenced in issue #75:

  1. Line 83 wasn't copying the config files over. I changed the docker cp command to make it do so.
  2. Line 88 wasn't working because the jvm.options file in the image actually has 1g rather than 2g so the sed didn't work.
  3. While debugging, the -v option in line 90 that mounts that config folder was repeated twice.
lebalz commented 4 years ago

@josegonzalez can this be merged?