[CONTROLLER-1372] Semantic errors in feature config files could be reported better Created: 17/Jun/15  Updated: 25/Nov/15  Resolved: 25/Nov/15

Status: Verified
Project: controller
Component/s: config
Affects Version/s: Lithium
Fix Version/s: None

Type: Bug
Reporter: Vratko Polak Assignee: Monika Verma
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All


External issue ID: 3769

 Description   

There was a more general CONTROLLER-922 which was fixed for syntactic errors.
There is a CONTROLLER-1349 concerning with a side effect of config subsystem stopping on semantic error, which was postponed to Beryllium.
This improvement concerns with question of reporting semantic errors, which would be nice to have in Lithium.

Steps to reproduce: As in CONTROLLER-1349 with the alternative step 11.

On current Lithium, there is no message (at level INFO or higher) which would inform the user that config-stopping error has even happened.
It would be better to have a log message specifying which feature/configfile/module/element contains the semantic error.



 Comments   
Comment by Vratko Polak [ 05/Nov/15 ]

Change 28985 was merged to Lithium (not Beryllium yet), which handles the reporting part.

Comment by Vratko Polak [ 25/Nov/15 ]

Merged to Berylium too. Here is what the message looks like now:

2015-11-25 12:40:12,647 | ERROR | config-pusher | ConfigPusherImpl

127 - org.opendaylight.controller.config-persister-impl - 0.4.0.SNAPSHOT Faile
d to apply configuration snapshot: 39-pcep-provider.xml(odl-bgpcep-pcep,odl-bgpcep-p
cep)
java.lang.IllegalStateException: Unable to resolve value AttributeConfigElement [def
aultValue=null, value=MappedDependency {namespace='urn:opendaylight:params:xml:ns:yan g:controller:pcep:topology:provider', serviceName='pcep-topology-stateful', refName= 'stateful06'}

] to attribute stateful-plugin

Generated at Wed Feb 07 19:55:23 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.