[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] iotests: update test owner contact information
From: |
Stefan Hajnoczi |
Subject: |
Re: [PATCH] iotests: update test owner contact information |
Date: |
Thu, 24 Mar 2022 10:36:33 +0000 |
On Wed, Mar 23, 2022 at 01:10:19PM +0100, Hanna Reitz wrote:
> On 23.03.22 09:39, Thomas Huth wrote:
> > On 22/03/2022 18.42, John Snow wrote:
> > > Quite a few of these tests have stale contact information. This patch
> > > updates the stale ones that I happen to be aware of at the moment.
> > >
> > > Signed-off-by: John Snow <jsnow@redhat.com>
> > > ---
> > > tests/qemu-iotests/025 | 2 +-
> > > tests/qemu-iotests/027 | 2 +-
> > > tests/qemu-iotests/028 | 2 +-
> > > tests/qemu-iotests/036 | 2 +-
> > > tests/qemu-iotests/039 | 2 +-
> > > tests/qemu-iotests/059 | 2 +-
> > > tests/qemu-iotests/060 | 2 +-
> > > tests/qemu-iotests/061 | 2 +-
> > > tests/qemu-iotests/062 | 2 +-
> > > tests/qemu-iotests/064 | 2 +-
> > > tests/qemu-iotests/066 | 2 +-
> > > tests/qemu-iotests/068 | 2 +-
> > > tests/qemu-iotests/069 | 2 +-
> > > tests/qemu-iotests/070 | 2 +-
> > > tests/qemu-iotests/071 | 2 +-
> > > tests/qemu-iotests/072 | 2 +-
> > > tests/qemu-iotests/074 | 2 +-
> > > tests/qemu-iotests/084 | 2 +-
> > > tests/qemu-iotests/085 | 2 +-
> > > tests/qemu-iotests/089 | 2 +-
> > > tests/qemu-iotests/090 | 2 +-
> > > tests/qemu-iotests/091 | 2 +-
> > > tests/qemu-iotests/094 | 2 +-
> > > tests/qemu-iotests/095 | 2 +-
> > > tests/qemu-iotests/097 | 2 +-
> > > tests/qemu-iotests/098 | 2 +-
> > > tests/qemu-iotests/099 | 2 +-
> > > tests/qemu-iotests/102 | 2 +-
> > > tests/qemu-iotests/103 | 2 +-
> > > tests/qemu-iotests/105 | 2 +-
> > > tests/qemu-iotests/106 | 2 +-
> > > tests/qemu-iotests/107 | 2 +-
> > > tests/qemu-iotests/108 | 2 +-
> > > tests/qemu-iotests/110 | 2 +-
> > > tests/qemu-iotests/111 | 2 +-
> > > tests/qemu-iotests/112 | 2 +-
> > > tests/qemu-iotests/113 | 2 +-
> > > tests/qemu-iotests/115 | 2 +-
> > > tests/qemu-iotests/117 | 2 +-
> > > tests/qemu-iotests/119 | 2 +-
> > > tests/qemu-iotests/120 | 2 +-
> > > tests/qemu-iotests/121 | 2 +-
> > > tests/qemu-iotests/123 | 2 +-
> > > tests/qemu-iotests/125 | 2 +-
> > > tests/qemu-iotests/126 | 2 +-
> > > tests/qemu-iotests/127 | 2 +-
> > > tests/qemu-iotests/135 | 2 +-
> > > tests/qemu-iotests/138 | 2 +-
> > > tests/qemu-iotests/140 | 2 +-
> > > tests/qemu-iotests/141 | 2 +-
> > > tests/qemu-iotests/143 | 2 +-
> > > tests/qemu-iotests/144 | 2 +-
> > > tests/qemu-iotests/146 | 2 +-
> > > tests/qemu-iotests/150 | 2 +-
> > > tests/qemu-iotests/153 | 2 +-
> > > tests/qemu-iotests/156 | 2 +-
> > > tests/qemu-iotests/162 | 2 +-
> > > tests/qemu-iotests/173 | 2 +-
> > > tests/qemu-iotests/176 | 2 +-
> > > tests/qemu-iotests/182 | 2 +-
> > > tests/qemu-iotests/192 | 2 +-
> > > tests/qemu-iotests/200 | 2 +-
> > > tests/qemu-iotests/216 | 2 +-
> > > tests/qemu-iotests/218 | 2 +-
> > > tests/qemu-iotests/224 | 2 +-
> > > tests/qemu-iotests/225 | 2 +-
> > > tests/qemu-iotests/228 | 2 +-
> > > tests/qemu-iotests/229 | 2 +-
> > > tests/qemu-iotests/231 | 2 +-
> > > tests/qemu-iotests/250 | 2 +-
> > > tests/qemu-iotests/251 | 2 +-
> > > tests/qemu-iotests/252 | 2 +-
> > > tests/qemu-iotests/258 | 2 +-
> > > tests/qemu-iotests/259 | 2 +-
> > > tests/qemu-iotests/261 | 2 +-
> > > tests/qemu-iotests/310 | 2 +-
> > > 76 files changed, 76 insertions(+), 76 deletions(-)
> > >
> > > diff --git a/tests/qemu-iotests/025 b/tests/qemu-iotests/025
> > > index 80686a30d5..5771ea9200 100755
> > > --- a/tests/qemu-iotests/025
> > > +++ b/tests/qemu-iotests/025
> > > @@ -20,7 +20,7 @@
> > > #
> > > # creator
> > > -owner=stefanha@linux.vnet.ibm.com
> > > +owner=stefanha@redhat.com
> >
> > Wow, these were really old ones ... I wonder whether these "owner" lines
> > really still make that much sense if they are neglected that much, or
> > whether the information should maybe rather be captured in MAINTAINERS
> > instead?
>
> Or maybe we should just drop the whole concept of ownership in the iotests
> altogether, I can’t remember it ever coming up. If a test fails, it never
> mattered to me who the “owner” is, I just did my best to fix it myself,
> usually. If I couldn’t, I used git-blame to figure out who to ask, because
> tests tend to be written by multiple people anyway.
>
> Anyway, that’d be more difficult, I suppose, because dropping ownership
> information would (I guess) require consent from everyone, so this is
> simpler for now.
>
> Thanks for the patch, I’ve applied it to my block branch:
>
> https://gitlab.com/hreitz/qemu/-/commits/block
For the record:
I'm happy fro my @linux.vnet.ibm.com email address to be removed or
replaced since the address is no longer in use. IBM copyright needs to
be kept.
Thanks,
Stefan
signature.asc
Description: PGP signature
Re: [PATCH] iotests: update test owner contact information, Eric Blake, 2022/03/25