[dpdk-ci] [PATCH v4] add script to decide best tree match for patches

Jeremy Plsek jplsek at iol.unh.edu
Fri Apr 19 18:21:09 CEST 2019


We've switched to using it a few days ago.
Some examples of it switching branches:
https://lab.dpdk.org/results/dashboard/patchsets/5503/
https://lab.dpdk.org/results/dashboard/patchsets/5499/
https://lab.dpdk.org/results/dashboard/patchsets/5498/

On Thu, Apr 18, 2019 at 4:16 AM Ali Alnubani <alialnu at mellanox.com> wrote:
>
> Hi,
>
> > -----Original Message-----
> > From: ci <ci-bounces at dpdk.org> On Behalf Of Ali Alnubani
> > Sent: Tuesday, April 9, 2019 7:13 PM
> > To: ci at dpdk.org
> > Cc: Thomas Monjalon <thomas at monjalon.net>; ferruh.yigit at intel.com;
> > jplsek at iol.unh.edu; Ori Kam <orika at mellanox.com>
> > Subject: [dpdk-ci] [PATCH v4] add script to decide best tree match for
> > patches
> >
> > The information provided in the MAINTAINERS file is used to find the tree
> > that matches the files changed in patches as follows:
> >  - For each file, it first tries to find a matching unix shell-style
> >    pattern. If one was found, it looks for a tree specified in the
> >    subsection containing the pattern, and that tree is returned.
> >    If no tree was found in that subsection, the script tries to
> >    find a tree specified under the name of the section containing
> >    the pattern.
> >  - If more than one tree was matched (for multiple files),
> >    a tree matching the common prefix of all these trees will be
> >    returned.
> >  - The main repo 'dpdk' will be returned if no other match was found.
> >
> > Results can be further improved by adding more information to the
> > MAINTAINERS file or by using the subject of the series/patch for matching
> > too.
> >
> > Bugzilla ID: 166
> >
> > Suggested-by: Thomas Monjalon <thomas at monjalon.net>
> > Signed-off-by: Ali Alnubani <alialnu at mellanox.com>
> > Signed-off-by: Ori Kam <orika at mellanox.com>
> > ---
> > Changes in v2:
> Typo.
> s/v2/v4/
> >   - Refactored the script to use classes, which makes it easier
> >     to be used inside other scripts.
> >   - Renamed the script so that it can be imported.
> >   - The script will always return a single tree.
> >     If multiple trees were matched (for multiple files),
> >     a tree matching the common prefix of all these trees will be
> >     returned.
> >   - Main repo is now reported if no other matches were found.
> >   - Only tree name will be returned without the full url.
> >   - Updated description and usage info.
> >
>
> Did you guys get a chance to test this version?
>
> Thanks,
> Ali



-- 
Jeremy Plsek
UNH InterOperability Laboratory


More information about the ci mailing list