Call built-in ProvisionerCleanup on vagrant destroy

frapposelli / vagrant-vcloud

Vagrant provider for VMware vCloud Director®

MIT License

67 stars 38 forks source link

Call built-in ProvisionerCleanup on vagrant destroy #111

Closed pfink closed 9 years ago

pfink commented 9 years ago

... otherwise plugins like vagrant-butcher aren't triggered. Compare with: https://github.com/mitchellh/vagrant/blob/v1.7.2/plugins/providers/virtualbox/action.rb#L97

Works fine with vagrant-butcher on multi-machine setup.

StefanScherer commented 9 years ago

LGTM!

tsugliani commented 9 years ago

@pfink Thanks for this PR, I think we will merge this one pretty quickly, obviously :+1:

@pfink do you mind doing this PR against our develop branch ? I'll let @frapposelli comment for that (and btw, we can probably apply this Patch on both vagrant-vcenter/vagrant-vcloudair too)