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? 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 >
