[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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



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


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]