)]}'
{"id":"openstack%2Fnova~770585","triplet_id":"openstack%2Fnova~stable%2Ftrain~Ia5e521e0f0c7a78b5ace5de9f343e84d872553f9","project":"openstack/nova","branch":"stable/train","topic":"bug/1864122","hashtags":[],"change_id":"Ia5e521e0f0c7a78b5ace5de9f343e84d872553f9","subject":"Use fair locks in resource tracker","status":"ABANDONED","created":"2021-01-13 18:01:39.000000000","updated":"2021-02-10 09:39:13.000000000","total_comment_count":1,"unresolved_comment_count":1,"has_review_started":true,"cherry_pick_of_change":711528,"cherry_pick_of_patch_set":2,"meta_rev_id":"ad9528b5b8478908adc5c84c25a53f5bb8d18ad8","_number":770585,"virtual_id_number":770585,"owner":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2021-01-14 18:15:51.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"}],"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":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"CC":[{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}],"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-01-13 18:01:39.000000000","updated_by":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"reviewer":{"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"},"state":"REVIEWER"},{"updated":"2021-01-13 18:06:10.000000000","updated_by":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-01-13 20:48:12.000000000","updated_by":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-01-13 20:57:59.000000000","updated_by":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"reviewer":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"state":"CC"},{"updated":"2021-01-14 18:15:51.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-02-10 09:39:13.000000000","updated_by":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"reviewer":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"state":"CC"}],"messages":[{"id":"144dd9241dfab98a9494e99b97f1a07126c79be3","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-01-13 18:01:39.000000000","message":"Patch Set 1: Cherry Picked from branch master.","accounts_in_message":[],"_revision_number":1},{"id":"78b82261b42cb9f26fbfb8c0759bc9670aa3d550","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-01-13 18:02:55.000000000","message":"Topic bug/1864122-stable/train removed","accounts_in_message":[],"_revision_number":1},{"id":"e483cba93f5803d19b03f2b790f8ce28a44996a2","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-01-13 18:03:13.000000000","message":"Topic set to bug/1864122","accounts_in_message":[],"_revision_number":1},{"id":"eeea1c382dba2132ef5300abab6df5cfa9998c70","tag":"autogenerated:jenkins-gerrit-trigger","author":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"date":"2021-01-13 18:06:10.000000000","message":"Patch Set 1:\n\nBuild Failed \n\n* SRIOV-nova-macvtap http://13.74.249.42/refs/changes/85/770585/1/SRIOV-nova-macvtap : FAILURE\n\n* SRIOV-nova-direct http://13.74.249.42/refs/changes/85/770585/1/SRIOV-nova-direct : FAILURE","accounts_in_message":[],"_revision_number":1},{"id":"34aa9e5372f91cdc974463fd56a4985ecb449f45","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-01-13 18:26:16.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"228bac3506120f2bfed98f426c025dde8f664b12","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-01-13 20:48:12.000000000","message":"Patch Set 1:\n\nBuild succeeded. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm- recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-bionic-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/85/770585/1/check/tempest-dsvm-full-bionic-py3/20b3757/ : SUCCESS in 1h 52m 41s","accounts_in_message":[],"_revision_number":1},{"id":"ecbd4699c8305aa59dd48ba5af662b517f147cb0","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-01-13 20:57:59.000000000","message":"Patch Set 1:\n\nFor rechecking only on the Cloudbase Nova Hyper-V CI, add a review comment with run-Cloudbase Nova Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/770585/1 : FAILURE in 2h 54m 43s","accounts_in_message":[],"_revision_number":1},{"id":"9b4b36f928fea93fbf6d9df1cf1f3c682e5f2b90","tag":"autogenerated:jenkins-gerrit-trigger","author":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"date":"2021-01-13 21:12:24.000000000","message":"Patch Set 1:\n\nBuild Failed \n\n* SRIOV-nova-macvtap http://13.74.249.42/refs/changes/85/770585/1/SRIOV-nova-macvtap : FAILURE\n\n* SRIOV-nova-direct http://13.74.249.42/refs/changes/85/770585/1/SRIOV-nova-direct : FAILURE","accounts_in_message":[],"_revision_number":1},{"id":"3c293357d57508da7288c9a508179740adfcf069","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-01-14 18:15:51.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- requirements-check https://zuul.opendev.org/t/openstack/build/7da55a5b018e4c068cbd7cd70f526e89 : SUCCESS in 3m 35s\n- grenade https://zuul.opendev.org/t/openstack/build/e0f15b3c5764402c8051b93617676d4f : SUCCESS in 44m 43s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/627cb17f5718404fac9f35a95a555d06 : SUCCESS in 1h 33m 51s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/d71580c072bc438b9c88b68038642ca1 : SUCCESS in 16m 13s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/4c0913da064e455b8c104dd59d539523 : SUCCESS in 13m 28s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/733ebfe80c1a472a9bf7cc0ff776533d : SUCCESS in 9m 47s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/e89e3333de69478ea8ba333028ae12ce : SUCCESS in 14m 19s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/cf0035d0d3174455b78f60ba3a967362 : SUCCESS in 15m 39s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/a3e382cac2994722a7d3952a19fe8252 : SUCCESS in 15m 11s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e5c0adf26bbd4586ad7a552aa8e73276 : SUCCESS in 9m 52s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/0370c20c21604ccf89af5320fe3f37dc : SUCCESS in 1h 22m 07s (non-voting)\n- devstack-plugin-ceph-tempest https://zuul.opendev.org/t/openstack/build/e801e425ca4348e49059e107c6ab568f : SUCCESS in 1h 10m 37s (non-voting)\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/7ad4eb12aa6b43bcb69c650173dc6723 : SUCCESS in 1h 18m 28s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/7d83b44619a34318867582e3daa9e0ea : SUCCESS in 1h 01m 10s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/1f0d91361e4c4dc9a57d7e6b651cd69a : SUCCESS in 48m 20s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/67e649acddda4160be92c954c13da927 : SUCCESS in 1h 29m 48s\n- nova-next https://zuul.opendev.org/t/openstack/build/b124c358821245fdbbe775e1883ef6d7 : SUCCESS in 1h 33m 53s\n- nova-tox-functional https://zuul.opendev.org/t/openstack/build/6df19bd22d2c4570a74a487ffc2acbf3 : SUCCESS in 20m 18s\n- nova-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/b285a56d96be40baba84b78b22660e6a : SUCCESS in 18m 33s\n- tempest-slow-py3 https://zuul.opendev.org/t/openstack/build/df3e31508e574936a78dfb29a291da9c : SUCCESS in 2h 02m 54s\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/d135d8745e3046e98d74ab68d142588e : SUCCESS in 52m 58s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/2625d3ea47684486ba69e5134c6d02e4 : SUCCESS in 56m 29s","accounts_in_message":[],"_revision_number":1},{"id":"26b411b3616c32d667d2b16994dd44a9ce069738","tag":"autogenerated:gerrit:abandon","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-01-14 23:59:43.000000000","message":"Abandoned\n\nI had a brain lapse thinking this could be backported. I cannot be, because it requires a bump of a lower constraint. I\u0027m abandoning the proposed backport patches accordingly.","accounts_in_message":[],"_revision_number":1},{"id":"ad9528b5b8478908adc5c84c25a53f5bb8d18ad8","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-02-10 09:39:13.000000000","message":"Patch Set 1:\n\nLooking at this from the context of [1]. If this is genuinely beneficial, what about the possibility of making this version specific? A quick test suggests that possible\n\n  \u003e\u003e\u003e import pbr.version\n  \u003e\u003e\u003e import oslo_concurrency.version\n  \u003e\u003e\u003e oslo_concurrency.version.version_info.semantic_version() \u003e pbr.version.SemanticVersion(4, 3, 0)\n  True\n  \u003e\u003e\u003e oslo_concurrency.version.version_info.semantic_version() \u003e pbr.version.SemanticVersion(4, 3, 1)\n  False\n  \u003e\u003e\u003e lock_kwargs \u003d {}\n  \u003e\u003e\u003e if oslo_concurrency.version.version_info.semantic_version() \u003e pbr.version.SemanticVersion(3, 29):\n  ...     lock_kwargs[\u0027fair\u0027] \u003d True\n  ...\n\n[1] https://review.opendev.org/c/openstack/nova/+/751368","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"8b9d8d3a9ee739c167140b176134dd31b2cf384c","revisions":{"8b9d8d3a9ee739c167140b176134dd31b2cf384c":{"kind":"REWORK","_number":1,"created":"2021-01-13 18:01:39.000000000","uploader":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"ref":"refs/changes/85/770585/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/85/770585/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/85/770585/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/85/770585/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/85/770585/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/85/770585/1"}}},"commit":{"parents":[{"commit":"b2037fc4e356b55949339a1358c16431a9ab8930","subject":"[stable-only] Cap bandit to 1.6.2 and raise hacking, flake8 and stestr","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b2037fc4e356b55949339a1358c16431a9ab8930"}]}],"author":{"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","date":"2020-02-27 16:37:34.000000000","tz":-360},"committer":{"name":"melanie witt","email":"melwittt@gmail.com","date":"2021-01-13 18:01:39.000000000","tz":0},"subject":"Use fair locks in resource tracker","message":"Use fair locks in resource tracker\n\nWhen the resource tracker has to lock a compute host for updates or\ninspection, it uses a single semaphore. In most cases, this is fine, as\na compute process only is tracking one hypervisor. However, in Ironic, it\u0027s\npossible for one compute process to track many hypervisors. In this\ncase, wait queues for instance claims can get \"stuck\" briefly behind\nlonger processing loops such as the update_resources periodic job. The\nreason this is possible is because the oslo.lockutils synchronized\nlibrary does not use fair locks by default. When a lock is released, one\nof the threads waiting for the lock is randomly allowed to take the lock\nnext. A fair lock ensures that the thread that next requested the lock\nwill be allowed to take it.\n\nThis should ensure that instance claim requests do not have a chance of\nlosing the lock contest, which should ensure that instance build\nrequests do not queue unnecessarily behind long-running tasks.\n\nThis includes bumping the oslo.concurrency dependency; fair locks were\nadded in 3.29.0 (I37577becff4978bf643c65fa9bc2d78d342ea35a).\n\nChange-Id: Ia5e521e0f0c7a78b5ace5de9f343e84d872553f9\nRelated-Bug: #1864122\n(cherry picked from commit 1ed9f9dac59c36cdda54a9852a1f93939b3ebbc3)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/8b9d8d3a9ee739c167140b176134dd31b2cf384c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/8b9d8d3a9ee739c167140b176134dd31b2cf384c"}]},"branch":"refs/heads/stable/train"}},"requirements":[],"submit_records":[],"submit_requirements":[]}