https://bugs.openldap.org/show_bug.cgi?id=10329
Issue ID: 10329 Summary: Additional issues with pcache, and a test Product: OpenLDAP Version: unspecified Hardware: All OS: All Status: UNCONFIRMED Keywords: needs_review Severity: normal Priority: --- Component: overlays Assignee: bugs@openldap.org Reporter: aweits@rit.edu Target Milestone: ---
Created attachment 1062 --> https://bugs.openldap.org/attachment.cgi?id=1062&action=edit test & patches
Hello again!
Further testing revealed some more issues in pcache [re: ITS#10270]. I've attached an update to test020-proxycache as well. These are based off the current git HEAD.
https://bugs.openldap.org/show_bug.cgi?id=10329
--- Comment #1 from Howard Chu hyc@openldap.org --- Not sure the test changes are workable. We can't really rely on consistent timing, especially when running tests on VMs.
https://bugs.openldap.org/show_bug.cgi?id=10329
Quanah Gibson-Mount quanah@openldap.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |2.5.20 Keywords|needs_review |
--- Comment #2 from Quanah Gibson-Mount quanah@openldap.org --- slapd code commits OK
tests need rewriting as timing cannot be guaranteed with the current test suite.
https://bugs.openldap.org/show_bug.cgi?id=10329
Quanah Gibson-Mount quanah@openldap.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Assignee|bugs@openldap.org |hyc@openldap.org
https://bugs.openldap.org/show_bug.cgi?id=10329
Howard Chu hyc@openldap.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |IN_PROGRESS Ever confirmed|0 |1
--- Comment #3 from Howard Chu hyc@openldap.org --- Code changes only https://git.openldap.org/openldap/openldap/-/merge_requests/770
https://bugs.openldap.org/show_bug.cgi?id=10329
--- Comment #4 from aweits@rit.edu aweits@rit.edu --- Apologies for not responding sooner - I can look at redoing the test and opening a new ITS for that in a few weeks or so (if that's appropriate)... Thanks for taking care of this!
https://bugs.openldap.org/show_bug.cgi?id=10329
Quanah Gibson-Mount quanah@openldap.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|2.5.20 |2.5.21
https://bugs.openldap.org/show_bug.cgi?id=10329
Quanah Gibson-Mount quanah@openldap.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|2.5.21 |2.6.11
https://bugs.openldap.org/show_bug.cgi?id=10329
--- Comment #5 from aweits@rit.edu aweits@rit.edu --- I just registered on gitlab, I have some additional feedback on the patches (the change in refresh_query() should be below the return code check from be_search()).
-
openldap-its＠openldap.org