-
Notifications
You must be signed in to change notification settings - Fork 229
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
Fix resource cloning deprecation for apt-transport-https
#351
Comments
We can't require Chef 12.9.1 just yet. |
What's preventing it atm? |
@eyalzek there are users who use older versions? |
Yeah but that's not stopping most of the other cookbooks in the chef supermarket to upgrade. This change seems due since chef 13 is already out and will become the standard soon. |
@eyalzek when "soon" happens, we can move to whatever's available in the core. A deprecation message is not a big deal. Using older cookbook versions means they won't get any other improvements in master. |
@eyalzek I think now we can require 12.9 or even 13. Please submit a PR? Note that there are funky things going on with the specs, I'm investigating but not getting very far just yet. |
This is to prepare for Chef 13 migration. References #351.
Sorry, don't have much time for a complete adaptation - also, I'm not entirely sure what are the full changes this requires |
@eyalzek I did a bit of work on this and getting closer to requiring at least Chef 12.9.1. Unfortunately, some dependency cookbooks used by Test Kitchen and/or ChefSpec seem to need updating for Chef 13. I am not deeply familiar with any of them so it takes time to figure out what part is problematic and whether there were new versions we can use. But I managed to get things moving towards requiring a recent Chef 12 version. |
Getting the following deprecation message when using the cookbook:
coming from this line:
https://github.com/michaelklishin/cassandra-chef-cookbook/blob/master/recipes/repositories.rb#L31
is this cookbook actually dependent of
apt
? most of theapt
resources are available as part of the chef core starting version 12.9.1...https://github.com/michaelklishin/cassandra-chef-cookbook/blob/master/metadata.rb#L12
The text was updated successfully, but these errors were encountered: