Re: [kvm-unit-tests PULL 00/11] s390x and generic script updates

[Thomas Huth <thuth@redhat.com>]

On 29/09/2020 10.49, Janosch Frank wrote:
> On 9/29/20 10:38 AM, Paolo Bonzini wrote:
>> On 28/09/20 19:49, Thomas Huth wrote:
>>>  Hi Paolo,
>>>
>>> the following changes since commit 58c94d57a51a6927a68e3f09627b2d85e3404c0f:
>>>
>>>   travis.yml: Use TRAVIS_BUILD_DIR to refer to the top directory (2020-09-25 10:00:36 +0200)
>>>
>>> are available in the Git repository at:
>>>
>>>   https://gitlab.com/huth/kvm-unit-tests.git tags/pull-request-2020-09-28
>>>
>>> for you to fetch changes up to b508e1147055255ecce93a95916363bda8c8f299:
>>>
>>>   scripts/arch-run: use ncat rather than nc. (2020-09-28 15:03:50 +0200)
>>>
>>> ----------------------------------------------------------------
>>> - s390x protected VM support
>>> - Some other small s390x improvements
>>> - Generic improvements in the scripts (better TAP13 names, nc -> ncat, ...)
>>> ----------------------------------------------------------------
>>>
>>> Jamie Iles (1):
>>>       scripts/arch-run: use ncat rather than nc.
>>>
>>> Marc Hartmayer (6):
>>>       runtime.bash: remove outdated comment
>>>       Use same test names in the default and the TAP13 output format
>>>       common.bash: run `cmd` only if a test case was found
>>>       scripts: add support for architecture dependent functions
>>>       run_tests/mkstandalone: add arch_cmd hook
>>>       s390x: add Protected VM support
>>>
>>> Thomas Huth (4):
>>>       configure: Add a check for the bash version
>>>       travis.yml: Update from Bionic to Focal
>>>       travis.yml: Update the list of s390x tests
>>>       s390x/selftest: Fix constraint of inline assembly
>>>
>>>  .travis.yml             |  7 ++++---
>>>  README.md               |  3 ++-
>>>  configure               | 14 ++++++++++++++
>>>  run_tests.sh            | 18 +++++++++---------
>>>  s390x/Makefile          | 15 ++++++++++++++-
>>>  s390x/selftest.c        |  2 +-
>>>  s390x/selftest.parmfile |  1 +
>>>  s390x/unittests.cfg     |  1 +
>>>  scripts/arch-run.bash   |  6 +++---
>>>  scripts/common.bash     | 21 +++++++++++++++++++--
>>>  scripts/mkstandalone.sh |  4 ----
>>>  scripts/runtime.bash    |  9 +++------
>>>  scripts/s390x/func.bash | 35 +++++++++++++++++++++++++++++++++++
>>>  13 files changed, 106 insertions(+), 30 deletions(-)
>>>  create mode 100644 s390x/selftest.parmfile
>>>  create mode 100644 scripts/s390x/func.bash
>>>
>>
>> Pulled, thanks (for now to my clone; waiting for CI to complete).
>> Should we switch to Gitlab merge requests for pull requests only (i.e.
>> patches still go on the mailing list)?
>>
>> Paolo
>>
> 
> Hrm, that would force everyone to use Gitlab and I see some value in
> having pull request mails on the lists. You just opened the Pandora's
> box of discussions :-)
> 
> If it's easier for you I'd be open to open a marge request and send out
> pull mails at the same time so people can comment without login to Gitlab.

... or maybe the people who already have a gitlab account could simply
include the URL to their CI run in their pull request cover letter...?

 Thomas