)]}'
{"id":"openstack%2Fkeystone~851979","triplet_id":"openstack%2Fkeystone~master~Idf4dd83527d077043a5b69c43209cc2d0420720b","project":"openstack/keystone","branch":"master","topic":"bug/1959674","attention_set":{},"removed_from_attention_set":{"28691":{"account":{"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"},"last_update":"2022-08-08 09:15:30.000000000","reason":"\u003cGERRIT_ACCOUNT_28691\u003e replied on the change","reason_account":{"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"}},"28619":{"account":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"last_update":"2022-09-24 15:25:41.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"Idf4dd83527d077043a5b69c43209cc2d0420720b","subject":"Fix application_credential identification in auth","status":"ABANDONED","created":"2022-08-03 10:23:22.000000000","updated":"2022-09-24 15:25:41.000000000","total_comment_count":3,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"2d56f0eb4eafce93cc6a2109520d5c3a2cd115ff","_number":851979,"virtual_id_number":851979,"owner":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"},{"tag":"autogenerated:zuul:check","value":1,"date":"2022-08-03 12:15:43.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":26892,"name":"Marcus Murwall","email":"marcus.murwall@cleura.com","username":"muran"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"recommended":{"_account_id":26892,"name":"Marcus Murwall","email":"marcus.murwall@cleura.com","username":"muran"},"all":[{"value":0,"date":"2022-08-08 09:15:30.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2022-08-03 10:40:18.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":26892,"name":"Marcus Murwall","email":"marcus.murwall@cleura.com","username":"muran"}],"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":"","value":1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":26892,"name":"Marcus Murwall","email":"marcus.murwall@cleura.com","username":"muran"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":26892,"name":"Marcus Murwall","email":"marcus.murwall@cleura.com","username":"muran"},{"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2022-08-03 10:38:34.000000000","updated_by":{"_account_id":26892,"name":"Marcus Murwall","email":"marcus.murwall@cleura.com","username":"muran"},"reviewer":{"_account_id":26892,"name":"Marcus Murwall","email":"marcus.murwall@cleura.com","username":"muran"},"state":"REVIEWER"},{"updated":"2022-08-03 11:02:56.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2022-08-03 11:04:01.000000000","updated_by":{"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"},"reviewer":{"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"},"state":"CC"},{"updated":"2022-08-03 11:05:12.000000000","updated_by":{"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"},"reviewer":{"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"},"state":"REVIEWER"},{"updated":"2022-08-03 12:15:43.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"269db7ec2f9328f3f68e61656fce3c000985db92","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"date":"2022-08-03 10:23:22.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"e95f2201502cd8c42fd94433dbeac98173e0cc6d","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"date":"2022-08-03 10:24:36.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"3d01f486017d97770ebaa43fef885bfff2a07cf1","author":{"_account_id":26892,"name":"Marcus Murwall","email":"marcus.murwall@cleura.com","username":"muran"},"date":"2022-08-03 10:38:34.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"052cb7dc1cd83923cf0ffc279b130cba70b76a17","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"date":"2022-08-03 10:39:11.000000000","message":"Patch Set 3: Commit message was updated.","accounts_in_message":[],"_revision_number":3},{"id":"047eca08aa509c06d5d116b8083ab1ad851431e4","author":{"_account_id":26892,"name":"Marcus Murwall","email":"marcus.murwall@cleura.com","username":"muran"},"date":"2022-08-03 10:40:18.000000000","message":"Patch Set 3: Code-Review+1","accounts_in_message":[],"_revision_number":3},{"id":"305976485ad3193ac36a4ff2017946b74098dd6e","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-08-03 11:02:56.000000000","message":"Patch Set 3:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/adc2dd33dac34beabe3127c62ccc2e01\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/ca3ebe094fb94cb5afa00be9e9e7f3a6 : SUCCESS in 23m 19s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/5f3c800af2754a54b337efc1ad03b27a : SUCCESS in 23m 17s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"3fcb4d699c395d5ff99bdcff6ebc554012dfe76a","author":{"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"},"date":"2022-08-03 11:05:05.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"bbac22f888a444b43ebe48d0517596525c71449f","author":{"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"},"date":"2022-08-03 11:05:12.000000000","message":"Patch Set 3: Code-Review-1","accounts_in_message":[],"_revision_number":3},{"id":"25eeb78ca297ddec4d27dcc13511bc7dc779ed72","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-08-03 12:15:43.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/92704cf691334e5aa86c41807a2369fe\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/8b35836be3d94397bcc71460df129948 : SUCCESS in 14m 29s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/284a2d2e3a464ab2a7e1adb4be409fbb : SUCCESS in 5m 35s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/4d9789d5707f4b0fb22f3253c01e60b6 : SUCCESS in 15m 35s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/68b52f719e3542209c99124ec277e09c : SUCCESS in 11m 40s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/a723794d046245e2beafe9ad7d0506ac : SUCCESS in 16m 05s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f18559e966fc4077a0f3a96fc78cebff : SUCCESS in 12m 39s\n- grenade https://zuul.opendev.org/t/openstack/build/d72931309aec46e88a7797ed97497c86 : SUCCESS in 1h 05m 49s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/08f3b2bd7e4844099cbc4445b977547d : SUCCESS in 1h 35m 16s\n- keystone-dsvm-py3-functional https://zuul.opendev.org/t/openstack/build/79c63245b4a24bdbb5e43e61f3003db5 : SUCCESS in 29m 01s\n- keystone-dsvm-py3-functional-fips https://zuul.opendev.org/t/openstack/build/572ea959779c4250b3617d3e10858e60 : SUCCESS in 55m 03s (non-voting)\n- keystone-dsvm-py3-functional-federation-ubuntu-focal https://zuul.opendev.org/t/openstack/build/209210d05fe0453c9f6a595f56e2b88b : SUCCESS in 32m 21s (non-voting)\n- keystone-dsvm-py3-functional-federation-ubuntu-focal-k2k https://zuul.opendev.org/t/openstack/build/fcfb020701ed4e6b8ec9b6c4866057cf : SUCCESS in 28m 22s\n- keystoneclient-devstack-functional https://zuul.opendev.org/t/openstack/build/ee684dbf135f4680aba70644206f9e08 : SUCCESS in 15m 26s (non-voting)\n- keystone-dsvm-ldap-domain-specific-driver https://zuul.opendev.org/t/openstack/build/1849f3f26d4d4a6aaa03dc18aebe0c47 : SUCCESS in 34m 20s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/b3205fb00fdd41398bda6fcf98ab11e0 : SUCCESS in 53m 25s\n- keystone-protection-functional https://zuul.opendev.org/t/openstack/build/41d071ed0db44fb28347f545b063ea54 : SUCCESS in 35m 32s","accounts_in_message":[],"_revision_number":3},{"id":"57885c1055d77685d106af68b9ee8a36991adcba","author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"date":"2022-08-05 10:18:11.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"1c76c768c50b67ca69830e71e95c2b713bc45932","author":{"_account_id":28691,"name":"Bo Tran","email":"ministry.96.nd@gmail.com","username":"ministry"},"date":"2022-08-08 09:15:30.000000000","message":"Patch Set 3: -Code-Review\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"2d56f0eb4eafce93cc6a2109520d5c3a2cd115ff","tag":"autogenerated:gerrit:abandon","author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"date":"2022-09-24 15:25:41.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"1dc210e5eb167247f56c01a48e092b0b74471833","revisions":{"66983a301614ba311bd3c8318158569ee1b48960":{"kind":"REWORK","_number":1,"created":"2022-08-03 10:23:22.000000000","uploader":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"ref":"refs/changes/79/851979/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/79/851979/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/79/851979/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/79/851979/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/79/851979/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/79/851979/1"}}},"commit":{"parents":[{"commit":"1dd6993d7b9b647810e6f495b62c37627c6e8658","subject":"Merge \"docs: Update docs to reflect migration to Alembic\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/1dd6993d7b9b647810e6f495b62c37627c6e8658"}]}],"author":{"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@ya.ru","date":"2022-08-03 10:22:45.000000000","tz":120},"committer":{"name":"Dmitriy Rabotyagov","email":"dmitriy.rabotyagov@citynetwork.eu","date":"2022-08-03 10:23:07.000000000","tz":120},"subject":"Fix application_credential identification in auth","message":"Fix application_credential identification in auth\n\nIn cases, when a token was created from application_credentials, and\na service tries to auth with such token, method_names would include\napplication_credential since it\u0027s a part of the reciept. At the same\nauth supposed to be processed with jsut a token, so it does not include\nassumed properties that application_credential does have which results\nin KeyError.\n\nWith that we change condition and try to get application_credential.id\nonly when application_credential is part of the auth instead of the\nmethod_names.\n\nChange-Id: Idf4dd83527d077043a5b69c43209cc2d0420720b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/66983a301614ba311bd3c8318158569ee1b48960"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/66983a301614ba311bd3c8318158569ee1b48960"}]},"branch":"refs/heads/master"},"3df82452504f564424ef29180d0c2e2a2766e612":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2022-08-03 10:24:36.000000000","uploader":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"ref":"refs/changes/79/851979/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/79/851979/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/79/851979/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/79/851979/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/79/851979/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/79/851979/2"}}},"commit":{"parents":[{"commit":"1dd6993d7b9b647810e6f495b62c37627c6e8658","subject":"Merge \"docs: Update docs to reflect migration to Alembic\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/1dd6993d7b9b647810e6f495b62c37627c6e8658"}]}],"author":{"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@ya.ru","date":"2022-08-03 10:22:45.000000000","tz":120},"committer":{"name":"Dmitriy Rabotyagov","email":"dmitriy.rabotyagov@citynetwork.eu","date":"2022-08-03 10:24:11.000000000","tz":120},"subject":"Fix application_credential identification in auth","message":"Fix application_credential identification in auth\n\nIn cases, when a token was created from application_credentials, and\na service tries to auth with such token, method_names would include\napplication_credential since it\u0027s a part of the reciept. At the same\nauth supposed to be processed with jsut a token, so it does not include\nassumed properties that application_credential does have which results\nin KeyError.\n\nWith that we change condition and try to get application_credential.id\nonly when application_credential is part of the auth instead of the\nmethod_names.\n\nCloses-Bug: #1959674\nChange-Id: Idf4dd83527d077043a5b69c43209cc2d0420720b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/3df82452504f564424ef29180d0c2e2a2766e612"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/3df82452504f564424ef29180d0c2e2a2766e612"}]},"branch":"refs/heads/master"},"1dc210e5eb167247f56c01a48e092b0b74471833":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2022-08-03 10:39:11.000000000","uploader":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"ref":"refs/changes/79/851979/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/79/851979/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/79/851979/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/79/851979/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/79/851979/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/79/851979/3"}}},"commit":{"parents":[{"commit":"1dd6993d7b9b647810e6f495b62c37627c6e8658","subject":"Merge \"docs: Update docs to reflect migration to Alembic\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/1dd6993d7b9b647810e6f495b62c37627c6e8658"}]}],"author":{"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@ya.ru","date":"2022-08-03 10:22:45.000000000","tz":120},"committer":{"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","date":"2022-08-03 10:39:11.000000000","tz":0},"subject":"Fix application_credential identification in auth","message":"Fix application_credential identification in auth\n\nIn cases, when a token was created from application_credentials, and\na service tries to auth with such token, method_names would include\napplication_credential since it\u0027s a part of the reciept. At the same\nauth supposed to be processed with just a token, so it does not include\nassumed properties that application_credential does have which results\nin KeyError.\n\nWith that we change condition and try to get application_credential.id\nonly when application_credential is part of the auth instead of the\nmethod_names.\n\nCloses-Bug: #1959674\nChange-Id: Idf4dd83527d077043a5b69c43209cc2d0420720b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/1dc210e5eb167247f56c01a48e092b0b74471833"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/1dc210e5eb167247f56c01a48e092b0b74471833"}]},"branch":"refs/heads/master","description":"Edit commit message"}},"requirements":[],"submit_records":[],"submit_requirements":[]}