Peter Jausovec
3f877581ed
Only log errors when we are deploying new services; don't log errors if we are scaling old services down.
2016-12-16 10:18:30 -08:00
Peter Jausovec
c8889e54c6
Merge pull request #27 from Microsoft/peterj/handle_killing_finished_events
...
Using deploymentMonitor only for logging events and not making any de…
2016-12-15 14:50:02 -08:00
Peter Jausovec
e07c6af3f7
only check for deployment succeeded event
2016-12-15 14:42:01 -08:00
Peter Jausovec
518c7a4e89
Using deploymentMonitor only for logging events and not making any decisions whether deployment failed or succeeded.
2016-12-15 12:05:38 -08:00
Peter Jausovec
fa2321f388
Merge pull request #25 from Microsoft/peterj/handle_killing_finished_events
...
Handle TASK_KILLING and TASK_FINISHED events.
2016-12-14 15:14:23 -08:00
Peter Jausovec
80860844b1
Handle TASK_KILLING and TASK_FINISHED events.
2016-12-14 15:14:07 -08:00
Peter Jausovec
cb208523e9
Merge pull request #24 from Microsoft/peterj/update_sseclient_version
...
update sseclient to 0.0.14
2016-12-14 14:22:33 -08:00
Peter Jausovec
45f6698d71
update sseclient to 0.0.14
2016-12-14 14:22:20 -08:00
Peter Jausovec
0fad6da526
Merge pull request #22 from Microsoft/peterj/get_logs_on_failure
...
Showing service statuses as they change during deployment. Logging th…
2016-12-14 13:40:35 -08:00
Peter Jausovec
9d047038ac
Update a comment
2016-12-14 13:40:20 -08:00
Peter Jausovec
7634dd8a95
Removing .stop()
2016-12-14 09:42:47 -08:00
Peter Jausovec
87e700e378
replacing threading.Event with a boolean
2016-12-14 09:33:13 -08:00
Peter Jausovec
22b2185f58
Addressing PR feedback
2016-12-13 14:47:35 -08:00
Peter Jausovec
a51d3752d9
Logging deployment_failed event. In case deployment completes (deploymentId is not in the deployments response anymore), we give the deployment monitor another second to catch up on the events, before we call the deployment completed. If we don't get the success/fail event we exit the loop. We check for fail/success from the monitor and if there's none, we check if deployment failed. if we can't determine that either, we say that the deployment ended.
2016-12-13 12:00:51 -08:00
Peter Jausovec
67fd963cf8
Changing to code so for Marathon we make the ssh tunnel to port 8080, while for Mesos we make it to port 80. Removing code that uses deploymentId to determine if deployment is completed and relying on events now. Updating unit tests to reflect the changes.
2016-12-13 09:21:53 -08:00
Peter Jausovec
deb5ec9b70
adding get_stream method
2016-12-12 15:53:12 -08:00
Peter Jausovec
7ec5a87841
Updating the DeploymentMonitor to take deploymentId to correctly check if deployment has succeeded. Adding DeploymentMonitor unit tests.
2016-12-08 17:03:55 -08:00
Peter Jausovec
d4b5a51da0
Update Mesos unit tests and add new unit tests for MarathonEvent
2016-12-08 16:10:25 -08:00
Peter Jausovec
cfedae8e7f
Using Marathon /events endpoint to get the deployment events and log them.
2016-12-08 15:56:06 -08:00
Stephen Provine
0d2ed35eb0
Better report docker command errors.
...
This ensures that if a docker command (either docker or docker-compose) fails, the contents of stderr is reported as a VSTS error so that it shows up in the build/release summary.
Also provide better error messages when no Dockerfile or docker-compose.yml files are found with the provided glob patterns.
Finally, fix an issue with docker-compose run action where it doesn't do docker-compose down if the run failed.
2016-12-08 11:30:06 -08:00
Peter Jausovec
0eae8a82c5
Downloading the file instead of just reading it and adding more unit tests.
2016-12-07 10:38:00 -08:00
Peter Jausovec
917f17e1c6
Fix the import name and removed func
2016-12-07 08:54:48 -08:00
Peter Jausovec
2b33770ca6
Showing service statuses as they change during deployment. Logging the stdout and stderr in case any task fails during deployment
2016-12-06 16:15:09 -08:00
Stephen Provine
b54cf69b42
Add "Lock service images" action.
...
This combines the "Write service image digests" and "Combine configuration" actions into a single, clearer action.
2016-12-06 13:34:32 -08:00
Peter Jausovec
660af9dd64
Merge pull request #20 from Microsoft/peterj/acs_client_tests
...
Add tests for ACSClient
2016-11-21 08:40:16 -08:00
Peter Jausovec
6292dab65d
Add tests for ACSClient
2016-11-16 16:11:17 -08:00
Stephen Provine
190aa1ced9
Fix nullref in Docker Compose build action.
...
Occurs when there are no additional image tags specified.
Also bulletproof against similar issue in Docker Compose "write service image digests" action in case there are no services present.
2016-11-16 13:36:41 -08:00
Stephen Provine
71bb69290a
Bump version numbers.
2016-11-15 22:23:02 -08:00
Stephen Provine
4d2ea19f86
Update docker.md and associated images.
2016-11-15 22:20:39 -08:00
Peter Jausovec
4548fe13ee
Adding user to the parameters. The 'user' setting at the root of marathon.json means the user that runs the tasks.
2016-11-15 20:10:36 -08:00
Stephen Provine
cb8995ad81
Minor fixes to external-nginx-lb app.
...
- Ensure HTTP Host is passed unchanged.
- Fix upgrade strategy to match marathon-lb defaults.
2016-11-14 11:56:07 -08:00
Stephen Provine
6356b2dcf1
Bump version numbers.
2016-11-10 17:24:34 -08:00
Stephen Provine
4235dc277b
Merge branch 'master' of https://github.com/microsoft/vsts-docker
2016-11-10 17:17:42 -08:00
Peter Jausovec
55f75909cc
Merge pull request #18 from Microsoft/peterj/external_vhost_port_80
...
External VHOST port should always be 80. The port specified in the la…
2016-11-10 17:10:59 -08:00
Peter Jausovec
41fbc58073
External VHOST port should always be 80. The port specified in the label is the container port under which the VIP ends up.
2016-11-10 17:00:16 -08:00
Peter Jausovec
b6ded21e9f
Merge pull request #17 from Microsoft/peterj/fix_exceptions
...
Fixing exceptions and cleanup
2016-11-10 16:54:17 -08:00
Peter Jausovec
95ddab7b5a
Only run cleanup if there was an exception
2016-11-10 16:53:53 -08:00
Peter Jausovec
e8f13112e2
Simplify - setting a boolean value for cleanup that we check before removing anything
2016-11-10 16:38:28 -08:00
Peter Jausovec
176edcec37
Fixing exceptions and cleanup
...
Bringing back the with statement as we need the __exit__ method to run and cleanup (if needed) and shutdown the ACS tunnel. While fixing this I found a couple of more issues (e.g. we were running cleanup if app already existed) and we weren't outputting anything because the logging was disabled in ACS client.
2016-11-10 14:40:21 -08:00
Stephen Provine
f408c8da80
Merge branch 'master' of https://github.com/microsoft/vsts-docker
2016-11-10 11:50:31 -08:00
Stephen Provine
f54ab01c83
SSH mode was not working.
2016-11-10 11:47:39 -08:00
Peter Jausovec
421798cf66
Merge pull request #16 from Microsoft/peterj/add_pytests_to_gulp
...
Add python tests to gulp
2016-11-10 08:50:00 -08:00
Peter Jausovec
3f89e099f8
Adding script to run Python tests and updating Gulp task to run them on 'gulp test'. Updating .gitignore to include .pyc and .coverage files
2016-11-09 16:47:59 -08:00
Stephen Provine
2e0c8e8cbb
Merge pull request #15 from Microsoft/deploy-task
...
Add deploy task, upgrade to version 0.3.0.
2016-11-09 12:06:08 -08:00
Stephen Provine
2ade8e9c30
Update Python scripts.
2016-11-09 11:42:55 -08:00
Stephen Provine
bb313a47ad
Sync the Python scripts one last time.
2016-11-08 15:46:11 -08:00
Stephen Provine
8d359b79c0
Update to latest Python deploy scripts.
2016-11-07 22:36:57 -08:00
Stephen Provine
e68980d3dd
Minor update to docker compose action names.
2016-11-07 16:22:55 -08:00
Stephen Provine
6aa5441bc3
Fix merge from master.
2016-11-06 08:09:25 -08:00
Stephen Provine
4560d1537e
Merge from master.
2016-11-06 08:08:57 -08:00