)]}'
{"id":"openstack%2Fnova~690734","triplet_id":"openstack%2Fnova~stable%2Fpike~I324193129acb6ac739133c7e76920762a8987a84","project":"openstack/nova","branch":"stable/pike","topic":"bug/1849409","hashtags":[],"change_id":"I324193129acb6ac739133c7e76920762a8987a84","subject":"Add functional regression test for bug 1849409","status":"MERGED","created":"2019-10-23 20:32:28.000000000","updated":"2019-12-13 20:48:05.000000000","submitted":"2019-12-13 20:48:05.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":"690734-1576270085104-b807434a","meta_rev_id":"8f7c299ed2005951e882ddbe443e1954d3287e3a","_number":690734,"virtual_id_number":690734,"owner":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"date":"2019-11-26 00:21:37.000000000","_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"date":"2019-11-25 22:05:54.000000000","_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2019-12-13 20:48:03.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":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"all":[{"value":0,"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},{"value":2,"date":"2019-12-13 19:28:55.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},{"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":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"all":[{"value":0,"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},{"value":1,"date":"2019-12-13 19:28:55.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},{"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":{"all":[{"value":0,"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},{"value":0,"_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":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-11-25 22:05:54.000000000","updated_by":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"reviewer":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"state":"REVIEWER"},{"updated":"2019-11-26 00:21:37.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":"REVIEWER"},{"updated":"2019-12-13 19:28:55.000000000","updated_by":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"reviewer":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"state":"REVIEWER"},{"updated":"2019-12-13 20:48:03.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":"ed79ca90aaee1a2979c59d1accac70ddb2714345","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-10-23 20:32:28.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"cdb75bbbdfedc49ad109ceccf56dd56fed4a4eb0","author":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"date":"2019-10-23 20:40:50.000000000","message":"Patch Set 1:\n\nBuild failed.\n\n- pmem-tempest-plugin-filtered http://52.27.155.124/34/690734/1/check/pmem-tempest-plugin-filtered/0835203/ : POST_FAILURE in 6m 59s","accounts_in_message":[],"_revision_number":1},{"id":"1e301808b6e333dc6f05bb7b5475c45d5574ef5f","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-23 22:20:02.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/d9e0cec6524b478596c600aca80460a2 : SUCCESS in 9m 57s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/d405769a9dcd4fa390d7b416323f6197 : SUCCESS in 17m 57s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/9fd915fba13647a1b6385e22d1c09b47 : SUCCESS in 13m 39s\n- build-openstack-sphinx-docs https://zuul.opendev.org/t/openstack/build/c687cef4fc0042d2a442950f120777da : SUCCESS in 9m 31s\n- nova-tox-functional https://zuul.opendev.org/t/openstack/build/1e34397ca54a4e9090f6430ce3a538d3 : SUCCESS in 22m 14s\n- nova-tox-functional-py35 https://zuul.opendev.org/t/openstack/build/37cd84c40bcd40efb22761fa55a2c491 : SUCCESS in 19m 46s","accounts_in_message":[],"_revision_number":1},{"id":"2cec9cfc595fbde8821c07f05de576b21c537959","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2019-10-24 00:10:45.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/690734/1 : FAILURE in 1h 15m 08s","accounts_in_message":[],"_revision_number":1},{"id":"b3becba08a57d32a23f17d215c560a9182ae9b32","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-10-28 15:37:36.000000000","message":"Patch Set 1: Workflow-1\n\nThis will have to be redone since the Rocky backport changed.","accounts_in_message":[],"_revision_number":1},{"id":"ca9d9b5904bf36e74642964377b5488acd916f51","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-11-25 21:52:46.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"a7e42aaa930a188e44b03a7c9c2a5aa424119703","author":{"_account_id":29963,"name":"Intel_Zuul","display_name":"Intel Corporation CI","email":"intel-openstack-ci@intel.com","username":"Intel_Zuul"},"date":"2019-11-25 22:05:54.000000000","message":"Patch Set 2:\n\nBuild failed.\n\n- pmem-tempest-plugin-filtered http://52.27.155.124/34/690734/2/check/pmem-tempest-plugin-filtered/1fd7c7b/ : POST_FAILURE in 6m 06s","accounts_in_message":[],"_revision_number":2},{"id":"0671c49bfe1db5a7a69e01d656d4d2fb54caa9b0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-11-25 22:39:25.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/0ce4b561609240c2833f97d758a5aae7 : SUCCESS in 9m 53s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/597c8a385af945c184d041c8b692ccf3 : SUCCESS in 14m 43s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/4e351009918444519f8c4b5f993482dc : SUCCESS in 14m 43s\n- build-openstack-sphinx-docs https://zuul.opendev.org/t/openstack/build/bbb3234a113d4026918252fa966ea532 : SUCCESS in 6m 06s\n- nova-tox-functional https://zuul.opendev.org/t/openstack/build/94d4f277be084bf69278a1e72c86df81 : SUCCESS in 28m 28s\n- nova-tox-functional-py35 https://zuul.opendev.org/t/openstack/build/98a43276d87b483a8ab73816b334e52d : SUCCESS in 20m 16s","accounts_in_message":[],"_revision_number":2},{"id":"926e5c644a3dfd9b263378fe38e094fd54737563","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2019-11-26 00:21:37.000000000","message":"Patch Set 2:\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/690734/2 : FAILURE in 23m 25s","accounts_in_message":[],"_revision_number":2},{"id":"d34ad6067d0697e591a9a44d12c167bacd8234cb","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2019-12-13 19:28:55.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"21ef47bb7ce2846be2a8e437724a5ad9acd8ec3b","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-12-13 19:37:29.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"5965c2396a160cd88f1d5c3a385400c54945a821","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-12-13 20:48:03.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/864e9639cbf0436aa8b718aedda7087f : SUCCESS in 9m 17s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/02eebd1304334eb8b070269ebb46d6c9 : SUCCESS in 19m 34s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/ccdd75208bfd4a0c878f7479264cc747 : SUCCESS in 14m 26s\n- build-openstack-sphinx-docs https://zuul.opendev.org/t/openstack/build/df57a50ca1fb48cebc12afb5a6cac0d6 : SUCCESS in 6m 35s\n- nova-tox-functional https://zuul.opendev.org/t/openstack/build/a88a550c7f6e4571b42df0508b5ac782 : SUCCESS in 25m 51s\n- nova-tox-functional-py35 https://zuul.opendev.org/t/openstack/build/5ce6c0c8811543eb9f6eaedd66c3e49e : SUCCESS in 17m 24s","accounts_in_message":[],"_revision_number":2},{"id":"cc46f062dceda9d2eae2d489347b5711af7188c4","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-12-13 20:48:05.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"e2ba87cb7b1cfd4226cfd8028436ecf87ad8a2be","revisions":{"d24cef6acb5d949e97e4c000adc59a146563d82e":{"kind":"REWORK","_number":1,"created":"2019-10-23 20:32:28.000000000","uploader":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"ref":"refs/changes/34/690734/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/34/690734/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/34/690734/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/34/690734/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/34/690734/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/34/690734/1"}}},"commit":{"parents":[{"commit":"6e2ec5c5a397b508cddb646e865b6f4d2132f1d6","subject":"Remove experimental job on openSUSE 42.3","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/6e2ec5c5a397b508cddb646e865b6f4d2132f1d6"}]}],"author":{"name":"Matt Riedemann","email":"mriedem.os@gmail.com","date":"2019-10-22 21:53:20.000000000","tz":-240},"committer":{"name":"Matt Riedemann","email":"mriedem.os@gmail.com","date":"2019-10-23 20:23:03.000000000","tz":-240},"subject":"Add functional regression test for bug 1849409","message":"Add functional regression test for bug 1849409\n\nChange I1aa3ca6cc70cef65d24dec1e7db9491c9b73f7ab in Queens,\nwhich was backported through to Newton, introduced a regression\nwhen listing deleted servers with a marker because it assumes\nthat if BuildRequestList.get_by_filters does not raise\nMarkerNotFound that the marker was found among the build requests\nand does not account for that get_by_filters method short-circuiting\nif filtering servers with deleted/cleaned/limit\u003d0. The API code\nthen nulls out the marker which means you\u0027ll continue to get the\nmarker instance back in the results even though you shouldn\u0027t,\nand that can cause an infinite loop in some client-side tooling like\nnova\u0027s CLI:\n\n  nova list --deleted --limit -1\n\nThis adds a functional recreate test for the regression which will\nbe updated when the bug is fixed.\n\nNOTE(mriedem): In this backport the test is modified to create the\nserver without networks using microversion 2.37 since the\nNeutronFixture in Pike is lacking niceties to create a server with\na specific network and generate a fake port.\n\nChange-Id: I324193129acb6ac739133c7e76920762a8987a84\nRelated-Bug: #1849409\n(cherry picked from commit 45c2752f2ce08b314012eff044b01aa7d626b43d)\n(cherry picked from commit 727d942b2830fb6970d99507f2b5eb1a28df01b2)\n(cherry picked from commit 47caaccd4a03660d7df144f2eadd821d36baeaa8)\n(cherry picked from commit 56c5753747564c1a93958f3094785d2e8e4b6598)\n(cherry picked from commit 7f3ba78c27cc167c74b3ffd65e2ad2df9d736c36)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/d24cef6acb5d949e97e4c000adc59a146563d82e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/d24cef6acb5d949e97e4c000adc59a146563d82e"}]},"branch":"refs/heads/stable/pike"},"e2ba87cb7b1cfd4226cfd8028436ecf87ad8a2be":{"kind":"REWORK","_number":2,"created":"2019-11-25 21:52:46.000000000","uploader":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"ref":"refs/changes/34/690734/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/34/690734/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/34/690734/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/34/690734/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/34/690734/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/34/690734/2"}}},"commit":{"parents":[{"commit":"b61f7a1d3a5a28e308efdb370477f79b2a2ceec0","subject":"Merge \"Delete instance_id_mappings record in instance_destroy\" into stable/pike","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b61f7a1d3a5a28e308efdb370477f79b2a2ceec0"}]}],"author":{"name":"Matt Riedemann","email":"mriedem.os@gmail.com","date":"2019-10-22 21:53:20.000000000","tz":-240},"committer":{"name":"Matt Riedemann","email":"mriedem.os@gmail.com","date":"2019-11-25 21:49:00.000000000","tz":-300},"subject":"Add functional regression test for bug 1849409","message":"Add functional regression test for bug 1849409\n\nChange I1aa3ca6cc70cef65d24dec1e7db9491c9b73f7ab in Queens,\nwhich was backported through to Newton, introduced a regression\nwhen listing deleted servers with a marker because it assumes\nthat if BuildRequestList.get_by_filters does not raise\nMarkerNotFound that the marker was found among the build requests\nand does not account for that get_by_filters method short-circuiting\nif filtering servers with deleted/cleaned/limit\u003d0. The API code\nthen nulls out the marker which means you\u0027ll continue to get the\nmarker instance back in the results even though you shouldn\u0027t,\nand that can cause an infinite loop in some client-side tooling like\nnova\u0027s CLI:\n\n  nova list --deleted --limit -1\n\nThis adds a functional recreate test for the regression which will\nbe updated when the bug is fixed.\n\nChange-Id: I324193129acb6ac739133c7e76920762a8987a84\nRelated-Bug: #1849409\n(cherry picked from commit 45c2752f2ce08b314012eff044b01aa7d626b43d)\n(cherry picked from commit 727d942b2830fb6970d99507f2b5eb1a28df01b2)\n(cherry picked from commit 47caaccd4a03660d7df144f2eadd821d36baeaa8)\n(cherry picked from commit 08337cccb060d0b3cad388004c1f802d5d053813)\n(cherry picked from commit f03f5075e3751ccd03a60999eab1d7f4bf7c4f02)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e2ba87cb7b1cfd4226cfd8028436ecf87ad8a2be"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e2ba87cb7b1cfd4226cfd8028436ecf87ad8a2be"}]},"branch":"refs/heads/stable/pike"}},"requirements":[],"submit_records":[],"submit_requirements":[]}