[dpdk-ci] script to determine target repo (was DPDK Lab)

Xu, Qian Q qian.q.xu at intel.com
Fri Jun 23 10:43:44 CEST 2017


Thomas/Bruce
1. For determining the repo tree to target, I don't believe that we can ever
> come up with a 100% accurate rule, as the tree to which a set is to be applied
> can be difficult to determine, so it may be done on the basis of on-list discussion.
> A 90% accurate rule it what we may have to accept.

-- Then if we find the performance issue, then maybe it's a false alarm due to apply to the wrong repo. So, we may face many false alarms according with the time. 
Then people may not treat the performance issue as a problem, so I still think we need to try 100% accurate to have a more trustable result when we send out the alarm. 


> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas at monjalon.net]
> Sent: Wednesday, June 21, 2017 4:33 PM
> To: Wei, FangfangX <fangfangx.wei at intel.com>
> Cc: ci at dpdk.org; Richardson, Bruce <bruce.richardson at intel.com>; Xu, Qian Q
> <qian.q.xu at intel.com>; O'Driscoll, Tim <tim.odriscoll at intel.com>; Eugene
> Voronov <eugene at mellanox.com>
> Subject: Re: [dpdk-ci] script to determine target repo (was DPDK Lab)
> 
> I agree with Bruce.
> 
> Sorry for not having written the scripts yet.
> Someone else in Mellanox should do it in July.
> In the meantime, do not hesitate to share your code if it speed up things.
> 
> Thanks a lot
> 
> 
> 21/06/2017 10:20, Richardson, Bruce:
> > Hi Fangfang,
> >
> > My comments on the questions you asked:
> >
> > 1. For determining the repo tree to target, I don't believe that we can ever
> come up with a 100% accurate rule, as the tree to which a set is to be applied
> can be difficult to determine, so it may be done on the basis of on-list discussion.
> A 90% accurate rule it what we may have to accept. However, since applying a
> patchset to a tree should not be a time-consuming operation, I suggest any
> script produce a list of possible trees in priority order to try. If not net, then try
> main, etc. etc.
> >
> > 2. Using the order from patch titles is correct.
> >
> > /Bruce


More information about the ci mailing list