Run the attached test case
in 1.3.6
Check error logs for messages like
ERR - cos-plugin - cos_cache_query_attr - cos attribute mailAlternateAddress failed schema check on dn: ou=no_cos_scope,dc=example,dc=com
3.
The logs appears on all entries evaluated even if they are out of the scope of the cos
This log should only be logged in plug-in debugging are enabled (nsslapd-errorlog-level: 65536)
This bug is a side effect of https://pagure.io/389-ds-base/issue/48978. The following commit https://pagure.io/389-ds-base/c/fb9b9ef0a74f64abd89e52c5c90fe7dfcea6ba71 change the schema check control from 'SLAPI_LOG_PLUGIN' to 'SLAPI_LOG_ERR'
The code of cos cache is quite complex and IMHO should be rewritten. A cos applies on the subtree of cos definition parent. So it is surprising to see the error message that is also logged on entries OUTSIDE of the scope of the cos. The code being complexe, schema is checked before the scope is tested. So at the end of the day, the cos attribute will not be added to an entry outside of the scope of the cos EVEN if the schema was checked on that entry.
Metadata Update from @tbordaz: - Custom field type adjusted to defect
Metadata Update from @tbordaz: - Issue assigned to tbordaz
Patch and test case added <img alt="0001-Ticket-49249-cos_cache-is-erroneously-logging-schema.patch" src="/389-ds-base/issue/raw/files/c437c633606eac03dbc0f72865eeb77f69400c727c2e418cb1ae3011ff6ffc3f-0001-Ticket-49249-cos_cache-is-erroneously-logging-schema.patch" />
Metadata Update from @tbordaz: - Custom field origin adjusted to IPA - Custom field reviewstatus adjusted to review - Custom field rhbz adjusted to https://bugzilla.redhat.com/show_bug.cgi?id=1437492
Metadata Update from @mreynolds: - Custom field reviewstatus adjusted to ack (was: review)
Metadata Update from @mreynolds: - Issue set to the milestone: 1.3.6.0
git push origin master
Counting objects: 11, done. Delta compression using up to 8 threads. Compressing objects: 100% (11/11), done. Writing objects: 100% (11/11), 2.59 KiB | 0 bytes/s, done. Total 11 (delta 7), reused 0 (delta 0) To ssh://git@pagure.io/389-ds-base.git 063b367..04635e4 master -> master
Metadata Update from @tbordaz: - Issue close_status updated to: fixed - Issue status updated to: Closed (was: Open)
git push origin 389-ds-base-1.3.6
Counting objects: 11, done. Delta compression using up to 8 threads. Compressing objects: 100% (11/11), done. Writing objects: 100% (11/11), 2.59 KiB | 0 bytes/s, done. Total 11 (delta 7), reused 0 (delta 0) To ssh://git@pagure.io/389-ds-base.git 2d07ca4..71a98aa 389-ds-base-1.3.6 -> 389-ds-base-1.3.6
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 issue has been cloned to Github and is available here: - https://github.com/389ds/389-ds-base/issues/2308
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: fixed)
Login to comment on this ticket.