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

Bug #5477

if site.pp does not exist on the first compile, puppet master will not detect when it is created

Added by Dan Bode over 3 years ago. Updated about 3 years ago.

Status:ClosedStart date:12/08/2010
Priority:HighDue date:
Assignee:Jesse Wolfe% Done:

0%

Category:parser
Target version:2.6.8
Affected Puppet version: Branch:http://github.com/jes5199/puppet/tree/ticket/next/5477
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

In 2.6, compile does not fail when site.pp does not exist.

However, if a catalog is compiled when site.pp does not exist, puppetmaster does not detect when site.pp is created. This requires a restart.

To recreate:

# ensure that site.pp does not exist
>rm -f /etc/puppet/manifests/site.pp
>puppet master --verbose --no-daemonize
>puppet agent --test
>echo 'notify{foo:}' > /etc/puppet/manifests/site.pp
>sleep 20
>puppet agent --test

changes in site.pp are not noticed, it requires a restart of the puppetmaster.


Related issues

Related to Puppet - Bug #15106: Missing site.pp can cause error 'Cannot find definition C... Closed 06/18/2012

History

#1 Updated by Nigel Kersten over 3 years ago

  • Status changed from Unreviewed to Accepted
  • Priority changed from Normal to High

#2 Updated by Nigel Kersten over 3 years ago

  • Target version changed from 69 to 2.6.5

#3 Updated by Nigel Kersten about 3 years ago

  • Target version changed from 2.6.5 to 2.6.x

#4 Updated by James Turnbull about 3 years ago

  • Target version changed from 2.6.x to 2.6.6

#5 Updated by James Turnbull about 3 years ago

  • Target version changed from 2.6.6 to 2.6.x

#6 Updated by Jesse Wolfe about 3 years ago

  • Assignee set to Jesse Wolfe
  • Branch set to http://github.com/jes5199/puppet/tree/ticket/next/5477

I’ve got a draft of a solution

#7 Updated by Jesse Wolfe about 3 years ago

  • Status changed from Accepted to Merged - Pending Release

Available in 2.6.next as of commit:a2ac367c2bea2b94b88f739250edb828da0091d4

Available in next as of commit:6985cc0f898d71dbd2ffbb353a2a482d9783cb42

#8 Updated by James Turnbull about 3 years ago

  • Target version changed from 2.6.x to 2.6.8

#9 Updated by Dominic Maraglia about 3 years ago

  • Status changed from Merged - Pending Release to Closed

Verified fixed in 2.6.next on: ec3178f7f3e7d4afd167089bc78ec99b88a0b664

Acceptance level test added to puppet-acceptance harness:

github.com:puppetlabs/puppet-acceptance.git 9fc64b9ff1087f5d1ea13b2d7e80d9741b00f2ce

Also available in: Atom PDF