Project

General

Profile

Actions
Copy link

Bug #7635

closed

syslog-ng's loghost regex does not match with set line

Added by Janos Mattyasovszky about 9 years ago. Updated almost 9 years ago.

Status:
Released
Priority:
1 (highest)
Assignee:
Nicolas CHARLES
Category:
Techniques
Target version:
3.0.13
Pull Request:
https://github.com/Normation/rudder-t...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

Hi,

Please check this:

https://github.com/Normation/rudder-techniques/commit/24ee9cd0dc7e80022d8f6d3f6ebfbe3d45c6c5b1
https://github.com/Normation/rudder-techniques/commit/39ca4bd861bbd4f8b2a11563de4e76191c87981b

The first commit changed the name of the loghost to rudder_loghost, but then the second fix reverts the regex-matching var to use a hard-coded version instead of the perviously changed variable...

-      "syslog_ng_conf_regex"  string => concat(escape("${syslog_ng_conf_prefix}"), "[^\"]+", escape("${syslog_ng_conf_suffix}"), ".*");
+      "syslog_ng_conf_regex"  string => concat("filter\ f\_local\_rudder\{facility\(local6\)\ and\ program\(\"rudder\.\*\"\)\;\}\;destination\ loghost\ \{(tcp|udp)\(\"", "[^\"]+", escape("${syslog_ng_conf_suffix}"), ".*");

The implemented Fix should please also fix the misconfigured logfiles :-)

Related issues 2 (0 open — 2 closed)

Related to Rudder - Bug #5773: Syslog configuration on agent can conflict with loghostsReleasedJonathan CLARKE2014-11-13Actions
Related to Rudder - Bug #5799: system technique can only setup log settings *once* on syslog-ngReleasedJonathan CLARKE2014-11-20Actions
Actions
Copy link
 #1

Updated by Vincent MEMBRÉ about 9 years ago

  • Related to Bug #5773: Syslog configuration on agent can conflict with loghosts added
Actions
Copy link
 #2

Updated by Vincent MEMBRÉ about 9 years ago

  • Related to Bug #5799: system technique can only setup log settings *once* on syslog-ng added
Actions
Copy link
 #3

Updated by Jonathan CLARKE about 9 years ago

  • Target version set to 3.0.13

Oh dear. Well spotted, Janos. Thanks for the detailed bug report. Fix coming.

Actions
Copy link
 #4

Updated by Jonathan CLARKE about 9 years ago

  • Status changed from New to In progress
  • Assignee set to Jonathan CLARKE
Actions
Copy link
 #5

Updated by Jonathan CLARKE almost 9 years ago

  • Translation missing: en.field_tag_list set to Quick and important
Actions
Copy link
 #6

Updated by Benoît PECCATTE almost 9 years ago

  • Assignee changed from Jonathan CLARKE to Benoît PECCATTE
Actions
Copy link
 #7

Updated by Benoît PECCATTE almost 9 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Benoît PECCATTE to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/rudder-techniques/pull/838
Actions
Copy link
 #8

Updated by Nicolas CHARLES almost 9 years ago

  • Assignee changed from Nicolas CHARLES to Jonathan CLARKE
Actions
Copy link
 #9

Updated by Jonathan CLARKE almost 9 years ago

  • Assignee changed from Jonathan CLARKE to Benoît PECCATTE
  • Pull Request changed from https://github.com/Normation/rudder-techniques/pull/838 to https://github.com/Normation/rudder-techniques/pull/839
Actions
Copy link
 #10

Updated by Jonathan CLARKE almost 9 years ago

  • Assignee changed from Benoît PECCATTE to Nicolas CHARLES
Actions
Copy link
 #11

Updated by Jonathan CLARKE almost 9 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100
Actions
Copy link
 #12

Updated by Vincent MEMBRÉ almost 9 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 3.0.13, 3.1.6 and 3.2.0 which were released today.

Actions
Copy link

Also available in: Atom PDF