GitHub PushEvent at 2014-04-04 20:59:48
Repository: bosh
Actor: rboshman
Payload: {"shas":[["92486c5d95c220e76522ea61ef12233c15b167fa","00100a4b0b14be5065514816c9e64ea19fb9d9cd@gmail.com","[director] Track failed job updates in event log\n\nThis commit fixes a bug that prevented failed instance updates to be\nrecorded in the event log. Instead of rescuing the error (so event_log\ndoesn't notice there has been an error and it will record a success update),\nnow we log the error and raise the exception. In this way, the track method\nin event_log will trap the error and it will record it in the event log file.\n\nAlso, instead of using a custom logic to check if the job update should be halted\n(due to an error, canary or not), use the standard ThreadPool mechanism: if it\ndetects an exception, it will stop processing new actions.\n\nRefactored also job_updater spec in order to detect all situatuibs and to check\nthe event log.\n\nSigned-off-by: Adam Stegman <astegman@pivotallabs.com>","Ferran Rodenas",true],["d6236f1839ec39049dcb1f6f97fad6499e4661d8","a39f51a90bfcf8ca23756ef12127e1cf20796cba@pivotallabs.com","Refactor job_updater_spec\n\nSigned-off-by: Adam Stegman <astegman@pivotallabs.com>","Maria Shaldibina",true],["054eb32b20a62ef70a2eadffc45689d8795d4b8c","a39f51a90bfcf8ca23756ef12127e1cf20796cba@pivotallabs.com","Do not assume current repo name, use relative path in tests\n\nSigned-off-by: Adam Stegman <astegman@pivotallabs.com>","Maria Shaldibina",true],["fdf40dbe45bdf0c919ac321b7043a0c95a594e8d","a39f51a90bfcf8ca23756ef12127e1cf20796cba@pivotallabs.com","Add an integration test for a failed job update\n\n* Integration tests use dummy-nats-job-supervisor which controls the state\nof the job\n\nSigned-off-by: Adam Stegman <astegman@pivotallabs.com>","Maria Shaldibina",true]],"size":4,"ref":"refs/heads/develop","head":"fdf40dbe45bdf0c919ac321b7043a0c95a594e8d"}