Skip to content

Add notification service to docker compose #22

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 59 commits into
base: master
Choose a base branch
from
Open

Add notification service to docker compose #22

wants to merge 59 commits into from

Conversation

@ghost
Copy link

@ghost ghost commented Jul 27, 2018

No description provided.

@ghost ghost self-requested a review July 27, 2018 14:34
@ghost ghost force-pushed the includeNotificationServiceInDockerCompose branch from 6810678 to d221cd3 Compare July 27, 2018 18:12
jusoto added 3 commits July 30, 2018 17:31
@jusoto
Changed Docker-Compose version to 2.3 to run tests in order to limit …
dd92e3d 
…memory usage per service.

Added a waiting line for trim-cmd, trip-query and gmaps-adapter
@jusoto
Changes:
d2d370a 
- docker-compose version to 2.3 to allow mem_limit per container due to excesive use of memory when launching docker-compose.yml services.
- added in IntegrationTests userId for CalculationService new required field
- added local profile in application.yml to allow debug edge-service locally
@jusoto
Added:
361b184 
bin/ folder to gitignore
Added environment variables for calculation-service
tripId defined as local variable instead of global for test cases
@jusoto jusoto force-pushed the includeNotificationServiceInDockerCompose branch from 7b45391 to 361b184 Compare December 5, 2018 21:10
jusoto added 20 commits December 6, 2018 23:02
@jusoto
added travis_wait for tests
c615144
@jusoto
Added ./gradlew clean build to travis_wait
fdaca35
@jusoto
Intregration tests created on independent setups to consume less reso…
c377a71 
…urces for travis
@jusoto
Added time_wait 30 min for ./gradle check
4a3e1a6
@jusoto
Show log info on travis
b830898
@jusoto
Added maximum time wait for running tests with docker containers
e5f139d
@jusoto
Added environment variables for tests
a162fa6
@jusoto
Output IP ports in logs
e0f627b
@jusoto
Added new testing variation of healthy container for travis
36ce474
@jusoto
test changes for travis
fa6a1ae
@jusoto
join together different test cases files into one for travis
19cba19
@jusoto
add previous setup for testing ports on containers for travis
ba62577
@jusoto
userservice test case port validation
4488eb3
@jusoto
test cases trip query containers validation port
71a8607
@jusoto
test cases included calculation port validation
01b860e
@jusoto
test cases use container name to create HTTP requests
1084fe4
@jusoto
test cases, use localhost instead of container name
d3fe1f4
@jusoto
added environment variables for dependencies host names
e5c759f
@jusoto
test cases remove localhost from userServiceURL
185ecb8
@jusoto
Added output aid to check active containers before running tests
5488400
jusoto added 30 commits December 12, 2018 11:11
@jusoto
Added flag for docker inspect on test cases
3f37e3c
@jusoto
test cases - added log feature with details of running containers
26be5c0
@jusoto
Tests: Added ip address 0.0.0.0 to communicate with containers
cac6b50
@jusoto
Tests: retrieve information about docker environment
7c56a8c
@jusoto
Tests: Removed container named URLs
c10cac5
@jusoto
Tests: fixed style issues.
7b48a60 
Travis: removed env variables
@jusoto
Tests: added docker info command to log
8253287
@jusoto
Test cases: added links for dependencies
3b62459
@jusoto
Test cases: changed to localhost
b0b9da3
@jusoto
Test cases: Added log info to track requests
9ec2613
@jusoto
Test cases: DOCKER_IP env changed to localhost
f5f0768
@jusoto
Test cases: exchanged localhost by env HOST
df7bc70
@jusoto
Test cases: replaced env HOST by inFormat function
17fd8b7
@jusoto
Test cases: changed HOST to DOCKER_HOST
fa07e6e
@jusoto
Test cases: double check user service
b36bba8
@jusoto
Test cases: removed initial check on user service
6ae0410
@jusoto
Test cases: fix https for http
f2a9956
@jusoto
Test cases: removed individual integration test files
c3cdb26
@jusoto
Test cases: replaced if for while for checking opened ports
ea031a3
@jusoto
Travis time_wait 15
2203a6c
@jusoto
Travis time_wait changed to 10
0d1471d
@jusoto
Removed travis time_wait configuration
146161f
@jusoto
Changed travis script to gradle instead of gradlew
7f0f214
@jusoto
removed travis time_wait
953caa0
@jusoto
Test cases: removed userservice check on dockercomposerule initializa…
69a9d42 
…tion
@jusoto
Test cases: included debug info with opened ports
cb43ead
@jusoto
Travis: Fix health checks for test cases
bec603c
@jusoto
Travis: remove health check for port 8761 in test cases
1eaa868
@jusoto
Travis: Pushing files from linux environment
3e47f97
@jusoto
travis: Added time sleep 20 seconds during test setup to allow pullin…
5681107 
…g images from docker hub
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dmssargent @Richardj52 @jusoto