Merge lp://staging/~sdn-charmers/charms/trusty/odl-controller/odl-cmds into lp://staging/~sdn-charmers/charms/trusty/odl-controller/trunk
Proposed by
Liam Young
Status: | Merged |
---|---|
Merged at revision: | 5 |
Proposed branch: | lp://staging/~sdn-charmers/charms/trusty/odl-controller/odl-cmds |
Merge into: | lp://staging/~sdn-charmers/charms/trusty/odl-controller/trunk |
Diff against target: |
171 lines (+69/-11) 2 files modified
hooks/odl_controller_hooks.py (+28/-11) hooks/odl_controller_utils.py (+41/-0) |
To merge this branch: | bzr merge lp://staging/~sdn-charmers/charms/trusty/odl-controller/odl-cmds |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
SDN Charmers | Pending | ||
Review via email: mp+262095@code.staging.launchpad.net |
To post a comment you must log in.
The post install commands that were being run no longer work on more recent versions of ODL because the features being requested have been renamed or removed.
This mp removes any default commands. Instead, a charm joined via the controller-api relation can request features so be installed or log settings to be set by setting odl-cmds to a string representation of json.
eg
{ install' : ['cosc- cvpn-ovs- rest', 'odl-netconf- connector- all'], cvpn-ovs- rest', 'odl-netconf- connector- all'],
'feature:
'log:set': {
'TRACE': ['cosc-
}
}
The above would request the installation of cosc-cvpn-ovs-rest and odl-netconf- connector- all and set logging to trace for cosc-cvpn-ovs-rest and odl-netconf- connector- all