-----Original Message----- From: Hugo Trippaers [mailto:trip...@gmail.com] Sent: Friday, 21 February 2014 1:15 AM To: dev@cloudstack.apache.org Cc: dev@cloudstack.apache.org; Hugo Trippaers; Ian Southam Subject: Re: Master blocker CLOUDSTACK-6130
Sent from my iPhone > On 20 feb. 2014, at 18:31, Srikanteswararao Talluri > <srikanteswararao.tall...@citrix.com> wrote: > > > >> On 20/02/14 7:14 pm, "Hugo Trippaers" <h...@trippaers.nl> wrote: >> >> Hey, >> >> Should be fixed by: >> commit ba7ff5369cc602c7e0c8d788b6617f27123c2328 >> Author: Hugo Trippaers <htrippa...@schubergphilis.com> >> Date: Thu Feb 20 14:04:51 2014 +0100 >> >> Fix parameter index in getCount function. >> >> >> @Talluri, Agreed. However with the current way of testing it is >> really hard to see which commit actually broke something. We should >> be able to tell which commits changed something when a particular >> test fails. Maybe we could start by supplying the list of changes >> between one run of the test suite and the next. That would at least >> narrow the number of changes developers have to look at to determine >> which change broke what,much like the current maven build. > > [Talluri] This can be done Hugo. I am working towards sending one > digest mail everyday to dev list on automated test runs. Any thoughts > or disagreements?? Sounds good to me, can you automate that? [Talluri] Sure, I am working on this. Cheers, Hugo > >> >> Cheers, >> >> Hugo >> >> >> On 20 feb. 2014, at 07:39, Srikanteswararao Talluri >> <srikanteswararao.tall...@citrix.com> wrote: >> >>> This is found by daily BVT run and sent out in daily BVT reporting mail. >>> It would be great if folks could actually pick up the bugs resulted >>> from BVT failures immediately. >>> >>> Thanks, >>> ~Talluri >>> >>>> On 20/02/14 10:20 am, "Sheng Yang" <sh...@yasker.org> wrote: >>>> >>>> Hi Hugo/Ian, >>>> >>>> I suspect https://issues.apache.org/jira/browse/CLOUDSTACK-6130 is >>>> caused by some of recent change of findbugs. It's likely got >>>> something wrong with GeneralDaoBase, and the most recently >>>> modifications are: >>>> >>>> commit 443acac92792cbffff2654c3414df276efac404f >>>> Author: Hugo Trippaers <htrippa...@schubergphilis.com> >>>> Date: Fri Feb 14 18:36:18 2014 +0100 >>>> >>>> Fix functional issue introduced by some findbug changes. >>>> >>>> commit 5a6ad11fa2ca3de64b94095c628bd3ba12e618c3 >>>> Author: Ian Southam <isout...@schubergphilis.com> >>>> Date: Wed Feb 12 17:08:52 2014 +0100 >>>> >>>> findbugs findings >>>> >>>> Signed-off-by: Hugo Trippaers <htrippa...@schubergphilis.com> >>>> >>>> From the error message, it shows statement as: >>>> >>>> SELECT COUNT(*) FROM user_ip_address INNER JOIN account ON >>>> user_ip_address.account_id=account.id INNER JOIN vlan ON >>>> user_ip_address.vlan_db_id=vlan.id WHERE >>>> user_ip_address.account_id=5 AND user_ip_address.allocated IS NOT >>>> NULL AND (account.type != 'VirtualNetwork' >>>> ) AND (vlan.vlan_type = ** NOT SPECIFIED ** ) >>>> >>>> And you can observed that some parameters have been shifted left by >>>> one position, because vlan.vlan_type should compare to >>>> "virtualNetwork". It should related to "i++"/"++i" change in the commit. >>>> >>>> After reverting these two commits, everything is fine. >>>> >>>> Could you take a look? >>>> >>>> --Sheng >