Drop Maven prerequisite

The Maven versions are enforced by odlparent, projects no longer need
to care about them (unless a particular project-specific plugin
requirement crops up).

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

ipv6: Use versions from odlparent

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

Fixes logging exceptions, plus few formatting changes

 + Logging exceptions now comply with [1]

 + Adds some 'final' qualifiers

 + Also adds some missing braces for 'if' blocks

 + Removes some unused imports

 [1] https://wiki.opendaylight.org/view/BestPractices/Logging_Best_Practices

Change-Id: Idf860e0e8b87dfa740e5d0f3d6e1dfe14cce564d
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

Revert "Bug 6831: support for l3 directly connected subnet"

This reverts commit 53edd46eb630dcc069c8378e2800b326f04c4239.

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

Remove duplicate lockmanager bean

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

BUG 6840: New karaf CLI commands

This review creates 4 new CLI command:
1> vpn-show
2> subnet-show
3> neutronvpn-ports-show
4> vpninstance-op-show

Change-Id: I18405b8888d0e46c378d17b557b9e1eb328e0648
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

Modification cloud-servicechain-state.yang key

Due to elan-service-chain-state is an augmentation of
elan-instance, elan-instance-name can be removed from
elan-to-pseudo-port-data list key. The new key will be
composed with elanLportTag and scfTag.

ScfTag is now defined as an unsigned 32-bit number

Change-Id: I153910b5e8a105818fc3a2a3480399efd4b78086
Signed-off-by: Marcos López Samaniego <marcos.lopez.samaniego@ericsson.com>

Bug 6861 : Fix for proper tableId in punt action

- Changed the InstructionType from write_actions to apply_actions

Change-Id: I04f71945c71025c28815bfd057f3fb4bd72dae81
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>

Clean up logging tests

In most cases, checking whether a logger is enabled before logging
isn't necessary: the first thing the logger does is to check whether
it's enabled, before processing SLF4J tokens. It's only useful if the
construction of the logged information requires expensive operations
(joining list items, formatting ARP messages...).

This patch includes some minor drive-by clean-up.

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

Fixes bgpmanager-api folder structure

 + My eclipse didn't like it. Me neither

Change-Id: I86bd0464a9c87d69c475122938e7a67407967fb8
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

BUG 6841: Few Remote flows not deleted on DPNs

Few Remote flows not deleted on DPNs after migration from
internal to external VPN.
For a vpn associate/dissociate, the vpn interface update event wait was
getting accumulated for every interface across vpns. So, vpn manger was
getting removed before interfaces were cleaned.
A label of 0 was being used to create unique ids across vpn engine if
IdManager failed to return a valid label.
Java Synchronize has been replaced by LockManager to handle events
gracefully across PLs during simultaneous northbound and southbound
events for subnet routes.

Change-Id: I3cf1d1553bd6b6b025a9cedbc860b7af246f1afe
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

Bug 6589 adding support for hwvtep devices ha

Added template code to copy the data between ha and child nodes.
When a child node connects copy its operational data to its ha operational
node.
When operational update is received on child node , copy that operational
update to its ha operational node.
When some config update happens on ha node, copy that config update to
child node.

An add/delete Macs.PhysicalLocator,TerminationPoint,Switch update from
child is immediately pushed to parent ha node.
Duplicate updates from childs is properly filtered out while copying to
the parent ha node.

Depends-On: I7ad598fe9604aae9ad1df71270f1cba5b212a634
https://git.opendaylight.org/gerrit/#/c/44670/
Patch was reverted until netvirt catches up.

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

Bug 6842 :  Incorrect error msg upon associating router to VPN with non-existing VPN-ID

Corrected to indicate absence of VPN and/or router as per the input
supplied.

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

Bug 6823 : Performance improvement in DHCP

- Making use of the API provided by interface manager to fetch the
  interface object from config Datastore. This will avoid unecessary reads
  on DS which is time consuming.
- Majority of the diff is in adding an if check before logging so that
  string formation can be avoided.

Change-Id: I9595732506ec3e143a1ff6ed84039279eda92b09
Depends-On: I03a5ed6af26a2f110b0712a6c1ced160a5cdebf0
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>

Bug 6770 - Fixes DjC for NPortCL + snmaps serialized + listed changes

Dependent review on top of
https://git.opendaylight.org/gerrit/#/c/44708/2.

1. Fixes the following in the parent review:
a) reducing reads/writes
b) done away with djc for reads since if data is being upfront and
used/changed much later, it will fail concurrency as the data can be
changed in between.
c) used the correct keys
d) improved/corrected logging
e) djc on port, enabling parallelism

2. Concurrency changes for calls made via add/remove/update for
neutronportchangelistener, except router interface add/remove

3. Added support for neutron port update to have new fixedIP across a
subnet

4. Eliminated unnecessary snmaps update and sending out notif for port
update with no change in params

5. Localized addtoNeutronRouterInterfacesMap calls within create/delete
VPN interface, called from 1 place now instead of 4 each.

6. Synchronized subnetmaps reads/writes to attain serialization.

7. Fixes extra route functionality for internal vpn.
Earlier router id was read from vpnmaps and used to determine if extra
route is to be added or not, but vpnmaps updated routerid only upon
router assc to vpn, now instead snmaps which is populated upon router
interface add is used to figure out the same.

8. Added djc for create/delete vpninterface

9. Optimized createL3InternalVpn - eliminated routerinfmap population.

10. Fixed network and router cache population upon add/remove/update.

11. Removes lockmanager dependency from neutronvpn and function signature
for lock/unlock utility to not use lockmanagerservice.

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

Bug 6825:- "BgpManager not started" error when trying to configure Bgp peer
For commands class, bgpmanager not supplied as parameter

Change-Id: I385869448c86cb8927d9d4479bcb3620ce17e67f
Signed-off-by: SivaKumar Perumalla <sivakumar.perumalla@ericsson.com>

minor fixes related to BGP
- command output: F-bit always set to true and fetch Stale-path time from config,
      show GR-stalepathTime as default in case not-configured.

Change-Id: Ibf7a535075c9b0b02a5868c50f05d669c51d499a
Signed-off-by: SivaKumar Perumalla <sivakumar.perumalla@ericsson.com>

set FBIT for bgp to true (always), as we expect to keep the
forwarding state (of CSS) eventhough the controller goes down.

Change-Id: I25d151a56c035aea1d70bf3edbc83488ccbed820
Signed-off-by: SivaKumar Perumalla <sivakumar.perumalla@ericsson.com>

modified stale route cleanup timer to 600Sec, in case nothing configured.
enabled route removal on stale-path timer expiry

Change-Id: I5f19a026c5ac76a215b8d0de73bd0386ef435b32
Signed-off-by: SivaKumar Perumalla <sivakumar.perumalla@ericsson.com>

BGP networks update callback is triggered even if the content remain same
Fix: On Update callback, verify old and new values and act on it

Change-Id: I75918e2f25f56e78f70dc4f43254fe6f02380dd3
Signed-off-by: SivaKumar Perumalla <sivakumar.perumalla@ericsson.com>

BUG 6839: Fixes for import/export RT and router dissociation in L3Vpn

A label of 0 was being used to create unique ids across vpn engine if
IdManager failed to return a valid label.
Java Synchronize has been replaced by LockManager to handle events
gracefully across PLs during simultaneous northbound and southbound
events for subnet routes.
To find out all the vpns who import routes of a particular vpn, we were
dependent on the configurational datastore of that Vpn Instance. On a vpn
instance deletion, this config data is lost. Hence, this data has to also
be stored in an operational datastore

Dependent on https://git.opendaylight.org/gerrit/#/c/45220/

Change-Id: I317a543bbc46c6b388962182075bbe112428b648
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

Bug 6673: DCN to DTCN changes

DCN to DTCN changes for NeutronVPn, VPNmanager and FIBmanager

Change-Id: I4f78888148a228ed2938dc995b5b1ed1180d83eb
Signed-off-by: Vyshakh Krishnan CH <vyshakh.krishnan.c.h@ericsson.com>

BGP-configuration read is failing as shard leader is not available
implemeted retry mechanism in bgp-get-config (100Seconds) for MDSAL read

Change-Id: Ieb2cfd3135e5fb3dc5d83b26fc50e750247205f8
Signed-off-by: SivaKumar Perumalla <sivakumar.perumalla@ericsson.com>

BUG 6725: fix contains below issues

        1) Broadcast group buckets are not properly programmed for l2gw
2) Delete dpn interface is not cleaning up other dpns bc groups.

Change-Id: I8cd4df83777bb8ca8113a7eb7b214333f14536e7
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>

Bug 6691: Fix exceptions in natservice for a dual-stack network

This patch also updates some debug messages.

Change-Id: I840b0782d3e6b5978df63d8885cdc7c3b6c8cc9e
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Bug 6831: support for l3 directly connected subnet

The directly connected subnet case has limitation of mandatory GW.
The ARP request is produced as well to each new target belongs to subnet.
The ARP reply should to add new route to existing VPN port.
It should use received new MAC in case of directly connected network.
Existing algorithm does it in wrong way by importing mac from old entries.

Change-Id: I3df5496d97cf5bb8c68ed6ce301fb95f2449f781
Signed-off-by: Konsta Pozdeev <konsta.pozdeev@hpe.com>

Flow Entries to match ARP packets in GwMacTable(19)

* ARP packets matched in L3_GW_MAC_TABLE instead of L3_INTERFACE_TABLE
* Matched packets are punted to controller and resubmitted to 17.
* Arp pipeline flow will be now as follows :
* 0->17>19->punt_to_ctrl,resubmit(17)

Change-Id: I04ce9d49eea17cb4e15ba289af19d4a6702bb8c7
Signed-off-by: eceghkl <manu.b@ericsson.com>

Bug 6721: first few ping requests to a floating IP are receiving multiple responses

This fix attempts to prevent multiple Datapaths replicating
an IP response from external network to floating ip hosted
inside the cloud.

As part of this fix, we install floatingipportmac into
MyMac Table (Table 19 at the time of this commit),
only on those hypervisors where that floating-ip hosted
VM is present.

Please note that the duplicate IP response packets
replication problem does not exists when external network
uses tunnels (instead of VLANs) as the floatingip destined
response packets from external network will be directed
to only one hypervisor (ie., only one datapath).

The fix also makes sure that when a floatingip is
disassociated the corresponding floatingipportmac entry
is removed from MyMac Table.

There are still general bugs where floatingip rules in some
tables (including MyMac table) are not removed when:

a. Floating IP is deleted
b. VM hosting the Floating IP is deleted (without disassociating).

The workaround for both the above is to disassociate floatingip
first before embarking on a (or) b . Issue a and b will
be addressed with a different bug, as it needs a more deeper
fix.

Change-Id: Ic0eb7994a9829aa89aee5d8400e0d0be94dfac36
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Bug 6773: Floating IP response answered from all

don't answer to ARP request if it arrives from different dpn.

Change-Id: I50942db1ef198ab919a34513775983d7683303d3
Signed-off-by: Olga <olga.schukin@hpe.com>

Performs a residual cleanup of ElanPseudoPort flows

 + When an ElanPseudoPort is removed on the Northbound, we
   must ensure that no elanPseudoPort related flow is left
   behind in any DPN.

Depends-On: https://git.opendaylight.org/gerrit/#/c/45799/

Change-Id: Ib2bdebfe03f574d5bc89abac989505ef81f0f381
Signed-off-by: Marcos López Samaniego <marcos.lopez.samaniego@ericsson.com>

Bug 6758: Remove inter-VPN link state even if error

When an inter-VPN link in error state is removed, the
inter-VPN link is removed from MDSAL but not the state.
The bug was due because, when the state is error, it has
not any attribute with the endpoint states.

Change-Id: Ic8324c6ed5a5b5b3cf9a65694aff7ff137f3ae40
Signed-off-by: Alvaro Gonzalez ARROYO <alvaro.gonzalez.arroyo@ericsson.com>

Bug 6673 : DCN to DTCN Changes for various modules

- Moved all DCN in DHCP module to DTCN.
- Also added DhcpSubnetListener and DhcpHwvtepListener to blueprint xml.
- Moved bind and unbind method from DhcpManager to DhcpServiceUtils.

Change-Id: I03a5ed6af26a2f110b0712a6c1ced160a5cdebf0
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>

Bug-6446: Concurrency changes related to NeutronPortChangeListener

In the current commit, we seperated MDSAL reads/writes into different
DataStoreJobCoOrdinator (which will offload job from listener to
pool of thread based on key-Port).

Change-Id: Ide49674ff76fb5bc929f6f0126ba72292667dd34
Signed-off-by: SivaKumar Perumalla <sivakumar.perumalla@ericsson.com>

Mask IPv6Prefix in ACL flows

When programming an OVS flow that includes an IPv6 prefix
(like in allowed_address_pairs), the IPv6 prefix has to be
properly masked, without which the flow is not programmed
and it results in an OFPT_ERROR.

example:
IPv6 prefix 2001:db8::111f:c0cf/64 has to be masked as
2001:db8::/64 in the flow that is programmed on the switch.

Change-Id: I162bd6e20ae513f103d0e50d9c0fa60085152bd5
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Bug 6668 -  Security Groups (all implementations) - port_security extension and default DHCP/ICMP drop rules

Removed service binding and default flows from transparent acl service
implementation.

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

Thrift interface changes to support EVPN operations over Quagga BGP stack

Quagga BGP stack enhancement to support EVPN leads to Thrift interface modifications, which specifically include the following:
Type changes:
a) new AFI parameter - L2VPN
b) new SAFI parameter - EVPN
c) encapsulation types as per RFC 5512
d) layer type - whether a VRF belongs to Layer-1 or Layer-2
e) protocol type - No Overlay config/L3VPN/EVPN/ANY
f) FIB update structure updated with new parameters: l2label, ethtag, esi, macaddress, routermac
API changes:
a) addVrf() includes new layer type parameter
b) pushRoute() includes new protocol type parameter
c) withdrawRoute() includes new protocol type, ethernet tag, esi and macaddress parameter
d) getRoutes() includes protocol type parameter
BGPUpdater Service API changes:
a) onUpdatePushRoute: includes protocol type parameter
b) onUpdateWithdrawRoute: includes nexthop, ethernet tag, esi, macaddressm, l2label, l3label protocol-type parameter

Change-Id: I0b9a397b8ed6b62aaec30ef753726f185d1afd88
Signed-off-by: Sumanth MS <sumanth.ms@ericsson.com>

Bug 6716:Fix NPE in NeutronvpnNatManager

Change-Id: If77b4ae30eb8e4608da571153f3dfdf28649d853
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Bug 6589 adding support for hwvtep devices ha

Added template code to copy the data between ha and child nodes.
When a child node connects copy its operational data to its ha operational
node.
When operational update is received on child node , copy that operational
update to its ha operational node.
When some config update happens on ha node, copy that config update to
child node.

An add/delete logical switch update from child is immediately pushed to
parent ha node.
Duplicate updates from childs is properly filtered out while copying to
the parent ha node.Adding unt Testcase file for l2gw.

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

bgp logging fixes

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

Fix WARNING when port is updated with allowed_address_pairs

When a port is updated with allowed_address_pairs, a WARNING message is
logged in karaf eventhough the necessary ACL flows are programmed on
the port. This patch fixes this issue.

Warning logs seen in karaf:
Acl action UPDATE invoking listener org.opendaylight.netvirt.
aclservice.StatefulIngressAclServiceImpl failed

Change-Id: I5e3b20032ecd32b52d55995bad98253990e2858a
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Fix 6693
-DHCP Server responds to DHCP requests punted from its table(60) only
-DHCP server should not run at all when the controller-dhcp-enabled flag is false

Change-Id: I5e39ba4c7ee982f98ccb5499e902659147881f06
Signed-off-by: Ravit Peretz <ravit.peretz@hpe.com>

Bug 6708
Neighbor NAPT switches group table buckets remain empty
Fix race by triggering NAPT neighbor group table update
upon tunnel interface state addition

Change-Id: If1774c91eb9a42a5da575ec25ca83935fbb178b2
Signed-off-by: Ravit Peretz <ravit.peretz@hpe.com>

Bug 6727
ExternalRouterListener ignore multiple routers implementation

Change-Id: I390a666dd61304816f73b7405cc3abbccc45028b
Signed-off-by: Ravit Peretz <ravit.peretz@hpe.com>

Bug 6628 - DMAC for L3 entities flows installation only after reversal

Will install router interface, router GW and floating IPs DMAC table flows
only when L2 is before L3 in the Netvirt pipeline.

Change-Id: Ibcb025ecf44811ce02259c1e1aaf4d32782ece16
Signed-off-by: yair <yair.zinger@hpe.com>

Clearer failure message in AclServiceTest if no flows are created

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

Upgrade AssertBeans version, intro. nicer syntax w.o. build()

Also incl. some general clean-up: now all code in *-impl instead of some
in *-api, as that is probably much cleaner, and simpler to follow when
reading.

The 3 classes that are in test.infra will eventually move somewhere else
(but let's anyway get this in as proposed here ASAP already).

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

Merge "Bug-6741: eth1 flows on table 0 are missing from d2 ovs"

Merge "Fix wiring issue in openstack.sfc-translator-impl"

Merge "Bug 6752: DHCP service is not bound"

Merge "Bug 6748: ACL mechanism uses reg5 instead of reg6."

Merge "Bug 6756: Fix related to missing ACL flows"

Fix & un-@Ignore AclServiceTest

It turns out that c/45261 re. "@Override init() instead @PostConstruct
start()" broke AclServiceTest (c/42109); somehow Mycila Guice
Jsr250Module interprets @PostConstruct in parent class, or protected VS
public, differently than Apache Aries Blueprint impl.

This fixes it by putting @PostConstruct back, for now.  I'll separately
investigate a cleaner fix (probably going either in Mycila Guice, Genius
or infrautils; not netvirt).

This problem was not seen pre-merge, and thus broke master, because we
are in the unhealthy habit of merging a bunch of changes without
re-re-rebasing EVERY change after EVERY merge.  While that certainly
adds some overhead for commiters and build infra, doing that would lead
to more stability in such cases, as well as avoid Guitar Hero git log
<https://twitter.com/henryhoffman/status/694184106440200192>, which in
turn would make it easier to locally try git revert and git bisect to
find out which commit cause what.

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

Bug 6756: Fix related to missing ACL flows

When two SG's having common rules are swapped for a VM, ACL flows related
to common rules were missing.

Fix:
+ When SG's having common rules are swapped for a VM, delete the flows
related to previous SG and then add flows related to new SG instead of the
reverse order.

+ Fixed a NPE in AclInterfaceCacheUtil.

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

Merge "Fixes default SG remote groups rules. 1))Remote default SG rules are not added with Ip addres asn same is fixed 2)Flow id is fixed for ipv4 and ipv6 rules."

Merge "Bug 6690 - when mixing dpdk & non-dpdk OVS with the same ODL no way to configure different datapath types"

Merge "Bug 6742 FloatingIPHanlder should use the external interface-name"

Bug 6732: ARP Replies Intermittent for Floating IP Addresses

Don't stop ARP responder logic in case there is an ARP cache entry
for targetIp.
Even though this cache entry should never exist for IPs that the ARP responder
is responsible for, there is no reason to stop the ARP request
processing because of that.

Disabled AclServiceTest#newInterface() because it was failing.
Will fix it in another patch since it has nothing to do with this patch.

Change-Id: I392d9d9132350201e8d5761fe99c348223ea858d
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 6742 FloatingIPHanlder should use the external interface-name

Change-Id: I9fb58b26139afec6d09fa7792b7e94570804244d
Signed-off-by: Ravit Peretz <ravit.peretz@hpe.com>

Bug 6748: ACL mechanism uses reg5 instead of reg6.

Also, removed match on MACs in learned flows

Change-Id: I7d686532abe60585c7f4294c27da044716127cb0
Signed-off-by: Tomer <tomer.pearl@hpe.com>

Merge "Bug 6707 - FIB table rules are not created when DPNTEPInfo is not available"

Bug 6752: DHCP service is not bound

After making the changes in genius for ACL as highest priority both ACL
and DHCP were sharing the same service index because of which the service
was not getting bound.

Fixed this by making use of the service index provided in NwConstants.

Change-Id: I7a110ce0d454bc33c2f7e8c00690a4e800178ab1
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>

Merge changes from topic 'bp/annotation'

* changes:
  aclservice-impl *Listener without dumb @PreDestroy super.close()
  aclservice-impl use infrautils AbstractLifecycle

aclservice end-to-end test, with a bunch of cool new patterns

This originally increased coverage of aclservice from 53% as of de3d70d
to 65% (but things have moved a bit since this work was started). The
new approach introduced should make it easy to push it up further.

Meant also as an example for replication elsewhere; illustrates:

* manual Xtend objects for Flows
* generated Xtend objects by XtendBeanGenerator
* AssertBeans helper for readable test object comparison
* Mockito with CallsRealOrExceptionAnswer etc. (formerly "Mikito")
* Wiring DI best practice pattern, based on Guice

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

Bug 6690 - when mixing dpdk & non-dpdk OVS with the same ODL
no way to configure different datapath types

Change-Id: I0c23e8ecc4bc998a1694d1ca2196666264a43011
Signed-off-by: Tali <tali.ben-meir@hpe.com>

Bug-6741: eth1 flows on table 0 are missing from d2 ovs

Change-Id: Iaf5b6a7b8787c89af9d13d23059f15a56b72231e
Signed-off-by: Olga <olga.schukin@hpe.com>

Bug 6707 - FIB table rules are not created when DPNTEPInfo is not available

When new TunnelEndPoint gets created, attempt to re-process
associated adjacencies that were skipped when the VM port
was initially broght up as the TEP was still missing

Change-Id: I67b0ce164dbcb6217bcbf0b16cd771e4249e0a4c
Signed-off-by: Tali <tali.ben-meir@hpe.com>

Fix wiring issue in openstack.sfc-translator-impl

The bundle is pulling in the rpc-service RenderedServicePathService.
But in order to use it, we should use the odl-sfc-provider feature,
providing the implementation of this rpc-service, and also, registering
it in the OSGi service regisry.

Using the odl-sfc-model is exposing the class RenderedServicePathService
but don't register any implementation for this service in the OSGi
service registry.

Change-Id: I8b583bae9cdff7804b204d63ee88765a24679441
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

BUG 6452: Error logs when deleting neutron network

Change-Id: Id3448e93d3dd0f8bcea83a178e6abe81de75a8da
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>

aclservice AclInterfaceStateListener update() TODO replace with comment

Change-Id: Ic3bd87e29e35b9d424c5fcfefed43fc88cc0bd1d
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Merge "Bug 6688 - Patch port is not correctly associated to ELAN"

Merge "Bug 6677: Create ext-routers when a router is created with ext-gw"

Merge "Bug 6687: Fix NPE when updating ExternalNetwork"

Code for myMAC in the L3VPN pipeline

Code to program the L3_GW_MAC_TABLE.

Change-Id: I02252a9904c24c8ff37338b4b52cef49e73e9382
Signed-off-by: eceghkl <manu.b@ericsson.com>

Bug 6688 - Patch port is not correctly associated to ELAN

Change-Id: Iaa96881879ea1fc32dfa3001abd45bbb0595ce91
Signed-off-by: Tali <tali.ben-meir@hpe.com>

Bug 6677: Create ext-routers when a router is created with ext-gw

When a router is created with an External Network, no ext-routers entry
is created. This flow is used by horizon & several tempest tests.

Change-Id: I33d520a24db1c6f455c98213e31e081c8c8509e9
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>

Bug 6687: Fix NPE when updating ExternalNetwork

Happens when there are routers which are not associated with
a DPN yet.

Change-Id: I80f25439484d2e8c9f813826bec880a406421ee2
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>

aclservice-impl *Listener without dumb @PreDestroy super.close()

and with @Override init() instead @PostConstruct start()

This goes with (requires, depends on)
https://git.opendaylight.org/gerrit/#/c/45258/, and complements
https://git.opendaylight.org/gerrit/#/c/45249/

I've manually tested this in Karaf and made sure BP (diag) is OK.

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

aclservice-impl use infrautils AbstractLifecycle

and do NOT use (repeat) @PostConstruct and @PreDestroy - they are on
AbstractLifecycle, which does correct exception handling etc.

Based on https://git.opendaylight.org/gerrit/#/c/44554/ &
https://git.opendaylight.org/gerrit/#/c/45248/ (follow-up); on which
this is dependent.

Also includes fixed up features.xml in line with above.

I've manually tested this in Karaf and made sure BP (diag) is OK.

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

Fixes default SG remote groups rules.
1))Remote default SG rules are not added with Ip addres asn same is
fixed
2)Flow id is fixed for ipv4 and ipv6 rules.

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

Merge "Bug 6666: Making sure no 0 datapathID is used when adding interfaces to the model, and when the node updated with the datapathID, create the relevant interfaces"

Merge "Bug 6628 - Handling missing router entities DMAC table flows"

Merge "Bug 6609: when 2 vm belonging to the same NETWORK/SUBNET get created in different COMOUTE NODE - ping between those 2 did not work"

Merge "Support multiple routers per external GW"

Bug 6666: Making sure no 0 datapathID is used when adding interfaces to the model, and when the node updated with the datapathID, create the relevant interfaces

Change-Id: I213afac94544dc4e806cb684fc12126e358747b3
Signed-off-by: Tomer <tomer.pearl@hpe.com>

Merge "Fix bad design of AclClusterUtil to make it pluggable for e2e tests"

Bug 6628 - Handling missing router entities DMAC table flows

This is part of the L2 L3 pipeline reversal.

Change-Id: If91c3cf3a06ddfb5dbf1317fc8dea0a72616e25f
Signed-off-by: yair <yair.zinger@hpe.com>

Bug 6609: when 2 vm belonging to the same NETWORK/SUBNET get created in different COMOUTE NODE - ping between those 2 did not work

Fix for avoiding race condition while programming remote BC
group in participating DPNs. With earlier code, there was a chance of this
group getting overwritten by another thread

Change-Id: I09ae213c69ae971011661e713aaa92788eec1ea7
Signed-off-by: Gobinath <gobinath@ericsson.com>

Support multiple routers per external GW

Was finally able to see the basic tempest scenario test
passing with this patch! (with a small tempest workaround
to disable ping to router interface, which is currently not
supported in netvirt)

It includes the following changes:
 * Create a vpninstance (with a new vpnId) for the neutron external network
 * Associate the external interface with that vpninstance
 * Partially revert changes done in [1] & [2], to make sure external
   floating IP & NAT rules are associated with the new vpnId
 * Fix ExternalNetworkGroupInstaller to look for the gateway IP in the
   under the new vpnId. Also do some refactoring to allow better debug messages
   for ARP resolving.
   Also, drop traffic sent to external gw traffic if its mac is not resolved (Bug 6653)

This code leverages current implementation of NAT tables to translate external vpnId
to internal vpnIds for specific floating / PNAT IPs.

[1] https://git.opendaylight.org/gerrit/#/c/43303/
[2] https://git.opendaylight.org/gerrit/#/c/43102/

Change-Id: I8d3019389d7fe6fc923a509082e7a13d61a91444
Signed-off-by: Koby Aizer <koby.aizer@hpe.com>

Fix bad design of AclClusterUtil to make it pluggable for e2e tests

Having AclClusterUtil as a "Bean" with a @PostContruct start() method
but then a static method (isEntityOwner()) which relies on that start()
is bad design - it's either static or it's not, not half-half.  It's
better not be static, and have an interface; this way the (upcoming)
end-to-end AclServiceTest can easily plug in an "always true"
implementation.

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

Merge "Fixes Bug 6624"

Bug 6623: Fix NPE in AclServiceUtils

This patch fixes couple of exceptions when an IPv6 address
is present in allowed_address_pairs.

Change-Id: If0660b299500b7c00609d97484ea42961bb73dd9
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Fixes Bug 6624

Null check is added for dpid and aclUuidList.

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

Merge "Fix Checkstyle violations related to exception handling in elanmanager."

Merge "Fix NPE and other exceptions"

Fix Checkstyle violations related to exception handling in elanmanager.

read
https://wiki.opendaylight.org/view/BestPractices/Coding_Guidelines#IllegalCatch

This change also includes a few additional non-Exception handling
related minor Checkstyle fixes, and can now enable enforcement of
Checkstyle for elanmanager.  (This is not directly related to the
exception handling fixes, but those were simply the last Checkstyle
violations left, and in order to ensure future non-regression, we can
now, finally, enable enforcement.)

Change-Id: If233f3d856b2f5402ef551f8ceceb242376b671c
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Fix NPE and other exceptions

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

Add config for neutronvpn

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

Merge "Use right docker instance"

Merge "Bug 6614: Fix NPE in AclInterfaceCacheUtil"

Merge "Fix FloatingIp add GARP handling"