Bug 9034: LockManager cancel() ReadWriteTransaction if it's not submit()

Based on Robert's analysis, this change will not fix Bug 9034, because
that's due to a leaked TransactionChain (not the ReadWriteTransaction
being fixed here) - but this could well introduce ANOTHER memory leak
in mdsal, so we should get this in nevertheless ASAP.

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

Genius CSIT Failure : Missing Mandatory Node Error

Due to a recent commit in yangtools, Genius CSIT has broken
as the code was missing to populate one of the mandatory nodes in
interface-state

https://git.opendaylight.org/gerrit/#/c/48672/

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

Use INFO for logging null DPID

OVS does not assign a DPID until later in the bridge reporting cycle so it isn't an error if the DPID is not set.

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

Bug 8938 - Resource-batch manager enhancement

Adding capabilities to return Futures in Resource batch manager calls.
This fetures helps the caller to know whether the submitted tx job has
been successful or not.
And upon success or failure , the caller can take more corrective
action.

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

Move start() method into the constructor

Move start() method into the constructor to avoid the use of a
PostConstruct method, following the recommendations in [1].

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

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

Enable checkstyle at common-parent

Remove the checkstyle plugin per pom and enable it at the common level

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

Replace *Index(" by *Index('

Replace lastIndex("") and indexOf("") by their character counterparts.
This improve performance.

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

Fix initialization logs in service-binding helpers

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

BUG 9000 - Fix ITM config file not installed

Change-Id: Id20ed77d3e7564eaef064a3bc9881e1f472560c0
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

BUG 9001 - Fix race condition on OpenFlow based tunnels

This change ensures for openflow based tunnels that the child interface
entry is created before both the parent and child interface states are
added. Otherwise the entry might be added after the inventory listener
is triggered so that child interface state is never added.

This is achieved creating the child interface entry within the ongoing
job, which key is changed yo the parent interface name in case of
openflow tunnels, so that it is synchronized with the interface state
operations.

Additionally, fix a problem when adding the child interface state if
it is not the first tunnel interface on the node and the parent
interface already exists for the openflow based tunnel.
InterfaceManagerCommonUtils#addStateEntry would fail to do so, using
OvsInterfaceStateAddHelper instead.

Change-Id: I212514b3fcedd588f6b59be3f02aae930bc74e16
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Fix namespace violations in lockmanager-impl

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

Fixed YANG key'ed list order probs in InterfaceManagerConfigurationTest

YANG keyed lists are backed by maps, and re-created whenever they're
de-serialised (which happens e.g. when getAugmentation() is called,
apparently). This patch rebuilds TerminationPoints before comparing
them, sorting a couple of keyed lists.

See
https://lists.opendaylight.org/pipermail/genius-dev/2017-July/001713.html

Change-Id: Iab6189ca8e02c2b48a8b2b7516a1fc66b11363e7
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Ignore newTunnelInterface tests in interafcemanager

As it is not working consistent, and hitting timing issues in master

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

Merge "Logging enchancements for genius"

Bump versions by x.(y+1).z for next dev cycle

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

Logging enchancements for genius

Change-Id: Ic4fccb25f03fc9fe5eaa0b05e89e33f335c39422
Signed-off-by: Edwin Anthony <n.edwin.anthony@ericsson.com>

Merge "Log enhancements for better debuggability"

Merge "Bump odlparent 2.0.2 to 2.0.4"

Merge "Spec for ServiceRecovery Framework"

Bump odlparent 2.0.2 to 2.0.4

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

Merge "Bug 8882 - With conntrack SNAT communication with PNF fails "

Log enhancements for better debuggability

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

Merge "Add private constructor to this utility class"

Merge "Cleanup"

Merge "Changing the default value of Interfacemanager statistics polling interval to 15 Minutes"

Merge "BUG 8876 - Fix to remove ELAN interface flows when external network deleted"

Bug 8882 - With conntrack SNAT communication with PNF fails

Make the load metadata  start and end bit configurable.

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

Merge "Add GPE option to ITM config"

Bug 7599 -hwvtep ucast mac consumption improvement

Caching ITM External Tunnels so that it would be faster to get from cache
then to read from Data store.

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

Merge "Bug 8859 : Table 220 programmed with wrong service-index"

Merge "IdManager: Use lock only when required"

Changing the default value of Interfacemanager statistics polling interval to 15 Minutes

Change-Id: Ia7dbc21971e60fd788956876b8457cbdc989a88a
Signed-off-by: Gowri R <gowri.r@ericsson.com>

Merge "Make DJC to be more concurrent and fixing signal issue b/w enqueue jobs"

Bug 8859 : Table 220 programmed with wrong service-index

Fix synchronization issues with service binding transaction submissions

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

BUG 8876 - Fix to remove ELAN interface flows when external network deleted

Add check for null when getting Interface object from Config DS.
This changeset is related to the patch:
https://git.opendaylight.org/gerrit/#/c/61131/

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

Bug 8882 - With conntrack SNAT communication with PNF fails

Added support for load metadata action.

depends on https://git.opendaylight.org/gerrit/#/c/60869

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

Make DJC to be more concurrent and fixing signal issue b/w enqueue jobs

and job queue handler

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

IdManager: Use lock only when required

It was observed that there were more than one thread waiting to acquire
lock for the uniqueKey. This can be avoided since there is a local map
which tracks the idValue allocated for every request until the idValue
is persisted in the DS.

During testing, there were some conflictModificationExceptions thrown.
This was because the arrayList for delayedIdEntries were not set as
CopyOnArrayList at certain places.

Also have done some refactoring for allocateIdFromLocalPool()

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

Fix Service-binding flowRef

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

Cleanup

- Add private constructor to utility class
- Remove useless code
- Replace size() == 0 by is empty.

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

Add private constructor to this utility class

Add private constructor to this utility class.

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

Fix IfmClusterUtils logs

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

Merge "Enqueued Job not getting cleaned up in some cases"

Merge "Remove derivation from controller config-parent"

Spec for ServiceRecovery Framework

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

Merge "Avoid .transform() NPEs"

Merge "hwvtep transaction batching separation across shards"

Merge "Bug 8476 Add support for logical switch replication mode"

Merge "Change method names to comply with naming conventions"

Merge "Guard some TRACE logs in DJC"

Merge "Add a private constructor to avoid instantiation"

Merge "Remove un-used TestDataStoreJobCoordinator"

Merge "Remove un-used newWriteOnlyTransaction() in removeTerminationEndPoint()"

Add GPE option to ITM config

Add a GPE option on ITM config as a straight forward way to enable the
VXLAN GPE extension system wide.

Change-Id: I3cf5ce42cddef15c071a0b80ab0cc95a83e2b4d1
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Bug 8476 Add support for logical switch replication mode

With OVS 2.6, the Logical Switch table, in the hardware_vtep schema, has
a new field, replication_mode. This bug will be used to add support to
netvirt, ovsdb, and genius for setting the replication_mode of logical
switches. The default will be for ODL to set the replication_mode
to "source_node" when creating logical switches.

Refer to [1] for more details on replication_mode.

[1] http://openvswitch.org/support/dist-docs/vtep.5.html

This patch depends on ovsdb patch:
https://git.opendaylight.org/gerrit/#/c/57209/

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

Capture flow addition to debug log

debug log any flows that get added to config DS
trace log flow deletion

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

Change method names to comply with naming conventions

Change method names to comply with naming conventions.

And minor cleanup.

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

Remove derivation from controller config-parent

Genius does not use the config subsystem so does nto need to
derive from controller config-parent.

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

Add a private constructor to avoid instantiation

Add a private constructor to avoid instantiation of this utility class.

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

Avoid .transform() NPEs

Guava’s Optional::transform doesn’t support null values returned by
the mapping function — it throws a NPE if that happens. java.util’s
Optional::map does support null values and returns an empty Optional.
Combined with the current YANG-based code’s ability to return null,
this means it’s better to use java.util’s method; this patch switches
all YANG-based Optional transforms to java.util.

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

Enqueued Job not getting cleaned up in some cases

Jobs were not getting cleaned up from JobQueue in case of Throwables.
This results in some residual jobs remaining in the JobQueue forever,
and unnecessarily hogging CPU.

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

Merge "Port Number Change on OVS restart not handled by IFM"

hwvtep transaction batching separation across shards

Hwvtep helpers were doing a single transaction for operations
spread across shards, which is not recommended, and hence refactoring
the same. Also, in case of continuous HWVTEP channel flap, there are
some issues observed with tunnels not getting programmed on TOR.
Fixing this by avoiding read on topology/oper DS to derive global nodeId.

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

@Ignore flaky newl2vlanInterfaceTests InterfaceManagerConfigurationTest

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

Remove un-used newWriteOnlyTransaction() in removeTerminationEndPoint()

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

Remove un-used TestDataStoreJobCoordinator

The TestDataStoreJobCoordinator is not used anymore anywhere in genius
and netvirt, and can and should thus be removed now.

It was my first take on the async test problem, but it has since been
replaced by the
org.opendaylight.genius.datastoreutils.testutils.TestableJobCoordinatorEventsWaiter,
which it *IS* actively used e.g. in genius' interfacemanager and
netvirt's aclservice component tests.

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

Merge "Fix missing getter and toString in ActionNxResubmit & tests for it"

Fix missing getter and toString in ActionNxResubmit & tests for it

This fixes a problem in TestIMdsalApiManager related to xtendbeans hit
in a failure of aclservice's
AclServiceStatelessTest>AclServiceTestBase.newInterface.

Also ActionLearn, ActionOutput & ActionNxConntrack, ActionSetArpOp,
ActionSetDestinationIp. ActionSetFieldDscp, ActionSetFieldMeta,
ActionSetFieldMplsLabel, ActionSetFieldPbbIsid, ActionSetFieldVlanVid,
ActionSetTunnelDestinationIp, ActionSetTunnelSourceIp all similarly
broken for xtendbeans tests due to missing getters - it would never have
correctly shown differences in these, and just silently ignored any.

as well as further missing correct overloaded toString() in ActionGroup,
ActionRegLoad, ActionLoadIpToSpa, ActionLoadMacToSha,
ActionNxLoadInPort, ActionRegMove, ActionSetDestinationIp,
ActionSetFieldEthernetDestination, ActionSetFieldEthernetSource,
ActionSetFieldTunnelId, ActionSetIcmpType, ActionSetIcmpv6Type,
ActionSetSourceIp, ActionSetSourceIpv6, ActionSetTcpDestinationPort,
ActionSetTcpSourcePort, ActionSetUdpDestinationPort and
ActionSetUdpSourcePort.

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

Exception in service-binding logic when a neutron port is deleted

With the recent changes in ACL, deletion of a vpninterface triggers
an unbind and bind of ACL service. Parallelly interface-config
delete triggers one more unbind of ACL service, and the final unbind
results in NPEs in service-binding logic. The last unbind is anyways
a NOP, and hence returning with an ERROR message. Whenever ACL improvises
their logic, this ERROR will also automatically go away.

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

Port Number Change on OVS restart not handled by IFM

If port number for an interface changes on OVS restart, interface-manager
was not handling the same, and this results in stale entries remaining in
interface-state DS. Making the code robust to handle OF port number
updates.

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

Merge "VM Migration: Handle VM migration only for OFPT_PORT_STATUS/OFPPR_ADD"

Merge "Bug 8677: Bump odlparent from 2.0.0 to 2.0.2"

Bug 8163: Use async DTCL verification in MdSalUtilTest

https://git.opendaylight.org/gerrit/#/c/60022/ is changing the
DTCL executor to MT so the DTCL verification in MdSalUtilTest needs
to be done asynchronously instead of assuming direct notification.

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

Guard some TRACE logs in DJC

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

Merge "Bug 8800: TestIMdsalApiManager assertFlowsInAnyOrder fail safe"

Bug 8677: Bump odlparent from 2.0.0 to 2.0.2

This lets Windows users build Karaf distributions
locally more easily (without requiring Git Bash/Cygwin).

see https://lists.opendaylight.org/pipermail/odlparent-dev/2017-June/001154.html

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

Merge "Removing unnecessary TRACE log in service-binding"

Bug 8800: TestIMdsalApiManager assertFlowsInAnyOrder fail safe

It *MUST* fail in case that:

  assertThat(sortedFlows).containsExactlyElementsIn(sortedExpectedFlows);

failed but:

  assertEqualBeans(sortedExpectedFlows, sortedFlows);

passed; any such case is likely a bug in ch.vorburger.xtendbeans.

netvirt aclservice tests (or their expected flows) must be fixed before
this change can be merged.  This doesn't fix anything, it would just let
us discover this kind of problem much earlier, if things ever broke
again.

The new FlowEntityAssertBeansTest adds a non-regression test for the
reoot cause problem; it fails as of the time of writing, but will pass
once the ch.vorburger.xtendbeans dependency is bumped from 1.2.3 to the
1.3.0 in https://git.opendaylight.org/gerrit/#/c/59950.

See bug 8800 for full background and details.

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

Removing unnecessary TRACE log in service-binding

This TRACE log is printing the whole tree of a change notification.
There are other informative logs already in place after the initial
processing, and this one is not needed.

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

Making ActionConverterUtil log to DEBUG

As these logs are filling up netvirt CSIT logs

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

Bug 8800: Prevent xtendbeans NPE warn.log in InstructionApplyActions

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

Remove odl-genius-ui from artifacts and features

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

DO NOT install or deploy the karaf artifact

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

BUG 8528: Fix duplicate Apply Actions Instruction

When an egress service sends it's own Apply Actions instruction, Genius
will generate two Apply Actions instructions southbound, one of it's
own containing the action to set REG6 and a different one containing the
service supplied actions. This is not correct per Openflow specification
and OVS rejects the operation.

Fixed by bundling the set REG6 action and the service provided
actions into the same Apply Actions instruction, taking care of
not rearranging the order of the service actions.

Change-Id: I48735857418fed0025b172b73eb02be9c8e9bbbe
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Upgrade to odlparent 2.0.0, karaf4 and Guava21

1. Change odlparent-lite to 2.0.0
2. Delete genius-features and rename features4-genius to genius-features
3. Change single-feature-parent to 2.0.0
4. Cleanup in karaf/pom.xml for karaf4
5. Fixes for Guava21
6. Removed odl-genius-ui feature, removing dependency on dlux

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

Merge "Genius wide blueprint xml path change: /OSGI-INF/ -> /org/opendaylight/"

Merge "avoiding logging full stack trace in case of id doesn't exist"

Genius wide blueprint xml path change: /OSGI-INF/ -> /org/opendaylight/

Also have removed fake "availability=optional"

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

Don’t load odl-genius-ui on boot

This drops odl-genius-ui as the Karaf 4 distribution’s local feature,
so that it’s no longer loaded as a boot feature. The Genius feature
repository will still be part of the distribution.

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

changing karaf3 to karaf4 for genius features

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

avoiding logging full stack trace in case of id doesn't exist

* This fix is to avoid filling karaf logs exception with full of stack
trace if source of exception is known by its exception type and message.

Change-Id: I057647f9a4ac90c32476e8d9eb2494a1b9d3c4a2
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Merge "Bug 8146:BFD tunnel monitoring interval update fix"

Remove uses of Optional for conditionals

Optional instances should never be constructed just to facilitate
conditional handling of the form

    Type variable = ...
    if (variable != null) {
        ...
    }

Turning this into

    Optional<Type> variable = Optional.ofNullable(...);
    variable.ifPresent(value -> ...);

causes more GC activity and makes the code harder to read (and
analyse).

This patch doesn’t address the issue of whether the values in question
are actually nullable.

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

Bug 8146:BFD tunnel monitoring interval update fix

Added support for dumping more caches
Caches were not getting cleared correctly. Fix to remove tunnels from Cache whenTEPs are deleted.

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

Merge "Use TestEntityOwnershipService that's now in controller"

Merge "Listener base classes should not throw exception if close()'d "

Migrate to odlparent 1.9.0

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

Fixing a typo in DJC log statement

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

Merge "Bug 8555 - Too many log info print to the console"

Use polymorphism instead of Optional

Optional shouldn’t be used instead of method polymorphism. This patch
deprecates
InterfaceManagerCommonUtils.createInterfaceChildEntry(String, String,
Optional) and replaces it with variants using either no transaction or
an explicit WriteTransaction.

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