NETVIRT-1586 - OVS Based NA Responder for IPv6 default Gateway


JamO Luhrsen
 

Hi Srinivas,

I noticed your CSIT patch failed to verify, but that might be a problem
with the verify job. I see something to do with python2 vs python3. I'll
try to figure that out.

As for where and how this new suite will run, you already added it to
the netvirt-1node-openstack.txt testplan, so it will run in all those
jobs using that testplan. However, if the feature is not installed by
default, you will want to add that to the feature list in the netvirt
JJB jobs. Like this:

https://github.com/opendaylight/releng-builder/blob/master/jjb/netvirt/netvirt-csit-multi-openstack.yaml#L35

I would create the releng/builder patch, then push a job to the sandbox
using that code. Then run the sandbox job with PATCHREFSPEC pointing
to your int/test patch. If everything is as expected we can merge both
releng/builder and int/test patches at the same time.

Thanks,
JamO


On 4/10/20 9:27 PM, srinivas.rachakonda@... wrote:

Hi Jamo,

A new feature is to be added in Aluminium release.

 

https://jira.opendaylight.org/browse/NETVIRT-1586

 

I have added the below suite for OVS Based NA Responder for IPv6 default Gateway.

https://git.opendaylight.org/gerrit/c/integration/test/+/88772

 

The design link is:

https://git.opendaylight.org/gerrit/c/netvirt/+/81533

 

The feature automation was done for the above design code to be pushed to CSIT.

But by default this feature won’t be enabled.

 

Please kindly suggest on where should this script be added so that it can be used whenever needed.

 

Thanks,

Srinivas

+91-9243478719

 


Srinivas <srinivas.rachakonda@...>
 

Hi Jamo,

 

Thanks for the mail and help provided.

 

The changes are xml based where we are introducing a flag "na-responder-mode" which can be either controller or switch based.

 

https://git.opendaylight.org/gerrit/c/netvirt/+/81533/25/ipv6service/impl/src/main/resources/initial/netvirt-ipv6service-config.xml

 

 

By default the flag "na-responder-mode" is set to controller and existing CSIT scripts would go through.

The suite that I have written is for when the flag "na-responder-mode" is set to "switch"

This is to check how the flows are generated and check traffic with switch mode.

 

I want the script to be in a separate testplan, something like tdd- (same as used by NAT/Conntrack suites) which will be run on demand basis.

 

Thanks,

Srinivas

+91-9243478719

 

From: Jamo Luhrsen <jluhrsen@...>
Sent: 13 April 2020 23:16
To: srinivas.rachakonda@...
Cc: karthikeyan.k@...; 'Dayavanti Gopal Kamath' <dayavanti.gopal.kamath@...>; 'Nishchya Gupta' <nishchyag@...>; ravikanth.ps@...; integration-dev@...
Subject: Re: NETVIRT-1586 - OVS Based NA Responder for IPv6 default Gateway

 

Hi Srinivas,

I noticed your CSIT patch failed to verify, but that might be a problem
with the verify job. I see something to do with python2 vs python3. I'll
try to figure that out.

As for where and how this new suite will run, you already added it to
the netvirt-1node-openstack.txt testplan, so it will run in all those
jobs using that testplan. However, if the feature is not installed by
default, you will want to add that to the feature list in the netvirt
JJB jobs. Like this:

https://github.com/opendaylight/releng-builder/blob/master/jjb/netvirt/netvirt-csit-multi-openstack.yaml#L35

I would create the releng/builder patch, then push a job to the sandbox
using that code. Then run the sandbox job with PATCHREFSPEC pointing
to your int/test patch. If everything is as expected we can merge both
releng/builder and int/test patches at the same time.

Thanks,
JamO

 

On 4/10/20 9:27 PM, srinivas.rachakonda@... wrote:

Hi Jamo,

A new feature is to be added in Aluminium release.

 

https://jira.opendaylight.org/browse/NETVIRT-1586

 

I have added the below suite for OVS Based NA Responder for IPv6 default Gateway.

https://git.opendaylight.org/gerrit/c/integration/test/+/88772

 

The design link is:

https://git.opendaylight.org/gerrit/c/netvirt/+/81533

 

The feature automation was done for the above design code to be pushed to CSIT.

But by default this feature won’t be enabled.

 

Please kindly suggest on where should this script be added so that it can be used whenever needed.

 

Thanks,

Srinivas

+91-9243478719

 


JamO Luhrsen
 

Do you mean the changes to enable the feature will be similar to this?

https://github.com/opendaylight/integration-test/blob/master/csit/scripts/set_snat_mode.sh

if so, you will want to create a file like that but specific to na-responder.

With that, you can add that new script to the netvirt configplan:

https://github.com/opendaylight/integration-test/blob/master/csit/configplans/netvirt.txt

Next in the releng/builder repo you will need to add a new variable to
control this feature. Just follow the example of ODL_SNAT_MODE since that
was the example in the script above.

Make the default "off", whatever that would mean for this feature so that
the new script can run in the netvirt.txt configplan and just be a noop.

Next you have to figure out if having this feature on is ok to run for the full
-tdd-wip- job. If so, just add the new variable in releng/builder to toggle
the feature "on" for the -tdd-wip- job.

Finally, add your suite to the -tdd-wip- testplan  (and remove it from the
netvirt-1node-openstack.txt testplan.

If it's *not* ok to have this feature enabled for the full -tdd-wip- job, you
will probably have to make a whole new job just for this suite. Better if
you don't have to so it saves some resources, but let me know if you do.

Thanks,
JamO

On 4/13/20 11:30 PM, srinivas.rachakonda@... wrote:

Hi Jamo,

 

Thanks for the mail and help provided.

 

The changes are xml based where we are introducing a flag "na-responder-mode" which can be either controller or switch based.

 

https://git.opendaylight.org/gerrit/c/netvirt/+/81533/25/ipv6service/impl/src/main/resources/initial/netvirt-ipv6service-config.xml

 

 

By default the flag "na-responder-mode" is set to controller and existing CSIT scripts would go through.

The suite that I have written is for when the flag "na-responder-mode" is set to "switch"

This is to check how the flows are generated and check traffic with switch mode.

 

I want the script to be in a separate testplan, something like tdd- (same as used by NAT/Conntrack suites) which will be run on demand basis.

 

Thanks,

Srinivas

+91-9243478719

 

From: Jamo Luhrsen <jluhrsen@...>
Sent: 13 April 2020 23:16
To: srinivas.rachakonda@...
Cc: karthikeyan.k@...; 'Dayavanti Gopal Kamath' <dayavanti.gopal.kamath@...>; 'Nishchya Gupta' <nishchyag@...>; ravikanth.ps@...; integration-dev@...
Subject: Re: NETVIRT-1586 - OVS Based NA Responder for IPv6 default Gateway

 

Hi Srinivas,

I noticed your CSIT patch failed to verify, but that might be a problem
with the verify job. I see something to do with python2 vs python3. I'll
try to figure that out.

As for where and how this new suite will run, you already added it to
the netvirt-1node-openstack.txt testplan, so it will run in all those
jobs using that testplan. However, if the feature is not installed by
default, you will want to add that to the feature list in the netvirt
JJB jobs. Like this:

https://github.com/opendaylight/releng-builder/blob/master/jjb/netvirt/netvirt-csit-multi-openstack.yaml#L35

I would create the releng/builder patch, then push a job to the sandbox
using that code. Then run the sandbox job with PATCHREFSPEC pointing
to your int/test patch. If everything is as expected we can merge both
releng/builder and int/test patches at the same time.

Thanks,
JamO

 

On 4/10/20 9:27 PM, srinivas.rachakonda@... wrote:

Hi Jamo,

A new feature is to be added in Aluminium release.

 

https://jira.opendaylight.org/browse/NETVIRT-1586

 

I have added the below suite for OVS Based NA Responder for IPv6 default Gateway.

https://git.opendaylight.org/gerrit/c/integration/test/+/88772

 

The design link is:

https://git.opendaylight.org/gerrit/c/netvirt/+/81533

 

The feature automation was done for the above design code to be pushed to CSIT.

But by default this feature won’t be enabled.

 

Please kindly suggest on where should this script be added so that it can be used whenever needed.

 

Thanks,

Srinivas

+91-9243478719