Merge lp:~james-page/charm-helpers/add-profile-name-to-params into lp:charm-helpers
Status: | Merged |
---|---|
Merged at revision: | 638 |
Proposed branch: | lp:~james-page/charm-helpers/add-profile-name-to-params |
Merge into: | lp:charm-helpers |
Diff against target: |
76 lines (+7/-19) 2 files modified
charmhelpers/contrib/openstack/context.py (+4/-2) tests/contrib/openstack/test_os_contexts.py (+3/-17) |
To merge this branch: | bzr merge lp:~james-page/charm-helpers/add-profile-name-to-params |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Liam Young (community) | Approve | ||
Review via email: mp+307032@code.launchpad.net |
Description of the change
Refactor AppArmorContext
The current design of the AppArmorContext class results in alot of
boilerplate configuration being written:
class NeutronMetadata
def __init__(self):
def __call__(self):
if not self.ctxt:
return self.ctxt
return self.ctxt
this might be better done as:
n_metadata = AppArmorContext
thus avoid creating a specific class when not really required.
LGTM, thanks