; create mode 100755 scripts/ci/org.centos/stream/8/x86_64/test-avocado
> create mode 100644 scripts/ci/org.centos/stream/README
>
Maybe it is too late, but just for the records:
Reviewed-by: Willian Rampazzo
Tested-by: Willian Rampazzo
CI job on a custom VM runner:
https://gitlab.com/willianrampazzo/qemu/-/jobs/1778451942
On Thu, Sep 30, 2021 at 5:55 AM Vladimir Sementsov-Ogievskiy
wrote:
>
> 9/30/21 11:47, Richard W.M. Jones wrote:
> > Under SELinux, Unix domain sockets have two labels. One is on the
> > disk and can be set with commands such as chcon(1). There is a
> > different label stored in memory (called
gned-off-by: Thomas Huth
> ---
> .gitlab-ci.d/buildtest.yml | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
Reviewed-by: Willian Rampazzo
On Wed, Jun 23, 2021 at 3:10 PM Philippe Mathieu-Daudé wrote:
>
> We don't use pow2ceil() anymore, remove it.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> tests/acceptance/boot_linux_console.py | 12
> 1 file changed, 12 deletions(-)
>
Reviewed-by: Willian Rampazzo
---
> tests/acceptance/boot_linux_console.py | 10 ++
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
Reviewed-by: Willian Rampazzo
9 +++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
Reviewed-by: Willian Rampazzo
4 +-
> 1 file changed, 9 insertions(+), 5 deletions(-)
>
Reviewed-by: Willian Rampazzo
hilippe Mathieu-Daudé
> ---
> tests/acceptance/boot_linux_console.py | 1 +
> tests/acceptance/ppc_prep_40p.py | 2 ++
> 2 files changed, 3 insertions(+)
>
Reviewed-by: Willian Rampazzo
gt; 3 files changed, 48 insertions(+), 13 deletions(-)
> create mode 100644 python/qemu/utils/feature.py
>
Based on my comments from the next patch of this series:
Reviewed-by: Willian Rampazzo
ature is "machine", option can be "q35".
> +type option: str
> +"""
> +try:
> + options_available = list_feature(self.qemu_bin, feature)
Looking at this code, the previous patch makes more sense the way it is now :)
Maybe, splitting it into multiple `list_` functions will make the code
here more complex.
Anyway, I'll let you decide.
Reviewed-by: Willian Rampazzo
On Tue, Jun 8, 2021 at 8:55 PM Cleber Rosa Junior wrote:
>
>
>
> On Tue, Jun 8, 2021 at 5:42 PM Wainer dos Santos Moschetta
> wrote:
>>
>> Hi,
>>
>> On 6/8/21 11:09 AM, Cleber Rosa wrote:
>> > Which can be used to check for any "feature" that is available as a
>> > QEMU command line option, and
On Tue, Jun 8, 2021 at 11:10 AM Cleber Rosa wrote:
>
> This introduces three different parts of a job designed to run
> on a custom runner managed by Red Hat. The goals include:
>
> a) serve as a model for other organizations that want to onboard
> their own runners, with their specific
gh the formal method of tracking machine/job maintainers have
> not been formalized, it should be known that the contacts/admins for
> this machine and job are:
>
> - Cleber Rosa
>
>clebergnu on #qemu
>
> - Willian Rampazzo
>
>willianr on #qemu
Acked-by: Willian Rampazzo
| 2 +-
> tests/vm/basevm.py| 3 ++-
> 7 files changed, 29 insertions(+), 13 deletions(-)
> create mode 100644 python/qemu/utils/__init__.py
> rename python/qemu/{machine => utils}/accel.py (100%)
>
Reviewed-by: Willian Rampazzo
/.flake8 (100%)
> create mode 100644 python/qemu/machine/__init__.py
> rename python/qemu/{ => machine}/accel.py (100%)
> rename python/qemu/{ => machine}/console_socket.py (100%)
> rename python/qemu/{ => machine}/machine.py (98%)
> rename python/qemu/{ => machine}/pylintrc (100%)
> rename python/qemu/{ => machine}/qtest.py (99%)
> rename python/qemu/{qmp.py => qmp/__init__.py} (96%)
>
This improves a lot the organization of the code.
Reviewed-by: Willian Rampazzo
do not have to add anything to SKIP_FILES.
>
> Signed-off-by: Max Reitz
> ---
> tests/qemu-iotests/297 | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
Reviewed-by: Willian Rampazzo
lways put two empty lines after a function
> - f'exec: cat > /dev/null' does not need to be an f-string
>
> Fix them.
>
> Signed-off-by: Max Reitz
> ---
> tests/qemu-iotests/tests/migrate-bitmaps-test | 38 ++-
> 1 file changed, 20 insertions(+), 18 deletions(-)
>
Reviewed-by: Willian Rampazzo
est | 3 +++
> 1 file changed, 3 insertions(+)
>
Reviewed-by: Willian Rampazzo
/qemu-iotests/297 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Reviewed-by: Willian Rampazzo
On 1/18/21 7:09 AM, Max Reitz wrote:
On 15.01.21 20:27, Willian Rampazzo wrote:
On Fri, Jan 15, 2021 at 2:43 PM Max Reitz wrote:
Instead of checking iotests.py only, check all Python files in the
qemu-iotests/ directory. Of course, most of them do not pass, so there
is an extensive skip
> +iotests.logger.debug('Files to be checked:')
> +iotests.logger.debug(', '.join(sorted(files)))
> +
> +print('=== pylint ===')
> +sys.stdout.flush()
> +
> +# Todo notes are fine, but fixme's or xxx's should probably just be
> +# fixed (in tests, at least)
> +env = os.environ.copy()
> +try:
> +env['PYTHONPATH'] += ':../../python/'
Do you have any objection to using os.path.dirname and os.path.join
here? This would make the code more pythonic.
> +except KeyError:
> +env['PYTHONPATH'] = '../../python/'
Same here. You could do it once, before the 'try' and use it inside.
Other than that,
Reviewed-by: Willian Rampazzo
this patch wants to prevent.
>
> Signed-off-by: Max Reitz
> Reviewed-by: Vladimir Sementsov-Ogievskiy
> ---
> tests/qemu-iotests/300| 1 -
> tests/qemu-iotests/iotests.py | 26 +-
> 2 files changed, 13 insertions(+), 14 deletions(-)
>
Reviewed-by: Willian Rampazzo
ged, 8 insertions(+), 11 deletions(-)
>
Reviewed-by: Willian Rampazzo
> tests/qemu-iotests/129 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Reviewed-by: Willian Rampazzo
Reitz
> Reviewed-by: Eric Blake
> Reviewed-by: Vladimir Sementsov-Ogievskiy
> ---
> tests/qemu-iotests/129 | 37 +
> 1 file changed, 13 insertions(+), 24 deletions(-)
Reviewed-by: Willian Rampazzo
> ---
> tests/qemu-iotests/129 | 27 +--
> 1 file changed, 25 insertions(+), 2 deletions(-)
Reviewed-by: Willian Rampazzo
-
> 2 files changed, 16 insertions(+), 4 deletions(-)
>
Reviewed-by: Willian Rampazzo
Ogievskiy
> ---
> tests/qemu-iotests/129 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Reviewed-by: Willian Rampazzo
On Thu, Jan 14, 2021 at 2:29 PM Max Reitz wrote:
>
> Signed-off-by: Max Reitz
> Reviewed-by: Vladimir Sementsov-Ogievskiy
> Reviewed-by: Eric Blake
> ---
> tests/qemu-iotests/129 | 2 ++
> 1 file changed, 2 insertions(+)
Reviewed-by: Willian Rampazzo
On Fri, Jan 15, 2021 at 6:30 AM Max Reitz wrote:
>
> On 14.01.21 21:02, Willian Rampazzo wrote:
> > On Thu, Jan 14, 2021 at 2:41 PM Max Reitz wrote:
> >>
> >> Signed-off-by: Max Reitz
> >> ---
> >> tests/qemu-iotests/129 | 4 ++--
> >>
part of mypy/pylint cleanup? It seems you are doing more
than that here. It would be good to have this listed in the commit
message. Despite that,
Reviewed-by: Willian Rampazzo
> '151', '152', '155', '163', '165', '169', '194', '196', '199', '202',
> '203', '205', '206', '207', '208', '210', '211', '212', '213', '216',
> '218', '219', '222', '224', '228', '234', '235', '236', '237', '238',
> --
> 2.29.2
>
>
console: / # reboot
> console: / # umount: devtmpfs busy - remounted read-only
> console: EXT4-fs (sda): re-mounted. Opts: (null)
> console: The system is going down NOW!
> console: Sent SIGTERM to all processes
> console: Sent SIGKILL to all processes
> console: Requesting system reboot
> console: sd 0:0:0:0: [sda] Synchronizing SCSI cache
> console: reboot: Restarting system
> PASS (8.44 s)
> RESULTS: PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 |
> CANCEL 0
> JOB TIME : 8.85 s
>
> [*] https://github.com/groeck/linux-build-test/
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> Cc: Guenter Roeck
> ---
> tests/acceptance/boot_linux_console.py | 30 --
> 1 file changed, 28 insertions(+), 2 deletions(-)
>
Reviewed-by: Willian Rampazzo
32 matches
Mail list logo