Rewrite Consul cookbook to use custom resources
emachnic opened this issue · comments
🗣️ Foreword
Thank for taking the time to fill this feature request fully. Without it we may not be able to , and the issue may be closed without resolution.
:person_frowning: Problem Statement
This cookbook is a mess of custom libraries that are difficult to maintain.
❔ Possible Solution
We should rewrite the Consul cookbook to use Chef custom resources
⤴️ Describe alternatives you've considered
➕ Additional context
I totally agree here, but unfortunately do not have time to do it myself.
I think its fair to assume major version breaking changes. 🏛️🚚
@johnbellone I've actually been working on a rewrite for this for a major version release that will leverage custom resources. It's been a long process thus far and I've had to refocus but hopefully be able to pick it back up
Instead of doing a complete rewrite in a single commit, I'll suggest that we gradually migrate individual features (like consulDefinition) one at a time in separate releases. This will help us ensure backward compatibility and also reduce the probability of bugs.
WDYT @johnbellone @emachnic ?
@theanirudhvyas that would normally be the best way but, after having gone through a lot of this cookbook, a complete rewrite is probably the most expeditious.