Issue #7917 has been updated by Jeff McCune.
Jeff McCune wrote: > (Copied from github comments) > > Daniel, > > I'm in agreement with Nan. > > We do make claims that puppet apply is a "first class citizen" and community > members like Jordan Sissel heavily rely on puppet apply being feature > complete with puppet agent. > > Furthermore, puppet apply is more suitable for situations like Vagrant where > a full master is not avaialable or necessary. > > Finally, as we integrate more with MCollective, the classes.txt file becomes > ever more important for filtering. > > For these reasons I think we should still write the file and we should write > it to the same file the agent writes to by default. > > If the end user desires, we support writing to multiple files as Nan > illustrated above. > > I'll re-open this once. If you still disagree, please feel free to close the > pull request and I won't re-open it. > > -Jeff I think it's worth noting; In my experience it's a far more common where end users run exclusively with puppet apply than it is end users mix puppet agent and puppet apply on the same node. Therefore, I think the more common case "wins" over the concern that puppet apply will "fight" with puppet agent. Does this make sense Daniel? -Jeff ---------------------------------------- Bug #7917: Puppet apply runmode should write classes.txt file https://projects.puppetlabs.com/issues/7917 Author: Jeff McCune Status: Needs Decision Priority: Normal Assignee: Nigel Kersten Category: settings Target version: Affected Puppet version: 2.7.0 Keywords: apply classes.txt classes mcollective agent puppet.conf conf main Branch: https://github.com/puppetlabs/puppet/pull/35 # Overview # The classes.txt file is only written when running Puppet in the agent run mode. It would be useful for integration with MCollective filtering if this file were also written when running Puppet the stand alone apply run mode. <pre> # The file in which puppet agent stores a list of the classes # associated with the retrieved configuration. Can be loaded in # the separate `puppet` executable using the `--loadclasses` # option. # The default value is '$statedir/classes.txt'. classfile = /var/lib/puppet/state/classes.txt </pre> This would be useful with MCollective's configuration setting of: classesfile = /var/lib/puppet/state/classes.txt # Impact Data # At least one community member has requested this feature. (See comments below). This ticket is important for mcollective integration with "puppet apply." -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
