-
Notifications
You must be signed in to change notification settings - Fork 18
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
rewind a resource that is already declared twice #6
Comments
Just noted that I have to rewind both instances of the resource to get the desired result:
Certainly not ideal |
This is a duplicate of #3, I guess. We've fixed the unwind case, but not (yet) the rewind case. |
Hi @thommay I don't think this is a dupe of that issue, for which the fix is unrelated to chef-rewind. |
Hm, you're right sorry. I'll construct some test cases and see what this looks like. |
I tested this, and found that (with current master) I just needed the second rewind block; the first was superfluous. Can you test with the 0.0.9 release and confirm this? |
The opsode apache cookbook actually clones the apache ports.conf template resource in the default and mod_ssl recipes:
If I want to rewind that resource in my own wrapper cookbook, and try to use chef-rewind, it appears to only find a single instance of the resource (maybe the first one), and rewind it. But I still end up with the template from presumably the second declaration of the resource
I know cloning the resource in the first instance is a bad thing, but is there some way to make chef-rewind match and rewind all instances of the resource, effectively consolidating them into a single resource declaration?
The text was updated successfully, but these errors were encountered: