Issue #12188 has been updated by Daniel Pittman. Status changed from Merged - Pending Release to In Topic Branch Pending Review Branch changed from https://github.com/puppetlabs/puppet/pull/415 to https://github.com/puppetlabs/puppet/pull/421
https://github.com/puppetlabs/puppet/pull/421 fixes a test break on Win32, where the absolute path wasn't qualified with the drive letter. ---------------------------------------- Bug #12188: agent when run --no-daemonize tries to remove the pid of a running daemon https://projects.puppetlabs.com/issues/12188 Author: R.I. Pienaar Status: In Topic Branch Pending Review Priority: Urgent Assignee: Daniel Pittman Category: plumbing Target version: 2.7.11 Affected Puppet version: Keywords: Branch: https://github.com/puppetlabs/puppet/pull/421 The fix in #5246 got the agent remove its pid after running. It's unfortunately too big a hammer as when the agent is also running daemonized the pid removal will fail and log an err level message. Before removing the pid file we should figure out if the pid file belongs to the current process and only then remove it. This is confirmed on 2.7.10 but might also be the case in 2.6 master -- 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.
