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>

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>