mbox

[PULL,0/4] qtests and gitlab-CI

Message ID 20200117110758.1995-1-thuth@redhat.com (mailing list archive)
State New, archived
Headers show

Pull-request

https://gitlab.com/huth/qemu.git tags/pull-request-2020-01-17

Message

Thomas Huth Jan. 17, 2020, 11:07 a.m. UTC
Hi Peter,

the following changes since commit fee0ec1fd11a6fb960517e18201ed8a686a0d7e8:

  Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' into staging (2020-01-14 14:11:34 +0000)

are available in the Git repository at:

  https://gitlab.com/huth/qemu.git tags/pull-request-2020-01-17

for you to fetch changes up to c82b95489f8c33318101d18e643f7b50259d8e4e:

  gitlab-ci.yml: Run tcg test with tci (2020-01-16 17:01:39 +0100)

----------------------------------------------------------------
* Various fixes for qtests
* Enable TCG tests with TCI in the gitlab CI
----------------------------------------------------------------

Laurent Vivier (1):
      migration-test: ppc64: fix FORTH test program

Shameer Kolothum (1):
      tests: acpi: update path in rebuild-expected-aml

Thomas Huth (2):
      tests/qtest/vhost-user-test: Fix memory leaks
      gitlab-ci.yml: Run tcg test with tci

 .gitlab-ci.yml                          |  1 +
 tests/data/acpi/rebuild-expected-aml.sh |  6 +++---
 tests/qtest/migration-test.c            |  4 ++--
 tests/qtest/vhost-user-test.c           | 11 ++++++++---
 4 files changed, 14 insertions(+), 8 deletions(-)

Comments

no-reply@patchew.org Jan. 17, 2020, 4:46 p.m. UTC | #1
Patchew URL: https://patchew.org/QEMU/20200117110758.1995-1-thuth@redhat.com/



Hi,

This series seems to have some coding style problems. See output below for
more information:

Message-id: 20200117110758.1995-1-thuth@redhat.com
Type: series
Subject: [PULL 0/4] qtests and gitlab-CI

=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===

From https://github.com/patchew-project/qemu
 * [new tag]         patchew/20200117110758.1995-1-thuth@redhat.com -> patchew/20200117110758.1995-1-thuth@redhat.com
Switched to a new branch 'test'
7fbbaf7 gitlab-ci.yml: Run tcg test with tci
1c4d104 tests/qtest/vhost-user-test: Fix memory leaks
2e1728c migration-test: ppc64: fix FORTH test program
247a6a0 tests: acpi: update path in rebuild-expected-aml

=== OUTPUT BEGIN ===
1/4 Checking commit 247a6a033937 (tests: acpi: update path in rebuild-expected-aml)
ERROR: line over 90 characters
#44: FILE: tests/data/acpi/rebuild-expected-aml.sh:34:
+echo '/* List of comma-separated changed AML files to ignore */' > ${SRC_PATH}/tests/qtest/bios-tables-test-allowed-diff.h

total: 1 errors, 0 warnings, 21 lines checked

Patch 1/4 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

2/4 Checking commit 2e1728c34f2b (migration-test: ppc64: fix FORTH test program)
3/4 Checking commit 1c4d10497e69 (tests/qtest/vhost-user-test: Fix memory leaks)
4/4 Checking commit 7fbbaf7a0df6 (gitlab-ci.yml: Run tcg test with tci)
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/20200117110758.1995-1-thuth@redhat.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
Peter Maydell Jan. 17, 2020, 5:27 p.m. UTC | #2
On Fri, 17 Jan 2020 at 11:08, Thomas Huth <thuth@redhat.com> wrote:
>
>  Hi Peter,
>
> the following changes since commit fee0ec1fd11a6fb960517e18201ed8a686a0d7e8:
>
>   Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' into staging (2020-01-14 14:11:34 +0000)
>
> are available in the Git repository at:
>
>   https://gitlab.com/huth/qemu.git tags/pull-request-2020-01-17
>
> for you to fetch changes up to c82b95489f8c33318101d18e643f7b50259d8e4e:
>
>   gitlab-ci.yml: Run tcg test with tci (2020-01-16 17:01:39 +0100)
>
> ----------------------------------------------------------------
> * Various fixes for qtests
> * Enable TCG tests with TCI in the gitlab CI
> ----------------------------------------------------------------
>
> Laurent Vivier (1):
>       migration-test: ppc64: fix FORTH test program
>
> Shameer Kolothum (1):
>       tests: acpi: update path in rebuild-expected-aml
>
> Thomas Huth (2):
>       tests/qtest/vhost-user-test: Fix memory leaks
>       gitlab-ci.yml: Run tcg test with tci


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/5.0
for any user-visible changes.

-- PMM