Autotest Weekly Hangout 2013-10-09 (CST edition)

Etherpad: https://autotest.etherpad.mozilla.org/1?

Meeting video: https://www.youtube.com/watch?v=6lTW4nVGGZU

Highlights

The problems on next from the last week were found and properly handled. Lots of processed pull requests, and more work is being dedicated to KVM Forum.

Administrivia

I'll be on vacation next week, and in the following week, myself, Cleber and Rudá will be at the KVM Forum, so we'll skip the next 2 instances of the weekly hangout.

Commits on next - autotest

The rpm spec file work is finally on the development branch. Kudos to cleber that got rid of all custom patches shipped by the RPMs

Cleber Rosa (12):
installation_support/autotest-install-packages-deps: return exist status code
      docs: fix usage of autodata
      global_config.ini: make it easier to be read by humans
      scheduler: add a option for how drone machines are deployed
global_config: add utility function/script to change global_config values
      conmux: try to use system conmux first
      utils/autotest-rh.init: make the PID_PATH line simpler
      README.fedora: add instructions for generating autotest packages
      RPM packages: add SPEC file
      server/hosts/serial.py: replace literal ".." with os.pardir
      autotest-framework.spec: added suggestions by Ross Brattain
installation_support/global_config_set_value.py: added suggestions by Ross Brattain

James Laska (1):
      Add README for deployment on Fedora

Lucas Meneghel Rodrigues (1):
      install-autotest-server.sh: Abort script if failed to install deps

Ross Brattain (10):
autotest-firewalld-add-service: move argparse to runtime instead of init
      autotest_firewalld_add_service_unittest: fix module name
      autotest_firewalld_add_service_unittest: mock out argparse
      autotest-firewalld-add-server: convert to Popen and add unit test
autotest-firewalld-add-service: improve handling of missing default zone
      autotest-firewalld-add-service: exit with status 1 when missing args
      autotest-firewalld-add-service: convert methods to static
autotest-firewalld-add-service: fix bug with newlines by adding rstrip
      autotest_firewalld_add_service_unittest: fix #! to use /usr/bin/env
autotest_firewalld_add_service_unittest: resolve pylint by splitting unit test

Rudá Moura (2):
Merge pull request #749 from autotest/setup-fix (with minor fix for PEP8 compliance) Started to create the "documentation" directory for virt-test, to match what we have on autotest. The documentation structure is the one that sphinx uses to build htmls and so on.


Commits on next - virt-test

Turns out that the problem of the past week was related to propcan, and we've been able to fix it.

Cleber Rosa (2):
      Simple koji "robustify" fixes
      Get rid of the installer registration concept

Dongsheng Yang (5):
      libvirt-snapshot: Improve virsh_snapshot test.
      Update snapshot-delete function in virsh module.
      Add more options to improve nodedev-dumpxml test.
      Improve nodedev-detach-reatach test.
      Add --paused option test for snapshot-revert.

Jiří Župka (1):
      openvswitch: Fix error due to moving with python modules

John Ferlan (2):
      virsh_list: Adjust test to SKIP remote test
      virsh_shutdown: Adjust test to SKIP remote test

Lucas Meneghel Rodrigues (23):
      Add JeOS.19.x86_64
      run: Update JeOS to version 19
      downloads: Add JeOS 19 download file
      bootstrap: Download JeOS 19 image rather than 17
      virttest: Move utils_koji to staging
      virttest.staging.utils_koji: Retry file downloads
      general: More docstring fixes
      virttest: Log top commit message
      libvirt.tests.virsh_cmd: Fix invalid syntax
      virttest.virsh: Fix void return of domiftune
      libvirt.virsh_cmd: virsh_desc PEP8 fixes
      libvirt.virsh_cmd: virsh_numatune PEP8 fixes
      libvirt.virsh_cmd: virsh_node_memtune PEP8 fixes
      libvirt.virsh_cmd: virsh_snapshot_disk PEP8 fixes
      run: PEP8 fixes
      shutdown: PEP8 fixes
      v2v: convert_remote_vm PEP8 fixes
      virttest.qemu_monitor: PEP8 fixes
      shared.scripts.virtio_console_guest: PEP8 fixes
      virttest.virsh: Remove obsolete part of the docstring
      qemu: virtio_console - Fix typo
      libvirt: virsh_cmd - Fix 'acpiphp_module_modprobe' function
      Moving build.cfg back to the qemu directory

Lukáš Doktor (11):
      qemu.tests.virtio_console: Fix variable overwriting
virttest.qemu_virtio_port: Allow to reload loss idx from outside the thread shared.scripts.virtio_console_guest: Remove sleeps from reconnect loopback
      qemu.tests.virtio_console: Harden the interrupted_loopback setting
      virttest.qemu_virtio_port: Prevent infinite loop
qemu.tests.virtio_console: Prevent infinite loop in some test on failure qemu.tests.virtio_console: Reorder interrupted_loopback and fix 80 chars
      qemu.tests.virtio_console: New interrupted loopback mode
      qemu.tests.virtio_console: Fix multi_open test to suit latest kernels
      qemu.tests.virtio_console: Reconnect guest worker in migrate test
      virttest.qemu_virtio_port: Check the guest script size

Martin Kletzander (1):
      libivrt: Fix virsh_domjobabort race condition

Pavel Hrdina (9):
      run: add libvirt specific options install and remove
      utils_cgroup: Strip the slash from cgroup path
      utils_cgroup: Fix regexp for cgroup controllers
virsh_attach_detach_disk: attach tmp disk with config option into offline guest virsh_attach_detach_disk: move add acpi module after attach/detach required disks virsh_attach_detach_disk: attach second temp disk for the detach_twice test
      virsh_attach_detach_disk: make the proper return status checking
      virsh_attach_detach_disk: improve 'acpiphp_module_modprobe' function
      virsh_attach_detach_disk: resume guest before we check the xml file

Rudá Moura (1):
      libvirt: virsh_domjababort: update code for PEP8 compliance.

Swapna Krishnan (1):
Updating build_install.py to work with newer changes in qxl, spice-vdagent and spice-gtk

Yunping Zheng (4):
      virt.qemu.test.systemtap_tracing : make the test more robust
      virt.tests: Updates to netperf
      qemu.test.netperf_udp: Some modify of netperf_udp
      virttest.qemu_vm: modify nic hotplug fd

yangdongsheng (2):
      libvirt-tests: Improve snapshot tests in libvirt.
      libvirt-tests: fix cleanup part in test for snapshot_disk.

Proposal - Pull request freeze, where only documentation pull requests could be accepted.

Pull requests still in review

Autotest

https://github.com/autotest/autotest/pull/752 RFC: Work in Progress: convert install script to python made by Ross - this one needs another review and updates


Virt Test

45 open pull requests, 2 down from last week, but at least 14 pull requests were resolved last week, so our team is able to keep up with the rate of new pull requests being added. This and the fact we've been able to find big regressions with automated testing means the work so far is slowing paying off.

The big work related to the PCI switch support made by ldoktor was presented as

https://github.com/autotest/virt-test/pull/910

Another pull request that is around for a while and we need to decide what is the best solution is:

https://github.com/autotest/virt-test/pull/769

Which deals with splitting utils_test. Many joined the discussion, my take on the subject is that the split could be done profitably by having:

from virttest import utils_test

utils_test.qemu.specific_api()

And generic functions can be added to utils_test/__init__.py.

I'd like to hear your opinions on that.

Grid maintenance

Although we still see random networking issues, in general, the jobs are stable, which means all the effort in maintaining the grid is paying off Working with Pavel, from the libvirt team to start running simple libvirt tests. There are 2 new boxes to be added to the grid that will be dedicated to libvirt.

Libvirt updates
Back into the swing this week after China Holiday
Happy to see picked right back up on healthy reviews + new pulls mix
Increasing interaction with libvirt eng. (Pavel a.k.a. Antique)
Discussions on how to TODO
serial problem: issue: 919 needs attention

Questions Corner

TODO

lmr
Finish demos and presentations for the KVM Forum
RFC about pull request freeze
Enjoy vacation

cevich
documentation
patch review
Tinker on github reporting code + aexpect


cleber
Software Installer code "moved" to autotest base libraries, differences:
first API layer is usable by regular code (GitRepoInstaller, KojiInstaller, etc) Idea is to unify all places where we have software installation functionality, such as utils/build_externals.py and utils/external_packages.py second API layer is for tests use (GitRepoTestInstaller, KojiTestInstaller, etc) where it makes sense, such as with GitRepoTestInstaller, the software components installed are automatically registered Will test a little further before pushing upstream, since it should also sync with virt-test "autotest.client.shared" namespace comment/question made during the hang out is related to the namespace on this code:
autotest.client.shared.installers.<other_level>
still usable, but having 5 levels is kind of a warning
arc has got a couple of new apis/commands: linuxdistro, regressionpoint, testenvironment, etc
further testing and documentation and pushing upstream
^ All this will make into the KVM Forum presentation

jzupka
same as last week
review pull-requests
bugs glusterfs
propcan

rmoura
review pull-requests
working on api documentation for autotest/virt-test

ldoktor
qemu_devices
PCI support for PCI switch/bridge/... - sent
simplify the insertion code (remove force, refactor, remove badbus, ...) - sent
Add support for qtree vs. qemu_devices verification - in progress
pci_devices test - in progress
various setups of PCI switch/bridge/RC + devices, qtree vs lspci
reviewing pull requests
fix bugs

kvm-qe (BJ):

Keep on bugs fix and case development. And each of us will force on one or two features in the following list:
    ypu: virtio console
    yunzheng: ipv6 supports and mq new cases
    xutian: timer device
    fyang: device assignment

_______________________________________________
Virt-test-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-test-devel

Reply via email to