Re: Maintaining patchtest


Andreas Müller
 

On Wed, Jan 8, 2020 at 1:59 PM Alexander Kanavin <alex.kanavin@...> wrote:

Thanks Paul, maintenance work is always greatly appreciated.

Alex

On Wed, 8 Jan 2020 at 13:49, Paul Barker <pbarker@...> wrote:

Hi all,

After discussion on the technical call yesterday I'm looking at
becoming the maintainer of patchtest
(http://git.yoctoproject.org/cgit/cgit.cgi/patchtest/) and
patchtest-oe (http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe/)
for Yocto Project.

Aside from a few minor issues I think patchtest is working pretty well
at the moment so I don't want to rock the boat too much. However I do
want to extend patchtest in a couple of ways:

* Support testing commits on a branch as well as patches in a
directory. This will allow integration with merge request workflows as
well as patches-by-email workflows.

* Support testing patches to other layers as well as oe-core. This
will require splitting the test cases into a "common" set which apply
to all layers (e.g. "Signed-off-by" must be present in a patch) and an
"oe-core specific" set (e.g. bugzilla reference validation which may
not be relevant for third-party layers). Other layers will then be
able to include their own test cases.

* Make it easier to run patchtest locally before submitting patches to the list.

I also want to simplify future maintenance of patchtest in a couple of ways:

* Move existing test cases from patchtest-oe into oe-core itself.
Archive off the old patchtest-oe repository.

* Consider if patchtest itself can be moved into oe-core as a script.
This is desirable but there would be a few issues to overcome, I need
to look into how patchtest is actually invoked by patchwork on the
server side first.

* If patchtest is to remain a separate repository then I'll move
development to GitLab with git.yoctoproject.org becoming a mirror.
This will allow automation of a few things like the deployment of new
patchtest versions to the server after committing to the master
branch.

I've also still got my eye on other ways we can support merge request
workflows for layers that want to use this but December got busy.

I'll give this a week for people to reply with feedback and then
submit patches to update the maintainers files in patchtest &
patchtest-oe.

Thanks,
Paul
Suggestions made sound like major enhancements to me. My favourites
are: extend to other layers and local operation :)

Thanks for taking care

Andreas

Join yocto@lists.yoctoproject.org to automatically receive all group messages.