[dts] [Patch V1] framework/tester.py : add this function to support ixia to send fixed number of packets during certain period it is only for test_suite nic_single_core_perf, not for common use

[Wang, FeiX Y]

Hi, Yong

Yes ,that's right,  the value " runtime" is only to make sure all the packets configured to be transmitted out within the period.
Ok, then I will add a check to see if the actual transmitted packets is as configured.

Thanks

fei  

-----Original Message-----
From: Liu, Yong 
Sent: Friday, September 22, 2017 1:51 PM
To: Wang, FeiX Y <feix.y.wang at intel.com>; dts at dpdk.org
Cc: Wang, FeiX Y <feix.y.wang at intel.com>
Subject: RE: [dts] [Patch V1] framework/tester.py : add this function to support ixia to send fixed number of packets during certain period it is only for test_suite nic_single_core_perf, not for common use

Fei,
Please make sure your code align to pep style.
I still has question for parameter "runTime", this value is depend on rate, total number and link rate.
I think the only value for this parameter is to make sure all packets transmitted.
Why not just check how many packets emitted and return when equal to parameter "packetNum".

Thanks,
Marvin

> -----Original Message-----
> From: dts [mailto:dts-bounces at dpdk.org] On Behalf Of wang fei
> Sent: Wednesday, September 20, 2017 8:31 PM
> To: dts at dpdk.org
> Cc: Wang, FeiX Y <feix.y.wang at intel.com>
> Subject: [dts] [Patch V1] framework/tester.py : add this function to 
> support ixia to send fixed number of packets during certain period it 
> is only for test_suite nic_single_core_perf, not for common use
> 
> Signed-off-by: wang fei <feix.y.wang at intel.com>
> ---
>  framework/tester.py | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/framework/tester.py b/framework/tester.py index 
> e7fb6bc..78202d0 100644
> --- a/framework/tester.py
> +++ b/framework/tester.py
> @@ -491,6 +491,17 @@ class Tester(Crb):
> 
>          return out.rpartition('[')[0]
> 
> +    def
> traffic_generator_send_packets(self,portList,ratePercent,packetNum,del
> ay=5
> ,runTime=60):
> +        """
> +        send fixed number of packets on specified ports during 
> + runTime
> time.
> +        Be note that this function is only used for 
> + nic_single_core_perf
> test case, not for common use
> +        """
> +        if self.check_port_list(portList, 'ixia'):
> +            return self.ixia_packet_gen.send_packets(portList,
> ratePercent, packetNum,delay,runTime)
> +        if not self.check_port_list(portList):
> +            self.logger.warning("exception by mixed port types")
> +            return None
> +
>      def traffic_generator_throughput(self, portList, 
> rate_percent=100,
> delay=5):
>          """
>          Run throughput performance test on specified ports.
> --
> 2.7.4