test suite reviews and discussions
 help / color / mirror / Atom feed
From: "Juraj Linkeš" <juraj.linkes@pantheon.tech>
To: Jun Dong <junx.dong@intel.com>, "dts@dpdk.org" <dts@dpdk.org>
Cc: "lijuan.tu@intel.com" <lijuan.tu@intel.com>,
	"qingx.sun@intel.com" <qingx.sun@intel.com>
Subject: RE: [V3 2/5] rename base classes 2
Date: Fri, 10 Jun 2022 12:41:29 +0000	[thread overview]
Message-ID: <a5e6561c7a5148fe84eac90279796850@pantheon.tech> (raw)
In-Reply-To: <20220610050810.1531-3-junx.dong@intel.com>

Reviewed-by: Juraj Linkeš <juraj.linkes@pantheon.tech>

> -----Original Message-----
> From: Jun Dong <junx.dong@intel.com>
> Sent: Friday, June 10, 2022 7:08 AM
> To: dts@dpdk.org
> Cc: lijuan.tu@intel.com; qingx.sun@intel.com; junx.dong@intel.com; Juraj
> Linkeš <juraj.linkes@pantheon.tech>
> Subject: [V3 2/5] rename base classes 2
> 
> From: Juraj Linkeš <juraj.linkes@pantheon.tech>
> 
> framework/*
> main.py
> nics/*
> tools/*
> 
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Signed-off-by: Jun Dong <junx.dong@intel.com>
> ---
> 
> V3:
>   - Fixed self string key word confuse through replace 'tg' to 'Traffic'
>   - Fixed self string key word confuse through replace 'sut' to 'Sut'
> 
> V2:
>   The original modification of rename
> 
>  framework/asan_test.py                        |  18 +-
>  framework/checkCase.py                        |  44 +--
>  framework/config.py                           | 192 ++++++------
>  framework/crbs.py                             |  37 ---
>  framework/debugger.py                         |   8 +-
>  framework/dts.py                              | 296 +++++++++---------
>  framework/excel_reporter.py                   |  74 ++---
>  framework/exception.py                        |   8 +-
>  framework/ixia_network/ixnet.py               |   2 +-
>  framework/json_reporter.py                    |  40 +--
>  framework/logger.py                           | 188 +++++------
>  framework/multiple_vm.py                      |  78 ++---
>  framework/{crb.py => node.py}                 | 120 +++----
>  framework/pktgen.py                           | 207 ------------
>  framework/plotting.py                         |   6 +-
>  framework/pmd_output.py                       |  26 +-
>  framework/project_dpdk.py                     |  72 ++---
>  framework/qemu_kvm.py                         |  88 +++---
>  framework/qemu_libvirt.py                     |  30 +-
>  framework/rst.py                              |   8 +-
>  .../{packet.py => scapy_packet_builder.py}    | 233 +++++++-------
>  framework/settings.py                         |  30 +-
>  framework/ssh_connection.py                   |  12 +-
>  framework/ssh_pexpect.py                      |  42 +--
>  framework/stats_reporter.py                   |  22 +-
>  framework/{dut.py => sut_node.py}             | 247 ++++++++-------
>  framework/test_case.py                        |  88 +++---
>  framework/test_result.py                      | 210 ++++++-------
>  .../{pktgen_ixia.py => tg_ixexplorer.py}      | 115 ++++---
>  ...pktgen_ixia_network.py => tg_ixnetwork.py} |  40 ++-
>  framework/{tester.py => tg_node.py}           | 275 ++++++++--------
>  framework/{pktgen_base.py => tg_perf.py}      | 266 ++++++++++++----
>  framework/{pktgen_trex.py => tg_trex.py}      |  54 ++--
>  framework/utils.py                            |  40 +--
>  framework/virt_base.py                        | 126 ++++----
>  framework/virt_common.py                      |   6 +-
>  framework/virt_resource.py                    |  36 +--
>  framework/virt_scene.py                       | 108 +++----
>  framework/{virt_dut.py => virt_sut.py}        | 102 +++---
>  main.py                                       |  12 +-
>  nics/net_device.py                            |  46 +--
>  nics/system_info.py                           |   6 +-
>  tools/dump_case.py                            |   6 +-
>  tools/setup.py                                | 134 ++++----
>  44 files changed, 1864 insertions(+), 1934 deletions(-)
>  delete mode 100644 framework/crbs.py
>  rename framework/{crb.py => node.py} (91%)
>  delete mode 100644 framework/pktgen.py
>  rename framework/{packet.py => scapy_packet_builder.py} (85%)
>  rename framework/{dut.py => sut_node.py} (89%)
>  rename framework/{pktgen_ixia.py => tg_ixexplorer.py} (95%)
>  rename framework/{pktgen_ixia_network.py => tg_ixnetwork.py} (83%)
>  rename framework/{tester.py => tg_node.py} (76%)
>  rename framework/{pktgen_base.py => tg_perf.py} (70%)
>  rename framework/{pktgen_trex.py => tg_trex.py} (95%)
>  rename framework/{virt_dut.py => virt_sut.py} (83%)
> 

  reply	other threads:[~2022-06-10 12:41 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-10  5:08 [V3 0/5] rename base classes Jun Dong
2022-06-10  5:08 ` [V3 1/5] rename base classes 1 Jun Dong
2022-06-10 12:41   ` Juraj Linkeš
2022-06-10  5:08 ` [V3 2/5] rename base classes 2 Jun Dong
2022-06-10 12:41   ` Juraj Linkeš [this message]
2022-06-10  5:08 ` [V3 3/5] rename base classes 3 Jun Dong
2022-06-10 12:41   ` Juraj Linkeš
2022-06-10  5:08 ` [V3 4/5] rename base classes 4 Jun Dong
2022-06-10 12:42   ` Juraj Linkeš
2022-06-10  5:08 ` [V3 5/5] rename base classes 5 Jun Dong
2022-06-10 12:43   ` Juraj Linkeš

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=a5e6561c7a5148fe84eac90279796850@pantheon.tech \
    --to=juraj.linkes@pantheon.tech \
    --cc=dts@dpdk.org \
    --cc=junx.dong@intel.com \
    --cc=lijuan.tu@intel.com \
    --cc=qingx.sun@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).