Age | Commit message (Collapse) | Author | Files | Lines |
|
Change-Id: Id5f6eb84ade0514bd9f1579e34c395994eeef4b2
Signed-off-by: Michal Cmarada <mcmarada@cisco.com>
|
|
Change-Id: I3c1ce39838c0166a31203461313152bb9d819460
Signed-off-by: Michal Cmarada <mcmarada@cisco.com>
|
|
Change-Id: I150ab8b52d1920da38909310df372711dcc67f4c
Signed-off-by: Michal Cmarada <mcmarada@cisco.com>
|
|
Change-Id: Ib8e820256daebd7d9bfc7f4a8610e72abe42bbb5
Signed-off-by: Michal Cmarada <mcmarada@cisco.com>
|
|
- updated dependencies
- compilation error fixes
Change-Id: Iad9d353d06c113a0da892ff11a442e4b41edae0d
Signed-off-by: Michal Cmarada <mcmarada@cisco.com>
|
|
Change-Id: I5e53b8404a437bf0d08eefcfc417e5d6d46fa0b4
Signed-off-by: Michal Cmarada <mcmarada@cisco.com>
|
|
- bumps ODL dependencies to neon
- migrates guava optional to java.utils.optional
Change-Id: Ibdaa365c7e21a341103aba88dfd67277e40d2969
Signed-off-by: Michal Cmarada <mcmarada@cisco.com>
|
|
Change-Id: Iaab95d1a821673f9138ba8a118c2e265826a6288
Signed-off-by: Michal Cmarada <mcmarada@cisco.com>
|
|
Change-Id: Ic3e48d39d570632743c66b9317cd638fba84a34f
Signed-off-by: Michal Cmarada <mcmarada@cisco.com>
|
|
This patch introduces ValidableTransaction interface,
used by Honeycomb's DataBroker to implement
Netconf's DOMDataTransactionValidator.
WriteTransaction implements ValidableTransaction
by delegating validate to DataModification
for both config and operational modifications.
This patch requires ODL Fluorine (HONEYCOMB-433),
which brings support for <candidate> validation to Netconf.
Alternatively custom ODL Oxygen build
with patches from stable/oxygen/oxygen-hc-improvements.
Change-Id: Ic3eaffaef0314d22da3ea37f9416f18ccb0b99ae
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
Signed-off-by: Michal Cmarada <mcmarada@cisco.com>
|
|
Change-Id: I142ebd2899272feff00abe7d4bae708f093ee3ec
Signed-off-by: Michal Cmarada <michal.cmarada@pantheon.tech>
|
|
Change-Id: Ia64b0869d1f2d9620d233f31077e72d887ef4716
Signed-off-by: Marek Gradzki <margradz@gmail.com>
|
|
Some of the writers do not support update,
also write/delete might not require validation.
Therefore, in order to reduce amount of boilerplate code,
make validateXXX methods default with empty implementation.
Change-Id: Id205b478900024c59834a6e6cb18bdbeb9b43792
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Sonar reports FIXMES as issues with higher priority as TODOs.
FIXMEs should only be used for known bugs,
not for improvements.
Also include JIRA issue ID in the TODO message.
Change-Id: I47461d948421ff17532f17ecec3ddae84239511f
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: I2ab95afb733885b3034dfc6a8cbad795fbf5ba6c
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Use xxxCurrentAttributes instead to simplify call graph.
Change-Id: I8201b115354dddd35ca0b63b2c72dfb4eac93646
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
This patch introduces FlatWriterRegistry.validateModifications.
Implementation iterates over writersOrder following bulkUpdate logic
to properly support subtree writers case.
Writers are now cabable of validating modifications.
Commonly used implementations (GenericWriter and GenericListWriter)
delegate validation capbility to Validators.
Change-Id: If7a0bb0838c0b8f2c0393c989f3b03853a2ea679
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
This patch introduces ModifiableDataTreeDelegator.validateCandidate
that translates DataTreeCandidate to DataObjectUpdates
and delegates validation to WriterRegistry (similarly as for bulk update).
ModifiableDataTreeManager.commit implementation
invokes validation before bulk update.
To make it efficient, DataObjectUpdates are computed once
and stored in DataTreeContext.
Change-Id: If4bd558e64ed84c11c9c50c7a98a2aaa8db841bb
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
This patch modifies contract of DataModification.validate
to make it idempotent.
ModifiableDataTreeManager.validate now invokes dataTree.validate
on a copy of DataTreeModification.
ModifiableDataTreeManager.validateCandidate was introduced
to allow additional validation.
Change-Id: I86fc101faff9b04afde2f3eb16fff4d4df2867ad
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Fixes issue reported by Sonar.
Change-Id: Ibf9bc89ee09f151a56766213714829abdb7012bb
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Utility classes should not be instantiated.
Change-Id: Iff821556ad6b593bce6b3e056241dc7e2bbe0c8b
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
this was used a while ago when we were using ODL dependencies with
config subsystem. Once ODL moved to Blueprint this become obsolete.
Several classes used in this provider are marked deprecated and are
removed in ODL Fluorine.
Change-Id: I5bd415a271a9d53ad40f785bdc3840c410f79249
Signed-off-by: Michal Cmarada <michal.cmarada@pantheon.tech>
|
|
The test does not use <validate> RPC
in contrast to ValidateTest from:
https://gerrit.fd.io/r/#/c/14040/
Change-Id: I57da44256515cf10afbd86ef7d7b1607caa3f52e
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: I2bed263c71b38e28706cb869232482b951158fd1
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Fixes issue reported by Sonar.
Change-Id: I083eb089f1ccd691a2015eaa401fe5acf7c59d49
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: Ie17b819efd381983754065a821771eb3aac11d06
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: I5944e9fd9388fbc533bf909b171b0672d798ad82
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: I4efbdcc4bc12e29bd767c7b8fa6696de62ac1d37
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: Ibd371b845debac895fad97b4e15732d858e00a3c
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: I25a22a194048637ea72dd35767e1e1701b5723e4
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
DataContainerNode is DataContainerChild.
Change-Id: I9122f9f3c27951f2d498f714d2ea15ebc96abdf3
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
The RouteWriter interface is used by LocRibChangeListener, which
never passes null parameters.
Change-Id: I33a775de6c67d24bc271c0af21c9634c661e85e9
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: Ied63ccd3fbc7e645ba3fa2db16528b3209319fa7
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: I17dba7ad77f4d7865f887056062d0748ca699d44
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
DataTreeModification.ready() used by DataModification.validate()
might throw IllegalArgumentException in case of missing mandatory nodes.
Use broader Exception type in WriteTransaction.submit()
to make sure contract defined by AsyncWriteTransaction is preserved.
Change-Id: I95cb3e1e8c6db36df90d2c78e7d63c854189e2fd
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Also fixes Sonar warning regarding
useless CapabilityChangeNotificationProducer instantiation.
Change-Id: I376b834618d34f13615051abf622e6b12e031050
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
ReflexiveListReaderCustomizer used default NOOP implementation,
which caused issues during serialization, because
BindingNormalizedNodeCodecRegistry.toNormalizedNode requrires key to be set.
Change-Id: I7822e07efe57cbdee2c539583776f8cc677ddab7
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: I3d65b568c3651e4b2474c25bdf80cf3fff416360
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: I84b338774c8175a7e59f03db3b81297607df0141
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
So far, when a subtree writer was registered on a list node
and ModificationDiff detected 2 or more updated list items for that writer,
FlatWriterRegistry just picked the first item in list, processed that one
and ignored the rest.
Change-Id: If66db1eaad5a3b5c35e5586f46fd83a0698e1728
Signed-off-by: Maros Marsalek <maros.mars@gmail.com>
|
|
Change-Id: Ibc269946c08702805531a62c40be295fecc6b4ce
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: I0883042a80525b72f33e66fdb0d5b9e600dcc713
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: Ib0b9a2a2369730cd8a21e35e4dce0b62e45e8ad3
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: I3160e5df4e66b3b3c732619fc1fc7599459fda76
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: I0c74799a7249777c3258be915e9a95836fe0265e
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Prevents invoking AutoCloseable.close(), which might not be idempotent.
Change-Id: Id28200fb68a4c6e38878cf4f0b8aaee298dbc37b
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: Ia7c0fc2d269ae88170297ef22476c8a61121238a
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
This change would allow using ShutdownHandlerImpl in integration tests.
Change-Id: I854f65d03b9e1a495fda7211fd1b7fd51b4e5ba3
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
The output statement of RPC operation is optional.
In such cases DefaultDOMRpcResult should be initialized with null result.
Change-Id: Ia832e480b3c72607b3636493bc044a9144f247be
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|
|
Change-Id: I46652c9f109a6ede60cfb3143d945fe8b6f1f042
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
|