summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-08-17AbstractGenericWriter: remove xxxCurrent methodsMarek Gradzki2-61/+25
Use xxxCurrentAttributes instead to simplify call graph. Change-Id: I8201b115354dddd35ca0b63b2c72dfb4eac93646 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-08-17HONEYCOMB-431: add validation support to WritersMarek Gradzki12-32/+404
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>
2018-08-17HONEYCOMB-431: delegate DataModification.validate to WriterRegistryMarek Gradzki8-36/+206
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>
2018-08-17HONEYCOMB-431: make DataModification.validate idempotentMarek Gradzki5-32/+125
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>
2018-08-13Hide ResourceLoader.ResourceLoaderIml's constructorMarek Gradzki1-0/+4
Fixes issue reported by Sonar. Change-Id: Ibf9bc89ee09f151a56766213714829abdb7012bb Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-08-13Hide TranslationUtil's constructorMarek Gradzki1-1/+3
Utility classes should not be instantiated. Change-Id: Iff821556ad6b593bce6b3e056241dc7e2bbe0c8b Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-08-13Bump maven-sonar-plugin version to 3.4.1.1170Marek Gradzki1-2/+2
Change-Id: Ic490d8b850fcd270ff49e1c682f40c0324afe3d6 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-08-10remove obsolete HoneycombDOMBrokerProviderMichal Cmarada3-191/+0
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>
2018-08-10HONEYCOMB-440: bump ODL dependencies to Oxygen-SR3Marek Gradzki2-8/+8
Change-Id: Ib178651a07b55f7e732a45751c31b678296b2083 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-08-09Rename TestValidate to EditConfigTestMarek Gradzki3-7/+7
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>
2018-08-06javadoc: add missing <name> in pom filesMarek Gradzki2-0/+2
Change-Id: I2bed263c71b38e28706cb869232482b951158fd1 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-08-06HoneycombDOMRpcService: remove unnecessary cast to FunctionMarek Gradzki1-3/+1
Fixes issue reported by Sonar. Change-Id: I083eb089f1ccd691a2015eaa401fe5acf7c59d49 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-08-06RWUtils: use lambdas instead of anonymous inner classesMarek Gradzki1-26/+5
Change-Id: Ie17b819efd381983754065a821771eb3aac11d06 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-08-03Do not use FQN of DataBroker in ModifiableDataTreeDelegatorMarek Gradzki1-2/+3
Change-Id: I5944e9fd9388fbc533bf909b171b0672d798ad82 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-07-31Remove code repetitions from GenericListWriterMarek Gradzki1-14/+8
Change-Id: I4efbdcc4bc12e29bd767c7b8fa6696de62ac1d37 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-07-20Relese notes: update jira project version for 1.18.10Marek Gradzki2-3/+3
Change-Id: I02b4f5948ae8818412d1ae0f7b3e1ac4acd55726 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-07-20Bump honeycomb version to 1.18.10-SNAPSHOTMarek Gradzki75-147/+147
Change-Id: Ibd371b845debac895fad97b4e15732d858e00a3c Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-06-27Register sutdown handler for Restconf server before startMarek Gradzki1-1/+1
Change-Id: I25a22a194048637ea72dd35767e1e1701b5723e4 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-06-27DelegatingRewriteDeleteProducer: remove useles instanceof checkMarek Gradzki1-1/+0
DataContainerNode is DataContainerChild. Change-Id: I9122f9f3c27951f2d498f714d2ea15ebc96abdf3 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-06-27bgp: mark parameters of RouteWriter @NonNullMarek Gradzki2-11/+9
The RouteWriter interface is used by LocRibChangeListener, which never passes null parameters. Change-Id: I33a775de6c67d24bc271c0af21c9634c661e85e9 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-06-21Cleanup WriterCustomizerMarek Gradzki1-5/+2
Change-Id: Ied63ccd3fbc7e645ba3fa2db16528b3209319fa7 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-06-13HONEYCOMB-439: bump ODL dependencies to Oxygen-SR2Marek Gradzki2-9/+9
Change-Id: I76cc110ac7e0a64cdad79a7614f9e8b5eb624428 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-06-11ModifiableDataTreeManager: remove redundant non-default constructorMarek Gradzki1-5/+1
Change-Id: I17dba7ad77f4d7865f887056062d0748ca699d44 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-06-11WriteTransaction: make sure TransactionCommitFailedException is thrownMarek Gradzki7-3/+298
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>
2018-06-07Bump odl.yang-binding version from 0.12.0 to 0.12.1Marek Gradzki1-1/+1
Change-Id: I37c247279c37c3fb066186c8390e827a9611b99e Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-06-07HONEYCOMB-432: bump ODL dependencies to Oxygen-SR1Marek Gradzki1-8/+8
Change-Id: Ibf01bcaf65bb048fcfae57b5ba0597faff5e4fe6 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-05-18Invoke CapabilityChangeNotificationProducer.close on shutdownMarek Gradzki1-2/+5
Also fixes Sonar warning regarding useless CapabilityChangeNotificationProducer instantiation. Change-Id: I376b834618d34f13615051abf622e6b12e031050 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-05-16HONEYCOMB-437: implement ReflexiveListReaderCustomizer.readCurrentAttributesMarek Gradzki2-41/+90
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>
2018-05-02javadoc: add missing <name> in pom filesMarek Gradzki24-1/+24
Change-Id: I3d65b568c3651e4b2474c25bdf80cf3fff416360 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-05-02bgp-extensions: fix ODL BGP links in javadocMarek Gradzki7-7/+6
Change-Id: I84b338774c8175a7e59f03db3b81297607df0141 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-04-25Collect all the updates for subtree writersMaros Marsalek3-7/+84
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>
2018-04-16WriteTransaction: log exception with error level when submit failsMarek Gradzki1-1/+1
Change-Id: Ibc269946c08702805531a62c40be295fecc6b4ce Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-04-10Fix various Sonar warningsMarek Gradzki6-17/+14
Change-Id: I0883042a80525b72f33e66fdb0d5b9e600dcc713 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-04-09release notes: set ODL project branch to stable-oxygenMarek Gradzki1-1/+1
Change-Id: I03b48e387d370a4e264fb8bf4e3b22a03656fbba Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-04-09Update jira project version in relese notes for 1.18.07Marek Gradzki2-3/+3
Change-Id: Ie86c53f0c5922704ea7cbb3e29ee258e53c2cc70 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-04-09Bump Honeycomb version to 1.18.07-SNAPSHOTMarek Gradzki75-147/+147
Change-Id: Ib0b9a2a2369730cd8a21e35e4dce0b62e45e8ad3 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-03-23HONEYCOMB-424: bump ODL dependencies to OxygenMarek Gradzki52-229/+253
Change-Id: I3160e5df4e66b3b3c732619fc1fc7599459fda76 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-03-07Fix Netconf TCP server shutdownMarek Gradzki1-9/+17
Change-Id: I0c74799a7249777c3258be915e9a95836fe0265e Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-03-07ShutdownHandlerImpl: remove registered components after shutdown is performedMarek Gradzki1-0/+5
Prevents invoking AutoCloseable.close(), which might not be idempotent. Change-Id: Id28200fb68a4c6e38878cf4f0b8aaee298dbc37b Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-03-07Fix Restconf server shutdownMarek Gradzki2-15/+36
Change-Id: Ia7c0fc2d269ae88170297ef22476c8a61121238a Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-03-07Move shutdown hook registration to dedicated providerMarek Gradzki4-7/+41
This change would allow using ShutdownHandlerImpl in integration tests. Change-Id: I854f65d03b9e1a495fda7211fd1b7fd51b4e5ba3 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-03-07Fix handling of RPCs that do not produce a resultMarek Gradzki2-14/+26
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>
2018-03-07Post HONEYCOMB-161 cleanupMarek Gradzki2-2/+0
Change-Id: I46652c9f109a6ede60cfb3143d945fe8b6f1f042 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-02-23HONEYCOMB-418: Bump ODL dependencies from Nitrogen SR1 to Nitrogen SR2Marek Gradzki3-10/+10
Change-Id: I98f73e0cce217202e8c335b1fa06f1b3fc8bce05 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-02-02Use XMLStreamNormalizedNodeStreamWriter from yang-data-codec-xmlMarek Gradzki2-1/+5
instead of old and deprecated version from yang-data-impl. Change-Id: If3f9dbffe83cdde29d2f4996dbd299a60bab90a9 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-02-02Use YangParserTestUtils instead of CrossSourceStatementReactor.BuildActionMarek Gradzki4-20/+22
Simplifies code and removes usages of deprecated YangStatementSourceImpl. Change-Id: I012e2de7a5a902927eaec87370f1850d54fbf4ee Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-02-02Remove usages of deprecated BindingNormalizedNodeSerializer from yantoolsMarek Gradzki8-8/+8
Use mdsal version: org.opendaylight.mdsal.binding.dom.codec.api.BindingNormalizedNodeSerializer instead of org.opendaylight.yangtools.binding.data.codec.api.BindingNormalizedNodeSerializer Change-Id: Idfb977acee95de40183d4574565c46ac9ab54a27 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-02-02Fix revision of hc-aug-test.yangMarek Gradzki4-39/+39
Change-Id: I1ca0e255c1467af31a39eb515c9cd4e7b4a4948a Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-02-02Improve error messages in distro testsMarek Gradzki2-5/+7
Change-Id: Ie0bc2e2add7940cb8f0ce2d5e4ea41d1e1c9d991 Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
2018-01-31Update project-odl-branch property to match ODL version usedMarek Gradzki1-1/+1
Change-Id: I2a036037245be07d2cd452a8036374967ca7622c Signed-off-by: Marek Gradzki <mgradzki@cisco.com>