Move StringValueObjectFactory to mdsal-binding-spec-util

This class does not need to reside in yang-binding, move it out
into mdsal-binding-spec-util.

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

Move DataObjectReadingUtil to mdsal-binding-spec-util

This moves access utility from yang-binding to mdsal-binding-spec-util,
as it has no place in yang-binding.

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

Move NotificationListenerInvoker to mdsal-dom-adapter

These classes should not be in yang-binding, as their concern lies
in runtime API realization. Move them to mdsal-dom-adapter.

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

Move RpcServiceInvoker to mdsal-dom-adapter

These classes should not be in yang-binding, as their concern lies
in runtime API realization. Move them to mdsal-dom-adapter.

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

Add NotificationListenerInvoker bridge class

This adds a migration bridge for controller, so we can safely
move NotificationListenerInvoker from yang-binding.

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

Add RpcServiceInvoker bridge class

This adds a migration bridge for controller, so we can safely
move RpcServiceInvoker from yang-binding.

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

Port WriteTransaction constants from controller

This adds the CREATE_MISSING_PARENTS and FAIL_ON_MISSING_PARENTS
constants present in controller’s implementation of WriteTransaction.
(They help make code easier to read.)

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

Remove odl-mdsal-models

All models have been split up into component odl-mdsal-model-*
features, remove odl-mdsal so it does not get misused for
'give me the world', which we cannot be achieved from here anyway.

JIRA: MDSAL-363
Change-Id: I1491790ce75de9950b3578a748277c03db23a333
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove odl-mdsal-models from features-mdsal

First step in removing odl-mdsal-models, remove it from our
master feature repository.

JIRA: MDSAL-363
Change-Id: I33ee69a10f0cb728f54e85bc0a838ffe1a1c55b2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move DOMRpcRouter implementation methods

This moves actual implementations of DOMRpc*Service into facades,
which allows us to properly support multiple interface, without
leaking them through DOMRpcRouter itself.

JIRA: MDSAL-283
Change-Id: Ie7d2515c35e160ff4827cd999d962bd961b5ad9a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Update *DataBrokerTest* classes from controller

The equivalent classes in the controller have diverged since
they were first copied so update them in mdsal.

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

Allow DataObjectModification to address grouping nodes

This adds the API and implementation required to address child
nodes introduced in cases by the use of a grouping.

Change-Id: I70871dcaa3441fbef81dfb15bfd965d133aa0894
JIRA: MDSAL-342
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert binding read to java.util.Optional

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

Change DOM read/exists to FluentFuture

CheckedFuture is deprecated. The only user(s) is in controller -
companion patch is https://git.opendaylight.org/gerrit/#/c/74128/.
This breaks the controller so either we merge them together or
coordinate one after the other with a (hopefully) short breakage window.

Change-Id: If69281a6a6710974c5625b2e09a09df611409129
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove DOMRpcRoutingTable.invoke()

Moving this method makes DOMRpcRoutingTable blissfully ignorant
of the invocation model, simplifying code flow and bringing us one
step towards reuse.

JIRA: MDSAL-283
Change-Id: I207362f00b7051f92f5303ef923eb6fe767f6130
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Shutdown DOMRpcRouter

Blueprint did not call close(), which means we were leaking threads.

JIRA: MDSAL-283
Change-Id: I96f2e58bf57645ecdba3b9b7fa05b2b86ce78c9a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Harden DOMRpcRouter shutdown

Use AbstractRegistration to get idempotenence. Also remove listener
register prior to shutting down the listener notifier.

JIRA: MDSAL-283
Change-Id: I948510a080c2540d309f81fa8a42591341db595e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Disallow DOMOperationInstance wildcards

Wildcards will complicate service lookup, let's postpone that to
when we actually need the capability.

JIRA: MDSAL-283
Change-Id: I445fd150424f41d242d7dca6875fe012e3c7891e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make Rpc/Action extend DOMOperationImplementation

This is an API omission preventing implementations from working
with AvailabilityExtension.

Change-Id: Iad2efc6aa5b4fb9c4ef4065c8f81d8eaf312d369
Signed-off-by: Jie Han <han.jie@zte.com.cn>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Do not retain reference to DOMRpcRouter in registrations

Once we have called removeListener() we will not interact with
the DOMRpcRouter, hence there is no need to retain a reference
to it.

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

Split out odl-mdsal-model-rfc7224

This is the feature to host iana-if-type, finishing move of
all odl-mdsal-models bundles into smaller chunks.

JIRA: MDSAL-363
Change-Id: Icfb3a924c11936ed9e288ee46a616217f6a5640f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Correct RFC7227/RFC7277 typo

Using 7227 is a typo, correct it before someone notices.

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

Split out odl-mdsal-model-draft-ietf-netmod-iana-afn-safi-00

This feature hosts the iana-afn-safi@2013-07-04.yang model,
which is ot going to go anywhere as the document has explired.

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

Remove ancient iana-if-type

This artifact is not used anywhere, remove it to prevent confusion.

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

Split out odl-mdsal-model-draft-ietf-lisp-lcaf-10

This splits out ietf-lisp-address-types into a separate feature.

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

Split out odl-l2-types

OpenDaylight L2 types are an independent model, allow it to be
consumed as such.

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

Split out odl-mdsal-model-draft-clemm-netmod-yang-network-topo-01-minimal

Most of our users are interested in the base topology model only, there
is no need to burden them with other models.

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

Rework model feature packaging

This patch breaks the monolithic odl-mdsal-models into multiple
features, one for each IETF document. This allows us to load only
models specific for a particular use case and documentes where
the modules where imported from.

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

Eliminate unneeded constant

Empty YangInstanceIdentifier is available as a constant, use that
instead of building our own.

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

Change binding read to FluentFuture

CheckedFuture is deprecated. There's no users of this
API yet so safe to change.

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

Add ActionService adapter

A mostly straightforward implementation of ActionService on top
of a DOMOperationService.

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

Teach DynamicBindingAdapter about properties

We needs to exert some control over how a we re-export services,
like ignoring re-exports and overriding properties.

Define the API contract in ServiceProperties and implement the two
features mentioned.

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

Remove unneeded .gitignore

This is not really needed and it keeps a bogus directory alive.

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

Generate stringValue() in union types

We need to vacate getValue() method name, as that may be specified
by user. Create a new stringValue() method, which additionally
contains a String rather a char[].

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

Teach binding-dom-codec about actions

Serialization of actions is slightly different than serialization
of RPCs or notifications. Teach BindingCodecContext how to deal
with them.

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

Speed up service adapter object methods

Rather than checking the method name twice inline co-locate signature
checks and result return.

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

Simplify DOMOperation{Service,Implementation}

In order to seamlessly bridge Binding->DOM->Binding invocation
without incurring thread safety hits we need to have a way
for the operation implementations to give us a handle.

We therefore should not be bridging the result through a callback,
but rather via a FluentFuture -- eliminate DOMOperationCallback
and make the former bridge methods the only API.

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

Add BindingLazyContainerNode

In order to cleanly migrate LazySerializedContainerNode from
binding-dom-adapter to codec, we need a baseline in codec.api.

BindingLazyContainerNode provides this baseline interface,
and AbstractBindingLazyContainerNode provides a reference
implementation.

We also improve type safety by requiring ContainerNode instead
of a raw NormalizedNode.

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

Remove unneeded lambda cast

This removes an explicit dependency on Guava's Function.

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

Log a message when we are falling back in QName search

Computing the QName should be mostly unneeded, log at debug
reasons for us having to fall back.

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

Speed up Action QName/QNameModule getter

Action has a QNAME field, hence we can access it directly.

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

Deprecate yang.binding.RpcImplementation

This is an old tagging interface, which is not used anywhere,
probably since we have started running binding RPCs on top
of DOM infrastructure.

Mark it deprecated for now, we will either remove it if it is not
useful or un-deprecate and documented it if we find a use for it.

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

Tag RPC input/output classes with RpcInput/RpcOutput

These interfaces are not just plain DataObjects, they have
a specialized meaning. This meaning was not useful when RpcInput
was a DataContainer, but with it being a DataObject, we can do
better.

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

RpcInput/RpcOutput should be DataObjects

RPC code already adds DataObject (instead of RpcInput) here, let's
make sure this is captured in the API contract.

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

Add ActionService and ActionProviderService

These are Binding API interfaces to register action implementations
and invoking actions.

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

Speed up dom-codec tests

SchemaContext and BindingRuntimeContext are classloader-invariants,
so we can safely cache them for all tests -- an improvement from
9 to 6 seconds.

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

Teach BindingToNormalizedNodeCodec action paths

This adds the first step in bridging DOM/Binding action invocation,
which is to acquire the SchemaPath of a particular action.

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

AdapterFactory should be @NonNullByDefault and @Beta

This is a new interface, subject to change, mark it as @Beta.
Also require implementations to deal will nullness.

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

Speed up RPC prototype lookup

Instead of mashing all the RPCs together, lookup the corresponding
module first and only look at that modules' RPCs.

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

Extract AdapterFactory interface and advertise as a service

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

Generate QNAME constant for actions

This field has been omitted, make sure we generate it to make
life easy for users.

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

Action should not be @NonNullByDefault

Since the codegen does not have the concept of annotation use,
hence we cannot use @NonNullByDefault and generate appropriate
annotations in generated code.

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

Require ContainerNode in input/output

Actions/RPCs really take a container, not anything else.

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

Add action tests

This adds a very basic test, which uncovered a failure to deal
with actions being multiply-inherited -- and fixes that issue.

Also a test model is added, so higher-layer components can be
tested.

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

Switch DOMExtensibleService to require getExtensions()

With all implementations switched over to provide getExtensions(),
remove getSupportedExtensions().

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

Add AbstractBindingLoadingAdapter

There are at least two adapters which are using a weak cache
to track child adapters. Introduce AbstractBindingLoadingAdapter
to host the common functionality.

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

Split out AbstractBindingAdapter

Most adapters are sharing same fields over and over, split
this common functionality into an abstract superclass.

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

Expose IpAddress(NoZone) string utilities

This exposes string utilities for dealing with unions, returning
string values.

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

Bump twirl-api to 1.3.15

twirl-api goes to 1.3.15

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

Introduce DOMExtensibleService.getExtensions()

This adds a properly-typed getExtensions() method, so end users
can acquire extensions in a type-safe manner.

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

Add binding ClusteredDataTreeChangeListener

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

Add dom-parent

Adding yangtools/mdsal declarations without running the plugin
is quite useful across our project and in downstreams.

Create dom-parent to hold exactly that.

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

Generate checkers for union member types

Update AbstractTypeProvider to propagate restrictions and update
UnionTemplate to take advantage of common methods already present
in BuilderTemplate.

Change-Id: I7e34c3fcb3859db270647ca434ac02658e2b1316
JIRA: MDSAL-360
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add DOM and binding PingPongDataBroker

Copied from the controller.

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

Add DOMOperationService and related interfaces

DOMOperationService supersedes DOMRpcService and unifies execution
of both RPCs and actions.

The design of this interface is inherently asynchronous, where the
system actually executing the action cannot be influenced by the
waiter blocking.

Change-Id: I8ff61f7b1d589cd3b7d32afba0cbd5042dbc5441
Signed-off-by: Jie Han <han.jie@zte.com.cn>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Package rfc8344 ietf-ip

This is the NMDA-compliant update, package it, but do not install
yet.

JIRA: MDSAL-358
Change-Id: Ie853d313fdf17facf1f6b306012b48f2ac642574
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Package rfc8343 ietf-interfaces

This is the NMDA-compliant update, package it, but do not install
yet.

JIRA: MDSAL-358
Change-Id: I67a33a511b50922d20b30489cdcc1c0d084a81ba
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Package rfc8342 ietf-datastores, ietf-origin

This is the core model of NMDA, defining how datastores are laid
out. Package it for the identities it provides, which will prove
useful in NMDA MD-SAL design.

JIRA: MDSAL-358
Change-Id: I5de5079495002bdff8677f750301aebf33dcde8d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clarify the event listener failure documentation

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

Take advantage of FluentFutures

FluentFutures allow

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

Optimize single-value range checks

For 'length 0' we generate a 'length >= 0 && length <= 0' check,
where we can simply use 'length == 0'. Fix that by teaching
the generator that min and max can be equal.

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

Add Binding{Data,RpcFuture}Aware bridge interfaces

We need to be able to communicate the binding future between two
incarnations of LazyDOMRpcResultFuture: controller and mdsal.

Create a migration interface for this purpose, which exposes
the required method. Convert RpcServiceAdapter to use it instead
of relying on LazyDOMRpcResultFuture class.

The same goes for LazySerializedContainerNode, bridged via
BindingDataAware.

JIRA: SFC-222
Change-Id: Icf5df0a2cfb4ec24d404ac6d077c2755692b9d5c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix ClusterSingletonServiceRegistration

This interface is intended to be a Registration, not an AutoCloseable,
and so is its only implementation.

Switch from AutoCloseable to ObjectRegistration, so users can take
advantage of the simplified API.

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

Fix GenericEntityOwnershipCandidateRegistration format

The javadoc has wrong indent, fix that.

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

Eliminate synthetic access to invoke0()

This method is invoked from InvocationStrategy, hence it being
private generates a synthetic accessor method. Change method
visibility to package-private, eliminating this inefficiency.

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

Fix xtend warnings

xtend-maven-plugin-2.14.0 flushes out a few more warnings, fix them.

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

Allow ipv4-address-no-zone to be parsed

Add methods which allow no-zone addresses to be parsed equally
efficiently as ip addresses.

JIRA: BGPCEP-790
Change-Id: Ic119430e3398777addeec2b41ca8e83eaead363b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove maven site configuration

We are not using maven site, remove its leftovers.

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

BUG-7697: add defences against nulls

Null listener is invalid, also make sure we do not ever set
prevRpcs to null.

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

Package rfc7952 ietf-yang-metadata

This is a base exptension required for NMDA models, package it here.

JIRA: MDSAL-358
Change-Id: I567f9d45ffa965b2eb3ea326692128f5af46ae74
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Repackage RFC8345 Network Topologies

This is an upgrade of ietf-network, which is not used anywhere
anymore to RFC8345 with proper modular artifacts.

JIRA: MDSAL-358
Change-Id: I898a1e7aa69d216972ce23dacb13f467ce6f9619
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor GeneratorJavaFile

This patch reworks GeneratorJavaFile so that it performs two-step
code generation. This allows us to understand what files are going
to be generated before actually generating them.

For bonus points we have removed a dependency on the ancient
plexus-container-default and made mdsal-binding-java-api-generator
maven-agnostic.

JIRA: MDSAL-350
Change-Id: Ie29865d93cce751a5ca265105a24a24049f4ccdb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump to yangtools 2.0.7

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

Generate code for action statements

RFC7950 defines actions, which are similar to RPCs in that they
can be invoked. This patch teaches binding codegen to generate
the interfaces for them.

JIRA: MDSAL-300
Change-Id: I8c2f9566e25948c8604d76ce52d86304b0f8be03
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Override xtend-maven-plugin dependencies

org.eclipse.equinox.common-3.10.0 seems to be required for some
reason.

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

Enforce patterns in supertype constructors

We have failed to enforce patterns when being constructed from
a supertype, which would allow creating invalid instances.

Fix that by enforcing patterns consistently.

JIRA: MDSAL-351
Change-Id: I16f4485ec8b6535c7094e3aac018cb6fbb0e5570
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Centralize code generation a bit more

BaseCompilationTest should be used as the baseline for creating
tests which create source code. Refactor it a bit, to allow for
wider reuse.

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

Use String.replace(char, char) instead of splitting

This code really wants to do a simple character substitution,
which has an elegant solution. This means we do not end up
compiling the replacement pattern and use a very efficient method.

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

Constify Types.listTypeFor(BaseYangTypes.STRING_TYPE)

We use this type for each constraint, hence we should be treating
it as a constant.

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

Hide parameterized Types constants

These constants should be accessed via their specializing static
factory methods. Hide them and fix the single offending caller.

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

Lower BindingTypes constant visibility

All of these types should be accessed via their concretizing
static factory methods, so hide them from outside world.

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

Fix case-related augment test

The test model used in this test is not valid, as it introduces
same container/leaf as both an augmentation of the list and as an
augmentation of that list's choice, leading to the leaf/container
being defined twice in the list's data tree hierarchy.

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

Bump odlparent to 3.1.2

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

Add Action and RpcOutput interfaces

This patch adds the binding interface prototype for actions.
Codegen will attach to specializations of this interface for
concrete generated types.

JIRA: MDSAL-300
Change-Id: I2fc97df50033b0d2d8f96b8281a82999a6b62869
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup AbstractForwardedDataBroker

Remove unused AutoCloseable and require codec and delegate to be
non-null.

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

Fix root choice targeting

InstanceIdentifier codec infrastructure cannot deal with choices
which have only grouping children due to the inability to properly
find the choice schema.

Use the case-assisted addressing mode to allow these items to be
properly resolved.

JIRA: MDSAL-45
Change-Id: I61f88e0baa854abc8286ca5c57edd572638ca63b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Allow base type to be passed down

We need this ability to generate RpcOutput-compliant containers.

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

ParameterizedTypeImpl should not allow nulls

Null parameterizations do not make sense, make sure we reject them
with a NPE.

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

include YANG module QName in "Schema for %s is not available." error

This is a repeat of I305299ffb3a40036dd72b858ef955ffcf45f6775,
which for some reason (human error or Gerrit bug?!) went into empty.

JIRA: MDSAL-354
Change-Id: Ie7014f824738a7e99799e1637a516617664d63d8
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Add binding blueprint XML

In order to start using the mdsal binding APIs,
we need them advertised.

Change-Id: I0ca67998c074697ef330eeb08df7d2f1abc5ffb0
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>