redisio icon indicating copy to clipboard operation
redisio copied to clipboard

Merge dynamic configuration with cookbook configuration?

Open oggy opened this issue 7 years ago • 2 comments

Like others here, I'm finding the breadcrumb mechanism a little unintuitive. Additionally, when using sentinel, it means you can't update the redis configuration after the first chef run without messing with the server state. So I'm wondering: is there a reason the provider can't merge its configuration with what the redis server wrote?

Redis delimits the configuration it writes with a comment: # Generated by CONFIG REWRITE, so the provider could easily append that to the configuration generated by the cookbook. I can submit a patch if this sounds good; just wanted to check if this idea has been considered/dismissed already.

oggy avatar Mar 27 '17 02:03 oggy

I'd like to see a fix for the breadcrumb concept. The initial PR missed updating the sentinel recipe so now I'm stuck having to manually delete this breadcrumb file in order to complete my chef run. I tested this out with even setting default['redisio']['default_settings']['breadcrumb'] to false.

poloka avatar Aug 13 '18 14:08 poloka

Marking stale due to inactivity. Remove stale label or comment or this will be closed in 7 days. Alternatively drop by the #sous-chefs channel on the Chef Community Slack and we'll be happy to help! Thanks, Sous-Chefs.

github-actions[bot] avatar Sep 30 '20 00:09 github-actions[bot]