)]}'
{"id":"openstack%2Fkeystone~976389","triplet_id":"openstack%2Fkeystone~master~I75bd568749ffb4b08ed5d00311d88a37c66a6867","project":"openstack/keystone","branch":"master","topic":"fix-protection-tests","hashtags":[],"change_id":"I75bd568749ffb4b08ed5d00311d88a37c66a6867","subject":"Fix protection tests for manager role","status":"ABANDONED","created":"2026-02-11 13:29:53.000000000","updated":"2026-02-11 13:42:14.000000000","total_comment_count":0,"unresolved_comment_count":0,"work_in_progress":true,"has_review_started":false,"meta_rev_id":"97faefad262595172c981132cfae97486f6018c4","_number":976389,"virtual_id_number":976389,"owner":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"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":"2d5b338e48f7c713c20dc07e3e45efeeaa0b3968","tag":"autogenerated:gerrit:newWipPatchSet","author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"date":"2026-02-11 13:29:53.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"bdb90bb94902249ec45a3e2c914f41748d6e0a9c","tag":"autogenerated:gerrit:newWipPatchSet","author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"date":"2026-02-11 13:32:50.000000000","message":"Uploaded patch set 2: New patch set was added with same tree, parent tree, and commit message as Patch Set 1.","accounts_in_message":[],"_revision_number":2},{"id":"94841b1ce2707aa802d16c03dc620b61074e6456","tag":"autogenerated:gerrit:newWipPatchSet","author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"date":"2026-02-11 13:36:47.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"97faefad262595172c981132cfae97486f6018c4","tag":"autogenerated:gerrit:abandon","author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"date":"2026-02-11 13:42:14.000000000","message":"Abandoned\n\nAbandoning in favor of rebasing https://review.opendev.org/c/openstack/keystone/+/886434 which already includes these test fixes.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"061909460c2ecb23add347bcbadde12f93e5f8fb","revisions":{"20b05653330e91d6b603c9f9eebdfc805a6a7aa3":{"kind":"REWORK","_number":1,"created":"2026-02-11 13:29:53.000000000","uploader":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"ref":"refs/changes/89/976389/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/89/976389/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/89/976389/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/89/976389/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/89/976389/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/89/976389/1"}}},"commit":{"parents":[{"commit":"7b5702f994139140ce195acbe15598dfcf4c8b98","subject":"Merge \"Replace deprecated warn method\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/7b5702f994139140ce195acbe15598dfcf4c8b98"}]}],"author":{"name":"Grzegorz Grasza","email":"xek@redhat.com","date":"2026-02-11 13:28:48.000000000","tz":60},"committer":{"name":"Grzegorz Grasza","email":"xek@redhat.com","date":"2026-02-11 13:29:36.000000000","tz":60},"subject":"Fix protection tests: update role count and run in CI","message":"Fix protection tests: update role count and run in CI\n\nThe bootstrap process now creates 5 roles (service, admin, manager,\nmember, reader), but test_roles.py was still asserting 5 total\n(4 bootstrap + 1 test-created). Update the expected count to 6\n(5 bootstrap + 1 test-created) to account for the manager role\nadded in commit bbbbd10e4.\n\nAlso add protection tests to the default tox testenv so they run\nalongside unit tests in CI, ensuring they don\u0027t silently regress.\n\nChange-Id: I75bd568749ffb4b08ed5d00311d88a37c66a6867\nSigned-off-by: Grzegorz Grasza \u003cxek@redhat.com\u003e\nCo-authored-by: Cursor \u003ccursoragent@cursor.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/20b05653330e91d6b603c9f9eebdfc805a6a7aa3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/20b05653330e91d6b603c9f9eebdfc805a6a7aa3"}]},"branch":"refs/heads/master"},"cc903a99185d25a3c6aea725076ddf232cbc8265":{"kind":"NO_CHANGE","_number":2,"created":"2026-02-11 13:32:50.000000000","uploader":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"ref":"refs/changes/89/976389/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/89/976389/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/89/976389/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/89/976389/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/89/976389/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/89/976389/2"}}},"commit":{"parents":[{"commit":"7b5702f994139140ce195acbe15598dfcf4c8b98","subject":"Merge \"Replace deprecated warn method\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/7b5702f994139140ce195acbe15598dfcf4c8b98"}]}],"author":{"name":"Grzegorz Grasza","email":"xek@redhat.com","date":"2026-02-11 13:28:48.000000000","tz":60},"committer":{"name":"Grzegorz Grasza","email":"xek@redhat.com","date":"2026-02-11 13:32:38.000000000","tz":60},"subject":"Fix protection tests: update role count and run in CI","message":"Fix protection tests: update role count and run in CI\n\nThe bootstrap process now creates 5 roles (service, admin, manager,\nmember, reader), but test_roles.py was still asserting 5 total\n(4 bootstrap + 1 test-created). Update the expected count to 6\n(5 bootstrap + 1 test-created) to account for the manager role\nadded in commit bbbbd10e4.\n\nAlso add protection tests to the default tox testenv so they run\nalongside unit tests in CI, ensuring they don\u0027t silently regress.\n\nChange-Id: I75bd568749ffb4b08ed5d00311d88a37c66a6867\nSigned-off-by: Grzegorz Grasza \u003cxek@redhat.com\u003e\nCo-authored-by: Cursor \u003ccursoragent@cursor.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/cc903a99185d25a3c6aea725076ddf232cbc8265"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/cc903a99185d25a3c6aea725076ddf232cbc8265"}]},"branch":"refs/heads/master"},"061909460c2ecb23add347bcbadde12f93e5f8fb":{"kind":"REWORK","_number":3,"created":"2026-02-11 13:36:47.000000000","uploader":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"ref":"refs/changes/89/976389/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/89/976389/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/89/976389/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/89/976389/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/89/976389/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/89/976389/3"}}},"commit":{"parents":[{"commit":"7b5702f994139140ce195acbe15598dfcf4c8b98","subject":"Merge \"Replace deprecated warn method\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/7b5702f994139140ce195acbe15598dfcf4c8b98"}]}],"author":{"name":"Grzegorz Grasza","email":"xek@redhat.com","date":"2026-02-11 13:28:48.000000000","tz":60},"committer":{"name":"Grzegorz Grasza","email":"xek@redhat.com","date":"2026-02-11 13:36:31.000000000","tz":60},"subject":"Fix protection tests for manager role","message":"Fix protection tests for manager role\n\nThe bootstrap process now creates 5 roles (service, admin, manager,\nmember, reader) and 3 implied role inferences (admin-\u003emanager,\nmanager-\u003emember, member-\u003ereader), but the protection tests were not\nupdated after the manager role was added in commit bbbbd10e4.\n\nUpdate test_roles.py expected count from 5 to 6 (5 bootstrap + 1\ntest-created) and test_implied_roles.py expected inferences from\n3 to 4 (3 defaults + 1 test-created).\n\nChange-Id: I75bd568749ffb4b08ed5d00311d88a37c66a6867\nSigned-off-by: Grzegorz Grasza \u003cxek@redhat.com\u003e\nCo-authored-by: Cursor \u003ccursoragent@cursor.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/061909460c2ecb23add347bcbadde12f93e5f8fb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/061909460c2ecb23add347bcbadde12f93e5f8fb"}]},"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":{}}}]}