Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hooks: s/storage-detached/storage-detaching/ #125

Merged
merged 1 commit into from Apr 30, 2015

Conversation

axw
Copy link
Contributor

@axw axw commented Apr 29, 2015

Rename storage-detached to storage-detaching. The hook will be run before storage is detached, to give charms an opportunity to cleanly release.

@axw axw force-pushed the charm-hooks-storage-detaching branch from 0eef354 to 82dda08 Compare April 30, 2015 02:39
@wallyworld
Copy link
Member

LGTM

axw added a commit that referenced this pull request Apr 30, 2015
hooks: s/storage-detached/storage-detaching/
@axw axw merged commit 7793941 into juju:v5 Apr 30, 2015
@rogpeppe
Copy link
Contributor

rogpeppe commented May 5, 2015

For the record, this is an API-breaking change and as such should not have landed
in charm.v5.

rogpeppe pushed a commit to rogpeppe-contrib/charm that referenced this pull request May 5, 2015
hooks: s/storage-detached/storage-detaching/
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants