https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/OPNFLWPLUG-646 OpenFlowPlugin <p>When someone wants custom FRM he needs to copy&amp;paste existing OpenFlowPlugin feature.xml. Please, modify existing stable/beryllium/features-li/**/features.xml that it is possible reference existing features (without existing FRM).</p> <p>The main problem is in feature "odl-openflowplugin-nsf-services-li" because id referes FRM.</p> <p>Operating System: All<br/> Platform: All</p> OPNFLWPLUG-646

FRM features.xml reorganize

Bug Resolved Done Andrej Leitner Jozef Slezák Tue, 22 Mar 2016 08:25:16 +0000 Mon, 27 Sep 2021 09:01:45 +0000 Wed, 13 Jul 2016 08:25:26 +0000 General 0 9 <p>patch: <a href="https://git.opendaylight.org/gerrit/#/c/37670/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/37670/</a></p> <p>Attachment featuresOrig.dot.svg has been added with description: current state</p> <p>Attachment featuresNew.dot.svg has been added with description: proposed patch</p> <p>Can you share more details Michal (Rehak) and Jozef Slezak?</p> <p>Hi guys,<br/> after discussion w/ Michal I made some regrouping of ofp-features in this patch.</p> <p>Forwarding-rules-manager bundle was extracted as separate feature on upper level (above the main southbound-li). The same is made with topology-lldp-discovery and topology-manager which are in addition merged along with lldp-speaker into new app feature named app-topology-li. Then the main feature ofp-southbound-li is not dependent on them, but it is dependency of them (see attached diagrams).</p> <p>So if someone would like to add and use e.g. custom FRM, it's possible to install only the main ofp-southbound-li feature without any default apps and then any particular features/apps/extensions he wants (actual or newly added).</p> <p>However actual behaviour is still preserved because in flow-services-li installation our default apps and main southbound feature (+dependencies) are included as before.</p> <p>In addition, I removed unnecessary flow-services-all-li and also nsf-services-li feature where extracted apps where located (unecessary then).</p> <p>As we decided at the meeting yesterday, it would be better to perform this change after he/li features swap.</p> <p>I like these changes. You will need a new patch after the He-Li swap as there will be no feature called -Li anymore but you will figure out very easily.</p> <p>Luis, is it okay removing feature directly rather than deprecating them first and then removing in next release ?</p> <p>Features is not as critical as API, as long as you make sure downstream projects are not impacted by these changes it is OK.</p> <p>okay, that's good. I would suggest then we should do the proper cleaning and have some best practices around it. Ideally we should allow user to pick up what they want, and also we should provide wrapper features as well that provides some dedicated functionality using the existing fine grain features. I added couple of comments on the patch as well.</p> <p>Hi, new patch is there. I made changes according to Anil's comments and also updated dependencies of drop-test feature - it is</p> <p>...not dependent on the wrapper feature now.</p> <p>Attachment featuresPatch2.dot.svg has been added with description: patch2</p> <p>rebased and updated after OFP design swap</p> <p>Attachment featuresPatch3.dot.svg has been added with description: patch 3</p> <p>Attachment FeaturesActualMasterDefault.dot.svg has been added with description: features actual master default</p> <p>Attachment FeaturesRegroupPatch6.dot.svg has been added with description: patch 6 final update</p> <p>merged</p> Development External issue ID 5572 External issue URL ODL SR Target Milestone Rank 0|i032fr: