Merge lp:~thedac/charms/trusty/swift-storage/setup-storage-on-config-changed into lp:~openstack-charmers-archive/charms/trusty/swift-storage/next

Proposed by David Ames
Status: Merged
Merged at revision: 90
Proposed branch: lp:~thedac/charms/trusty/swift-storage/setup-storage-on-config-changed
Merge into: lp:~openstack-charmers-archive/charms/trusty/swift-storage/next
Diff against target: 10 lines (+1/-0)
1 file modified
hooks/swift_storage_hooks.py (+1/-0)
To merge this branch: bzr merge lp:~thedac/charms/trusty/swift-storage/setup-storage-on-config-changed
Reviewer Review Type Date Requested Status
Corey Bryant (community) Approve
Review via email: mp+274477@code.launchpad.net

Description of the change

Run setup_storage() on config-changed hook in case block-device has changed.

To post a comment you must log in.
Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_lint_check #11901 swift-storage-next for thedac mp274477
    LINT OK: passed

Build: http://10.245.162.77:8080/job/charm_lint_check/11901/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_unit_test #11067 swift-storage-next for thedac mp274477
    UNIT OK: passed

Build: http://10.245.162.77:8080/job/charm_unit_test/11067/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_amulet_test #7340 swift-storage-next for thedac mp274477
    AMULET OK: passed

Build: http://10.245.162.77:8080/job/charm_amulet_test/7340/

Revision history for this message
Corey Bryant (corey.bryant) wrote :

Reviewed and tested successfully. Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hooks/swift_storage_hooks.py'
2--- hooks/swift_storage_hooks.py 2015-10-13 19:05:02 +0000
3+++ hooks/swift_storage_hooks.py 2015-10-14 22:24:00 +0000
4@@ -79,6 +79,7 @@
5 status_set('maintenance', 'Configuring ipv6')
6 assert_charm_supports_ipv6()
7
8+ setup_storage()
9 ensure_swift_directories()
10 setup_rsync()
11

Subscribers

People subscribed via source and target branches