On Jul 21, 7:25 am, Jon Jaroker <goo...@jaroker.com> wrote: > Hello Alessandro, > > I relocated the dependency declaration into the native types and also > removed two stray semi-colons. After reviewing the dot graph, I also > made "Secure Check File" a dependency of "Validate Check File". The > updated class is below. > > Puppet still does not honor the "Validate Check File" -> "Make Sudo > File Live" dependency, even with these changes. > > The sudo check failed because my sudo template was missing an EOL > character. I am able to prevent the failure by appending '<% %>' to > the bottom of the template. > > Thank you for checking the class. I think it is correct and this > dependency problem is a bug.
That might very well be, in which case Puppetlabs would probably appreciate a bug report. Before submitting one, however, it would be worthwhile to check whether the catalog the node applies (incorrectly) is indeed the result of compiling the manifest you show. There are several ways in which the node might end up trying to apply a different catalog, whether an old, cached one or one built from a different version of the manifest. Just insert a Notice resource into the class, and make sure it is reflected in the node's log. It might also be amusing to test whether the problem remains if you change Exec['Validate_Check_File'] to have refreshonly => false. I don't see much else in your class that seems a promising explanation for why a feature that generally works reliably to fail for you. Don't mistake me: I think it *should* work as you expect with refreshonly => true; I'm just saying that there's a little dark corner there where a bug might hide. Good luck, John -- You received this message because you are subscribed to the Google Groups "Puppet Users" group. To post to this group, send email to puppet-users@googlegroups.com. To unsubscribe from this group, send email to puppet-users+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/puppet-users?hl=en.
