Fixup Augmentable and Identifiable methods changing

This is a fixup of the change in binding codegen, adjusting:
- getKey() -> key()
- setKey() -> withKey()
- getAugmentation() -> augmentation()

Change-Id: I19a391660f0bc30b0ec8d043859d9d20b81f3dfd
JIRA: TSC-101
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Conflicting modification exception for 47->21 flow

Description : Observed this issue can happen when both
ExtenalNetworkChangeListener and ExternalRouterListener try to install the
47->21 flow for external-network simultaneously.

Changes done to synchrornize these events by using jobordinator having
extnernal-network id as key.

Issue : NETVIRT-1135

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

NETVIRT-1137: NPE: Direct Exception

Elan Interface Manager calls for interface details by making an API call
to Interface Manager which returns a null object because interface is
alreay deleted. Elan Interface Manager is now checks for null before
proceeding.

Change-Id: I89bf739b3cc5a98a686ff89a26f41937ed3779b9
Signed-off-by: Dhiman <dhiman.ghosh@ericsson.com>

Fail builds on warnings

Also remove the unnecessary echo statement. Sphinx already prints a
message to indicate where the docs are at the end of the build so is
redundant and the echo fails Windows builders.

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

Resolve Sphinx build warnings

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

unblock broken build by replacing some but not all Xtend from aclservice

This rewrites some of the Xtend with Java.  It only replaces those
*Builder classes, but does not (yet?) replace the expected objects with
some TBD not Xtend based approach.

As it is something related to Java generics in those Builder classes
which seems to (suddenly) somehow be causing the build failure, this is
sufficient, for now, to unblock broken build.

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

Remove use of InstanceIdentifier.builder(InstanceIdentifier)

This method has been deprecated for 4+ years, do not use it.

Change-Id: Ia01331bd67d8769e9247871c1c0ebfa9622145ee
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

NETVIRT-1163:No option to configure IPv6 on ODL
No option was available to configure IPv6 in ConfigCli , so adding one

Change-Id: If7b6ec769f7aa41c1494e0bd6b6f793120661a19
Signed-off-by: Loshmitha <loshmitha@ericsson.com>

Handling NPE

Issue : NETVIRT-1276

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

NETVIRT-1255: Corrected wrong goto table number specified with two of the default flows

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

Remove AsyncWriteTransaction.submit() impls

The submit() method is now defaulted in the interface
so implementations don't need to implement it.

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

New Route Origin For Routes learnt dynamically in the data-path

Invisible ips learnt in the data-path are currently being denoted as
"static" in FIB. This review introduces "dynamic" for such routes.

This is introduced with reference to:
http://study-ccna.com/connected-static-dynamic-routes/

Area#4
Review#1
l3vpn-robustness spec : https://git.opendaylight.org/gerrit/69526

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

NETVIRT 1202 : AS number issue during display
Value of AS number was an integer which gave rise to negative values so
changing it to long to increase the range of the value
Signed-off-by: Loshmitha <loshmitha@ericsson.com>
Change-Id: I587cd867e55a0535d09b1c3b082fe71455c20eff
Signed-off-by: Loshmitha <loshmitha@ericsson.com>

QoS Alert code cleanup

1. Decouple NBI neutron commands and QoS alert cache creation.
2. Port is added/deleted from the cache based on ovsdb notification.
3. Handle the case where counters get wrapped.

Change-Id: I2172613574f343cd13860065ee30ca517bfd61df
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>

controller-punt-protection: yang changes

yang changes for configuring hard timeout parameter in learnt flows

link: https://git.opendaylight.org/gerrit/#/c/71023
Issue: NETVIRT-1218

Change-Id: Ic4194b534203950c863617cc1d6700bc6a17b1a4
Signed-off-by: Ravindra Thakur <ravindra.nath.thakur@ericsson.com>

NETVIRT-1241 : Preprovisioning of l2gw conf. fails

Fixed the usecase of device connected after l2gw is
already provisioned.

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

NETVIRT-1227 : Refactoring segment creation

For multisegment network , the code is unnecessarly using multiple loops
and varibles , increasing the probability of NP and
ArrayIndexOutOfBoundExceptions.
Hence refactoring the code.

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

handle tunnel port add/up event

tunnel port is not a neutron port
do not put in port cache if it is tunnel port

dhcpManager.getNeutronPort(interfaceName) throws exception
when the supplied name is not a proper uuid.

in case of tunnel port port name is not a uuid

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

Adding counters for HwVtep Listeners.

Added infra utils counter in hwvtep southbound listeners.

Change-Id: I51ab3509149cfd3f6f64011149a94de6a0cfc693
Signed-off-by: A K Sahu <a.k.sahu@ericsson.com>

NETVIRT-1282 updating default BGP config server port 7644
modified default log file location of bgp

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

The NFIP to FIP traffic is failing when there are multiple route

The issue occurs when the FIP vm is atached to router1, NFIP is attached
router 2, and there is no port in router2 subnet present in FIP vm
compute node.

NETVIRT-1238 failure in Ping Vm Instance1 Floating IP From SNAT VM
Instance1

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

Revert "Use managed transactions in elanmanager-impl"

This reverts commit 8611eb3618e0831475f940d0fcc8e3c7135faed8, which
introduces an exception because two transactions are submitted with
conflicting changes.

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

Enable SNAT flows on router update

Currently, flows are removed on router update, and never
re-installed. This patch ensures that the flows are enabled.

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

Forbid manual transactions throughout

Now that all modules use managed-tx-parent, merge it with
binding-parent so all future modules are automatically protected.

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

Use managed transactions in vpnmanager-impl

This also enforces restrictions on newReadWriteTransaction and
newWriteOnlyTransaction calls, to prevent new code introducing
unmanaged transactions.

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

Use managed transactions in elanmanager-impl

This also enforces restrictions on newReadWriteTransaction and
newWriteOnlyTransaction calls, to prevent new code introducing
unmanaged transactions.

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

Use managed transactions in natservice-impl

This also enforces restrictions on newReadWriteTransaction and
newWriteOnlyTransaction calls, to prevent new code introducing
unmanaged transactions.

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

Eliminate InstanceIdentifier.toInstance() reference

This method has been long-deprecated, do not use it.

Change-Id: I1df75a9072c3be3113357f338d651124a990f957
Signed-off-by: Robert Varga <nite@hq.sk>

netvirt-1115: ecmp configuration on non bgpvpn router is possible again

The issue consists in allowing ecmp configurations for non bgpvpn
routers. Actually, there is a filtering mechanism with BGPVPN that
consists in limiting the number of ECMP entries with the number of
router distinguisher configured on the VPNInstance. In cases, where the
VPNInstance is a router, there is no need to do that.

Change-Id: Ic6ccc5419d6e9228651cbdaf8519a788560358e9
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

NETVIRT-1261: ACL Stats: Drop stats is 0 sometimes

Fixed issue mentioned with this bug. With the fix drop stats should be
returned properly from ACL Stats RPC in all scenarios.

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

added dhcp packet coutnters

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

Avoid UT failures in AclServiceIPV6 tests due to errors in shutting
down of JobCoordinatorImpl, before next tests are initiated

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

Adjust to RPC method signature update

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

adding support for evpn rt3

Type 3 routes are required for Broadcast, Unknown Unicast and Multicast
traffic in an EVPN environment.

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

Enable discovery of Virtual IPs (or Movable IPs)

VNF's like "load-balancer, high-availability",
require the vMAC/IP remain same even after multiple
incarnation at different OF-ports. ODL MUST provide
seamless connectivity to all these VNF's.

JIRA: NETVIRT-1253
Change-Id: I4ee60f22599d52177fb624bc11b70aaa73c2513a
Signed-off-by: Siva Kumar Perumalla <sivakumar.perumalla@gmail.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Signed-off-by: A K Sahu <a.k.sahu@ericsson.com>

Revert "NETVIRT-1197: Stale entries exists in ACL caches"

This reverts commit 74e32b83f8ed5c58ddd323edcc6587fa99aca5d7.

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

NETVIRT-1080 : Fix PNF discovery failure after FIP detached

Problem:
  In conntrack NAT mode, deleting last FIP port on dpn also
  deleting the PNF flow entries ion the OVS node.
Solution:
  The problem is caused by missing ExternalFixedIP entry
  in the vpn-to-dpn-list for the external subnet on the
  NAPT switch. The existence of this entry will prevent
  deleting the PNF flows when the FIP is disassociated.
  The patch fix the problem by adding/deleting the
  ExternalFixedIP entry when the external subnet is
  associated/disassociated with the router.

Change-Id: Ib46e860448138c6808105573ae4c52be0bc2b437
Signed-off-by: Vinh Nguyen <vinh.nguyen@hcl.com>

added spec for evpn and l2gw integration

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

Retain Earlier Elected Napt Switch After Upgrade for SNAT

This Spec attempts to retain the earlier elected Napt-Switch for a
given Router(VRF) before upgrade as Napt-Switch even after the upgrade
of OpenDaylight Controller.

JIRA: NETVIRT-1222

Change-Id: I2fc9bd917c3bdf65e7cd788f9bd4435e0a3ad3c6
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>
Signed-off-by: Shaik Zakir <shaikzakir119@gmail.com>

NETVIRT-1197: Stale entries exists in ACL caches

This patch is to add SecurityGroupListener to remove stale entries in acl caches

Currently, ACL module receives add/update events on ACL object via
NeutronSecurityRuleListener, but could not get remove event.
This new SG listener would help ACL module to receive remove event also.

Change-Id: I914eda387cd7dcd07b4c4eb9194630edc3970535
Signed-off-by: Tarun Thakur <tarun.t@altencalsoftlabs.com>

Fix for VM Migration Table 51 stale entry

Description: An issue is observed when
ElanLearnVpnVip MIP MAC and Interface
MAC are same. During migration,
ElanLearntVpnVipToPortListener remove
deletes MAC entry from ElanInterfaceForwardingEntries,
which is referred by ElanInterfaceStateChangeListener
remove workflow, which is started after the
ElanInterfaceForwardingEntries thread
completes its course of action. Both threads
have an action to delete DMAC flow entries,
but the first thread refers InterfaceManager for
InterfaceInfo which returns null, hence skips the
delete DMAC workflow, but deletes the MAC entry.
Other thread refers MAC entry, which has been deleted
by first thread, hence this too skips delete workflow.

This fix address the problem by deleting MAC entry in
first thread, only if InterfaceInfo is not null so that second
thread can delete DMAC entry as well delete MAC entry
during its course of action.

Change-Id: Icc7d15b62f6aaa3b1bae1844d9a013043b8d541d
Signed-off-by: ashwinbc <ashwin.b.c@ericsson.com>

NETVIRT-1254: ext-nw VRF not published to Quagga

IP family is not being set for Internet BGPVPN with external network
associated. Hence due to no exchange of RD, iRT, eRT information, routes are
not getting published to quagga.

This commit fixes the same.

Change-Id: Ic259afe889120b48baedd58538238cb7b63b6111
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>

NETVIRT-1200: Upstreaming fixes

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

Spec for Tap as a Service

Included the review link for pipeline changes
Addressed review comments
Rebased

Change-Id: I095c7caedc66aa58693210c63bb27a6bca1b476a
Signed-off-by: Hema Gopalakrishnan <hema.gopalkrishnan@ericsson.com>

Spec: OVS Based NA Responder for IPv6 default Gateway

This spec addresses OVS Based Neighbor Advertisement(NA) responder for
IPv6 default Gateway. Neighbor Solicitation(NS) request which has
initiated from IPv6 configured/enabled Data Center(DC) VMs are served by
OVS based NA responder.

Issue: NETVIRT-1228

Change-Id: I2cabf7ae45d59c737216f50e0d4b941c4b06d2a4
Signed-off-by: Karthikeyan Krishnan <karthikeyangceb007@gmail.com>

The SNAT FIB flow miss entry id is made more unique

NETVIRT-1157 Stale flows after clearing gateway info from a router

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

Spec to realize Neutron BGPVPN APIs within OpenDaylight Netvirt

This specification addresses several gaps in existing
L3VPN Service implementation in NetVirt, such that
L3VPN Service can robustly realize VPNs managed via
the Openstack Neutron BGPVPN APIs.

Change-Id: I2e15416f15055b27245cfd0ddd60640689e9007f
Signed-off-by: Vivekanandan Narasimhan <vivek.konnect@gmail.com>
Signed-off-by: Sam Hague <shague@redhat.com>

spec: controller punt protection

The spec aims to address the issue of potential overloading of the
controller due to excessive punting of packets done via different
flow tables in netvirt pipeline.

Issue: NETVIRT-1218

Change-Id: I8b879b8ccf26bdc3b4e90e43d59b812a45a2db46
Signed-off-by: Ravindra Thakur <ravindra.nath.thakur@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bucket entries are not updated after
karaf restart:

After controller restart, NPE was coming from
getPrefixToInterface which was disturbing
bucket entries (getBucketsForRemoteNexthop).

This is fixed by adding a null check for
getting prefix-to-interface.

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

NETVIRT-1224 host routes not updated in VM if subnet is updated with host routes

Root cause: NeutronSubnetChangeListener implements AsyncDataTreeChangeListener
   and therefore subnet update is sent only to config leader and local cache on
   other nodes is not updated. If DHCP packet is processed by non config leader
   node, it replies with old/stale subnet host routes.

Fix: Instead of using NeutronvpnUtils cache, read subnet data from config
   datastore. This will not add any performance impact because in normal working
   case when entry is not found, data is retrieved from config datastore only.

Change-Id: If563ed264c1aadb14cfbdb8e5086d84068d2dfd2
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>

Migrate some old Java constructs

- Replace lambdas to method references
- Remove some redundancies

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

Qos Policy Instance Recovery will now handle cases where incorrect
Policy Uuid is provided.

Change-Id: I72c91ceef29b56560b2825b0c9dc953bf72e2cf5
Signed-off-by: Meenakshi <meenakshi.c@ericsson.com>

Netvirt-QoS: new listener for bw rules programming

Currently bandwidth limit rules are programmed from interface listener
which should only be used for flow programming which in case of QoS
does dscp marking flow programming.

Now, ovsdb termination point listener is introduced to program the
bandwidth rules on the VM ports. This will also make qos alert cache
management better since QoS policy can be applied to both ports and subports
but bw limit rules are only applied to ports.

Change-Id: Ib2be2b59eb7a6b4d739f9827086581365ce90455
Signed-off-by: Ravindra Thakur <ravindra.nath.thakur@ericsson.com>

Spec: Subnet Routing for hidden IPv6 addresses

This spec addresses Subnet Routing feature for hidden
IPv6 traffic wherein ODL would learn the
hidden/invisible IPv6 addresses and set up forwarding
rules for both intra-DC and inter-DC communication.

Issue: NETVIRT-1213

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

NETVIRT-1252: NPE while creating internet VPN interface

Due to an NPE introduced as part of
https://git.opendaylight.org/gerrit/#/c/57411/, VPN interfaces belonging to
internet based VPNs are not getting created. This is part of a DJC job which
creates IETF interface for the neutron port as well. Due to this failure,
table 0 entry is ultimately not getting realized.

This commit fixes the same.

Change-Id: I8cdcb273c47c2627583cd367fa4dbfba446298a2
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>

Update odltools version to 0.1.8

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

Set docs version to Fluorine

Master branch is currently Fluorine.

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

Update odltools version to 0.1.7

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

Optimize tox options

JIRA: NETVIRT-1232
Change-Id: I5f1e3c70678b98a97144ca1f7db3b3d6cba1f83b
Signed-off-by: Sam Hague <shague@redhat.com>

Fix some prints

JIRA: NETVIRT:1232
Change-Id: I9318bf47b635960aa8c4e8956b8c00565560f0f4
Signed-off-by: Sam Hague <shague@redhat.com>

Add flake8 support and fix issues

To run flake8:
- cd to root of odltools project, .../odtools
- tox -e flake
* tox by itself can be used but will also run the unit tests

JIRA: NETVIRT-1232
Change-Id: I2702c1073474b9e3d12b9ae65c5252b3887ebd5b
Signed-off-by: Sam Hague <shague@redhat.com>

Use ifname in place of ifName

JIRA: NETVIRT: 1232
Change-Id: Ia65f2dd43ed2cfd14259617b644663b31381835b
Signed-off-by: Sam Hague <shague@redhat.com>

Use transport in cli for http/https

This also allows using tcp or ssl when ovs is wired in.

JIRA: NETVIRT-1232
Change-Id: Icde4ae6684be3adfa03b87d91c952649471adac9
Signed-off-by: Sam Hague <shague@redhat.com>

Refactor Model class

- reduce some of the boilerplate code from the classes
- move the get_container and the by_key methods to the
the base Model class

- Added tox integration to run tests
- Changed the way test resources are found so it works
for both tox and other test runners

JIRA: NETVIRT-1232
Change-Id: I86b7c67dd14a4dfdea7b016c21807c47a69539bf
Signed-off-by: Sam Hague <shague@redhat.com>

NETVIRT-1227 : L2gw with multisegment network fail

The starting index from segmentation index is 1 not 0 which is another
reason for failure.

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

ODLTools: Misc CLI improvements

* Display table name alongwith table id in flow outputs
* Add option to print flow meta info only
* Add option to print flow URLs
* Add option to use https
* Clean-up a stray print of elan interfaces

JIRA: NETVIRT-1232
Change-Id: I719138a6945d854f650931b9e3244861dc64a99f
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Migrate some old Java constructs

- Remove unboxing
- Replace lambdas to method references
- Remove some redundancies

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

NETVIRT-1225 QOS bandwidth rule not pushed to ovs

Root cause: Configuration fails because config change is not applied
            in child node if parent does not exist.

Fix: Use write config transaction 'merge' method instead of 'put'
     pass a flag to true to create the parent if does not exist.
     this flag is default false in case of 'put'

Change-Id: I0f1a1ae6c6eaf28719969998a7ac147c7be2c236
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>

NETVIRT-1207 QoS Alert not working

Now karaf uses Log4J2, custom qos alert logger configuration
no longer works. Moved to log4j2 logger and appender config.

Change-Id: Id6105d6734851b6c9d213bdfe8d966a4a42ca9fa
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>

Add doc for debugging port status update

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

Fix outdir to be path arg

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

ODLTools: Misc fixes and additions

1. Add using ifindexes for mapping lport to interface
2. Fix tunnel state always showing none
3. Fix 'show flows elan'
4. Fix dpn to hostname mapping

JIRA: NETVIRT-1232
Change-Id: I9bda2ce3d77c3b011fee8ed308b46957cf199229
Signed-off-by: Vishal Thapar <vthapar@redhat.com>

Use managed transactions in fibmanager-impl

This also enforces restrictions on newReadWriteTransaction and
newWriteOnlyTransaction calls, to prevent new code introducing
unmanaged transactions.

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

Use managed transactions in neutronvpn-impl

This also enforces restrictions on newReadWriteTransaction and
newWriteOnlyTransaction calls, to prevent new code introducing
unmanaged transactions.

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

NETVIRT-1198: Acl Stats does not support Anti Spoof drop stats.

Anti-spoofing stats support was not present currently. Support has been
added, and for implementing that all the drops from Anti Spoofing table
has been redirected to acl committer table by modifing 4th bit in
metatdata.
A new flow has been introduced in committer table with this new
match criteria and their we are dropping all the drop packets.

This new flow will contains all the anti spoof drops and using the cookie, we will
be pulling the table details and displaying anti spoof stats

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

NETVIRT-1171:IPv4 entries appearing in FIB though not associated to Router

Problem Description:
====================
IPv4 entries are appearing in FIB when subnet is not associated to router
in a dualstack network

Solution Description:
=====================
Dual Stack VM has both IPv4 and IPv6 Address, IPv6 or IPv4 subnet only
associated to router means it should show only IPv6 or IPv4 address based
on the subnet address family. To do this behaviour we need to create the
VPN-Interface adjacency based on the subnet associated to router. Have
done the code changes accordingly in VPN-Interface adjacency.

Change-Id: Id1a4712dc1b8be0713fe7dcb526555c5e5021e39
Signed-off-by: Karthikeyan Krishnan <karthikeyangceb007@gmail.com>

NETVIRT-1194 : NAT has two Listener on Interrface-State Oper DS

Description : Currently, NAT has two listeners on interface-state
Operational DS. One performing operation related to SNAT and other
related to DNAT.

These two Listeners can be combined to one which reduces querying on
certain DS queries commonly done by both of these listeners.

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

Fix ifname to be ifName

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

Implement AsyncWriteTransaction.commit() in BatchedTransaction

Implement the new commit() method which uses FluentFuture
instead of the deprecated CheckedFuture. The commit() method
is defaulted in the interface but, as soon as all implementations
implement commit(), the deprecated submit() method will be defaulted
instead.

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

Provide a class in VPN to manage cluster ownership.

The cluster ownership management can be accomplished
by few approaches.  However, this initial review
attempts to decouple the ownershipmanager from the
rest of the vpn system.

Area 1
Review #1
Refer l3vpn-robustness spec: https://git.opendaylight.org/gerrit/69526

Change-Id: Id3f4258ea960728f438d09d41e38a4a677f9362f
Signed-off-by: Vivekanandan Narasimhan <vivek.konnect@gmail.com>

Use managed transactions in qosservice-impl

This also enforces restrictions on newReadWriteTransaction and
newWriteOnlyTransaction calls, to prevent new code introducing
unmanaged transactions.

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

Use managed transactions in statistics-impl

This also enforces restrictions on newReadWriteTransaction and
newWriteOnlyTransaction calls, to prevent new code introducing
unmanaged transactions.

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

Prevent new code from using the DataBroker

This introduces a new Checkstyle check to fail the build if code
*calls* the newReadWriteTransaction or newWriteOnlyTransaction methods
in DataBroker. I haven’t found a nice way to make this check
conditional, so for now it’s added via a new parent module,
managed-tx-parent, which is used instead of binding-parent in all
modules which currently support it.

This patch is self-contained, but the plan is to move the Checkstyle
implementation to infrautils and the Checkstyle configuration to
Genius or MD-SAL, alongside ManagedNewTransactionRunner. Once all the
existing code has been converted to use only
ManagedNewTransactionRunner, we’ll be able to apply the configuration
to binding-parent and do away with managed-tx-parent.

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

NETVIRT-1201 Upstream fixes to master --- 7

Read Vrf Table within the DJC to establish/clear VPN footprint on a
DPN.

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

NETVIRT-1190: Local Adjacency for extra-route is not cleaned up from the
vpn-netxhop data-store

Extra routes with one next-hop should be treated as non-ECMP routes. This
fix rectifies a condition to update local-nextHops for extraroutes with
only one next-hop

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

Use ifName instead of ifname

JIRA: NETVIRT-1232
Change-Id: I835e92c5a352e8bc482858739ef1673efa5bdce3
Signed-off-by: Sam Hague <shague@redhat.com>

Move ovs under netvirt

JIRA: NETVIRT-1232
Change-Id: Ibc9f5460a42bca1005402b196d3c16c03f53677f
Signed-off-by: Sam Hague <shague@redhat.com>

Wire in analzye inventory

JIRA: NETVIRT-1232
Change-Id: I8052cb5aa0f160157ed5801065f7f49fb2bac6ce
Signed-off-by: Sam Hague <shague@redhat.com>

Wire in flow commands and parser

JIRA: NETVIRT-1232
Change-Id: Ife96728f41012a4fc1d002e461d19eab652b085a
Signed-off-by: Sam Hague <shague@redhat.com>

Elan Junits fixes
follow up to:https://git.opendaylight.org/gerrit/#/c/68187/

Change-Id: Ia20fa0493bfacd1d7ef231d17a28e9ff9017ebbd
Signed-off-by:Edw7n <n.edwin.anthony@ericsson.com>

Bump odltools version to 0.1.2

Change-Id: I5e291fafc021299f439e97f95c6a55808530509e
JIRA: NETVIRT-1232
Signed-off-by: Sam Hague <shague@redhat.com>

Add more show and analyze commands

Also renamed the resource model files to not have
: in the name since that is a special character
with Windows.

JIRA: NETVIRT-1232
Change-Id: I2d1dd093451854da1caedffd8d5a2a1ca5602db8
Signed-off-by: Sam Hague <shague@redhat.com>

Use quotes consistently

JIRA: NETVIRT-1232
Change-Id: If7d435a8f5d58b2a5dc0befbea524357ca69df2a
Signed-off-by: Sam Hague <shague@redhat.com>

Add setup clean option

JIRA: NETVIRT-1232
Change-Id: I1eb28dfd2c1e70ff284ddf6f63eba28c816d0d1b
Signed-off-by: Sam Hague <shague@redhat.com>

itm-direct tunnel related changes for scaling
depends on genius RPC implementation:
https://git.opendaylight.org/gerrit/#/c/68397/

Change-Id: Ib65733e577182067d02d0e90234c06591da09074
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>

Router internal address not reachable

The router internal address was not reachable since ovs is marking the
packet as -trk. Now -trk packets will be resubmitted to ct to have
second lookup.

This is a workaround until ovs bug is fixed. Now in table 239 ct_clear
is called for both tracked and untracked packet.

table=244, n_packets=39, n_bytes=3822, priority=62030,ct_state=-trk
actions=ct_clear,resubmit(,242)

table=239, n_packets=5, n_bytes=378, priority=100,ipv6
actions=ct_clear,goto_table:240
table=239, n_packets=246, n_bytes=35726, priority=100,ip
actions=ct_clear,goto_table:240

NETVIRT-1064 No connectivity to a router's internal address

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

VMs on tennant vlan network not getting an IP when spwaned after
the networks are added to the router.

The router port elandpnlist didn't have a state. So when the first port
is router port in the list, the elan flows fails.

Check is added to skip router port while checking if a port is the first
one in the list.

NETVIRT-1208 An Instance doesn't get an IP after when a network is added
to a router

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

Move files around to fit pypi better

JIRA: NETVIRT-1232
Change-Id: I0632fbcaa83c20f8ab5fd2f2ccadfe97b389d600
Signed-off-by: Sam Hague <shague@redhat.com>

rm odltools egg files

JIRA: NETVIRT-1232
Change-Id: I62d29b48c79e76a3a4a548e9b74967be071389a2
Signed-off-by: Sam Hague <shague@redhat.com>

NETVIRT-1157 Stale flows after clearing gateway info from a router

The CentralizedSwitchChangeListener was not able to retrieve the router
object for delete hence added cache for it.

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