Comments (12)
Updating your metadata.rb to reflect this dependency would be a start. The master branch still says chef_version '>= 12.14'
.
Also, despite pinning an older version of this cookbook (4.0.6), presumably before you dropped support for Chef 12, I still ran into this issue. It only worked when I manually pinned all of my dependencies to known good versions.
If you're saying that NO versions of this cookbook work on Chef 12 any more, that's one thing, but if older versions are still supposed to work on 12 and they're not that's still an issue. Suddenly breaking and then telling people to pin their apt cookbook manually is pretty crappy. You should be responsible for pinning the right versions of your dependencies in your metadata.rb for each cookbook version. And >=
is not good enough.
from sensu-chef.
That's fair enough, but the Readme should be in any case updated to reflect that this cookbook doesn't support Chef 12 anymore and is Chef 13+. We've pinned the apt cookbook for now and we're in the process of moving to 14 anyway, but this was still a pretty sudden break.
from sensu-chef.
Chef 12 is officially end of life at this point since Chef 14 shipped. You're going to see a large part of the Chef ecosystem begin to drop support for Chef 12. If you still want to continue to use it you'll need to pin apt on your own. It's pretty easy to do this either in your environment or in a wrapper cookbook, but that's really outside the scope of this cookbook. Pinning here would just break users of currently supported Chef releases who wanted to pull down the latest cookbooks.
from sensu-chef.
@tas50 this should work on chef 12.14 from what I recall in #608 is that not the case? If not then we need to either update the metadata or revert.
from sensu-chef.
Confirmed that apt_preference
did not exist before 13.3 here so the question is rollback or update it and say we support chef 13.3 >=
.
from sensu-chef.
If you're saying that NO versions of this cookbook work on Chef 12 any more, that's one thing, but if older versions are still supposed to work on 12 and they're not that's still an issue. Suddenly breaking and then telling people to pin their apt cookbook manually is pretty crappy. You should be responsible for pinning the right versions of your dependencies in your metadata.rb for each cookbook version. And >= is not good enough.
I wholeheartedly agree and had I known at the time that we need at least chef 13.3 I would have pushed back on the changes. What's done is done and there is no sense crying over spilled milk, I am very conflicted on the best course of action at this point is. As the intent of the PR was to bring the community back up to speed with changes and chef 12 is EOL (will never receive another update) and the change was made in a major release (indicating breaking changes which were properly called out in the CHANGELOG) I don't think we should revert it. I think we should update the chef metadata.rb
to indicate the versions we support and I agree that we should have pinned the apt
cookbook (as well as many others) to at least a major version using pessimistic version constraints. Not using >=
comes with its fair share of problems as well (such as not supporting new major versions of chef-client or cookbooks before we have had time to test it) so no matter what someone is affected by the choices we make as far as pinning versions. If someone would like to submit a PR to fix this up it would be greatly appreciated otherwise I can submit a fix this week which should give people time to voice any conerns/disagreement. If there is enough disagreement on this being the right path forward what we could do is add the requirement for apt
back in and would allow us to work with any chef 13.x (not forcing 13.3+
which has everything but apt_preferences
) but I guess if you have already upgraded from 12 -> 13 an upgrade to >= 13.3
will not be too painful so I am not sure if that really helps anyone out but is an option to soften the impact.
from sensu-chef.
As I have not heard anyone stating that we should roll back I have created the above PR to fix this. after it has been reviewed and merged I will release and link to it here.
from sensu-chef.
released: https://supermarket.chef.io/cookbooks/sensu/versions/5.1.2
from sensu-chef.
what are the changes need to upgrade chef 12 to chef14 . could you please give me the how can i check the modifications manually,
from sensu-chef.
@ajaydevops2019 its not quite a straightforward answer as it depends on what cookbooks you are using but here are some good materials to help you get started:
- https://docs.chef.io/upgrade_client.html
- https://docs.chef.io/release_notes.html
- https://blog.chef.io/2017/03/29/upgrading-chef-client-13
- https://blog.chef.io/2018/02/16/preparing-for-chef-14-and-chef-12-end-of-life
from sensu-chef.
from sensu-chef.
@ajaydevops2019 I am not sure what to say, since I don't have access to your codebase I have no idea what you need to do. I linked you to generic resources to help you determine that for yourself. If that is not enough I suggest your company engage in some chef professional services or hire a contractor to assist your team. In the meantime you can pin the version of your cookbook to 4.4.0
as this was the last version of this cookbook to support chef 12. As chef dropped 12.x support on April 30, 2018
it is highly unlikely that we will release any further 4.x changes, the only changes I might see us backporting are any security issues with the chef code itself.
from sensu-chef.
Related Issues (20)
- Execute integration tests on Travis HOT 2
- Consistency with Sensu Plugins development HOT 1
- testing slack integration.
- Enterprise data bag for enterprise repo credentials does not seem to work HOT 3
- Amazon Linux 2 support HOT 4
- Document how to install gems into a non embedded ruby setup HOT 5
- Changelog for 5.1.2 missing HOT 3
- No such file or directory - /sbin/status HOT 14
- server components should be broken out into a unique cookbook HOT 13
- rabbitmq-server Fails to Start After Updating Erlang Packages HOT 8
- Package install is never up to date HOT 5
- How do I add keys to a `sensu_check` resources that are not part of the spec? HOT 2
- Yum package versioning is busted HOT 3
- i want to create a document for chef14. HOT 1
- Sensu server and api restart every run since Chef 14
- Sensu installation fails due to rabbitmq cookbook HOT 3
- Unsigned packages can't be installed on systems that support gpg key checking HOT 3
- The repository 'https://repositories.sensuapp.org/apt cosmic Release' does not have a Release file. HOT 1
- Install fails on RHEL8 clients
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from sensu-chef.