Hi All, Building 4.2 is failing, Posted the possible fix for the issue. Thanks Rajesh Battala
-----Original Message----- From: Rajesh Battala [mailto:nore...@reviews.apache.org] On Behalf Of Rajesh Battala Sent: Monday, August 19, 2013 11:56 AM To: Sateesh Chodapuneedi; Edison Su; Devdeep Singh Cc: Rajesh Battala; cloudstack Subject: Review Request 13641: Fix for build failure in 4.2 ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13641/ ----------------------------------------------------------- Review request for cloudstack, Devdeep Singh, edison su, and Sateesh Chodapuneedi. Bugs: CLOUDSTACK-4391 Repository: cloudstack-git Description ------- 4.2 build is failing as there is an issue with the Unittest for LibvirtComputingResourceTest with recent commit to LibvirtComputingResouce class. Diffs ----- plugins/hypervisors/kvm/test/com/cloud/hypervisor/kvm/resource/LibvirtComputingResourceTest.java 56d6536 Diff: https://reviews.apache.org/r/13641/diff/ Testing ------- Created a possible fix for the issue Applying the patch, KVM unittess are running successful. KVM tests are passing successfully. ------------------------------------------------------- T E S T S ------------------------------------------------------- Running com.cloud.hypervisor.kvm.resource.LibvirtVifDriverTest log4j:WARN No appenders could be found for logger (com.cloud.hypervisor.kvm.resource.LibvirtComputingResource). log4j:WARN Please initialize the log4j system properly. log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info. Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.034 sec Running com.cloud.hypervisor.kvm.resource.LibvirtComputingResourceTest Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.058 sec Running com.cloud.hypervisor.kvm.resource.LibvirtVMDefTest Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.018 sec Results : Tests run: 13, Failures: 0, Errors: 0, Skipped: 0 Thanks Rajesh Battala Thanks, Rajesh Battala