)]}'
{"id":"openstack%2Fkeystone~983209","triplet_id":"openstack%2Fkeystone~master~I3b27e19087d28ca424ee7617a9b4c63d92a01654","project":"openstack/keystone","branch":"master","hashtags":[],"change_id":"I3b27e19087d28ca424ee7617a9b4c63d92a01654","subject":"Fix LDAP paged results with size limits","status":"ABANDONED","created":"2026-04-02 14:02:54.000000000","updated":"2026-04-02 14:07:40.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"72c64bedf7bf402ac14f743d9be2b56ac24f7927","_number":983209,"virtual_id_number":983209,"owner":{"_account_id":38589,"name":"Moutaz Chaara","display_name":"Moutaz Chaara","email":"moutaz.chaara@sap.com","username":"tz3","status":"SAP SE"},"actions":{},"labels":{"Verified":{"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{},"pending_reviewers":{},"reviewer_updates":[],"messages":[{"id":"bc006bea9367317a3808ace05a8402229d2520cd","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":38589,"name":"Moutaz Chaara","display_name":"Moutaz Chaara","email":"moutaz.chaara@sap.com","username":"tz3","status":"SAP SE"},"date":"2026-04-02 14:02:54.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"72c64bedf7bf402ac14f743d9be2b56ac24f7927","tag":"autogenerated:gerrit:abandon","author":{"_account_id":38589,"name":"Moutaz Chaara","display_name":"Moutaz Chaara","email":"moutaz.chaara@sap.com","username":"tz3","status":"SAP SE"},"date":"2026-04-02 14:07:40.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"5fa1722827cae333f41a6a978e9ebf3bc0e25799","revisions":{"5fa1722827cae333f41a6a978e9ebf3bc0e25799":{"kind":"REWORK","_number":1,"created":"2026-04-02 14:02:54.000000000","uploader":{"_account_id":38589,"name":"Moutaz Chaara","display_name":"Moutaz Chaara","email":"moutaz.chaara@sap.com","username":"tz3","status":"SAP SE"},"ref":"refs/changes/09/983209/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/09/983209/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/09/983209/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/09/983209/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/09/983209/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/09/983209/1"}}},"commit":{"parents":[{"commit":"3526925ae34a66e626bec985628a421a7694168e","subject":"Merge \"Keystone identity mapping to support project definition as a JSON\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/3526925ae34a66e626bec985628a421a7694168e"}]}],"author":{"name":"Moutaz Chaara","email":"moutaz.chaara@sap.com","date":"2026-04-02 14:01:47.000000000","tz":120},"committer":{"name":"Moutaz Chaara","email":"moutaz.chaara@sap.com","date":"2026-04-02 14:02:34.000000000","tz":120},"subject":"Fix LDAP paged results with size limits","message":"Fix LDAP paged results with size limits\n\nThe FakeLdap test harness\u0027s result3() method didn\u0027t properly\nimplement pagination. When a page size was set, it would just\ntruncate results without tracking pagination state across\nmultiple requests.\n\nThis fix:\n- Extracts and parses the cookie to determine starting index\n- Returns only the requested page of results\n- Generates response control with updated cookie\n- Supports both old and new python-ldap APIs\n\nRefactored into helper methods for better maintainability:\n- _get_paging_cookie(): Extract cookie from control\n- _create_paging_control(): Create response control\n\nThis ensures LDAP paging works correctly even when the LDAP\nserver has size limits configured.\n\nChange-Id: I3b27e19087d28ca424ee7617a9b4c63d92a01654\nSigned-off-by: Moutaz Chaara \u003cmoutaz.chaara@sap.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/5fa1722827cae333f41a6a978e9ebf3bc0e25799"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/5fa1722827cae333f41a6a978e9ebf3bc0e25799"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{}}}]}