-
-
Notifications
You must be signed in to change notification settings - Fork 605
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
elasticsearch_install action :remove failure #688
Comments
Hey there -- I'd be glad to drop the path from the dpkg_package :remove action; it does seem like it's probably unnecessary. Alternately, I'll note you could also use the regular install action with Hope that helps. Feel free to submit a PR to fix that path in the remove action 👍 |
I think that is a good option as well, although removal should still probably work. PR is up |
When attempting to remove an
elasticsearch_install
with the block below, it fails as it attempts to telldpkg
to remove the package version listed while prepending#{Chef::Config[:file_cache_path]}
to the name, which if you are just trying todpkg
uninstall something, fails.My question is: I know this cookbook does not support upgrading, but should it not allow the following for sake of people writing their own upgrade recipe? I worked around this by having an explicit
dpkg_package
block withaction :remove
I am happy to submit a PR for this, just wanted to gauge the response first. Thanks!
The spec I was trying is:
Here is some output from
test-kitchen
for my wrapper cookbookThe text was updated successfully, but these errors were encountered: