Issue #4224 has been updated by Jeff McCune. Status changed from Needs More Information to Closed
# Closing in favor of 16637 We discussed this in the puppet-dev IRC channel today and we feel we've missed the boat for fixing this bug in the 2.7 series. We do still accept responsibility for fixing this issue, but we do not intend to fix it in 2.7 because it would likely be an unwelcome surprise. Instead, we plan to fix it in 3.x as soon as possible. <pre> [4:55pm] zaphod42: jmccune: I noticed that #16637 referenced #4224, which seems to be the exact same thing but against 2.7.x. What are your thoughts on what should be done on #4224? [4:55pm] gepetto: zaphod42: jmccune: #16637 is http://projects.puppetlabs.com/issues/show/16637 "Bug #16637: Puppet confdir and vardir are wrong when running non-root - Puppet. It has a status of In Topic Branch Pending Review and is assigned to -" [4:55pm] jmccune: looking [4:55pm] zaphod42: gepetto doesn't understand me talking about multiple bugs.... #4224 [4:55pm] zaphod42: #4224 [4:55pm] gepetto: zaphod42: #4224 is http://projects.puppetlabs.com/issues/show/4224 "Bug #4224: vardir and confdir should be in ~/.puppet if not run as root - Puppet. It has a status of Needs More Information and is assigned to -" [4:56pm] jmccune: Yeah [4:56pm] zaphod42: gepetto-- [4:56pm] jmccune: I ran across that on Saturday and my eyes almost rolled out of my head. [4:56pm] jmccune: I mean, we sort of missed the boat on that one for 2.7 [4:56pm] drewmania joined the chat room. [4:56pm] zaphod42: that is what I'm thinking [4:56pm] jmccune: Given that we've carried 20 releases of the master explicitly using the system directories. [4:56pm] derpops joined the chat room. [4:57pm] zaphod42: I think the 2.7 bug should just be closed [4:57pm] jmccune: I think it would be an unwelcome surprise for users even if we did accept it as a bug over 2 years ago [4:57pm] jmccune: Yeah [4:57pm] jmccune: Doing that [4:57pm] zaphod42: cool </pre> ---------------------------------------- Bug #4224: vardir and confdir should be in ~/.puppet if not run as root https://projects.puppetlabs.com/issues/4224#change-72150 Author: Matt Robinson Status: Closed Priority: Normal Assignee: Category: Target version: 2.7.x Affected Puppet version: Keywords: defaults settings Branch: There's currently logic (which_dir) in lib/puppet/util/run_mode.rb that uses /var/lib/puppet and /etc/puppet if you're running puppetmaster regardless of if you're running it as root. James agrees this is not correct behavior. I guess the logic at some point was that if you're running puppetmaster you must need root. That seems backwards to me since master doesn't really need root, except to maybe switch to run as the puppet user, and if anything this logic should apply to the agent, but perhaps not even then. def var_dir which_dir( (Puppet.features.microsoft_windows? ? File.join(Dir::WINDOWS, "puppet", "var") : "/var/lib/puppet"), "~/.puppet/var" ) end def which_dir( global, user ) #FIXME: we should test if we're user "puppet" # there's a comment that suggests that we do that # and we currently don't. expand_path case when name == :master; global when Puppet.features.root?; global else user end end -- 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 puppet-bugs@googlegroups.com. To unsubscribe from this group, send email to puppet-bugs+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
