[BGPCEP-795] PCEP module register missing from Blueprint Created: 07/May/18 Updated: 22/May/18 Resolved: 22/May/18 |
|
| Status: | Resolved |
| Project: | bgpcep |
| Component/s: | None |
| Affects Version/s: | Nitrogen, Carbon |
| Fix Version/s: | None |
| Type: | Bug | Priority: | High |
| Reporter: | Kevin Wang | Assignee: | Kevin Wang |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
| Description |
|
After the controller is restarted, somehow the rootRuntimeBeanRegistratorWrapper in "pcep/topology-provider/target/generated-sources/config-binding/org/opendaylight/controller/config/yang/pcep/topology/provider/AbstractPCEPTopologyProviderModule.java" doesn't got populated properly. It seems that the config module didn't start correctly, which caused blueprint fail to inject register runtimeBeanRegistrator. Related code in "pcep/topology-provider/target/generated-sources/config-binding/org/opendaylight/controller/config/yang/pcep/topology/provider/AbstractPCEPTopologyProviderModule.java"
public org.opendaylight.controller.config.yang.pcep.topology.provider.PCEPTopologyProviderRuntimeRegistrator getRootRuntimeBeanRegistratorWrapper(){
return rootRuntimeBeanRegistratorWrapper;
}
@Override
public void setRuntimeBeanRegistrator(org.opendaylight.controller.config.api.runtime.RootRuntimeBeanRegistrator rootRuntimeRegistrator){
this.rootRuntimeBeanRegistratorWrapper = new org.opendaylight.controller.config.yang.pcep.topology.provider.PCEPTopologyProviderRuntimeRegistrator(rootRuntimeRegistrator);
}
|
| Comments |
| Comment by Claudio David Gasparini [ 10/May/18 ] |
|
Hi Kevin, Please add affected version, and should not this be opened against Controller maybe? Regards, |
| Comment by Kevin Wang [ 20/May/18 ] |
|
Found the issue. Basically when ServerSessionManager was closed, the RuntimeRegistration got set to null. However, when it's started again, the registration doesn't got reset |
| Comment by Kevin Wang [ 20/May/18 ] |
|
cdgasparini I have pushed a patch here. Please review it. |
| Comment by Claudio David Gasparini [ 21/May/18 ] |
|
Hi kevinxw, carbon/nitrogen are not longer supported. Regards, |
| Comment by Claudio David Gasparini [ 22/May/18 ] |
|
Carbon and Nitrogen are not longer supported, therefore there is no more build or test jobs for such branch. Fix patches won't be merged therefore. |