Project

General

Profile

Bug #1128

MiddlewareErrors really shouldn't be in syslog for non-debug builds

Added by Anonymous almost 9 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Important
Assignee:
-
Category:
Middleware
Target version:
-
Seen in:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
Yes
Needs Doc:
Yes
Needs Merging:
Yes
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

We're already short on space for /var . Having [[MiddlewareErrors]] be reported in syslog for valid operations (that should be trickled up the stack) just seems like a really bad idea, and this will blow up release builds that have been up and running for months where transient, correctable errors are being reported on a periodic basis.

History

#1 Updated by William Grzybowski almost 9 years ago

Do you really think that is a bad idea?

Otherwise there is no other idea yet to view errors that happened in the past

TBH the argument about short on space for /var is very weak, those errors are not frequent and would hardly eat up more than 10k

#2 Updated by Anonymous almost 9 years ago

Replying to [comment:1 william]:

Do you really think that is a bad idea?

Otherwise there is no other idea yet to view errors that happened in the past

TBH the argument about short on space for /var is very weak, those errors are not frequent and would hardly eat up more than 10k

Ok then. We can leave it on for the 8.2 release, but really... we need to revisit this item in the future once we have a sane logging subsystem. It discourages me from using this because I like using Exceptions in python code to trap unwanted scenarios and trickle it up the stack, instead of worrying about testing return values, etc. Using well defined Exceptions will help us create better APIs that will withstand the test of time, instead of continually rototilling things, and thus will help us design a more robust platform.

#3 Updated by William Grzybowski over 8 years ago

  • Status changed from Unscreened to Closed

Also available in: Atom PDF