-
Notifications
You must be signed in to change notification settings - Fork 300
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HPCC-23896 Missing ESP logging in cloud #13651
HPCC-23896 Missing ESP logging in cloud #13651
Conversation
@jakesmith Please review |
https://track.hpccsystems.com/browse/HPCC-23896 |
Signed-off-by: Richard Chapman <rchapman@hpccsystems.com>
47f940d
to
d5c70a7
Compare
Automated Smoketest: ✅ Unit tests result:
Regression test result:
HPCC Stop: OK
|
Automated Smoketest: ✅ Unit tests result:
Regression test result:
HPCC Stop: OK
|
@@ -2350,6 +2351,11 @@ void setupContainerizedLogMsgHandler() | |||
IPropertyTree * logConfig = queryComponentConfig().queryPropTree("logging"); | |||
if (logConfig) | |||
{ | |||
if (logConfig->getPropBool(logQueueDisabledAtt, false)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perhaps should also disable if traceLevel = 0?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
traceLevel is not something known about at jlib level (but yes, whole area of traceLevel vs loggingDetail is a bit of a mess)
@@ -715,6 +715,8 @@ int CCD_API roxie_main(int argc, const char *argv[], const char * defaultYaml) | |||
traceLevel = topology->getPropInt("@traceLevel", runOnce ? 0 : 1); | |||
if (traceLevel > MAXTRACELEVEL) | |||
traceLevel = MAXTRACELEVEL; | |||
if (traceLevel && topology->hasProp("logging/@disabled")) | |||
topology->setPropBool("logging/@disabled", false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If a config file had disabled=true, I'd have thought that would override other logging settings like traveLevel.
But this means if traveLevel>0, it ignores @disabled flag.
Is this logic required?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's so that I can set traceLevel=1 when running a standalone program (where default logging is disabled)
@richardkchapman - looks good, a couple of questions. |
@jakesmith Questions answered @ghalliday Please review/merge |
Signed-off-by: Richard Chapman rchapman@hpccsystems.com
Type of change:
Checklist:
Smoketest:
Testing: