Bug #1910

Logcheck: "Compiled configuration" changed to "Compiled catalog"

Added by micah - about 3 years ago. Updated almost 3 years ago.

Status:Closed Start date:01/28/2009
Priority:Normal Due date:
Assignee:James Turnbull % Done:

0%
Category:unknown
Target version:0.24.8
Affected Puppet version:0.24.6 Branch:
Keywords:logcheck
Votes: 0

Description

Looks like there was a slight message change in 0.24.6 which requires a small logcheck change.

The notice on the puppetmaster that used to say:

Compiled configuration for node.domain.net in 40.93 seconds

now says:

Compiled catalog for node.domain.net in 40.93 seconds

patch attached.

logcheck_catalog.diff (756 Bytes) micah -, 01/28/2009 02:45 am

logcheck2.diff (650 Bytes) micah -, 02/25/2009 11:54 pm

logcheck_1910_fix.diff (827 Bytes) micah -, 03/04/2009 04:34 am

History

#1
Updated by James Turnbull about 3 years ago

  • Status changed from Unreviewed to Closed
  • Target version changed from 0.24.8 to 0.25.0

Fixed in commit:a065aeb567896be04ebac978a380e6becf4518eb in branch master.

#2
Updated by micah - almost 3 years ago

  • File logcheck2.diff added
  • Target version changed from 0.25.0 to 0.24.8

This should be targeted for 0.24.8 as its a trivial change and 0.24.7/0.24.8 are affected by it.

Additionally, since upgrading to 0.24.7 there are more puppetmaster logcheck entries that now occur:

Feb 23 20:09:28 puppetmaster puppetmasterd[3529]: Searched for resource params and tags in 0.54 seconds
Feb 23 20:09:29 puppetmaster puppetmasterd[3529]: Handled resources in 2.09 seconds
Feb 23 20:09:29 puppetmaster puppetmasterd[3529]: Resource comparison took 1.27 seconds
Feb 23 20:10:18 puppetmaster puppetmasterd[3507]: Searched for resources in 0.35 seconds
Feb 23 20:11:59 puppetmaster puppetmasterd[3529]: Searched for host in 0.00 seconds

Please apply the attached patch to handle these as well.

#3
Updated by James Turnbull almost 3 years ago

Pushed in commit:8c010e0ccc8b007f8657a1025d2de36e2310e550 in branch 0.24.x

#4
Updated by micah - almost 3 years ago

  • File logcheck_1910_fix.diff added
  • Status changed from Closed to Re-opened
  • Assignee set to James Turnbull
  • 3 changed from Unknown to Trivial
  • Keywords set to logcheck

Hi James,

Can you push this minor change to the previous diff, it makes it actually work as intended, instead of not… sorry about that.

#5
Updated by James Turnbull almost 3 years ago

  • Status changed from Re-opened to Closed

Pushed in commit:a790ee3b487a7eac6668e45747e338d44d75da9e in branch 0.24.x

Also available in: Atom PDF