-
Notifications
You must be signed in to change notification settings - Fork 25
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
Updating Docker files for UPS #8
Conversation
matzew
commented
Jul 21, 2015
- removing quickstarts image
- Using Centos7, WF9 and Java8 (inheriting from 9.0.0.Final WF image) for UPS images
- explicitly referencing 9.0.0.Final instead of latests
- DB references via container links for MySQL
- README updates
@danielpassos @secondsun @sebastienblanc @abstractj wanna take a look ? |
followed the instructions in the readme, got this when I fired up the UPS container |
did you do the linking ? On Wed, Jul 22, 2015 at 5:06 PM, Hoyt Summers Pittman <
Matthias Wessendorf blog: http://matthiaswessendorf.wordpress.com/ |
My 2 cents matzew#1 |
Formatter/Fix README
@matzew |
summers, you need to build the wildfly image too! your is still java7 mine goes Centos7, wf9 and jdk8 ;) On Wednesday, July 22, 2015, Hoyt Summers Pittman notifications@github.com
Sent from Gmail Mobile |
|
||
```shell | ||
docker run --name ups-dev \ | ||
--link unifiedpushDEV:unifiedpush |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing a '' after unifiedpush
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch
What I did? docker build -t aerogear/unifiedpush-wildfly .
docker run --name keycloak \
-p 5306:3306 \
-e MYSQL_USER=unifiedpush \
-e MYSQL_PASSWORD=unifiedpush \
-e MYSQL_DATABASE=unifiedpush \
-e MYSQL_ROOT_PASSWORD=supersecret \
-d mysql
docker run --name unifiedpush \
-p 3306:3306 \
-e MYSQL_USER=unifiedpush \
-e MYSQL_PASSWORD=unifiedpush \
-e MYSQL_DATABASE=unifiedpush \
-e MYSQL_ROOT_PASSWORD=supersecret \
-d mysql
docker run --name ups \
--link unifiedpush:unifiedpush \
--link keycloak:keycloak \
-p 8443:8443 \
-it aerogear/unifiedpush-wildfly |
So, maybe is a good idea add it in the README |
After build |
@matzew Running the second run command (first went fine), I get: FATA[0002] Error response from daemon: Cannot start container 0916d9f7e2498c42dfcdd9404aa439ce101674a4f1a3bcd84a4536947d2b581a: Error starting userland proxy: listen tcp 0.0.0.0:3306: bind: address already in use [ Fedora 21, Docker version 1.6.0, build 350a636/1.6.0] |
@matzew |
Worth mentioning that there is an error in the log (it does not block the deployment): 10:47:36,114 ERROR [org.hibernate.tool.hbm2ddl.SchemaUpdate](ServerService Thread Pool -- 67) HHH000388: Unsuccessful: create index deviceTokenIndex on Installation (deviceToken) |
@vibe13 the INDEX is a known thing - we have JIRA somewhere for that :) |
@matzew ok great :-) |
@vibe13 what Docker version are you using ? @secondsun had issues w/ 1.6.x on his Fedora-21 |
@matzew I'm running 1.7.1 now. Still have the same issues. |
I tested the unifiedpush-wildfly from a VM, that comes w/ Fedora:
and, well - works there |
Ok so here are my tests :
** UPDATE ** Fedora 21 - Fedora Provided 1.6.0 - runs correctly |
One thing that I noticed : If you build from the master branch and someone updates the master branch docker won't pull the updates because it will still be using the cached steps from a previous build. It looks like there are workarounds to make sure the clone step runs every time, but that will be something to investigate |
@secondsun mind giving this a new try?
That step was failing for you in the past |
Dev is working for me on Fedora 22 now. |
Stable looks good, +1 |