[libvirt] [test-API 00/10] Reorgnize/refactoring round 2

Osier Yang jyang at redhat.com
Wed Apr 18 15:00:25 UTC 2012


On 2012年04月18日 13:49, Osier Yang wrote:
> This should be the second round to reorgnize/refactor the codes.
>
> What this round does:
>    * New directory "conf"; Move $root/*.cfg into it
>    * New directory "src"; Move $root/*.{py,css,xls} into it (except
>      the main program libvirt-test-api.py)
>    * Rename libvirt-test-api.py to libvirt-test-api
>    * Move install-autotest-server into "scripts"
>    * Rename BUGSKIP to SKIP, and modify the skipping syntax so
>      that it skips whole test case, but not test step(s). See the
>      explaination in the patch to get why to do it.
>    * Destroy the codes to skip test steps.
>    * Other small fixes.
>
> TODO in next round:
>    * Support run all cases in a bundle
>    * Support to skip test case with the new skipping syntax
>    * A new config property for the scratch directory, (now
>      it always use $(pwd)/$guest_name to download the files,
>      we should have something like '/var/cache/libvirt-test-API'
>      instead).
>    * Other cleaning up.
>
> Osier Yang (10):
>    Create new dir 'conf' and move conf files into it
>    Fix the typo in usage string
>    Remove the suffix .py for libvirt-test-api.py
>    New directory 'src' and move *.{py,css,xls} files into it
>    Fix the typo which mispells 'shutil' as 'shutils'
>    Move install-autotest-server into 'scripts'
>    Update logging stuffs so that it works after source tree reorgnizing
>    Add document to tell the default log output file
>    Rename and modify the syntax of BUGSKIP
>    Destroy the codes to skip case steps.
>
>   BUGSKIP                                            |    4 -
>   SKIP                                               |   26 ++++++++
>   env.cfg =>  conf/env.cfg                            |    0
>   kickstart.cfg =>  conf/kickstart.cfg                |    0
>   dist/redhat/env_inspect.py                         |    2 +-
>   libvirt-test-api.py =>  libvirt-test-api            |   50 ++++++++--------
>   repos/domain/attach_disk.py                        |    2 +-
>   repos/domain/attach_interface.py                   |    2 +-
>   repos/domain/autostart.py                          |    2 +-
>   repos/domain/balloon_memory.py                     |    2 +-
>   repos/domain/blkstats.py                           |    2 +-
>   repos/domain/console_io.py                         |    2 +-
>   repos/domain/console_mutex.py                      |    2 +-
>   repos/domain/cpu_affinity.py                       |    2 +-
>   repos/domain/cpu_topology.py                       |    2 +-
>   repos/domain/create.py                             |    2 +-
>   repos/domain/define.py                             |    2 +-
>   repos/domain/destroy.py                            |    2 +-
>   repos/domain/detach_disk.py                        |    2 +-
>   repos/domain/detach_interface.py                   |    2 +-
>   repos/domain/domain_blkinfo.py                     |    2 +-
>   repos/domain/domain_id.py                          |    2 +-
>   repos/domain/domblkinfo.py                         |    2 +-
>   repos/domain/dump.py                               |    2 +-
>   repos/domain/eventhandler.py                       |    2 +-
>   repos/domain/ifstats.py                            |    2 +-
>   repos/domain/install_image.py                      |    4 +-
>   repos/domain/install_linux_cdrom.py                |   10 ++--
>   repos/domain/install_linux_check.py                |    4 +-
>   repos/domain/install_linux_net.py                  |   12 ++--
>   repos/domain/install_windows_cdrom.py              |   12 ++--
>   repos/domain/migrate.py                            |    2 +-
>   repos/domain/ownership_test.py                     |    2 +-
>   repos/domain/reboot.py                             |    2 +-
>   repos/domain/restore.py                            |    2 +-
>   repos/domain/resume.py                             |    2 +-
>   repos/domain/save.py                               |    2 +-
>   repos/domain/sched_params.py                       |    2 +-
>   repos/domain/shutdown.py                           |    2 +-
>   repos/domain/start.py                              |    2 +-
>   repos/domain/suspend.py                            |    2 +-
>   repos/domain/undefine.py                           |    2 +-
>   repos/domain/update_devflag.py                     |    2 +-
>   repos/interface/create.py                          |    2 +-
>   repos/interface/define.py                          |    2 +-
>   repos/interface/destroy.py                         |    2 +-
>   repos/interface/undefine.py                        |    2 +-
>   repos/libvirtd/qemu_hang.py                        |    2 +-
>   repos/libvirtd/restart.py                          |    2 +-
>   repos/network/autostart.py                         |    2 +-
>   repos/network/create.py                            |    2 +-
>   repos/network/define.py                            |    2 +-
>   repos/network/destroy.py                           |    2 +-
>   repos/network/network_list.py                      |    2 +-
>   repos/network/network_name.py                      |    2 +-
>   repos/network/network_uuid.py                      |    2 +-
>   repos/network/start.py                             |    2 +-
>   repos/network/undefine.py                          |    2 +-
>   repos/nodedevice/detach.py                         |    2 +-
>   repos/nodedevice/reattach.py                       |    2 +-
>   repos/nodedevice/reset.py                          |    2 +-
>   repos/npiv/create_virtual_hba.py                   |    2 +-
>   .../multiple_thread_block_on_domain_create.py      |    4 +-
>   repos/remoteAccess/tcp_setup.py                    |    2 +-
>   repos/remoteAccess/tls_setup.py                    |    8 +-
>   repos/sVirt/domain_nfs_start.py                    |    2 +-
>   repos/snapshot/delete.py                           |    2 +-
>   repos/snapshot/file_flag.py                        |    2 +-
>   repos/snapshot/flag_check.py                       |    2 +-
>   repos/snapshot/internal_create.py                  |    2 +-
>   repos/snapshot/revert.py                           |    2 +-
>   repos/storage/activate_pool.py                     |    2 +-
>   repos/storage/build_dir_pool.py                    |    2 +-
>   repos/storage/build_disk_pool.py                   |    2 +-
>   repos/storage/build_logical_pool.py                |    2 +-
>   repos/storage/build_netfs_pool.py                  |    2 +-
>   repos/storage/create_dir_pool.py                   |    2 +-
>   repos/storage/create_dir_volume.py                 |    2 +-
>   repos/storage/create_fs_pool.py                    |    2 +-
>   repos/storage/create_iscsi_pool.py                 |    2 +-
>   repos/storage/create_logical_volume.py             |    2 +-
>   repos/storage/create_netfs_pool.py                 |    2 +-
>   repos/storage/create_netfs_volume.py               |    2 +-
>   repos/storage/create_partition_volume.py           |    2 +-
>   repos/storage/define_dir_pool.py                   |    2 +-
>   repos/storage/define_disk_pool.py                  |    2 +-
>   repos/storage/define_iscsi_pool.py                 |    2 +-
>   repos/storage/define_logical_pool.py               |    2 +-
>   repos/storage/define_mpath_pool.py                 |    2 +-
>   repos/storage/define_netfs_pool.py                 |    2 +-
>   repos/storage/define_scsi_pool.py                  |    2 +-
>   repos/storage/delete_dir_volume.py                 |    2 +-
>   repos/storage/delete_logical_pool.py               |    2 +-
>   repos/storage/delete_logical_volume.py             |    2 +-
>   repos/storage/delete_netfs_volume.py               |    2 +-
>   repos/storage/delete_partition_volume.py           |    2 +-
>   repos/storage/destroy_pool.py                      |    2 +-
>   repos/storage/pool_name.py                         |    2 +-
>   repos/storage/pool_uuid.py                         |    2 +-
>   repos/storage/undefine_pool.py                     |    2 +-
>   .../install-autotest-server                        |    0
>   activityfilter.py =>  src/activityfilter.py         |    0
>   casecfgcheck.py =>  src/casecfgcheck.py             |    2 +-
>   env_clear.py =>  src/env_clear.py                   |    2 +-
>   exception.py =>  src/exception.py                   |    2 +-
>   generator.py =>  src/generator.py                   |   61 ++++----------------
>   log.css =>  src/log.css                             |    0
>   log.xsl =>  src/log.xsl                             |    2 +-
>   logxmlparser.py =>  src/logxmlparser.py             |    4 +-
>   mapper.py =>  src/mapper.py                         |    0
>   parser.py =>  src/parser.py                         |    6 +-
>   process.py =>  src/process.py                       |    0
>   proxy.py =>  src/proxy.py                           |    2 +-
>   sharedmod.py =>  src/sharedmod.py                   |    0
>   utils/env_parser.py                                |    2 +-
>   115 files changed, 188 insertions(+), 205 deletions(-)
>   delete mode 100644 BUGSKIP
>   create mode 100644 SKIP
>   rename env.cfg =>  conf/env.cfg (100%)
>   rename kickstart.cfg =>  conf/kickstart.cfg (100%)
>   rename libvirt-test-api.py =>  libvirt-test-api (91%)
>   rename install-autotest-server =>  scripts/install-autotest-server (100%)
>   create mode 100644 src/__init__.py
>   rename activityfilter.py =>  src/activityfilter.py (100%)
>   rename casecfgcheck.py =>  src/casecfgcheck.py (99%)
>   rename env_clear.py =>  src/env_clear.py (99%)
>   rename exception.py =>  src/exception.py (97%)
>   rename generator.py =>  src/generator.py (78%)
>   rename log.css =>  src/log.css (100%)
>   rename log.xsl =>  src/log.xsl (98%)
>   rename logxmlparser.py =>  src/logxmlparser.py (99%)
>   rename mapper.py =>  src/mapper.py (100%)
>   rename parser.py =>  src/parser.py (99%)
>   rename process.py =>  src/process.py (100%)
>   rename proxy.py =>  src/proxy.py (99%)
>   rename sharedmod.py =>  src/sharedmod.py (100%)
>
> Regards,
> Osier
>

Pushed without 2/10, with SKIP kept, we will need it
anyway in future, and it doesn't hurt us now. Also
with the nit of 10/10 fixed. Thanks.

Osier




More information about the libvir-list mailing list