On 16.07.19 17:51, Thomas Huth wrote: > On 16/07/2019 17.41, Max Reitz wrote: >> On 16.07.19 14:28, Thomas Huth wrote: >>> Since most iotests are now run during "make check" already, we do not >>> need to test them explicitly from the gitlab-ci.yml script anymore. >>> And while we're at it, add some of the new non-auto tests >= 248 instead. >>> >>> Signed-off-by: Thomas Huth <[email protected]> >>> --- >>> .gitlab-ci.yml | 12 +++--------- >>> 1 file changed, 3 insertions(+), 9 deletions(-) >>> >>> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml >>> index c63bf2f822..fa5d094453 100644 >>> --- a/.gitlab-ci.yml >>> +++ b/.gitlab-ci.yml >>> @@ -45,15 +45,9 @@ build-tcg-disabled: >>> - ./check -raw 001 002 003 004 005 008 009 010 011 012 021 025 032 033 048 >>> 052 063 077 086 101 104 106 113 147 148 150 151 152 157 159 160 >>> 163 170 171 183 184 192 194 197 205 208 215 221 222 226 227 236 >>> - - ./check -qcow2 001 002 003 004 005 007 008 009 010 011 012 013 017 018 >>> 019 >>> - 020 021 022 024 025 027 028 029 031 032 033 034 035 036 037 038 >>> - 039 040 042 043 046 047 048 049 050 051 052 053 054 056 057 058 >>> - 060 061 062 063 065 066 067 068 069 071 072 073 074 079 080 082 >>> - 085 086 089 090 091 095 096 097 098 099 102 103 104 105 107 108 >>> - 110 111 114 117 120 122 124 126 127 129 130 132 133 134 137 138 >>> - 139 140 141 142 143 144 145 147 150 151 152 154 155 156 157 158 >>> - 161 165 170 172 174 176 177 179 184 186 187 190 192 194 195 196 >>> - 197 200 202 203 205 208 209 214 215 216 217 218 222 226 227 >>> 229 234 >>> + - ./check -qcow2 028 040 051 056 057 058 065 067 068 082 085 091 095 096 >>> 102 >>> + 124 127 129 132 139 142 144 145 147 151 152 155 157 165 194 196 >>> + 200 202 203 205 208 209 216 218 222 227 234 248 250 254 255 256 >> >> This removes 197 and 215. Why? > > As mentioned in the cover letter, I've seen problems with 197 and 215 in > the gitlab CI pipelines (since the two tests apparently cause a lot of > memory pressure). At least they were causing trouble when "make check" > was running with other tests in parallel. Maybe they still work fine > when they run alone here. I'll give it another try...
OK. Shouldn’t this be mentioned in the commit message here, too? ;-) Reviewed-by: Max Reitz <[email protected]>
signature.asc
Description: OpenPGP digital signature
