-
Improvement
-
Resolution: Unresolved
-
Medium
-
None
-
None
-
None
The ITM CLI Commands should all be migrated from the old to the new Karaf API.
They also should be all moved into a new TBD itm-cli bundle instead of mixed into itm-impl. (The itm-impl should contain no System.out or CommandSession; no dependency to Karaf CLI; anything related to that should only be in itm-cli, which should use OSGi services exposed by itm-impl, or even just itm-api.)
This will require some clean up, as currently non-CLI code pathes uses the same TepCommandHelper and there is weird code which if there is no CommandSession throws exceptions, but if there is prints to the console (for CLI commands). That should be cleaned up - probably by simply always throwing exceptions and catching them in the CLI.
https://git.opendaylight.org/gerrit/#/c/72204/ is a first stab at this. Who will replicate it?