Project

General

Profile

Bug #13427

On error on nodecompliance SQL, logs are huge

Added by François ARMAND about 1 year ago. Updated about 1 year ago.

Status:
Released
Priority:
N/A
Category:
Performance and scalability
Target version:
Severity:
Major - prevents use of part of Rudder | no simple workaround
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Very Small
Priority:
112
Tags:

Description

One user reported an error on insertion for "nodecompliance" table. That resulted in the full row content being logged, and that content is huge (it is the JSON serialisation of detailled compliance).

We must only log the error without details, and only add details for debug level (or trace).

The problematic insert was:

[2018-09-07 02:22:56] ERROR report - Error when saving node compliances: Batch entry 0 insert into nodecompliance (nodeid, runtimestamp, endoflife, runanalysis, summary, details)
values (‘9c8bab57-d6e2-475b-99f9-df2e8e0521cf’, ‘2018-09-07 02:22:32+02’, ‘2018-09-07 02:32:32+02’, ’{

...
huge json
...

}’) was aborted: ERROR: permission denied for relation nodecompliance  Call getNextException to see other errors in the batch.

Related issues

Related to Rudder - Bug #13429: Make nodecompliance ttl data NewActions

Associated revisions

Revision 4161f9e4 (diff)
Added by François ARMAND about 1 year ago

Fixes #13427: On error on nodecompliance SQL, logs are huge

History

#1

Updated by François ARMAND about 1 year ago

  • Description updated (diff)
#2

Updated by François ARMAND about 1 year ago

  • Target version set to 4.1.15
#3

Updated by François ARMAND about 1 year ago

  • Status changed from New to In progress
#4

Updated by François ARMAND about 1 year ago

  • Related to Bug #13429: Make nodecompliance ttl data added
#5

Updated by François ARMAND about 1 year ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/rudder/pull/2028
#6

Updated by Rudder Quality Assistant about 1 year ago

  • Assignee changed from Nicolas CHARLES to François ARMAND
#7

Updated by François ARMAND about 1 year ago

  • Status changed from Pending technical review to Pending release
#8

Updated by Vincent MEMBRÉ about 1 year ago

  • Status changed from Pending release to Released
  • Priority changed from 113 to 112
This bug has been fixed in Rudder 4.1.15, 4.3.5 and 5.0.1 which were released today.
Changelog
Changelog
Changelog

Also available in: Atom PDF