[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