swift-storage-relation-broken hook not implemented

[Jill Rouleau]

Swift-storage charm lacks a hook to remove-relation to swift-proxy. When trying, the swift-storage unit logs:

2015-07-27 20:09:31 INFO juju.worker.uniter uniter.go:480 running "swift-storage-relation-departed" hook
2015-07-27 20:09:31 INFO juju.worker.uniter context.go:260 skipped "swift-storage-relation-departed" hook (not implemented)
2015-07-27 20:09:31 INFO juju.worker.uniter uniter.go:497 skipped "swift-storage-relation-departed" hook (missing)
2015-07-27 20:09:31 INFO juju.worker.uniter uniter.go:505 committing "relation-departed" hook
2015-07-27 20:09:31 INFO juju.worker.uniter uniter.go:520 committed "relation-departed" hook
2015-07-27 20:09:31 INFO juju.worker.uniter uniter.go:480 running "swift-storage-relation-broken" hook
2015-07-27 20:09:31 INFO juju.worker.uniter context.go:260 skipped "swift-storage-relation-broken" hook (not implemented)
2015-07-27 20:09:31 INFO juju.worker.uniter uniter.go:497 skipped "swift-storage-relation-broken" hook (missing)
2015-07-27 20:09:31 INFO juju.worker.uniter uniter.go:505 committing "relation-broken" hook
2015-07-27 20:09:31 INFO juju.worker.uniter uniter.go:520 committed "relation-broken" hook

And the relationship remains. The swift-proxy charm does have the necessary hooks.

[Chris MacNaughton (chris.macnaughton)]

Looking briefly at what's done in the relation-joined/changed hooks, it looks like all the relation-broken hook needs to do it remove the files at:

- /etc/rsync-juju.d/050-swift-storage.conf
- /etc/swift/swift.conf

Would that match your expectations?

[Jill Rouleau (jillrouleau)]

Hey sorry for the late reply. I believe that would do it, yeah. Thanks.