----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12223/ -----------------------------------------------------------
(Updated July 3, 2013, 3:58 p.m.) Review request for cloudstack, Marcus Sorensen and Murali Reddy. Changes ------- Changes requested added. The "event type" is now the async job's action, not the underlying job (moved to the even description). Bugs: CLOUDSTACK-3190 Repository: cloudstack-git Description ------- Updated AsyncJobManagerImpl to publish async job events when async jobs are created, updated and completed. I am currently stashing the command event description in the commandInfo structure, and then pulling it back out as needed. I could switch this to make a database change, but that seemed like a more invasive change. I have further diffs to clean up ActionEvent and AlertEvent as well. Diffs (updated) ----- api/src/com/cloud/event/EventCategory.java cee6529 server/src/com/cloud/api/ApiServer.java 0cd1d61 server/src/com/cloud/async/AsyncJobManagerImpl.java 0101a8a Diff: https://reviews.apache.org/r/12223/diff/ Testing ------- Manual testing only at this point. I am more than willing to write a python test using marvin, but I'm unsure if marvin has rabbitmq library support or not yet. Please advise. Thanks, Ryan Dietrich