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

Run provisioner cleanup when destroying VMs #272

Merged
merged 1 commit into from
Apr 15, 2016

Conversation

Sharpie
Copy link
Contributor

@Sharpie Sharpie commented Feb 23, 2016

Vagrant v1.3.0 introduced the ability for provisioners to define cleanup tasks
that get run during VM destruction in order to clean up bits of global state.
This patch adds the ProvisionerCleanup task to the OpenStack destroy action.

Vagrant v1.3.0 introduced the ability for provisioners to define cleanup tasks
that get run during VM destruction in order to clean up bits of global state.
This patch adds the ProvisionerCleanup task to the OpenStack destroy action.
@ggiamarchi
Copy link
Owner

@Sharpie I suppose you use the Chef client provisionner ? Up to now it seems to be the only one that implement this feature. Unfortunately i don't have an up and running Chef server to test it on my side. Can you confirm you successfully tested it with the Chef client provisionner ?

@Sharpie
Copy link
Contributor Author

Sharpie commented Mar 1, 2016

I use a Puppet provisioner which de-registers a machine that is being destroyed, along with a hostname provisioner that manages /etc/hosts and removes a machine's entries from other nodes when it is destroyed.

I also use multiple providers on a regular basis: VirtualBox, AWS, OpenStack, vSphere. So, having consistent provisioner cleanup behavior between all is important as well.

@ggiamarchi ggiamarchi self-assigned this Apr 15, 2016
@ggiamarchi ggiamarchi added this to the 0.7.2 milestone Apr 15, 2016
@ggiamarchi ggiamarchi added the bug label Apr 15, 2016
@ggiamarchi ggiamarchi merged commit 70aabcc into ggiamarchi:master Apr 15, 2016
@Sharpie Sharpie deleted the enable-provisioner-cleanup branch April 18, 2016 02:57
Sharpie pushed a commit to Sharpie/vagrant-openstack-provider that referenced this pull request Jul 25, 2016
…anup

Run provisioner cleanup when destroying VMs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants