NETVIRT-221: Fixed rules are not added when no SG is associated with the
port.

    1. When we add a port with SG disabled and no SG associated, the
    port does not have InterfaceAcl Augmentation. this is causing
    NullPointerException when port is updated with SG enable. So handled
    the NPE, fixed the issue and update flow will add the required fixed
    rules.

Change-Id: I0199f7eabad7f1b6caec8ce53412d86e50f63f51
Signed-off-by: kiranvasudeva <kirankumar.v@altencalsoftlabs.com>

NETVIRT-1227 : L2gw with multisegment network fail

Because of wrong index being captured in debug mode , failing with
ArrayIndexOutOfBoundException and elan instance creation is failing.

Change-Id: I0ef479bb5e545e4c8587dfa077ab4334afe778b3
Signed-off-by: eaksahu <a.k.sahu@ericsson.com>

NETVIRT-1200: Upstreaming fixes 2

VM interface IP not advertised to DCGW via BGP

Issue:
When compute server(CSS) is rebooted , the port state was
transitioning from UP->UNKNOWN and UNKNOWN->UP.The check for
original/updated state in InterfaceStateChange listener update() method
prevented the handling of states.

Fix:
When compute reboots, port state transition can have various
combination. Graceful transition would be UP->DOWN->UNKNOWN and
UNKNOWN->DOWN->UP. But as per the logs we saw UP->UNKNOWN and UNKNOWN->UP.
To handle all posible state transition , now State Transition Table is
introduced.

Change-Id: Ibd79866196f085dab30baaa11d8c5c526567e2fd
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>

Initial packaging support

Change-Id: I8d93396986ca24a1bb7f267d4fb348008a08f8c2
Signed-off-by: Sam Hague <shague@redhat.com>

reduce LOG.error() to debug() in ElanUtils.waitForTransactionToComplete

JIRA: NETVIRT-1215
Change-Id: I32f6273cce59a5cc0b1b03489bab31d2c9622e52
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

fix error handling in elanmanager-impl workers

It should not log confusing ERROR messages if it will retry (which it
currently does).  It should however fail the jobs, and the let the
JobCoordinator log the problem (and increments its failure metrics!) if
the caught RuntimeException is anything else than caused by a
TransactionCommitFailedException (currently that will lead to
addToListenableFutureIfTxException doing nothing and the worker to error
log it, but the JobCoordinator to think the job passed, when actually it
really failed).

This is "take 2" with a self counter proposal to the initial attempt in
change Idf603c6ec8679125a9659378af6627c5fb3eb49e, which would be
abandoned if this was deemed to be a more acceptable solution.

JIRA: NETVIRT-1215
Change-Id: I23be5fae88bf908273c47ade1e74348d1e20dada
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Wire in subparsers

Change-Id: I7f4f64ffb4914342e73ea1d35ebd1a74501f2f54
Signed-off-by: Sam Hague <shague@redhat.com>

integrate various updates

- move model map to models.py
- add json pretty_print for model dumps to files

Change-Id: I0553da82f3fa4f5ca441b45ee3122c6859c8deea
Signed-off-by: Sam Hague <shague@redhat.com>

integrate model operations

Change-Id: Ie1dc0946bbe906549a0ea71e59adec69cd2171f8
Signed-off-by: Sam Hague <shague@redhat.com>

ODLTools: Capture dumps for all data models

This adds all data models as captured in netvirt CSIT to
get_all_dumps()

Change-Id: Id7a0222915595ee6cf5f74a466bf2038bb10b046
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Fixes and additions

1. Fixes the lport and vrfid masks
2. Adds more dds models to constants for data collection

Change-Id: I652463b1a10a8c011809ae51080657bfd5c1f3f6
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Ensure NeutronPCL txs are closed

This is a minimal set of fixes to ensure that all write transactions
opened in NeutronPortChangeListener are closed.

Change-Id: I705da839bf39e2b6f832e28ba7fcfd625a086387
JIRA: NETVIRT-1210
Signed-off-by: Stephen Kitt <skitt@redhat.com>

reduce debug logging

Change-Id: Ieec658adc5a0f3ccfe31daf1d8ee79178c8036d7
Signed-off-by: Sam Hague <shague@redhat.com>

change logger format

Change-Id: Ib937127e05f9e735b9e8e28fb3f9cf050e667a75
Signed-off-by: Sam Hague <shague@redhat.com>

move deprecated files to old dir

Change-Id: I2c76bc58bb1a42581e5ac6b0dbd8491cc8f82abf
Signed-off-by: Sam Hague <shague@redhat.com>

updating spec with related yang changes.

Change-Id: I4f7ceed89fbca2321eaffbfab010768b70a31885
Signed-off-by: Siva Kumar Perumalla <sivakumar.perumalla@gmail.com>

Refactor odltools to use commands

- changed layout of packages to be more modular
- refactored logging to use a common root config

Change-Id: I88cc012e5599ddc29d75940bea396a4861403097
Signed-off-by: Sam Hague <shague@redhat.com>

Fix tox errors in existing specs

Change-Id: Id235926666a2623f943b13bfabc6b43a20eebc5e
Signed-off-by: Sam Hague <shague@redhat.com>

Minor fixes and addendums

1. Fix definition for itm-config
2. Add option to specify tmp data directory for logs
3. Changed json file names to .json from .log to distinguish from other logs
4. Add method to generate all json files

Change-Id: I459c66b9c5e64d6de95ab54a7f30a7e57605f003
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

providing range values for min-rx, min-tx of BGP-BFD.
removed threashold,cbit from yang.

Change-Id: I65a45066735646f23c07bc92b098a62af2637638
Signed-off-by: Siva Kumar Perumalla <sivakumar.perumalla@gmail.com>

Deleted ACL xtend-gen files

Looks like xtend generated files were unintentionally added to git,
hence removing them.

Change-Id: I27066534fbfc68dc4bbcb06970a880d7dbf24280
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>

Bump to odlparent 3.1.0 and yangtools 2.0.3

Change-Id: I06d9dadd753b64c3d717b028bb1ddfbcc71a6bf4
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Update robotfiles for latest csit format

Change-Id: Id7427b5e164e1ea010c4a208403532e3e2c65b5d
Signed-off-by: Sam Hague <shague@redhat.com>

No option to configure IPV6 on ODL
IPV6 option was not able in ConfigCli so adding the option in it

Issue: NETVIRT-1163
Change-Id: I59262991fba566ad6bbf08c56e9a99529c3d0389
Signed-off-by: Loshmitha <loshmitha@ericsson.com>

NETVIRT-1193: ACL dropping IPv6 RA packets from external router.

When an IPv6 subnet is created without specifying ipv6_ra_mode,
it is expected that the VMs with this subnet obtains global IPv6
address from non-OpenStack router using SLAAC.

Hence added below ACL flow to allow IPv6 RA packets from external
router if ipv6_ra_mode is not specified.
Since ipv6_src for RA packets are always link-local address, flow
contains match ipv6_src=fe80::/10 to allow from entire link-local
prefix.

cookie=0x6900000, duration=12.117s, table=240, n_packets=0, n_bytes=0, priority=63010,icmp6,reg6=0x400/0xfffff00,ipv6_src=fe80::/10,icmp_type=134,icmp_code=0 actions=resubmit(,220)

Change-Id: I030a99dd2e4385748a6b49cb2735e154b229da01
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>

fix unit test failure

Change-Id: I0fdcb1dd54cd59dea5d88734349c7a871cbb34ed
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

Add Trunks and duplicate flow analysis

Change-Id: Ieb3cadec7608e58153e0840f4abfa466564c4782
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Fix NPE in startArpMonitoring for a dual-stack network

Issue: NETVIRT-1206
Change-Id: Ic4ac243fe8378b7fa70606dcaf0bb7937f3fc80b
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

ELAN L2GW Junit Test Cases

1. verifyL2gw1Connection
   verify the dpn macs are programed in tor remote ucast
   verify dpn remote broadcast group includes tor endpoint
   verify tor1 mcast group includes dpns
2. verifyL2gwMac1InDpns
   verify tor mac is programed in dpn
3. verifyL2gw2Connection
   verify tor1 mac is programed in tor2
   verify tor1 mcast group includes tor2 tep
4. verifyL2gwMac2InTors
   verify new tor1 mac is programed in tor2
5. verifyL2gwMacDeleteInTors
   verify tor1 mac is deleted from tor2
6. verifyAddDpnAfterL2gwConnection
   verify tor macs are programed in new dpn
7. verifyDeleteDpnAfterL2gwConnection
   verify tor macs are removed from dpn
   verify tor mcast group does not contain deleted dpn
   verify dpn mac is removed from tor
8. verifyDeleteL2gw1Connection
   verify dpn bc group does not contain tor
   verify dpn macs are removed from tor

Change-Id: I648e246079d0c432037a4b6851275e7986ebf753
Signed-off-by: Yugandhar Reddy Kaku <yugandhar.reddy.kaku@ericsson.com>

Fix some minor issues found through static analysis

Change-Id: I314bef2f98fceaca399c3a9358b371deae6188eb
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Revert "Disable building sfc by default"

This reverts commit 03b638ad869fc47929e08dfd0464eee13ad357c2.

Change-Id: Id79e47d877428a070573c59c08bd55a78d2025d2
Signed-off-by: Sam Hague <shague@redhat.com>

Enable building of sfc by default

Change-Id: I7e1dace00f188e9426aee9b433cdce365155d7ab
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

fix BindingToNormalizedNodeCodec NoClassDefFoundError in sfc.classifier

Change-Id: I4bd5433160af22b8e59ed1dcf95a2439070ddfbd
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Missed one listener migration to genius/tools

Change-Id: I7d833da23221c4272834179b2281381cc0c11fd1
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Migrate from genius mdsalutil-api/utils to tools-api/utils

Change-Id: I5fa864aa9c206ddbfbdec0012c21f18db2c2abc1
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Clean up VPN data on VPN delete.

1. When a VRFTable is not created in FIB due to VPN addition, the VRFTable will
not be available during VPN deletion.
2. The prefix-to-interface, l3-nexthop datastores will not have any data for
a vpn, until a vpn-interface is up and running in the network.
Under such conditions VPN deletion causes
ModifiedNodeDoesNotExistException on trying to delete such datastores
which causes entire transaction which handles clean-up to fail, leave
stale-data in the system.

This further causes new VPN-create with same RDs to fail. This review
fixes the issue.

Change-Id: I3011d81907eb32693de3079bd118cfaaf1b9dfcc
Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>

NETVIRT-1158 ping to floating IPs of VMs start working only after
pinging from within VM to the outside first

When the packet returned to pipeline after sending to conntrack had
metadata which ofplugin was not able to parse. Now the metadata
associated with conntrack is cleared before resubmitting dispatcher at
table 214.

table=214, priority=62030,ct_state=-new-est+rel-inv+trk,ct_mark=0x1/0x1
actions=ct_clear,resubmit(,17)
table=214, priority=62030,ct_state=-new+est-rel-inv+trk,ct_mark=0x1/0x1
actions=ct_clear,resubmit(,17)

Change-Id: I0c01bb2da5fdc95f6f44e03e2f2d21881a55958b
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
(cherry picked from commit 2c548a85205e2bef3d004e6811423acba277b86e)

Revert "Revert "NETVIRT-430 Ping responder on tenant network and FIP-FIP traffic between vm in same n/w in same compute is not working in stateful SG mode.""

This reverts commit 5cc1d18d2fae5d5590b80e82416170641a786ff0.

Change-Id: I0a8df5496b04c76b71ead3fd2a0006c64d72393b
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

NETVIRT-1181: 3 issues related to port-udpate with Allowed address pairs
are addressed in this fix.
1. AAP added as port-udpate, is added for L2Broadcast allow rule.
2. AAP deleted as port-update, is handled to remove L2Broadcast allow
rule only if there is no other aap with same mac.
3. AAP deleted as port-udpate, is handled to remove Dhcp v4 & v6 allow
rule only if there is no other aap with same mac

Change-Id: Id422f2dd828265d12da4cc03b41f6cd4e6967569
Signed-off-by: kiranvasudeva <kirankumar.v@altencalsoftlabs.com>

Add fluorine and oxygen spec dirs

Change-Id: Iabcf865ff32ab6effa0af726bce56d5a61c57000
Signed-off-by: Sam Hague <shague@redhat.com>

NETVIRT-942, NETVIRT-945 : Conflicting modification for path...

*Conflicting path modification exception is seen for
elan forwarding tables when two methods, removeElanInterface
and removeElanStateForInterface, are trying two remove the
same elan forwarding tables entry

*This is fixed by adding djc for each elan interface for
remove method

Change-Id: I8139e53fe08a236348975e653bf7cad2a32f65da
Signed-off-by: eswanit <swati.udhavrao.niture@ericsson.com>

Convert ManagerListener to non-static HwvtepNodeHACache

This one popped up after the original patches to convert to
non-static HwvtepNodeHACache were submitted while ago.

Let's get this patch and the subsequent genius patch that
removes the static version in ASAP to finally put this to bed.

Change-Id: I657aa9d1b2065ac531117fd1236fe44865a8ae1a
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

NETVIRT-1173 : Prevent SNAT packets unnecessary looping

Problem Description:
======================
We noticed lot of SNAT packets are
getting punted to ODL which is causing delayed installation of certain
flows.

On analysis, it's been obseved that under one more scenario, packets
unnecessarily getting looped for 4 sec between ODL and OVS (threshold time we wait to flow
install before making drop decision)

1) N1-> subnet1 (10.0.0.0/24)
2) R1 and subnet1 added to R1.
3) VPN1(100:1) and R1 added to this VPN.
4) Ext-NET1 created and associated with Ext-BGPVPN.
5) Router-gw-set R1 with Ext-Net1

With this set-up, if following configuration is done,
certain packets are looped due to existing NAT bug

6) create N2(20.0.0.0/24) and associated this with VPN1.
7) Initiate TCP/UDP traffic from VMs of N2.

When first packet punted to ODL Controller, we make an
entry(<routerid>:<VM-IP>:<TCP/UDP Port>) in a map and try to find if
internal-to-external mapping available which will not be available for
N2's subnet(as this is not associated with R1) and we just drop that
packet. But, at this point we not taking care of removing of earlier added
entry. Hence, when the 2nd packet is punt to ODL Controller, this packet simply loops
between ODL<->OVS for 4 sec(same will happened with subsequent packets
too).

Solution Proposal:
==================
Changes done to prevent such packet being send back to OVS which is
causing congestion between ODL<->OVS.

Change-Id: I3215a5230f49e05e820b057d6fba415acad084a8
Signed-off-by: Shaik Zakir <shaikzakir119@gmail.com>

NETVIRT-1188: ACL: Stats not working for ingress direction

Fixed this problem by using lport tag in Cookie for drop flows.

Earlier, metadata matches which was available in pipeline egress side (VM
Ingress) is no longer supported. So, added lport details in Cookie itself
now to uniquely identify drop flows. Aligning with aforementioned changes,
removed metadata matches from direct stats input which only has Cookie
matches now.

Change-Id: Ifac03e75eb42b2a6f9dbd0d14579f462873d9285
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>

NETVIRT-1068: Upstreaming fixes 2

Issue-1 : When VM is configured with extra-route, refcount in l3nexthop is
incremented. It gets incremented further due to following reasons:

(a) After initial extra-route configuration using command - neutron
router-update RouterA destination=IP-A,nexthop=prefix-A , if another
update is done using command - neutron router-update RouterA
destination=IP-A,nexthop=prefix-B , neutron router listener calls update
on prefix-A as well as prefix-B. On prefix-A , secondary adj (IP-A) is
removed , where as its added on prefix-B. This back-to-back update creates
race-condition in Vrf Engine , leading inconsistencies in l3nexthop,
VpnExtraRoute, VpnInterfaceOp DS. (b) After initial extra-route
configuration using command - neutron router-update RouterA
destination=IP-A,nexthop=prefix-A, if cluster reboot is performed ,
TEP-ADD event triggered the update of FIB entry for IP-A. Update call in
FIB leads to increase in refcount of l3nexthop for prefix-A.

After refcount has reached high number due to issue-(a) and (b) , if VM is
migrated , group will not be programmed on destination DPN,, thus leading
to VM becoming un-reachable.

Fix: For issue-(a) , a temporary fix of 2sec delay is introduced in
neutron. A better fix/design need to be thought to avoid race condition.
For issue-(b) , after cluster reboot , secondary-adj FIB updated is
avoided due to wrong check in updateVpnInterfaceOnTepAdd method. refcount
is removed from l3nexthop yang because , after cluster reboot , due to
multiple Add/Update replays for a given prefix, refcount goes higher than
it should be. Hence prefixes using a group themselves are stored in
l3nexthop , so that even if spurious Add/Update are triggered post cluster
reboot for same prefix , its not updated.

Change-Id: I4618303345db1241b1826b018424b3df0f8bd9ec
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

NETVIRT-1187 Imported FIB Route is not being translated to flows

Caused due to NPE by calling a getter on a null extra-route object while
processing an imported FIB.
Also, remote table-21 flow creation was broken for imported routes.

Change-Id: Ib3783f9a83cf40d4560e10ba593600f47d3a176b
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

Migrate HwvtepHACache users to HwvtepNodeHACache

The static HwvtepHACache instance is replaced by the
HwvtepNodeHACache singleton service so migrate users.

Change-Id: I3bab48856df6d1ccb4aad0d34c90b4d68d64d668
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Java 8 migration

* Use method references instead of lambda expressions where
  appropriate.
* Use lambda expressions instead of anonymous functions where
  appropriate.
* Use Map::forEach instead of streaming over entry sets and retrieving
  matching values.
* Use Map::computeIfAbsent instead of manually handling missing
  entries.

Change-Id: I815d20d26ad7d7dfdf36b8f11d4d813ebc402457
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Drop unnecessary explicit types

Change-Id: Id6ea9e86b9c0a3bf9086d2efb7af89e277e2c56c
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Java 5 migration

* Use foreach loops instead of indexed for loops or iterating while
  loops.
* Use String::contains instead of String::indexOf.
* Use StringBuilder instead of StringBuffer, or a collector when the
  StringBuffer is only used to join a string collection.

This patch also simplifies the return value handling in
FibHelper::doesPrefixBelongToSubnet.

Change-Id: I13c63380e056cdeaa96090c16fee06316db782d9
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Ignore xtend-gen files

Change-Id: I0d198170d15416532c4ea1d67d257b4ca2fc6474
Signed-off-by: Sam Hague <shague@redhat.com>

Removal of deprecated ACL table constants.

With new ACl pipeline old ACL table constants are not deprecated same were
present in Unit tests. Renaming them with new pipeline table names.

Change-Id: I7419897ce5970d14539a0fe442b0abc3700df195
Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>

Disable building sfc by default

Change-Id: Ie64bb828f27d3e6ffc5ecc052fbb06c488f6d62e
Signed-off-by: Sam Hague <shague@redhat.com>

NETVIRT-1155: L3nexthop group count is not decremented for extra-route

Change-Id: I666c065ee72bbf604f596ec6c51a745f9466091f
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>
(cherry picked from commit 596786022adfadabe39b7a4fd30b1a1e3c87139b)

ELAN Service Recovery

Below service recovery options are supported
for ELAN:
* ELAN service recovery (deregister and register
  ElanInterface Listener again)
* ELAN Interface recovery

Change-Id: I7ded46eb34ec6c7942cd4a7903f2ad0a2a01df5d
Signed-off-by: eswanit <swati.udhavrao.niture@ericsson.com>

Add debug log for port status update

Make this puppy a little more debug-able.

Change-Id: Icd927169945112c4b06ff7f2d596a380a455bba3
Signed-off-by: Josh <jhershbe@redhat.com>

NETVIRT-1166:IPv6 NA packet is setting with wrong solicited flag

Problem Description:
====================
Currently ODL IPv6 Service is doing Neighbor Advertisement (NA) for
Neighbor Solicitation (NS) request.
NS packet may contains IPv6 Source Address as below possible way:
:: --> Unspecified
SLL --> Source Link Local Address

When NS packet is punted to the ODL controller, It will prepare the NA
packet and it will send this NA to NS initiated nodes.
NA packet contains IPv6 Destination Address as below possible way based on
the NS packet received.
FF02::1 --> All node multicast IP address (FLAGS: R:1, S:0, O:1)
Dest_LLA --> Destination LLA taken from NS (FLAGS: R:1, S:1, O:1)

Currently ODL controller is setting Solicited Flag=1 for All node
multicast IPv6 address. Which is wrong.

Solution:
===========
ODL IPv6 Service Defined constant with wrong value is the root cause of
this issue. To address this issue have made the changes properly with IPv6
constant.

Change-Id: I0b0868ad78a904c8e7ea4d0d157b16d6ef309974
Signed-off-by: karthikeyan <karthikeyangceb007@gmail.com>

Remove config import from inter-vpn-link.yang

CSS remnant - not needed

Change-Id: Icffb5ad18286663862443b2b8e4c1e323c8d96ba
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Remove last use of AbstractDataBrokerTest

This has been deprecated in favour of
AbstractConcurrentDataBrokerTest. Note that BgpManagerTest’s tests are
actually useless since they assert that 1 equals 1, but that’s
something for another patch.

Change-Id: I0d9b2a8013754cb56fc2eb111b225a750bdf2e02
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Remove all dlux dependencies

These were already disabled, this patch removes them.

Change-Id: I5e56495e15e70981382c177700868c3289c5da71
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Opening update server port with specified IP address (if configured)
update server port shall depend on EoS-entity-BGP.

Change-Id: Iad8bb1e1daf318c811453c2ddd622fb7e6187390
Signed-off-by: Siva Kumar Perumalla <sivakumar.perumalla@gmail.com>

Remove deprecated AsyncWriteTransaction#commit method

This method has been deprecated for quite some time and now has a
default implementation so can be safely removed from sub-classes.

Change-Id: I837cd59fcc40dff5fc9b69d7316046de382e697f
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

NETVIRT-1169 : Upgrade - GW-MAC flows are sometimes not programmed on some of the DPNs

Caused due to OLFE between vrfEntryListener.populateFibOnNewDpn() and
vrfENtryListener.createRemoteFibEntries().
This fix makes use of RetryingManagedNewTransactionRunner to ensure that
transaction is successful.

Change-Id: I0ca16484d71a6d47e5d64acb465fd6f0f55e7ef3
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

NETVIRT-1170: Deleting VM with a learnt IP does not clean up
prefix-to-interface DS for the VM

A check that ignored non-primary adjacencies while processing an
operational vpn interface delete was causing this failure. This is
observed in cases where a non-primary adjacency is THE LAST adjacency to
get deleted for the VM interface.

Change-Id: I114a1e0b176c246edecda9d8f07ef38af96224e4
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

RELENG-63: Move COE bundle to odl-netvirt-coe

odl-netvirt-impl currently includes the coe-impl bundle, which causes
issues for two reasons:

* coe-impl depends on coe-northbound-api (from the coe project), but
  odl-netvirt-impl doesn't depend on that (directly or indirectly); this
  results in the odl-netvirt-impl including a coe bundle which is
  incorrect;
* the coe-northbound-api is included in coe’s odl-coe-api feature, so
  the above results in two features containing the coe-northbound-api,
  which is also incorrect.

Moving coe-impl to odl-netvirt-coe ensures that odl-netvirt-impl itself
stays independent of COE, and ensures that no NetVirt feature directly
includes a COE bundle.

Change-Id: Ibe12e48eda592ff58af97f0b1df3da295b8f3f0c
Signed-off-by: Faseela K <faseela.k@ericsson.com>

NETVIRT-1062: refactor SubnetmapChangeListener

* SubnetmapChangeListener:

    don't need to call vpnSubnetRouteHandler.onSubnetAddedToVpn method
    second time if Subnetmap.getInternetVpnId attribute was set.
    According to initial design,
    vpnSubnetRouteHandler.onSubnetAddedToVpn method doesn't take in
    account InternetVpnId, when it creates/updates/removes
    SubnetOpDataEntry for the given Subnetmap.

* ExternalSubnetVpnInstanceListener:

    fix potential race condition. It is better at first to create an
    Internal VPN instance for the given provider subnet and only then
    try to update all corresponding objects.

* NeutronvpnManager:

  ** updateVpnInternetForSubnet:
    should always keep set VpnId attribute for updating Subnetmap.

  ** associateExtNetworkToVpn and disassociateExtNetworkFromVpn:
    to avoid race conditions it is better at first to update BgpVpnType
    of VPN instance and only then to update associated/disassociated
    SubnetMaps with a new InternetVpnId/null.

JIRA: NETVIRT-1062
Change-Id: Ia388b9112f2ca683864c2c15d8ac44ed0e0c25b8
Signed-off-by: Valentina <valentina.krasnobaeva@6wind.com>

Service-Recovery: Below service recovery actions are supported for ACL
(a) ACL service recovery: As part of this, all ACL listeners are
re-registered (unregistered and registerd again)
(b) ACL Interface recovery
(c) ACL Instance recovery

Updated with ACL-Instance recovery changes.

Service recovery actions supported are defined in SRM module with below
patch set:
https://git.opendaylight.org/gerrit/#/c/68987/

Depends-On: I4c91cd52e7595b92b494356dfa715dbdf15da3c5

Change-Id: I5517effbec7f00e8fd48e656d9443f525fc74eeb
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>
Signed-off-by: kiranvasudeva <kirankumar.v@altencalsoftlabs.com>
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>

Fixed typo changing "doesnt" to "doesn't"

My first delivery to the Netvirt project :-).

Change-Id: If4596d1031f15258cd490f4145e40d13725cde78
Signed-off-by: Ariel Adam <aadam@redhat.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

Revert "Service-Recovery: Below service recovery actions are supported for ACL (a) ACL service recovery: As part of this, all ACL listeners are re-registered (unregistered and registerd again) (b) ACL Interface recovery (c) ACL Instance recovery"

This reverts commit fb00e158d0f667046d89d66c996ace9b423af444.

Change-Id: Ib73043c75b70ddfeed132b0ef0485e9b737ab13c
Signed-off-by: Sam Hague <shague@redhat.com>

Revert "NETVIRT-430 Ping responder on tenant network and FIP-FIP traffic between vm in same n/w in same compute is not working in stateful SG mode."

This reverts commit 137537a26188f2139ee44eb35ad6ac945eb26a8b.

Change-Id: I61eb74d861245f61e6dceaae35911a370a0a3094
Signed-off-by: Sam Hague <shague@redhat.com>

NETVIRT-430 Ping responder on tenant network and FIP-FIP traffic between
vm in same n/w in same compute is not working in stateful SG mode.

Replace the dummy commit with zone 0 with ct_clear action. This clears
the ct state in the current packet and will submit the packet to ct
again. So in cases where ct(table=x, zone=2) action twice in the
pipeline( with same zone) , the second one also reaches the netfilter.

Requires ovs2.9 with kernel module installed.

Depends-on: Change-Id: I163ed99d5af8d3643749ac2a278f5c8170d7ae29
Depends-on: Change-Id: If86203e73a5ec516ff487f3f5c33fbd7dbfd12e1
Change-Id: Iab842c6232476d499a3d8c8602d5b2f57f9e4e1f
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Unit Test Cases Changes

With the new table changes in ACL current ACL unit tests are not working,
changes required accordingly to make them work.

Change-Id: Ia127caf792420fce141978c7309c524e0fb537b0
Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>

NETVIRT-1076: refactor arrow anti-patterns

* NeutronvpnUtils:

    ** getSubnetMapsforNetworkRoute() method is replaced by
       getPrivateSubnetsToExport(), which obtains the list of private
       subnets from router in more simple way;
    ** addExternalNetworkToVpn() and removeExternalNetworkFromVpn() were
       moved to NeutronvpnManager to avoid import collisions.

* NeutronvpnNatManager:

    ** use getPrivateSubnetsToExport() instead of
       getSubnetMapsforNetworkRoute();
    ** fix log messages.

* NeutronvpnManager:

    ** refactor arrow anti-patterns in associateNetworksToVpn() and
       dissociateNetworksFromVpn() methods;
    ** add a support of use-case, when BGPVPN can be associated with more
       than one Provider Network;
    ** fix log messages;

JIRA: NETVIRT-1076
Change-Id: Ifea2e0d2c2631ac87a3883f5a59d5fd3f189f812
Signed-off-by: Valentina <valentina.krasnobaeva@6wind.com>

Stop using MDSALDataStoreUtils

This patch removes all uses of MDSALDataStoreUtils, either using
SingleTransactionDataBroker, or a managed transaction.

Change-Id: Ia012cc825a9eb7768de91c1882f76127d44c3595
Signed-off-by: Stephen Kitt <skitt@redhat.com>

CLI to print DHCP caches.

Added dhcpcli to print dhcp caches .

Change-Id: I8df01ff9830dae6d8660e8430721a7ceeac41ddf
Signed-off-by: eaksahu <a.k.sahu@ericsson.com>

Service Recover for QoS Service as a whole & QoS Policy Instance.

For QoS service Recovery, listeners in the QoS service will be
re-regisered. Following command will do service recovery for QoS
service.
-----
srm:recover service qos
-----

For Qos Policy Instance Recovery, The policy id giving in the following
command will be reapplied on all the networks and port on which it was
applied earlier.
-----
srm:recover instance policy <policy-id>
-----

Change-Id: If02bc0a610d216f4387832ac21dac80aafb7d70e
Signed-off-by: Naveen Kumar Verma <naveen.kumar.verma@ericsson.com>

Switch to using lfdocs-conf to pull in theme

lfdocs-conf is a new project that allows us to centrally manage
project theme configuration without having to duplicate it
everywhere or have the "docs" project become a large submodule
project.

We will push to ReadTheDocs separately and then create a new path
for this doc under docs.opendaylight.org/projects/integration-packaging

For further details refer to:
http://docs.releng.linuxfoundation.org/en/latest/project-documentation.html

Change-Id: Iefbb38dd3fa4714c5018d6553bfde41e45e2bd37
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Service-Recovery: Below service recovery actions are supported for ACL
(a) ACL service recovery: As part of this, all ACL listeners are
re-registered (unregistered and registerd again)
(b) ACL Interface recovery
(c) ACL Instance recovery

Updated with ACL-Instance recovery changes.

Service recovery actions supported are defined in SRM module with below
patch set:
https://git.opendaylight.org/gerrit/#/c/68987/

Depends-On: I4c91cd52e7595b92b494356dfa715dbdf15da3c5

Change-Id: I21be0f9ed05eb3bf17ddb422ed10196e122b5cc8
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>
Signed-off-by: kiranvasudeva <kirankumar.v@altencalsoftlabs.com>

NETVIRT-1148:SNAT Traffic is not working with Dual Stack Router

Problem Description:
======================
Existing IPv4 NAT functionality is not working with Dual Stack Router
configuration scenario. Packets are getting dropped at ODL Controller.

Solution Proposal:
==================
Due to IPv4 and IPv6 Subnets are part of the single SNAT router, ODL
controller is not able to handle IPv6 Subnet properly. Due to this reason
IPv4 SNAT traffics are not working. To address this issue have made the
changes in IPv4 NAT service code to skip the IPv6 subnet process for
NATing. Since IPv6 NAT is not required. All IPv6 address are GUA
address's.

Change-Id: I7be6a999542e156047368203eee8e865b2c2eaac
Signed-off-by: karthikeyan <karthikeyangceb007@gmail.com>

NETVIRT-1144: PNF failure: Ping External Network PNF from Vm Instance 1

The rotuer<->bgpvpn swap procedure was being triggered even for addition
of a new adjacency to the vpn-interface, leading to a delay of two
seconds(the sleep present in the swap logic) in learning the PNF route.

Change-Id: I1178545da1d11dd7635fbdeb7f5848b25d2ade1a
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

NETVIRT-1132: NPE in TunnelInterfaceStateListener

Fix NPE caused by auto boxing by initializing
boolean in yang model to "false" so that the
Boolean object will not be null when checked,
for example, in if statements.

Details of analysis can be found in the JIRA
https://jira.opendaylight.org/browse/NETVIRT-1132

Change-Id: Iffa77778c178d7362e69b7cde3b0243a3d481de6
Signed-off-by: Victor Pickard <vpickard@redhat.com>

csit error log cleanup

when l2gw use case is not executed oper elan instance is not present
do not put error log if oper elan instance is absent.

Change-Id: Ie462ee96ce9239d5159b3a5809e0cce1558423f8
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

NETVIRT-1134 : ModifiedNodeDoesNotExistException bgp/networks in CSIT

This exception was being thrown when a delete was performed on a
non-existent node.
This was because of the data-remaining in the data-stores that were
created during the initial_create-networks on devstack bringup.

This patch aims at proper handling of deletion of those bgp routes that
have no nexthop.
It also prevents withdrawal of extra-routes from bgp when such the router
for the extra-routes is not associated to a bgpvpn

Change-Id: Ia666014c03b3f3454878487d3d42ef756a8bd2b2
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

NETVIRT-1141: Remote ACL count missing in acl stats data

While displaying the stats for acl it is not considering remote acl
drop in the count.
Providing implementation to consider that also.

Jira Id:https://jira.opendaylight.org/browse/NETVIRT-1141

Change-Id: I72fa8ad261ada769a65ae7e992e670523ed99b83
Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>

Switch to InstanceIdDataObjectCache

Change-Id: Ie6a2d4676d978d3f6ad192cfc1fb913b137b0b93
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Revert "NETVIRT-1134 : ModifiedNodeDoesNotExistException bgp/networks in CSIT"

This reverts commit eead978ce7038d1709ff0456134650ca603acf72.

Change-Id: I73248f5fef1cd799c87e3a2b593e0c22c702aef9
Signed-off-by: Sam Hague <shague@redhat.com>

NETVIRT-1146 : Exception IllegalArgumentException due to null subnet-id
during extra-route vrf-entry deletion

A bug in one of the checks during extra-route creation was leading
to exceptions in the delete path.
Label-Mapper is a bgpvpn exclusive data-store and should not be populated
for router based interal-vpn adjacencies. Such a population lead to
iRT/eRT code execution during the delete extra route life-cycle for plain
router case, leading to this exception.

Change-Id: I9bbfcd55a56f64793a322c647ad63f60c2a8ca1a
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

NETVIRT-1144 : Using JC for ExternalNetGroup Installation

Description:
Due to OptimisticLockFailedException during installtion of external
network subnet group, the installation of default FIB rule for
external-subnet CIDR been not installed resulting in PNF UC failure.

Change done to use JobCordinator fo installing this group there by
preventing multiple listeners attempting to install the same group
symultanously.

Change-Id: I667077eb2750e9e48019e5a2ad70520d3ada4d8b
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

NETVIRT-1134 : ModifiedNodeDoesNotExistException bgp/networks in CSIT

This exception was being thrown when a delete was performed on a
non-existent node.
This was because of the data-remaining in the data-stores that were
created during the initial_create-networks on devstack bringup.

This patch aims at proper handling of deletion of those bgp routes that
have no nexthop.

Change-Id: I0035866fdd4154680ca3f5a2f1ff67900b26a569
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

NETVIRT-1045 No connectivity between instances on different
networks connected to the same router when using VLAN setup

A pseudo port is added to ElanDPNList  when a vlan becomes a part of the
VPN for all the dpn which has the VPN. This ensures that the patch
ports to the physical interfaces are created and necessary FIB flows are
programmed.

The pseudo port will be add a subnet becomes a part of the VPN or when
the first vpn port is added to the DPN.

Change-Id: I722ac54c33d091e7f3820a32e86422bc29511905
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

statemanager: document the unused dependencies

StateManager appears to have a number of service dependencies which it
doesn’t use; but its whole point is to signify that the set of
services it depends on is ready, as a proxy for the system as a whole
being ready. This patch suppresses unused warnings.

Change-Id: I2777d89381ea9a02321645afe45d166a33ff21ad
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Enable fingbugs-slf4j enforcement in binding-parent

This ensures we won’t have any regressions in log handling in modules
based on binding-parent. We will update the dependency on odlparent
once the 3.0.3 or 3.1.0 release is available.

Change-Id: I91f21504c9a18580733a1eedd3ba379de79d37c2
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Fix logging issues in sfc

Change-Id: I251aafafc48dd3b52de968750869c3ac3253d347
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Fix logging issues in vpnmanager

Change-Id: I06ada5f51eb2afcdfaa9516a7943a4962dc3f7ca
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Fix logging issues in alarm

Change-Id: I1701c010e1646b70b76a01a0704ce45769854074
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Fix logging issues in cloud-servicechain

Change-Id: Iea8c9652d0350345c2148e1c18730a185cc176ad
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Fix logging issues in neutronvpn

Change-Id: Ib1a7e63a5bef3158ad73f632acf825548ab66fcc
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Fix logging issues in natservice

Change-Id: I249b8d5805c552e4f569a16aa764e864f00f8950
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Fix logging issues in coe

Change-Id: I5792e150df21ec22584e8c1307317f529e06fa86
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Fix logging issues in fibmanager

Change-Id: I392f9d26b237ac3de756efff1da26bcd0a1a6a87
Signed-off-by: Stephen Kitt <skitt@redhat.com>