[TSC-181] ITM Yang Models Cleanup Created: 23/Nov/18 Updated: 25/Jul/19 Resolved: 25/Jul/19 |
|
| Status: | Resolved |
| Project: | tsc |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Sodium |
| Type: | Weather Item | Priority: | Medium |
| Reporter: | Chintan Apte | Assignee: | Apurba Mukherjee |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| ODL Project: | genius |
| ODL Release: | |
| ODL Impacted Projects: | Netvirt, SFC |
| ODL Expected Date: |
| Description |
|
See here for details: - https://git.opendaylight.org/gerrit/#/c/77605/ This Spec change will address the ITM yang Models cleanup, removing some dead/unwarranted code. |
| Comments |
| Comment by Ariel Adam [ 29/Nov/18 ] |
|
Hi there. Why is this a TSC weather item? |
| Comment by Chintan Apte [ 30/Nov/18 ] |
|
Hi Ariel, This weather item will bring yang changes to the itm models, which are fundamental in nature. This change will need CSIT changes in Genius as well and perhaps in Netvirt. Hence this is a weather item. Thanks, Chintan |
| Comment by Vishal Thapar [ 30/Nov/18 ] |
|
To add to what Chintan said, this will also require changes in CSIT, ODLTools and possibly SFC too. Genius user documentation will also need to be updated for any external users of Genius. |
| Comment by Chintan Apte [ 30/Nov/18 ] |
|
Have started pushing pertaining changes to this yang :- Here is the first one and is a WIP |
| Comment by Chintan Apte [ 04/Dec/18 ] |
|
https://git.opendaylight.org/gerrit/78416 this is the review link for the changes and is a WIP. |
| Comment by Brady Johnson [ 04/Dec/18 ] |
|
If APIs are changed/removed, they must first be deprecated in one release, then removed/changed in the subsequent release. Were these APIs deprecated in Fluorine or some previous release? Brady
|
| Comment by Daniel Farrell [ 06/Dec/18 ] |
|
Can you folks retarget this at the next release, if it's not really a weather item for Neon? |
| Comment by Faseela K [ 09/Dec/18 ] |
|
Brady, The Deprecation was not done, and hence, we will deprecate for this release, and target this weather item for the next release! |