#49822 Issue 49640 - Cleanup plugin bootstrap logging
Closed by spichugi. Opened by spichugi.
spichugi/389-ds-base plugin_error_fix  into  master

Download 49822.patch

Bug Description: We add PBKDF2_SHA256 password storage schema two times. During:
1. the dse.ldif parsing;
2. the bootstrap plugin operation.
It causes the error to appear during the startup.

Fix Description: Make plugin_setup() function report the error to TRACE log level
if the plugin already exists. We will report the error in ERR log level during
the config bootstrap anyway (code path for the 1st option from bug description).
For 2nd option, report the error to TRACE if it is 'already exist' issue
and to ERR if it is any other case.

Make the plugin_setup returns more consistent.

https://pagure.io/389-ds-base/issue/49640

Reviewed by: ?

rebased onto 7ecfefe478266fae0fb6dd792342a2df50a5597f

Is this logging line redundant from what you did in plugin.c?

Is this logging line redundant from what you did in plugin.c

I was thinking that it can potentially help with the debugging.
It points out the function name 'slapd_bootstrap_config' and the configfile where the issue is happening. So we can narrow down the problem if it would happen.

But we use TRACE log level rarely because it's really noisy... So if you think it is too much redundant, I'll remove it.

Good point, ack

rebased onto 2fa04088931450237ad6580fd5fc5903878bdbb0

Pull-Request has been merged by spichugi

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/2881

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

Metadata