The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com

Bug #5445

Mac pkg creation script should create /var/lib/puppet

Added by Nigel Kersten almost 4 years ago. Updated over 2 years ago.

Status:ClosedStart date:12/02/2010
Priority:NormalDue date:
Assignee:Gary Larizza% Done:

0%

Category:OSX
Target version:2.7.12
Affected Puppet version: Branch:https://github.com/glarizza/puppet-1/tree/bug/2.7.x/5445_create_varlib
Keywords:

We've Moved!

Ticket tracking is now hosted in JIRA: https://tickets.puppetlabs.com

This issue is currently not available for export. If you are experiencing the issue described below, please file a new ticket in JIRA. Once a new ticket has been created, please add a link to it that points back to this Redmine ticket.


Description

EOM.


Related issues

Related to Puppet - Bug #5530: puppet master fails to create "$vardir/rrd" Duplicate 12/13/2010

History

#1 Updated by Paul Berry over 3 years ago

  • Status changed from Accepted to Code Insufficient

#2 Updated by Paul Berry over 3 years ago

See discussion on the puppet-dev mailing list.

#3 Updated by Gary Larizza over 2 years ago

  • Category set to OSX
  • Target version set to 2.7.x
  • Branch set to https://github.com/glarizza/puppet-1/tree/bug/2.7.x/5445_create_varlib

I checked out the discussion on the Dev list, and made the changes that were suggested (added notes to the commit and made the change to createpackage.sh that was suggested).

Pull request is here https://github.com/puppetlabs/puppet/pull/384

#4 Updated by Gary Larizza over 2 years ago

  • Status changed from Code Insufficient to In Topic Branch Pending Review

#5 Updated by Anonymous over 2 years ago

  • Status changed from In Topic Branch Pending Review to Needs More Information
  • Assignee changed from Nigel Kersten to Gary Larizza

Isn’t this a non-standard place to put files on MacOS? I thought it preferred /var/db or so for these files, and /var/lib was a Linux-ism?

#6 Updated by Gary Larizza over 2 years ago

That was discussed http://groups.google.com/group/puppet-dev/browse_thread/thread/d200f8bbe91c9/db1fa48992a7b227?lnk=gst&q=%235445#db1fa48992a7b227 but basically, we’ve used /var/lib/puppet since .25.5 and the change from /var/lib/ should probably be left to a major release and not to a minor release. CURRENTLY, /var/lib/puppet IS the location for $vardir, so we need to ensure that it exists before puppet runs, but I’d be happy to change that in the next major version (should we all decide that this is the best location for it).

#7 Updated by Anonymous over 2 years ago

  • Status changed from Needs More Information to In Topic Branch Pending Review

Gary Larizza wrote:

That was discussed http://groups.google.com/group/puppet-dev/browse_thread/thread/d200f8bbe91c9/db1fa48992a7b227?lnk=gst&q=%235445#db1fa48992a7b227 but basically, we’ve used /var/lib/puppet since .25.5 and the change from /var/lib/ should probably be left to a major release and not to a minor release. CURRENTLY, /var/lib/puppet IS the location for $vardir, so we need to ensure that it exists before puppet runs, but I’d be happy to change that in the next major version (should we all decide that this is the best location for it).

Awesome. Can you follow up with another pull request targetting Telly, and implementing the change of location? If not, a ticket to the effect we need to satisfies. Thanks.

#8 Updated by Gary Larizza over 2 years ago

I think a new ticket might be better for this. I could CHANGE the createpackage.sh script to create /var/db/puppet, but we would need further work to ensure that OS X uses /var/db/puppet as the $vardir. I’ll create that now.

#9 Updated by Gary Larizza over 2 years ago

Actually, wouldn’t this be covered in #9407?

#10 Updated by Nigel Kersten over 2 years ago

Yes.

#11 Updated by Anonymous over 2 years ago

  • Status changed from In Topic Branch Pending Review to Merged - Pending Release

#12 Updated by Matthaus Owens over 2 years ago

  • Status changed from Merged - Pending Release to Closed
  • Target version changed from 2.7.x to 2.7.12

Released in 2.7.12rc1

Also available in: Atom PDF