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