Re: Maintaining patchtest


Armin Kuster
 

Paul,

Thanks for stepping up as the Maintainer.

- Armin

On 1/8/20 4:48 AM, Paul Barker 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

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#47886): https://lists.yoctoproject.org/g/yocto/message/47886
Mute This Topic: https://lists.yoctoproject.org/mt/69526934/1024635
Group Owner: yocto+owner@...
Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub  [akuster@...]
-=-=-=-=-=-=-=-=-=-=-=-

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