Hi Lucas,

Below is some feedback for avocado-vt-plugin base on my testing with latest released avocado-vt-plugin RPM package.

1. It's better to print formatted trace-back like autotest does when exception raise at load avocado-vt-plugin, it will very helpful to do bug fix.
   like below exception,
Test discovery plugin <avocado.core.plugins.virt_test.VirtTestLoader object at 0x23e5390> failed: 'Namespace' object has no attribute 'netdst' Above exception raise but not trace-back print, it's hard to know where this exception is thrown for a new user.(Most of team member in our team is new user now)

2. vt_nettype in avocado-virt-0.27.0-1.el7.centos.noarch can't be pass to virt_test plugin from virt-test.conf because exception mentioned in item 1. It caused by a typo in process options function and it has be fixed in upstream master but not include in latest released RPM.

3. I suggest to use a stable code to do release, make RPM packages. And each release should have details release notes and detail change log for relased RPM packages.

4. vt-config option works but customized configuration can't update default configuration from virt-test.conf, which break my test becase test VM is not the expect VM.

And I will continue my testing and provider feedback to you.

Thanks,
Xu


On 08/05/2015 01:46 PM, xutian wrote:


On 08/04/2015 10:40 AM, Lucas Meneghel Rodrigues wrote:
Hello everyone:

After a lot of work and many reviews later, we have virt-test and avocado-vt merged:

https://github.com/avocado-framework/avocado-vt/commit/20dd39ef00db712f78419f07b10b8f8edbd19942


What was done there?

* The virttest library calls to autotest APIs were converted to equivalent avocado API calls. There's still a tiny bit of autotest dependencies there to ensure correct test reporting * The project was adapted to be installable system-wide, through the means of distro packages, such as RPM. By default, the user files are all created in ~/avocado/data/avocado-vt (the ~/avocado location is configurable). * Avocado 0.27.0 and Avocado VT 0.27.0 were released, and both are available in our COPR repo so you can install them at your leisure.


What does this mean from now on?

It officially means that virt-test is now avocado-vt, and that new development should be done in the avocado-vt repo. virt-test's repo is supposed to be left where it is, but we'll disable commit access to it. Our current virt-test maintainers will be granted access to the avocado-vt's repo, and we'll work to help our contributors to rebase their PRs against avocado-vt.
Do you have a clear plan when will you begin to rebase opening PR to avocado-vt ? And when will you disable commit access permission on virt-test repo? We (KVM-QE team) plan to develop memory hotplug/unplug and multi-host migration related test case in this and next month, so we hope changes on virt-test repo will not impact our works.


Future

* The next step is to port our test providers to use avocado libraries, instead of the autotest ones, and move the repos location to avocado. * avocado-vt and our test providers will be maintained normally from this point forward. * Myself and my team will eventually move to focus on avocado-virt, the successor to avocado-vt. This is still ways to go before it can be considered ready for production (stability and feature set good enough for daily virt testing). * Somewhere down the line, avocado-vt will be in maintenance mode, and avocado-virt should be the focus of development.

We ask our awesome contributor and maintainer's patience while we work out eventual issues that came with the transition. We'd love to hear your criticisms and bug reports, and will work as quickly as we can to solve these issues.


Notes

* Please add my COPR repo:

https://copr.fedoraproject.org/coprs/lmr/Autotest/

Check there for install instructions. To test avocado-vt, you can follow:

http://avocado-vt.readthedocs.org/en/latest/GetStartedGuide.html

That should do the trick. Any problems, please report them on:

https://github.com/avocado-framework/avocado-vt/issues/new


Cheers!


Lucas

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


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


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

Reply via email to