)]}'
{"id":"openstack%2Fglance~749513","triplet_id":"openstack%2Fglance~stable%2Fussuri~Iff3dfbfcbfb956a06d77a144e5456bdb556c5a2c","project":"openstack/glance","branch":"stable/ussuri","topic":"bug/1884596","hashtags":[],"change_id":"Iff3dfbfcbfb956a06d77a144e5456bdb556c5a2c","subject":"Cleanup import status information after busting a lock","status":"MERGED","created":"2020-09-02 14:15:19.000000000","updated":"2020-09-18 21:40:23.000000000","submitted":"2020-09-18 21:40:23.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":"749513-1600465223651-b2ad87a5","meta_rev_id":"f63b8b7a11c1bfc2afcd022d4e749cd5bb83eb64","_number":749513,"virtual_id_number":749513,"owner":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-09-18 21:40:23.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":2,"date":"2020-09-18 18:57:25.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":2,"date":"2020-09-03 15:02:03.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":{"approved":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":1,"date":"2020-09-18 18:57:25.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":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"recommended":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":1,"date":"2020-09-03 15:02:03.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Top Priority / Holds Gate"},"description":"","value":1,"default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-09-02 15:03:02.000000000","updated_by":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"reviewer":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"state":"REVIEWER"},{"updated":"2020-09-18 18:57:25.000000000","updated_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"},{"updated":"2020-09-18 21:40:23.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":"f2e7a07e02cfc829a049c3431d94d1a9497e9eec","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2020-09-02 14:15:19.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"0234fc2ec4ac51f12b72925d3f56fa2b3fbc82ab","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2020-09-02 15:03:02.000000000","message":"Patch Set 1: Code-Review+2 Review-Priority+1\n\nImportant backport!","accounts_in_message":[],"_revision_number":1},{"id":"d198141f10f56075eae5b5c00501e38f8611f033","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-02 17:26:07.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- grenade https://zuul.opendev.org/t/openstack/build/6a2f579258a14ffa99aa9392d8592590 : SUCCESS in 1h 15m 09s\n- tempest-integrated-storage https://zuul.opendev.org/t/openstack/build/453b9948809c45228b4b481142372fb8 : SUCCESS in 1h 08m 19s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/340aeca782474166995d73d5f6ec1952 : SUCCESS in 5m 43s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/0023447883df48dfadb19165bc5fe0a4 : FAILURE in 5m 40s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/c667dc3e618e47deb96a7205e43f8dab : SUCCESS in 5m 42s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/ddfa777d22514d9b9324fc6092d36420 : SUCCESS in 6m 20s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/563283c467cc45ca9f1ec26e576371e9 : SUCCESS in 7m 21s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/0de0712b0f4d4e979e9a79b7712f434b : SUCCESS in 11m 06s\n- openstack-tox-functional-py37 https://zuul.opendev.org/t/openstack/build/f83f6618be444d53b192e961c050f02b : SUCCESS in 10m 16s\n- glance-code-constants-check https://zuul.opendev.org/t/openstack/build/199351d057654074ab4c5425fb2ed470 : SUCCESS in 7m 43s\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/9418b7ced2434cf5b86683e51fc737b1 : SUCCESS in 1h 06m 59s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/2160c44d66d34ba3856cbfa9660c9cb4 : SUCCESS in 56m 54s\n\nWarning:\n  Comments left for invalid file glance/tests/utils.py","accounts_in_message":[],"_revision_number":1},{"id":"b14bce015da81ddae2a30f48fa179418fc2bfdf8","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2020-09-03 15:02:03.000000000","message":"Uploaded patch set 2: Patch Set 1 was rebased.","accounts_in_message":[],"_revision_number":2},{"id":"2053b0356a39cb5d65e1df49900bc577acdde9fe","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-03 18:16:56.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/adc6836a3df242c896c5ce132cd390bf : SUCCESS in 1h 09m 56s\n- tempest-integrated-storage https://zuul.opendev.org/t/openstack/build/db34aa1705f34df994740931c5906968 : SUCCESS in 1h 03m 47s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/31801e8c3aa24b1d947c0738f292901d : SUCCESS in 4m 55s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/4965e8e36d0a40838fec5a39a32bd36f : SUCCESS in 5m 36s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/1f47a616bd034157b3c538dab17f784a : SUCCESS in 4m 52s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/1eb21708a76c418f89902c537ce09e30 : SUCCESS in 7m 48s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/a3621112ed094a28a55cdb1536dda0b1 : SUCCESS in 7m 09s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/95668c5f447140c486ea8c62843b3f59 : SUCCESS in 8m 13s\n- openstack-tox-functional-py37 https://zuul.opendev.org/t/openstack/build/1bdd0c6d90c7400c94997e2cf08f44ec : SUCCESS in 11m 50s\n- glance-code-constants-check https://zuul.opendev.org/t/openstack/build/5548888ed16e4ac39ec4f962c80450e6 : SUCCESS in 4m 17s\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/30c030d9b5a440598ba36100dcfdb12b : SUCCESS in 1h 00m 49s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/9e3e641eb9d5442ca1a5909c9230d2db : SUCCESS in 56m 47s","accounts_in_message":[],"_revision_number":2},{"id":"52a3e8895a9dc8e6e6d440c44fcb6ef8c622ddd2","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2020-09-18 18:57:25.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"5507e218013de509e7a6bcb058808ef06da6507f","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-18 18:57:42.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"a85796d63c760916ae6218dc9d5f7e09ff7eca61","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-18 21:40:23.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2},{"id":"421ebc06e8dacba29e87d088c561ce2c92eda871","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-18 21:40:23.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/bb7c42b3239b49bc89df7dcc054c64ca : SUCCESS in 1h 05m 41s\n- tempest-integrated-storage https://zuul.opendev.org/t/openstack/build/81009cd06edb43ec8fa3ffa9afc26ebe : SUCCESS in 1h 17m 12s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/24c16ced348842bc8b56317e3be51765 : SUCCESS in 5m 31s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/1697fea5973a4511bb028ed5f3dcc874 : SUCCESS in 5m 35s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/9e3a1429c8ab483b9b579ebd8475c9b8 : SUCCESS in 6m 05s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/521b98be88d94bfa8cc7b973de0168cf : SUCCESS in 6m 02s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/bcac802a58e348c086224bcd49a5bcf3 : SUCCESS in 7m 06s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/de7d9df8b75a4fc982580598722d6ce1 : SUCCESS in 8m 05s\n- openstack-tox-functional-py37 https://zuul.opendev.org/t/openstack/build/340011824b654265bef7ba31169d5c41 : SUCCESS in 8m 11s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/027db559e477420b971038d098d6409f : SUCCESS in 1h 01m 14s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"5b681da16520bac664f37107b088ef38519b22bc","revisions":{"7ac1f68c20aee3d056c29438c1edacaacdc95b3a":{"kind":"REWORK","_number":1,"created":"2020-09-02 14:15:19.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/13/749513/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/13/749513/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/13/749513/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/13/749513/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/13/749513/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/13/749513/1"}}},"commit":{"parents":[{"commit":"0bd0eca1c440e47d71a46c26133630decd477f02","subject":"Add ImageLock to base flow checks","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/0bd0eca1c440e47d71a46c26133630decd477f02"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2020-08-17 15:29:43.000000000","tz":-420},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2020-09-02 14:14:56.000000000","tz":-420},"subject":"Cleanup import status information after busting a lock","message":"Cleanup import status information after busting a lock\n\nWhen we bust a lock, we now own the image for that time period\nand may exclude the other task (if still running) from updating\nthe import status information. If not still running, we should\ntake responsibility of that cleanup since we know what task we\nstole the lock from. We should, however, only do that if we\nsucceed in grabbing the lock to avoid racing with another thread\nwhich might be trying to do the same thing.\n\nChange-Id: Iff3dfbfcbfb956a06d77a144e5456bdb556c5a2c\n(cherry picked from commit 552da8440070da50e67bd477ca13b5429e22dc30)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/7ac1f68c20aee3d056c29438c1edacaacdc95b3a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/7ac1f68c20aee3d056c29438c1edacaacdc95b3a"}]},"branch":"refs/heads/stable/ussuri"},"5b681da16520bac664f37107b088ef38519b22bc":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2020-09-03 15:02:03.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/13/749513/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/13/749513/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/13/749513/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/13/749513/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/13/749513/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/13/749513/2"}}},"commit":{"parents":[{"commit":"03a11dbbfb4c1f60946712413db449f8b52a4503","subject":"Add ImageLock to base flow checks","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/03a11dbbfb4c1f60946712413db449f8b52a4503"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2020-08-17 15:29:43.000000000","tz":-420},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2020-09-03 15:00:35.000000000","tz":-420},"subject":"Cleanup import status information after busting a lock","message":"Cleanup import status information after busting a lock\n\nWhen we bust a lock, we now own the image for that time period\nand may exclude the other task (if still running) from updating\nthe import status information. If not still running, we should\ntake responsibility of that cleanup since we know what task we\nstole the lock from. We should, however, only do that if we\nsucceed in grabbing the lock to avoid racing with another thread\nwhich might be trying to do the same thing.\n\nChange-Id: Iff3dfbfcbfb956a06d77a144e5456bdb556c5a2c\n(cherry picked from commit 552da8440070da50e67bd477ca13b5429e22dc30)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/5b681da16520bac664f37107b088ef38519b22bc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/5b681da16520bac664f37107b088ef38519b22bc"}]},"branch":"refs/heads/stable/ussuri"}},"requirements":[],"submit_records":[],"submit_requirements":[]}