)]}'
{"id":"openstack%2Fnova~797975","triplet_id":"openstack%2Fnova~stable%2Fstein~I333b3d85deed971678141307dd06545e308cf989","project":"openstack/nova","branch":"stable/stein","topic":"bug/1910466","hashtags":[],"change_id":"I333b3d85deed971678141307dd06545e308cf989","subject":"Test numa and vcpu topologies bug: #1910466","status":"ABANDONED","created":"2021-06-24 16:19:49.000000000","updated":"2022-10-05 15:44:22.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"2a66f88a60f08b31bf503b45b9096e54ea001e46","_number":797975,"virtual_id_number":797975,"owner":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"tag":"autogenerated:zuul:check","value":-1,"date":"2021-06-24 16:43:51.000000000","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":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_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":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-06-24 16:19:49.000000000","updated_by":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"},{"updated":"2021-06-24 16:43: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"}],"messages":[{"id":"aeceaad08800ffa92d77f56df5a1c2f9f6f4f053","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-06-24 16:19:49.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"41e5e4b66283b439aeb3e0a8e56aa2fb269be422","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2021-06-24 16:23:52.000000000","message":"Uploaded patch set 2: Patch Set 1 was rebased.","accounts_in_message":[],"_revision_number":2},{"id":"5fb951a9ceb981eac34298763b768a1f2c2cb42b","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-24 16:43:51.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/eafac2fa07cd45b38584eadd5b2a420b : SUCCESS in 17m 31s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/3532c3cc51624aaab523fed52a064c10 : SUCCESS in 15m 31s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/480d93f16e474629a4b6cf83ae999e50 : FAILURE in 12m 14s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/25ab8c13cffd486b8afac205b90739e6 : SUCCESS in 14m 40s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/be5b7705d2134e16877f8a6250205cfa : SUCCESS in 15m 09s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/c6cf53dfeaae486caa3bd3ab6d7b8c54 : SUCCESS in 13m 36s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/a55f9dc6279b474c813a21ce4a34bec6 : SUCCESS in 14m 20s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/142d07bd7b084c81b0fc822423aa5f54 : SUCCESS in 10m 11s\n- nova-tox-functional https://zuul.opendev.org/t/openstack/build/1227d20505c841c0bed8c5403352d89a : SUCCESS in 18m 33s\n- nova-tox-functional-py35 https://zuul.opendev.org/t/openstack/build/6918e981e1f64cd8a7b196a958bbb905 : SUCCESS in 18m 32s","accounts_in_message":[],"_revision_number":2},{"id":"2a66f88a60f08b31bf503b45b9096e54ea001e46","tag":"autogenerated:gerrit:abandon","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2022-10-05 15:44:22.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"34d48d3d3732061f6be6e4842a09847d57d7f37a","revisions":{"c24897b2388afbbf0a6602f000d13248539a01a8":{"kind":"REWORK","_number":1,"created":"2021-06-24 16:19:49.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/75/797975/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/75/797975/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/75/797975/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/75/797975/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/75/797975/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/75/797975/1"}}},"commit":{"parents":[{"commit":"e96ee3034fc3a3544e4bc7bb61b9d3c09f3762e7","subject":"libvirt: Start checking compute usage in functional tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e96ee3034fc3a3544e4bc7bb61b9d3c09f3762e7"}]}],"author":{"name":"Sean Mooney","email":"work@seanmooney.info","date":"2021-01-06 18:56:39.000000000","tz":0},"committer":{"name":"Stephen Finucane","email":"stephenfin@redhat.com","date":"2021-06-24 14:27:57.000000000","tz":60},"subject":"Test numa and vcpu topologies bug: #1910466","message":"Test numa and vcpu topologies bug: #1910466\n\nThis change reproduces bug #1910466\nWhen hw:cpu_max_[sockets|cores|threads] is configured\nin addition to an explict numa topologies and cpu pinning\nnova is currently incapable of generating the correct\nvirtual CPU topology resulting in an index out of range\nerror as we attempt to retrieve the first topology\nfrom an empty list.\n\nThis change reproduces the error via a new functional\ntest.\n\nConflicts:\n  nova/tests/functional/libvirt/test_numa_servers.py\n\nChanges:\n  nova/tests/functional/libvirt/test_numa_servers.py\n\nNOTE(stephenfin): Conflicts are due to the absence of changes related to\nthe \u0027cpu-resources\u0027 spec, which moved pinned CPU tracking to placement,\nsuch as change I0996c7c5b76deec759ebff325038ded216306908 (\"tests:\nAdditional functional tests for pinned instances\"), as well as a couple\nof related bug fixes and fixups. The change is because we\u0027re missing\nchange Ifcda7336d56c9b623720ee018ec5697740986273 (\"fakelibvirt: Stop\ndistinguishing between NUMA, non-NUMA\") and need to replace a reference\nto \u0027HostInfo\u0027 with the older \u0027NUMAHostInfo\u0027\n\nRelated-Bug: #1910466\nChange-Id: I333b3d85deed971678141307dd06545e308cf989\n(cherry picked from commit fe25fa13e0572aafb62b28bf7fa843a1860e5010)\n(cherry picked from commit 5f994a18b3105047a2231231815d09f96c920958)\n(cherry picked from commit e9526265455e3189b400bc7877b89ca3f59777cf)\n(cherry picked from commit 334a310e2096e699d3c327e95b1c0a83cf77691b)\n(cherry picked from commit 2e785340207c8c3dd318525626facd6812295449)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c24897b2388afbbf0a6602f000d13248539a01a8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c24897b2388afbbf0a6602f000d13248539a01a8"}]},"branch":"refs/heads/stable/stein"},"34d48d3d3732061f6be6e4842a09847d57d7f37a":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2021-06-24 16:23:52.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/75/797975/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/75/797975/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/75/797975/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/75/797975/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/75/797975/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/75/797975/2"}}},"commit":{"parents":[{"commit":"77b433b61e239e0b79736c7e9994a9b445b90d00","subject":"Merge \"Validate id as integer for os-aggregates\" into stable/stein","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/77b433b61e239e0b79736c7e9994a9b445b90d00"}]}],"author":{"name":"Sean Mooney","email":"work@seanmooney.info","date":"2021-01-06 18:56:39.000000000","tz":0},"committer":{"name":"Stephen Finucane","email":"stephenfin@redhat.com","date":"2021-06-24 16:23:39.000000000","tz":60},"subject":"Test numa and vcpu topologies bug: #1910466","message":"Test numa and vcpu topologies bug: #1910466\n\nThis change reproduces bug #1910466\nWhen hw:cpu_max_[sockets|cores|threads] is configured\nin addition to an explict numa topologies and cpu pinning\nnova is currently incapable of generating the correct\nvirtual CPU topology resulting in an index out of range\nerror as we attempt to retrieve the first topology\nfrom an empty list.\n\nThis change reproduces the error via a new functional\ntest.\n\nConflicts:\n  nova/tests/functional/libvirt/test_numa_servers.py\n\nChanges:\n  nova/tests/functional/libvirt/test_numa_servers.py\n\nNOTE(stephenfin): Conflicts are due to the absence of changes related to\nthe \u0027cpu-resources\u0027 spec, which moved pinned CPU tracking to placement,\nsuch as change I0996c7c5b76deec759ebff325038ded216306908 (\"tests:\nAdditional functional tests for pinned instances\"), as well as a couple\nof related bug fixes and fixups. The change is because we\u0027re missing\nchange Ifcda7336d56c9b623720ee018ec5697740986273 (\"fakelibvirt: Stop\ndistinguishing between NUMA, non-NUMA\") and need to replace a reference\nto \u0027HostInfo\u0027 with the older \u0027NUMAHostInfo\u0027\n\nRelated-Bug: #1910466\nChange-Id: I333b3d85deed971678141307dd06545e308cf989\n(cherry picked from commit fe25fa13e0572aafb62b28bf7fa843a1860e5010)\n(cherry picked from commit 5f994a18b3105047a2231231815d09f96c920958)\n(cherry picked from commit e9526265455e3189b400bc7877b89ca3f59777cf)\n(cherry picked from commit 334a310e2096e699d3c327e95b1c0a83cf77691b)\n(cherry picked from commit 2e785340207c8c3dd318525626facd6812295449)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/34d48d3d3732061f6be6e4842a09847d57d7f37a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/34d48d3d3732061f6be6e4842a09847d57d7f37a"}]},"branch":"refs/heads/stable/stein"}},"requirements":[],"submit_records":[],"submit_requirements":[]}