)]}'
{"id":"openstack%2Fkeystone~783405","triplet_id":"openstack%2Fkeystone~stable%2Fstein~I75590c20e90170ed862f46f0de7d61c7810b5c90","project":"openstack/keystone","branch":"stable/stein","topic":"bug/1885753","hashtags":[],"change_id":"I75590c20e90170ed862f46f0de7d61c7810b5c90","subject":"Retry update_user when sqlalchemy raises StaleDataErrors","status":"MERGED","created":"2021-03-26 17:50:53.000000000","updated":"2021-04-28 21:32:03.000000000","submitted":"2021-04-28 21:23:17.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"783405-bug/1885753","meta_rev_id":"30b37700dba265dd5247c8a207001891efa12d5c","_number":783405,"virtual_id_number":783405,"owner":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-04-28 21:23:15.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"value":0,"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":16465,"name":"Kristi Nikolla","email":"knikolla@bu.edu","username":"knikolla"}],"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":{"approved":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"recommended":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2021-04-15 21:34:36.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"value":2,"date":"2021-04-28 14:08:00.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"value":2,"date":"2021-04-28 16:38:04.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":1,"date":"2021-04-14 14:52:53.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":16465,"name":"Kristi Nikolla","email":"knikolla@bu.edu","username":"knikolla"}],"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":{"approved":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"value":0,"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"value":1,"date":"2021-04-28 16:38:04.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":16465,"name":"Kristi Nikolla","email":"knikolla@bu.edu","username":"knikolla"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":16465,"name":"Kristi Nikolla","email":"knikolla@bu.edu","username":"knikolla"},{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-03-26 19:35:13.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"},{"updated":"2021-03-29 19:33:48.000000000","updated_by":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"},{"updated":"2021-03-29 19:33:48.000000000","updated_by":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"reviewer":{"_account_id":16465,"name":"Kristi Nikolla","email":"knikolla@bu.edu","username":"knikolla"},"state":"REVIEWER"},{"updated":"2021-04-15 21:34:36.000000000","updated_by":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"reviewer":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"state":"REVIEWER"}],"messages":[{"id":"92fa79f47592f561fa7701beb85200921a3f2bfd","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2021-03-26 17:50:53.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"39b2a46b392f76a9d95954fd7030126ed091bf29","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-26 19:35:13.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/2a8a814ab5f143618185a2d85aed7298 : FAILURE in 35m 13s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/89a38ae13e134e2e92dcdc2638e6d35e : FAILURE in 32m 53s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/0da20432b66f416695dbc0a26785dda5 : SUCCESS in 6m 07s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/ec551dabd3b540898dcfbfc7efa9a3a0 : FAILURE in 29m 30s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/12105b5b15274e4bb3c99e160f4e75eb : FAILURE in 30m 01s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/f86e00ff4dde4fbd92ace9684377ebd1 : FAILURE in 30m 35s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/58d9e81b261841ee8a60b88721817aba : SUCCESS in 10m 17s\n- tempest-full https://zuul.opendev.org/t/openstack/build/af8c862759484a53b7851830a2f62531 : SUCCESS in 1h 23m 55s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/84d04e6e3d39463e93aa8e84ee6dbba3 : SUCCESS in 50m 27s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/619d6293a1a9401fa42c424d31fec7eb : SUCCESS in 1h 32m 31s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/5ed586754f564320bd8d6cac911d0b0e : SUCCESS in 7m 10s\n- keystone-dsvm-functional https://zuul.opendev.org/t/openstack/build/caead0cc33a949b49cb59e12108fd812 : SUCCESS in 33m 38s\n- keystone-dsvm-py3-functional https://zuul.opendev.org/t/openstack/build/68e7fd4d0e4d45578e03b9d7a69acff6 : SUCCESS in 34m 48s\n- keystone-dsvm-functional-federation-opensuse15 https://zuul.opendev.org/t/openstack/build/a0e390482a68455ea9541f3185aa2e22 : FAILURE in 13m 17s (non-voting)\n- keystone-dsvm-py3-functional-federation-opensuse15 https://zuul.opendev.org/t/openstack/build/83ac7da245344ece84e60c82626907e5 : FAILURE in 13m 33s (non-voting)\n- keystone-dsvm-py3-functional-federation-opensuse15-k2k https://zuul.opendev.org/t/openstack/build/94c7294186d549c1bdc0837934ea50f0 : FAILURE in 14m 55s (non-voting)\n- keystoneclient-devstack-functional https://zuul.opendev.org/t/openstack/build/e947967d6321463f8f45ad8a907e8488 : SUCCESS in 18m 51s (non-voting)\n- keystone-dsvm-ldap-domain-specific-driver https://zuul.opendev.org/t/openstack/build/a732ec023e8d464eb77fec976bb9b46d : SUCCESS in 41m 36s (non-voting)\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/be4cbfe4a5944cbc883ecbdffea2130f : SUCCESS in 1h 00m 39s","accounts_in_message":[],"_revision_number":1},{"id":"ce4e081a80326b6d1a7f2ae44d910312c7e40e6a","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2021-03-29 19:27:05.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"109d0580c7cb2eefaaa6f72b9bcfaaf644d4c421","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-29 21:39:44.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/c1c114621c3d4206a507dcf2ae6aca88 : SUCCESS in 30m 26s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/2a092b95d72a4bffa8c6572653f8fa48 : SUCCESS in 27m 55s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/baffafaa957a4a4cbdedfd510efb9c6e : SUCCESS in 8m 36s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/0aeed62988b74a2faa2fd9a1fe10e37e : FAILURE in 20m 37s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/cf75aca017784f7891b3607a034bea0a : SUCCESS in 25m 20s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/386096ad5fe948c380959c8a5604d185 : SUCCESS in 27m 23s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/09bc91ef6b054e448443136b840a2686 : SUCCESS in 10m 38s\n- tempest-full https://zuul.opendev.org/t/openstack/build/234192f3d6d448c8b11a8deee0dbffeb : TIMED_OUT in 2h 06m 43s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/3dd67241860f45a8a48c1c21a414d3a8 : SUCCESS in 48m 04s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/8698247ff39e40a9bb2d27eba8957355 : SUCCESS in 1h 15m 20s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/61b5f1987ef346abaef8e54380c00336 : SUCCESS in 8m 16s\n- keystone-dsvm-functional https://zuul.opendev.org/t/openstack/build/b6be41f25d004750af7186195753ef94 : SUCCESS in 34m 22s\n- keystone-dsvm-py3-functional https://zuul.opendev.org/t/openstack/build/a6d6718af3c84db99bb314ccfdfbce57 : SUCCESS in 28m 42s\n- keystone-dsvm-functional-federation-opensuse15 https://zuul.opendev.org/t/openstack/build/be238100225248e8a03f90a52e404d22 : FAILURE in 18m 16s (non-voting)\n- keystone-dsvm-py3-functional-federation-opensuse15 https://zuul.opendev.org/t/openstack/build/c7df9284e4ad4a8495cd28b918c899aa : FAILURE in 14m 01s (non-voting)\n- keystone-dsvm-py3-functional-federation-opensuse15-k2k https://zuul.opendev.org/t/openstack/build/4618222c448b47ee974bd0bee3431e60 : FAILURE in 18m 26s (non-voting)\n- keystoneclient-devstack-functional https://zuul.opendev.org/t/openstack/build/459940cb832045af8ab3abe844e0c93b : SUCCESS in 14m 09s (non-voting)\n- keystone-dsvm-ldap-domain-specific-driver https://zuul.opendev.org/t/openstack/build/32e6fa9f93d74e1aa7348e04b5f15b4d : SUCCESS in 32m 17s (non-voting)\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/a97c256ad6064b4596901fbd3c787c41 : SUCCESS in 1h 03m 21s","accounts_in_message":[],"_revision_number":2},{"id":"ef4767eb042e242b817eefb488099b1f839b69da","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2021-03-30 19:38:09.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"0036aafbee430707b948303098ed99759528d090","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-31 01:58:12.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/ecc6861e68054720a47bc4a837b1af86 : FAILURE in 29m 43s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/75dca0cbdefe46b5aea6191c66fa81d2 : FAILURE in 32m 24s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/919024d95182417987ae88c977e47c92 : SUCCESS in 7m 27s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/914bdfeee70c4e9992b8c4478e8e4df7 : SUCCESS in 25m 53s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/0cd2961656244d78aab8ab3d5ef964d0 : FAILURE in 31m 41s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/0a1b3f26a3fc442dba8203c6f269d591 : FAILURE in 28m 29s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ffcda50c40704e9995ba75988362cb14 : SUCCESS in 12m 01s\n- tempest-full https://zuul.opendev.org/t/openstack/build/8f1e34622e084cdb9ae056ca9b624bac : SUCCESS in 1h 42m 26s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/3c939ecc43a94372ae0d8d1f665b2d8c : SUCCESS in 1h 00m 17s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/4d013e89dfee4e7787e043aeb32f39b2 : SUCCESS in 1h 21m 46s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/292d46b179354080b06d2490350a11bd : SUCCESS in 9m 03s\n- keystone-dsvm-functional https://zuul.opendev.org/t/openstack/build/033b82e35e024818b48f789407faf168 : SUCCESS in 36m 34s\n- keystone-dsvm-py3-functional https://zuul.opendev.org/t/openstack/build/7858b8a0eab44bd189d7975258b4b4f4 : SUCCESS in 31m 29s\n- keystone-dsvm-functional-federation-opensuse15 https://zuul.opendev.org/t/openstack/build/21bcffbf5fd04c99b13583f9010392cc : FAILURE in 13m 46s (non-voting)\n- keystone-dsvm-py3-functional-federation-opensuse15 https://zuul.opendev.org/t/openstack/build/afa440d15a95499c96ecb8dab715577c : FAILURE in 9m 38s (non-voting)\n- keystone-dsvm-py3-functional-federation-opensuse15-k2k https://zuul.opendev.org/t/openstack/build/d04845cad0174ae6b5765ad7e8f67469 : FAILURE in 15m 04s (non-voting)\n- keystoneclient-devstack-functional https://zuul.opendev.org/t/openstack/build/b8e4c62037cf4d9abd49046a2843b6c8 : SUCCESS in 17m 48s (non-voting)\n- keystone-dsvm-ldap-domain-specific-driver https://zuul.opendev.org/t/openstack/build/2889e3370bd743bdb6fae8d84dac57f6 : SUCCESS in 47m 42s (non-voting)\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/3d0c1283cb35461ba0a9bcad47e9cce1 : SUCCESS in 53m 50s","accounts_in_message":[],"_revision_number":3},{"id":"b14dafbc50062937d54ab585ed5f40b36b82f002","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2021-03-31 12:35:34.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"27f9f78809fd624fd9a86d08fcadcbf4abbf7706","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-31 14:02:50.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/bea5f3a1b1d44d8ea297cdb7350a746d : SUCCESS in 33m 32s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/b88cf4299f524532a6b08ca96bfb1b81 : SUCCESS in 28m 22s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/70a876907fff4899a32a4ffab9968b99 : SUCCESS in 5m 57s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/4e29796b514e49d99e95e80acf3ac540 : SUCCESS in 21m 17s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/d6af82b277cf4fd5ad47639e8d590f88 : SUCCESS in 25m 49s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/50d9dffb4dde4bffb2b1e0324503e59c : SUCCESS in 27m 31s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/3bfbca181405437a9cfab874159fd4bb : SUCCESS in 9m 35s\n- tempest-full https://zuul.opendev.org/t/openstack/build/8aad0f1f54ad4696b811b4ec41276176 : SUCCESS in 1h 22m 21s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/ffbd9998f75b4848a8b8f8a4e590e2c3 : SUCCESS in 55m 53s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/229f323bf2644445b1fc3299e18982da : SUCCESS in 1h 12m 18s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/bc0a185e634b4231ace1b5a7c973ed95 : SUCCESS in 8m 31s\n- keystone-dsvm-functional https://zuul.opendev.org/t/openstack/build/57d1012cddea43dcb4f94587487d2476 : SUCCESS in 29m 54s\n- keystone-dsvm-py3-functional https://zuul.opendev.org/t/openstack/build/fa5b2ea3ed704309999c207226920e44 : SUCCESS in 34m 30s\n- keystone-dsvm-functional-federation-opensuse15 https://zuul.opendev.org/t/openstack/build/9ba1df00914b404982f0cfe11cdc27f2 : FAILURE in 13m 41s (non-voting)\n- keystone-dsvm-py3-functional-federation-opensuse15 https://zuul.opendev.org/t/openstack/build/7c39f3f0da4e4a6f8bb452b54074b045 : FAILURE in 13m 47s (non-voting)\n- keystone-dsvm-py3-functional-federation-opensuse15-k2k https://zuul.opendev.org/t/openstack/build/ac5f4bc17e77404e8c4507aa1bd210ae : FAILURE in 14m 56s (non-voting)\n- keystoneclient-devstack-functional https://zuul.opendev.org/t/openstack/build/423bf9d4ba9844929b4f61e5b59f6620 : SUCCESS in 18m 43s (non-voting)\n- keystone-dsvm-ldap-domain-specific-driver https://zuul.opendev.org/t/openstack/build/faf7928afb7b4a6f8fe541aae89989f0 : SUCCESS in 37m 56s (non-voting)\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/ba8beacb9ea440809938dbff547933cd : SUCCESS in 1h 02m 23s","accounts_in_message":[],"_revision_number":4},{"id":"f9070d30af20f1ce9239a0d7308c041252efffd2","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2021-04-05 14:43:51.000000000","message":"Patch Set 4: Code-Review+1\n\nLooks good, waiting on a +1 from Kristi, a review from Sean, and the stable/train patch to land.","accounts_in_message":[],"_revision_number":4},{"id":"497a43f0930521e2db4603600becf19eed9b1457","author":{"_account_id":16465,"name":"Kristi Nikolla","email":"knikolla@bu.edu","username":"knikolla"},"date":"2021-04-14 14:52:53.000000000","message":"Patch Set 4: Code-Review+1","accounts_in_message":[],"_revision_number":4},{"id":"20418438e464ea296ca13c8b4b64374c2f178adf","author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"date":"2021-04-15 21:34:36.000000000","message":"Patch Set 4: Code-Review+1","accounts_in_message":[],"_revision_number":4},{"id":"2fe778fd20987b0227eb9a33a7ab891ee86b9bd2","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2021-04-28 14:08:00.000000000","message":"Patch Set 4: Code-Review+2\n\nI\u0027ll let Sean kick this through.","accounts_in_message":[],"_revision_number":4},{"id":"a88610845eebc9b3fb4d3bd92b52f21560805d5e","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2021-04-28 16:38:04.000000000","message":"Patch Set 4: Code-Review+2 Workflow+1\n\nLooks good.","accounts_in_message":[],"_revision_number":4},{"id":"0e9254f0db448c00e96054b5b7d6593fead3c7f1","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-28 16:38:41.000000000","message":"Patch Set 4: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":4},{"id":"838f464cf2162c69c2b1e5486ae643c50faa9e24","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-28 21:23:15.000000000","message":"Patch Set 4: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/16428c73304b40fb8929742427c04e39 : SUCCESS in 28m 36s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/18b0471c146c487a92bace5dd78c5142 : SUCCESS in 6m 39s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/269c1f98cc3b498fa1ed56fc332ae54a : SUCCESS in 15m 53s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/b90dd897faa540aba2158d82396165eb : SUCCESS in 26m 10s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/fb7ed04e036c47ce8b3d2cb8348f9d70 : SUCCESS in 23m 34s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/5fcad820f6b1465085f00d62b3d8a695 : SUCCESS in 13m 36s\n- tempest-full https://zuul.opendev.org/t/openstack/build/d440dbd07bad4d0180309dec233e7dca : SUCCESS in 1h 05m 50s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/01ce4bc3c5a8412e85ade36560965917 : SUCCESS in 45m 55s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/46220dbab9ff41f09a01ba0c485d3acf : SUCCESS in 1h 22m 33s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/0ac0e872db614c5c8299c45a71f34c61 : SUCCESS in 7m 08s\n- keystone-dsvm-functional https://zuul.opendev.org/t/openstack/build/392102f2c9f84c1fa8f9e19eeba7b803 : SUCCESS in 30m 58s\n- keystone-dsvm-py3-functional https://zuul.opendev.org/t/openstack/build/441a213e4de64ddea02a02d8b28cbe61 : SUCCESS in 25m 53s\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/5d5884b09fbf41a9a5ff35550a0411e4 : SUCCESS in 59m 04s","accounts_in_message":[],"_revision_number":4},{"id":"91f2c05461729115b763ba86a2659525d1266c02","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-28 21:23:17.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":4},{"id":"30b37700dba265dd5247c8a207001891efa12d5c","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-04-28 21:32:03.000000000","message":"Patch Set 4:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/45c4db18d08b4f40a46837c06b58342b : SUCCESS in 1m 29s\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/7b829eeff54e4cd7b38adbaa8e84db4b : SUCCESS in 1m 06s","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"f36034c8a6530b16f6b6eb88ee97f540c974ba00","revisions":{"0345cf175971a89f73e61f1f84e3ef8f89770f18":{"kind":"REWORK","_number":1,"created":"2021-03-26 17:50:53.000000000","uploader":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"ref":"refs/changes/05/783405/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/05/783405/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/05/783405/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/05/783405/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/05/783405/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/05/783405/1"}}},"commit":{"parents":[{"commit":"a249bb79137cd1137ecf55a4c7877e7808f794ff","subject":"Merge \"Delete system role assignments from system_assignment table\" into stable/stein","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/a249bb79137cd1137ecf55a4c7877e7808f794ff"}]}],"author":{"name":"Lance Bragstad","email":"lbragstad@gmail.com","date":"2020-06-30 16:50:41.000000000","tz":-300},"committer":{"name":"Lance Bragstad","email":"lbragstad@gmail.com","date":"2021-03-26 17:49:20.000000000","tz":0},"subject":"Retry update_user when sqlalchemy raises StaleDataErrors","message":"Retry update_user when sqlalchemy raises StaleDataErrors\n\nKeystone\u0027s update_user() method in the SQL driver processes a lot of\ninformation about how to update users. This includes evaluating password\nlogic and authentication attempts for PSI-DSS. This logic is evaluated\nafter keystone pulls the user record from SQL and before it exits the\ncontext manager, which performs the write.\n\nWhen multiple clients are all updating the same user reference, it\u0027s\nmore likely they will see an HTTP 500 because of race conditions exiting\nthe context manager. The HTTP 500 is due to stale data when updating\npassword expiration for old passwords, which happens when setting a new\npassword for a user.\n\nThis commit attempts to handle that case more gracefully than throwing a\n500 by detecting StaleDataErrors from sqlalchemy and retrying.  The\nidentity sql backend will retry the request for clients that have\nstale data change from underneath them.\n\nChange-Id: I75590c20e90170ed862f46f0de7d61c7810b5c90\nCloses-Bug: 1885753\n(cherry picked from commit 6b581bfe43811379e9ceea2770242f05f0c9fa66)\n(cherry picked from commit 22119176d612d4441d9cf25e76adde52ec4f88e2)\n(cherry picked from commit 20a10ebc50fda0ac3cfab378293f9f2c27168c96)\n(cherry picked from commit 72e1cab0a47f493d47dbe482a03da269b4238967)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/0345cf175971a89f73e61f1f84e3ef8f89770f18"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/0345cf175971a89f73e61f1f84e3ef8f89770f18"}]},"branch":"refs/heads/stable/stein"},"13ad966c459267e91985ff54b26f7a67993e62af":{"kind":"REWORK","_number":2,"created":"2021-03-29 19:27:05.000000000","uploader":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"ref":"refs/changes/05/783405/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/05/783405/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/05/783405/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/05/783405/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/05/783405/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/05/783405/2"}}},"commit":{"parents":[{"commit":"a249bb79137cd1137ecf55a4c7877e7808f794ff","subject":"Merge \"Delete system role assignments from system_assignment table\" into stable/stein","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/a249bb79137cd1137ecf55a4c7877e7808f794ff"}]}],"author":{"name":"Lance Bragstad","email":"lbragstad@gmail.com","date":"2020-06-30 16:50:41.000000000","tz":-300},"committer":{"name":"Lance Bragstad","email":"lbragstad@gmail.com","date":"2021-03-29 19:25:23.000000000","tz":0},"subject":"Retry update_user when sqlalchemy raises StaleDataErrors","message":"Retry update_user when sqlalchemy raises StaleDataErrors\n\nKeystone\u0027s update_user() method in the SQL driver processes a lot of\ninformation about how to update users. This includes evaluating password\nlogic and authentication attempts for PSI-DSS. This logic is evaluated\nafter keystone pulls the user record from SQL and before it exits the\ncontext manager, which performs the write.\n\nWhen multiple clients are all updating the same user reference, it\u0027s\nmore likely they will see an HTTP 500 because of race conditions exiting\nthe context manager. The HTTP 500 is due to stale data when updating\npassword expiration for old passwords, which happens when setting a new\npassword for a user.\n\nThis commit attempts to handle that case more gracefully than throwing a\n500 by detecting StaleDataErrors from sqlalchemy and retrying.  The\nidentity sql backend will retry the request for clients that have\nstale data change from underneath them.\n\nConflicts:\n      keystone/tests/unit/test_backend_sql.py due to import order\n      differences between train and ussuri.\n\nChange-Id: I75590c20e90170ed862f46f0de7d61c7810b5c90\nCloses-Bug: 1885753\n(cherry picked from commit ceae3566e83b26fd6a1679154eae9b0cef29da64)\n(cherry picked from commit f47e635b8041542faa05e64606e66d2fbbc5f284)\n(cherry picked from commit 5b7d4c80d484262018f937083050844648f07a11)\n(cherry picked from commit 07d3a3d3ff534a5295842d4f236042b30536cd82)\n(cherry picked from commit f3feadfa045f1ec9f74c74e9044ec53a8824f8de)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/13ad966c459267e91985ff54b26f7a67993e62af"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/13ad966c459267e91985ff54b26f7a67993e62af"}]},"branch":"refs/heads/stable/stein"},"658f66be864c13c43b2395c829bce6c6c30578b7":{"kind":"REWORK","_number":3,"created":"2021-03-30 19:38:09.000000000","uploader":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"ref":"refs/changes/05/783405/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/05/783405/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/05/783405/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/05/783405/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/05/783405/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/05/783405/3"}}},"commit":{"parents":[{"commit":"a249bb79137cd1137ecf55a4c7877e7808f794ff","subject":"Merge \"Delete system role assignments from system_assignment table\" into stable/stein","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/a249bb79137cd1137ecf55a4c7877e7808f794ff"}]}],"author":{"name":"Lance Bragstad","email":"lbragstad@gmail.com","date":"2020-06-30 16:50:41.000000000","tz":-300},"committer":{"name":"Lance Bragstad","email":"lbragstad@gmail.com","date":"2021-03-30 19:35:23.000000000","tz":0},"subject":"Retry update_user when sqlalchemy raises StaleDataErrors","message":"Retry update_user when sqlalchemy raises StaleDataErrors\n\nKeystone\u0027s update_user() method in the SQL driver processes a lot of\ninformation about how to update users. This includes evaluating password\nlogic and authentication attempts for PSI-DSS. This logic is evaluated\nafter keystone pulls the user record from SQL and before it exits the\ncontext manager, which performs the write.\n\nWhen multiple clients are all updating the same user reference, it\u0027s\nmore likely they will see an HTTP 500 because of race conditions exiting\nthe context manager. The HTTP 500 is due to stale data when updating\npassword expiration for old passwords, which happens when setting a new\npassword for a user.\n\nThis commit attempts to handle that case more gracefully than throwing a\n500 by detecting StaleDataErrors from sqlalchemy and retrying.  The\nidentity sql backend will retry the request for clients that have\nstale data change from underneath them.\n\nConflicts:\n      keystone/tests/unit/test_backend_sql.py due to import order\n      differences between train and ussuri. Also adjust the expected log\n      message since the method path is different compared to older\n      releases, which have the driver name in them (e.g., Identity).\n\nChange-Id: I75590c20e90170ed862f46f0de7d61c7810b5c90\nCloses-Bug: 1885753\n(cherry picked from commit ceae3566e83b26fd6a1679154eae9b0cef29da64)\n(cherry picked from commit f47e635b8041542faa05e64606e66d2fbbc5f284)\n(cherry picked from commit 5b7d4c80d484262018f937083050844648f07a11)\n(cherry picked from commit 07d3a3d3ff534a5295842d4f236042b30536cd82)\n(cherry picked from commit d4f48fc4e53f71d653e133104854f064fbb1b25f)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/658f66be864c13c43b2395c829bce6c6c30578b7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/658f66be864c13c43b2395c829bce6c6c30578b7"}]},"branch":"refs/heads/stable/stein"},"f36034c8a6530b16f6b6eb88ee97f540c974ba00":{"kind":"REWORK","_number":4,"created":"2021-03-31 12:35:34.000000000","uploader":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"ref":"refs/changes/05/783405/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/05/783405/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/05/783405/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/05/783405/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/05/783405/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/05/783405/4"}}},"commit":{"parents":[{"commit":"a249bb79137cd1137ecf55a4c7877e7808f794ff","subject":"Merge \"Delete system role assignments from system_assignment table\" into stable/stein","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/a249bb79137cd1137ecf55a4c7877e7808f794ff"}]}],"author":{"name":"Lance Bragstad","email":"lbragstad@gmail.com","date":"2020-06-30 16:50:41.000000000","tz":-300},"committer":{"name":"Lance Bragstad","email":"lbragstad@gmail.com","date":"2021-03-31 12:33:43.000000000","tz":0},"subject":"Retry update_user when sqlalchemy raises StaleDataErrors","message":"Retry update_user when sqlalchemy raises StaleDataErrors\n\nKeystone\u0027s update_user() method in the SQL driver processes a lot of\ninformation about how to update users. This includes evaluating password\nlogic and authentication attempts for PSI-DSS. This logic is evaluated\nafter keystone pulls the user record from SQL and before it exits the\ncontext manager, which performs the write.\n\nWhen multiple clients are all updating the same user reference, it\u0027s\nmore likely they will see an HTTP 500 because of race conditions exiting\nthe context manager. The HTTP 500 is due to stale data when updating\npassword expiration for old passwords, which happens when setting a new\npassword for a user.\n\nThis commit attempts to handle that case more gracefully than throwing a\n500 by detecting StaleDataErrors from sqlalchemy and retrying.  The\nidentity sql backend will retry the request for clients that have\nstale data change from underneath them.\n\nConflicts:\n      keystone/tests/unit/test_backend_sql.py due to import order\n      differences between train and ussuri. Also adjust the expected log\n      message since the method path is different compared to older\n      releases, which have the driver name in them (e.g., Identity).\n\nChange-Id: I75590c20e90170ed862f46f0de7d61c7810b5c90\nCloses-Bug: 1885753\n(cherry picked from commit ceae3566e83b26fd6a1679154eae9b0cef29da64)\n(cherry picked from commit f47e635b8041542faa05e64606e66d2fbbc5f284)\n(cherry picked from commit 5b7d4c80d484262018f937083050844648f07a11)\n(cherry picked from commit 07d3a3d3ff534a5295842d4f236042b30536cd82)\n(cherry picked from commit d4f48fc4e53f71d653e133104854f064fbb1b25f)\n(cherry picked from commit 328cf33aab61775301adbb4c1f6abaa2f331cd94)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/f36034c8a6530b16f6b6eb88ee97f540c974ba00"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/f36034c8a6530b16f6b6eb88ee97f540c974ba00"}]},"branch":"refs/heads/stable/stein"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}