Commit Graph

174 Commits

Author SHA1 Message Date
Mano Marks
2cd3d55fb8
Merge pull request #100 from thaJeztah/fix-javaworker-result
Fix javaworker result service not starting
2018-01-04 10:25:19 -08:00
Mano Marks
ae9ea759bc
Merge pull request #101 from thaJeztah/improve-result-dockerfile
Optimize result image
2018-01-04 10:25:01 -08:00
Mano Marks
e5dfa4747b
Merge pull request #102 from thaJeztah/improve-java-worker
Optimize java-worker image
2018-01-04 10:24:25 -08:00
Sebastiaan van Stijn
8b6432d58d
Optimize java-worker image
This patch optimizes the java-worker image;

- Use multi-stage build to separate the build-stage from the "deploy" stage
- Switch to the official Maven image, instead of manual installation of Maven.
  The official Maven image is also based on the `openjdk` repository on Docker
  Hub, which replaces the (now deprecated) `java` repository.
- Use `COPY` instead of `ADD` to follow best-practice
- Add `-XX:+UnlockExperimentalVMOptions -XX:+UseCGroupMemoryLimitForHeap` arguments
  so that JAVA will take memory-limits into account
- Use a JRE base image for the final build-stage

This change brings the size of the final image down from 184MB to 87MB

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2018-01-04 12:39:17 +01:00
Sebastiaan van Stijn
45788a5f1c
Optimize result image
This patch makes some optimizations in the result service's image;

- switch to `node:8.9-alpine`, which a smaller base image
- use `https://` for the registry
- use `COPY` instead of `ADD`, following best pracice
- cleanup npm cache, and combine steps to reduce image size

These changes bring down the image size from 248MB to 81.2MB

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2018-01-04 12:26:14 +01:00
Sebastiaan van Stijn
a8daa2d956
Fix javaworker result service not starting
The `--debug` option is deprecated and caused the result
service to not start:

    result_1  | [nodemon] 1.14.7
    result_1  | [nodemon] to restart at any time, enter `rs`
    result_1  | [nodemon] watching: *.*
    result_1  | [nodemon] starting `node --debug server.js`
    result_1  | (node:20) [DEP0062] DeprecationWarning: `node --debug` and `node --debug-brk` are invalid. Please use `node --inspect` or `node --inspect-brk` instead.
    result_1  | [nodemon] app crashed - waiting for file changes before starting...

This patch removes the `--debug` option, as it's not needed,
and no longer present in the other example stacks.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2018-01-04 11:49:58 +01:00
Mano Marks
762543a06d
Merge pull request #98 from akimd/shellcheck
tests: please shellcheck
2017-12-14 10:14:27 -08:00
Akim Demaille
8db887ee3a tests: please shellcheck
Signed-off-by: Akim Demaille <akim.demaille@docker.com>
2017-12-14 10:49:06 +01:00
Mano Marks
9652ca5e1c
Merge pull request #89 from oherrala/redisconn
Don't lose Redis connection on reconnect
2017-11-29 11:39:59 -08:00
Mano Marks
86aa3afa7b
Merge pull request #87 from ikisusi/startup-order
fix start-up order, depend worker on redis in docker-compose.yml
2017-11-29 11:30:43 -08:00
Mano Marks
f803307f5f
Merge pull request #96 from ManoMarks/master
adding warning about switching to Linux containers
2017-11-29 10:49:15 -08:00
Mano Marks
ee4eebab16 adding warning about switching to Linux containers 2017-11-29 10:47:53 -08:00
Mano Marks
b380900fac
Merge pull request #95 from ManoMarks/master
removing --debug flag from nodemon
2017-11-29 10:45:11 -08:00
Mano Marks
cfb85eabc0 removing --debug flag from nodemon 2017-11-29 10:43:31 -08:00
Mano Marks
9785312bf5
Merge pull request #90 from ubinix-warun/master
Patched: .NET 2.0.0 SDK for Worker
2017-11-21 13:37:49 -08:00
Mano Marks
3a5539927b
Merge pull request #92 from BretFisher/master
prevent dotnet worker from consuming all cpu
2017-11-21 13:37:17 -08:00
Mano Marks
ea465a712d
Merge branch 'master' into master 2017-11-21 13:36:41 -08:00
Mano Marks
f664995f25
Merge branch 'master' into master 2017-11-21 13:36:04 -08:00
Mano Marks
5d9127d152
Merge pull request #93 from BretFisher/result-test-fix-and-node-update
updating result app node version and fixing tests
2017-11-21 13:34:59 -08:00
Bret Fisher
ddd39f1699 updating node versions and fixing result tests 2017-11-21 13:48:10 -05:00
Bret Fisher
9fa3e4360f updating node versions and fixing result tests 2017-11-21 04:35:01 -05:00
Bret Fisher
f5a84ab40f prevent dotnet worker from consuming all cpu 2017-11-21 02:11:55 -05:00
ubinix-warun
3ab5145d4d FEATURED: .NET 2.0.0 SDK for Worker 2017-10-02 11:50:53 +07:00
Ossi Herrala
c2eeb7ab73 Don't lose Redis connection on reconnect
Keep Redis connection stored so that it's not lost and we avoid
reconnecting all the time.
2017-10-01 17:26:01 +03:00
Marko Laakso
0f9eee5879 fix start-up order, depend worker on redis in docker-compose.yml 2017-10-01 14:36:10 +03:00
Mano Marks
a58579e442 Update docker-stack.yml 2017-07-21 14:05:21 -07:00
Mano Marks
44efef623c Update ISSUE_TEMPLATE 2017-05-04 13:30:48 -07:00
Mano Marks
28a509b736 Create ISSUE_TEMPLATE 2017-05-04 13:30:03 -07:00
Mano Marks
0d4d850d88 Merge pull request #78 from ManoMarks/master
updating to latest version of dotnet
2017-04-07 12:34:39 -07:00
Mano Marks
5be0813483 updating to latest version of dotnet 2017-04-07 12:33:22 -07:00
Mano Marks
c33b6ae23b Merge pull request #67 from marcosnils/patch-2
Update replicas to 1 in result service
2017-01-31 08:22:32 -08:00
Marcos Nils
45eb3aaf9d Update replicas to 1 in result service
If a loadbalancer is used for this service, then socket.io will fail as the session will be created only in one container.
2017-01-31 08:40:55 -03:00
Sophia Parafina
14ba5b9cb3 Merge pull request #62 from ManoMarks/master
adding a placement constraint on visualizer
2017-01-25 14:32:11 -06:00
Mano Marks
8666fce704 adding a placement constraint on visualizer so it runs on a manager node 2017-01-25 12:29:38 -08:00
Sophia Parafina
36ff673463 Merge pull request #61 from ManoMarks/master
updating compose files to v3
2017-01-24 14:50:38 -06:00
Mano Marks
a01d01da63 Updating title of app 2017-01-24 12:48:36 -08:00
Mano Marks
88d606ed58 Updating README to include stack deploy instructions 2017-01-24 12:47:35 -08:00
Mano Marks
1d7bae103b updating compose files to v3 2017-01-23 17:44:16 -08:00
Mano Marks
96e20cf595 Merge pull request #51 from mbelsak/master
Establishe redis connection after its lost after a few minutes
2017-01-20 17:03:47 -08:00
Mano Marks
21781de893 Merge pull request #57 from ggtools/patch-1
Fix docker-compose-javawork.yml
2017-01-20 15:12:37 -08:00
Mano Marks
8e798dec3a Merge pull request #60 from docker/duplicate_key
note about one vote per client
2017-01-20 14:08:50 -08:00
sophia parafina
67e2e0800f note about one vote per client 2017-01-20 15:58:22 -06:00
Mano Marks
006f294799 Merge pull request #58 from ManoMarks/stack
Adding in docker-stack.yml
2017-01-18 09:46:32 -08:00
Christophe Labouisse
81f18a27f6 Fix docker-compose-javawork.yml
With docker-compose 1.10-rc1 build from this file was failing. Fix this by using the new build syntax with explicit `context` & `dockerfile`.
2017-01-15 22:25:58 +01:00
Mano Marks
d53ab169ac adding placement restraint to add visualizer, must run on a manager 2017-01-13 18:15:26 -08:00
Mano Marks
a34564cb55 updating to use dockersamples/visualizer:stable instead of manomarks/visualizer 2017-01-10 18:29:36 -08:00
Mano Marks
0285545512 adding before tag to vote and result images 2017-01-10 17:57:43 -08:00
Mano Marks
fc316a130a adding a line where one was missing 2017-01-09 14:44:01 -08:00
Mano Marks
4743b464a6 changing docker-stack.yml to more closely match the rest of the app, and add a visualizer 2017-01-09 14:42:37 -08:00
Mano Marks
48bdf36336 adding docker-stack.yml 2017-01-09 14:23:30 -08:00