----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18067/ -----------------------------------------------------------
Review request for cloudstack, daan Hoogland and Hugo Trippaers. Repository: cloudstack-git Description ------- Used FindBugs to find some possible scary bugs and wrote some unit tests for them. Diffs ----- plugins/hypervisors/xen/src/com/cloud/ha/XenServerFencer.java 5604db8 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java cf5c6d6 plugins/hypervisors/xen/test/com/cloud/ha/XenServerFencerTest.java PRE-CREATION plugins/hypervisors/xen/test/com/cloud/hypervisor/xen/resource/CitrixResourceBaseTest.java c56cc6d Diff: https://reviews.apache.org/r/18067/diff/ Testing ------- Refactored some code and wrote unit tests Thanks, Sander Botman