[dts] [PATCH] IXIA: Exit on failure of clear/take port ownership

[Chen, Zhaoyan]

Thanks, Vukkisala,

Good fixing on etgen.py, however, we are going to remove the etgen API in future. 
Because there is a new pktgen API developed in DTS framework, 
which support not only IXIA but also TREX (SW). They have unified packet generator API,
so you need to modify nothing in the test suite, if you change the packet generator in test environment.

So we encourage you to try this new unified API for your performance test suite.
We have several documents for you to migrate from old etgen API under doc folder
- migrate_from_etgen_to_pktgen.rst
- pktgen_prog_guide.rst
- trex_known_issue.rst

Furthermore, your fixing seems also valid for the new pktgen API. Could you please generate 1 more patch for it?

BTW, how many test suite you are using with etgen in DTS? What are they?

Any problem, please let me know. Thanks.


Regards,
Zhaoyan Chen


> -----Original Message-----
> From: dts [mailto:dts-bounces at dpdk.org] On Behalf Of Tu, Lijuan
> Sent: Monday, August 12, 2019 3:00 PM
> To: pvukkisala at marvell.com; dts at dpdk.org
> Cc: avijay at marvell.com; fmasood at marvell.com
> Subject: Re: [dts] [PATCH] IXIA: Exit on failure of clear/take port ownership
> 
> Applied, thanks
> 
> > -----Original Message-----
> > From: dts [mailto:dts-bounces at dpdk.org] On Behalf Of
> > pvukkisala at marvell.com
> > Sent: Friday, August 9, 2019 8:51 PM
> > To: dts at dpdk.org
> > Cc: avijay at marvell.com; fmasood at marvell.com; Phanendra Vukkisala
> > <pvukkisala at marvell.com>
> > Subject: [dts] [PATCH] IXIA: Exit on failure of clear/take port
> > ownership
> >
> > From: Phanendra Vukkisala <pvukkisala at marvell.com>
> >
> > Signed-off-by: Phanendra Vukkisala <pvukkisala at marvell.com>
> > ---
> >  framework/etgen.py |    8 ++++++--
> >  1 file changed, 6 insertions(+), 2 deletions(-)
> >
> > diff --git a/framework/etgen.py b/framework/etgen.py index
> > 212b9c2..a47d8cd 100644
> > --- a/framework/etgen.py
> > +++ b/framework/etgen.py
> > @@ -427,10 +427,14 @@ class IxiaPacketGenerator(SSHConnection):
> >          out = self.send_expect("set chasId [ixGetChassisID %s]" %
> > self.tclServerIP, "% ")
> >          self.chasId = int(out.strip())
> >
> > -        self.send_expect("ixClearOwnership [list %s]" % string.join(
> > +        out = self.send_expect("ixClearOwnership [list %s]" %
> > + string.join(
> >              ['[list %d %d %d]' % (self.chasId, item['card'],
> > item['port']) for item in self.ports], ' '), "% ", 10)
> > -        self.send_expect("ixTakeOwnership [list %s] force" % string.join(
> > +        if out.strip()[-1] != '0':
> > +            return False
> > +        out = self.send_expect("ixTakeOwnership [list %s] force" %
> > + string.join(
> >              ['[list %d %d %d]' % (self.chasId, item['card'],
> > item['port']) for item in self.ports], ' '), "% ", 10)
> > +        if out.strip()[-1] != '0':
> > +            return False
> >
> >          return True
> >
> > --
> > 1.7.9.5