Merge lp://staging/~adam-collard/charms/trusty/swift-proxy/lib-in-python-package into lp://staging/~openstack-charmers-archive/charms/trusty/swift-proxy/next
Proposed by
Adam Collard
Status: | Merged |
---|---|
Merged at revision: | 107 |
Proposed branch: | lp://staging/~adam-collard/charms/trusty/swift-proxy/lib-in-python-package |
Merge into: | lp://staging/~openstack-charmers-archive/charms/trusty/swift-proxy/next |
Diff against target: |
396 lines (+120/-52) 13 files modified
.coveragerc (+1/-0) Makefile (+3/-3) charm-helpers-hooks.yaml (+1/-1) charmhelpers/contrib/openstack/utils.py (+4/-0) charmhelpers/contrib/storage/linux/ceph.py (+2/-11) charmhelpers/core/kernel.py (+68/-0) hooks/swift_hooks.py (+1/-1) lib/swift_context.py (+7/-1) setup.cfg (+1/-1) unit_tests/__init__.py (+0/-2) unit_tests/test_swift_context.py (+7/-7) unit_tests/test_swift_hooks.py (+2/-1) unit_tests/test_swift_utils.py (+23/-24) |
To merge this branch: | bzr merge lp://staging/~adam-collard/charms/trusty/swift-proxy/lib-in-python-package |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Chris Glass (community) | Approve | ||
Landscape | Pending | ||
Review via email: mp+270304@code.staging.launchpad.net |
Description of the change
With the addition of actions to the OpenStack charms, there's a need to share code between actions and hooks. This branch moves library code to a top-level ./lib/ and then symlinks that into ./hooks/lib, likewise for charm-helpers.
This should be a plain refactor with no change in behaviour, but laying the ground for an actions/lib symlink and an actions/
(see https:/
To post a comment you must log in.
charm_lint_check #9540 swift-proxy-next for adam-collard mp270304
LINT OK: passed
Build: http:// 10.245. 162.77: 8080/job/ charm_lint_ check/9540/