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

This issue tracker is now in read-only archive mode and automatic ticket export has been disabled. Redmine users will need to create a new JIRA account to file tickets using https://tickets.puppetlabs.com. See the following page for information on filing tickets with JIRA:

Bug #15337

Do not merge configuration files in Telly

Added by Anonymous almost 4 years ago. Updated over 3 years ago.

Status:ClosedStart date:07/02/2012
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:settings
Target version:3.0.0
Affected Puppet version: Branch:https://github.com/puppetlabs/puppet/pull/1006
Keywords:settings puppet.conf configuration merge

We've Moved!

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


Description

Overview

In 3.0rc the user puppet.conf is merged with the system puppet.conf. This behavior is undesirable because it makes it difficult to know where to install modules, where to load extensions from, and where settings are actually being defined.


Related issues

Related to Puppet - Refactor #7749: Bootstrapping Puppet in Ruby code has nasty scope cycles... Closed 06/01/2011
Related to Puppet - Bug #13429: Puppet::Resource fails when used from scripts Closed 03/26/2012
Related to Puppet - Bug #16637: Puppet confdir and vardir are wrong when running non-root Closed 09/29/2012

History

#1 Updated by Anonymous almost 4 years ago

Seems to be related to ticket #7749 [1]. The change looks to have happened in commit e465c16 [2] in pull request 571 [3]

#2 Updated by Anonymous over 3 years ago

  • Target version changed from 3.0.0 to 3.x

#3 Updated by Anonymous over 3 years ago

Picking this back up today.

An old topic branch I had is at https://github.com/jeffmccune/puppet/tree/ticket/3.0rc/15337_do_not_merge_configuration_files_in_telly

I’m going to rebase this off of 3.x and continue on.

#4 Updated by Anonymous over 3 years ago

7789 and 13429 are slightly related and in this order:

$ git lg | ack '7749|13429'
* \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \   895de3b Merge pull request #710 from cprice-puppet/bug/master/13429-faces-as-libraries 3 months ago Daniel Pittman 
| | * | | | | | | | | | | | | | | | | | 812b3cd (#13429) refactor some settings initialization 3 months ago Chris Price  (cprice-puppet/bug/master/13429-faces-as-libraries)
* | | | | | | | | | | | | |   c20c81a Merge pull request #571 from puppetlabs/integration/master/7749-bootstrapping-puppet 4 months ago Daniel Pittman 
| * | | | | | | | | | | | | e78a77e (#7749) doc / tests cleanup 4 months ago cprice 
| * | | | | | | | | | | | 1aa71a9 (#7749) rename "Settings#setdefaults" to "define_settings" 5 months ago cprice 
| * | | | | | | | | | | | 274847c (#7749) get rid of $puppet_application_name and $puppet_application_mode globals 5 months ago cprice 
| * | | | | | | | | | | | 4751522 (#7749) get rid of "set_run_mode" method in Application class 5 months ago cprice 
| * | | | | | | | | | | | e465c16 (#7749) Improvements to handling settings, bootstrapping puppet 5 months ago cprice 
| * | | | | | | | | | | | cb3ce74 (#7749) Parse command line args and config file before loading app/face 5 months ago cprice 
| * | | | | | | | | | | | dc23404 (#7749) Reduce weird scope cycles when bootstrapping settings 5 months ago cprice 

#5 Updated by Anonymous over 3 years ago

  • Status changed from Accepted to In Topic Branch Pending Review
  • Branch set to https://github.com/puppetlabs/puppet/pull/1006

#6 Updated by Anonymous over 3 years ago

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

Merged in commit 6236218

#7 Updated by eric sorenson over 3 years ago

  • Target version changed from 3.x to 3.0.0

#8 Updated by Matthaus Owens over 3 years ago

  • Status changed from Merged - Pending Release to Closed
  • Affected Puppet version deleted (3.0.0rc3)

Released in Puppet 3.0.0-rc4

Also available in: Atom PDF