)]}'
{"id":"openstack%2Fnova~794837","triplet_id":"openstack%2Fnova~master~Iab55d6a1fcde763ca859d0d1ecfb11a11cf35738","project":"openstack/nova","branch":"master","topic":"configure-direct-snapshot","hashtags":[],"change_id":"Iab55d6a1fcde763ca859d0d1ecfb11a11cf35738","subject":"Allow configuration of direct-snapshot feature","status":"NEW","created":"2021-06-04 14:21:15.000000000","updated":"2021-07-08 09:47:29.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":true,"submittable":false,"total_comment_count":7,"unresolved_comment_count":1,"has_review_started":true,"meta_rev_id":"ae9b1f791c77405bd8a8f02fcd14ccaae894812e","_number":794837,"virtual_id_number":794837,"owner":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2021-06-22 18:07:20.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}],"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":{"disliked":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":-1,"date":"2021-07-08 09:47:29.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}],"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":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}],"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":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"CC":[{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"}],"REVIEWER":[{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-06-04 15:00:58.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":"CC"},{"updated":"2021-06-04 17:17:48.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-06-04 17:18:48.000000000","updated_by":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-06-04 18:38:38.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-06-04 20:57:15.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-06-05 12:15:44.000000000","updated_by":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"state":"CC"},{"updated":"2021-06-09 17:19:02.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":"CC"},{"updated":"2021-06-28 09:01:28.000000000","updated_by":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"reviewer":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"state":"CC"},{"updated":"2021-07-08 09:47:29.000000000","updated_by":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"reviewer":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"state":"REVIEWER"}],"messages":[{"id":"3c26924e5339d47a9025578f39635bf28fabfe00","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"date":"2021-06-04 14:21:15.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"c461dac2ddc81863925c2f3268981b2833d32334","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-04 15:00:58.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/fb23d84d84ae41baba7cf0ff65bcd31d : SUCCESS in 24m 18s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/3c0cbc743e2c428b9802d297151932ec : SUCCESS in 32m 34s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"02b821e78491c541a2dda04cd6dc962a810ebeff","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-04 17:17:48.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/d35705db12bb4b3f9079b5b919d5064e : SUCCESS in 1h 26m 15s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/c7fd9bd5b40e43cd8aff5a631c0737de : SUCCESS in 19m 22s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/8cf3463686bc46a992ce7843892a0b24 : SUCCESS in 16m 46s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/28fc30b370124688b4009e67acae5779 : SUCCESS in 9m 06s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/4e027a11fe714a68a71bd000db04287c : SUCCESS in 13m 07s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/2748c529eec644ea9a9a4a6b2290b929 : SUCCESS in 13m 23s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/dd01b2dffdc541198eaafbceb9e49ad9 : SUCCESS in 13m 46s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ec31b36b612440c2a356e8764c87379c : SUCCESS in 9m 40s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/83c59a84cfe541109cbaa138fc0a1074 : SUCCESS in 16m 05s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/b6d242f897e14c2e80b8bf8274769278 : SUCCESS in 1h 12m 28s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/9047ed9498f648e3b69c6f23fefbfa87 : SUCCESS in 1h 16m 00s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/2801776024c948c997761e1255bb1879 : SUCCESS in 58m 31s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/b09ef6dfec1549ed86e5cfa071e41b0b : SUCCESS in 56m 41s (non-voting)\n- nova-lvm https://zuul.opendev.org/t/openstack/build/457865843d814a60af0830186a066bdf : SUCCESS in 1h 07m 39s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/54ab15e3c80c49088cb865e567260295 : SUCCESS in 1h 50m 46s\n- nova-next https://zuul.opendev.org/t/openstack/build/8095f4ff86154837993739e7662366d0 : SUCCESS in 1h 26m 47s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/51a528dd5c164f92ab858a125a02dc02 : SUCCESS in 18m 47s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/56aca0c8faba48e29a8a6de310b2b355 : SUCCESS in 32m 49s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/3c5b189bd3ef410eb0c696da9c0d6a7b : SUCCESS in 1h 11m 47s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/e42d821e84144b199f23d699f88c7a0d : SUCCESS in 1h 01m 41s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/3279a368160f41d0a7fdd4126fa39cf8 : SUCCESS in 48m 38s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/df63cf8a03f84423a1cd48258eed0e3c : SUCCESS in 25m 53s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/4c0e5fd3804f4a1da673d2ca14729981 : SUCCESS in 32m 20s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"5e65130db5911c690f94afcc3906792b9ca17a31","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-06-04 17:18:48.000000000","message":"Patch Set 1:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- EMC_PowerFlex_NOVA http://publiclogs.emc.com/37/794837/1/check/EMC_PowerFlex_NOVA/01df47b/EMC_PowerFlex_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC PowerFlex CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_PowerFlex_CI","accounts_in_message":[],"_revision_number":1},{"id":"0057ac1b59eb7c997f34f34898a410adfb6aebae","tag":"autogenerated:cloudbase-ci","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-06-04 18:38:38.000000000","message":"Patch Set 1:\n\nBuild succeeded. For checking Nova with Ceph, add a comment with run-Cloudbase Nova Ceph Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/794837/1/iscsi : SUCCESS in 2h 15m 31s","accounts_in_message":[],"_revision_number":1},{"id":"53115fac09e7de5e0e93db2ab39e18ba51ad674e","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-06-04 20:57:15.000000000","message":"Patch Set 1:\n\nBuild failed. 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-focal-py3 finger://zuul-v3-executor.novalocal/967750d77ed9427585178092d70513cb : POST_FAILURE in 1h 50m 58s","accounts_in_message":[],"_revision_number":1},{"id":"c7107ede5435644bfde466786880c802b4a61c72","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-06-05 12:15:44.000000000","message":"Patch Set 1:\n\nBuild succeeded\n\n- dsvm-nova http://207.189.188.190/logs/794837/794837/1/check-not-vote/ext-nova-zuul/e99b0aa : SUCCESS in 56m 25s","accounts_in_message":[],"_revision_number":1},{"id":"0c919c0d8c50c3a512b833f23e8e70e748d1c5f3","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2021-06-09 17:19:02.000000000","message":"Patch Set 1:\n\n(3 comments)\n\nThis seems not super controversial, and I think there\u0027s likely some real-world use-case that makes it make sense. I do wonder if you\u0027re trying to evacuate RBD, why you wouldn\u0027t just disable the glance locations API to make everything start detaching (i.e. even new image boots).","accounts_in_message":[],"_revision_number":1},{"id":"6f47a48aea3ba6bfe57c7242347b018efc1e830b","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"date":"2021-06-10 08:59:02.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"424132afa71e4a3c48ecebecb783e650b249d9ac","author":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"date":"2021-06-10 09:09:50.000000000","message":"Patch Set 2:\n\n(3 comments)\n\n\u003e Patch Set 1:\n\u003e \n\u003e (3 comments)\n\u003e \n\u003e This seems not super controversial, and I think there\u0027s likely some real-world use-case that makes it make sense. I do wonder if you\u0027re trying to evacuate RBD, why you wouldn\u0027t just disable the glance locations API to make everything start detaching (i.e. even new image boots).\n\nI don\u0027t really get what you mean by \"disable glance\u0027s location API\", but my understanding of the code, at least for the RBD one, is: Try to guess the glance image pool by looking at the parent of the vm disk (https://github.com/openstack/nova/blob/b0cd985f0c09088098f74cc0cb1df616cc0ef12b/nova/virt/libvirt/imagebackend.py#L1110), then only after call glance API and try to look into all the image locations if any is located into the same ceph cluster.\n\nOne solution can effectively to flatten manually all the vm disk to remove every relation between the disk and the image + disable glance location API, but it is really cumbersome, and purpose of my patch is just to make that more clearer for operators who need to perform such kind of operation during life of their deployments","accounts_in_message":[],"_revision_number":2},{"id":"d954470e97ae59a2d2b03f9cd27dc9502cfc989d","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"date":"2021-06-10 09:13:17.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"bf20bf34187f10a5b20eadb0bc43f3d029c85aa5","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-06-10 10:15:02.000000000","message":"Patch Set 3:\n\nBuild succeeded\n\n- dsvm-nova http://207.189.188.190/logs/794837/794837/3/check-not-vote/ext-nova-zuul/d29185c : SUCCESS in 59m 55s","accounts_in_message":[],"_revision_number":3},{"id":"bb36bfc7c1943670a71d67c30153b33ec9acf378","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-10 10:43:20.000000000","message":"Patch Set 3:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/38130035dda943fbb1343e439832fc6b : SUCCESS in 32m 37s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/1dd0ad692404434a80e641c7a4c35a7c : SUCCESS in 31m 50s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"17a42180cc213286ecfe15e381c9332b608fbacf","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-10 11:45:08.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- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/0c385443664041cda72ca244670ed39e : SUCCESS in 1h 10m 54s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/2e5b30d757e44e3cb37ac05a3f73c7b0 : SUCCESS in 16m 03s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/be698cd93f4b45af8d0bbe3062f2f705 : SUCCESS in 13m 59s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/9e35db926631433789a4f3a9ba841e82 : FAILURE in 8m 06s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/5c8de7ab94d74d668721f3e15c2c227f : SUCCESS in 13m 07s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/a177c21d653d4aeab15992a837688fb8 : SUCCESS in 12m 51s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/174b31c3ac39431fbd5a6d3a3481456b : SUCCESS in 13m 49s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/a247582519634da7af395c18fbc42c87 : SUCCESS in 10m 06s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/7308aea835384249816c14de8dcde3b8 : SUCCESS in 17m 07s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/32c72deb281e4fd4b9f04f07bb483e78 : SUCCESS in 1h 12m 00s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/28565984f17f449784d0887052961b0c : SUCCESS in 1h 13m 42s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/290593a2ad234bdd8c74265030fed5cb : SUCCESS in 54m 19s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/aaccdd057a9f45829ad5b1538d89c804 : SUCCESS in 59m 33s (non-voting)\n- nova-lvm https://zuul.opendev.org/t/openstack/build/28efa9d2e8df46fe8a25d40113ebc085 : SUCCESS in 52m 19s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/332cebc3cac24f52b9c0cfdf8904cbeb : SUCCESS in 1h 21m 16s\n- nova-next https://zuul.opendev.org/t/openstack/build/3372b9936f08449a9af82b4eb24d1398 : FAILURE in 1h 44m 19s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/15c8bba0251345bd954889b4e0ea1e11 : SUCCESS in 23m 02s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/e3860b5e01a44c73b09db82754e5859e : SUCCESS in 19m 25s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/c57bb801783f44b6b9ecc2b56f8a4cd5 : SUCCESS in 1h 22m 31s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/5a2b0c1d7fcd465db671870966f6771b : SUCCESS in 57m 24s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/666417faff704e16b613739d06001c90 : SUCCESS in 48m 50s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/7220e50bfa7c4166a2a06b844267be8b : FAILURE in 30m 03s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/7dce0db580284033ba4b100d86109151 : FAILURE in 27m 19s (non-voting)\n\nWarning:\n  Comments left for invalid file nova/crypto.py","accounts_in_message":[],"_revision_number":3},{"id":"3b04758aca26b1fa3fb565de9f4637b7011ced41","author":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"date":"2021-06-10 11:59:11.000000000","message":"Patch Set 3:\n\nrecheck","accounts_in_message":[],"_revision_number":3},{"id":"85ae930204ba20d2b04407dfb2751c8ccf405f0d","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-06-10 12:00:51.000000000","message":"Patch Set 3:\n\nBuild failed\n\n- dsvm-nova http://207.189.188.190/logs/794837/794837/3/check-not-vote/ext-nova-zuul/5b8a5a1 : FAILURE in 0s","accounts_in_message":[],"_revision_number":3},{"id":"17ff60812703e409c53cc742b60f7fb6f99fffd9","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-06-10 12:10:32.000000000","message":"Patch Set 3:\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-focal-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/37/794837/3/check/tempest-dsvm-full-focal-py3/0ea83ae/ : SUCCESS in 1h 42m 12s","accounts_in_message":[],"_revision_number":3},{"id":"dc4e0d75b43716d36eb66b87bceca6032163ea47","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-10 12:26:52.000000000","message":"Patch Set 3:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/05a2817410a84c7ea1822930a3b0703b : SUCCESS in 20m 00s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/081bc7cfc86e4e759023959344215a32 : SUCCESS in 21m 13s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"e4a557acd8851e2ba09c5cc509c29f45fa0e3ad9","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-10 13:34:23.000000000","message":"Patch Set 3:\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- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/42a63028ab214c38bc385e355f637ac9 : SUCCESS in 1h 24m 23s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/b609dbf565f6472cb87d92c59b04b7ec : SUCCESS in 14m 50s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/0862fd9bb1714cee99eba598ad52cc65 : SUCCESS in 16m 10s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/dafabb5b7fb941109f934a0c59959976 : FAILURE in 7m 57s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/1d0868fd0a774440b41de9550a6de7c7 : SUCCESS in 12m 45s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/79c11240043b4e3398d3e3964dd9f6e7 : SUCCESS in 12m 23s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/6226ceac981e4c02a1772aed68658405 : SUCCESS in 12m 56s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e0f06bca5c6649f9917ce3429ae4bc2c : SUCCESS in 8m 53s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/2dfd7934948d47f6ac67b3cfe6c427ca : SUCCESS in 15m 51s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/e9704a17f7b340cfa2dd17084855e814 : SUCCESS in 1h 11m 45s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/d2a03a3fca464801854da6dbd2dc202b : SUCCESS in 1h 21m 08s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/f2a19c1c99194fbca6a6b0c54ca199a3 : FAILURE in 49m 25s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/d2311a14c4bc4743a2d2e0a2dab33c56 : SUCCESS in 56m 05s (non-voting)\n- nova-lvm https://zuul.opendev.org/t/openstack/build/907cb769d1ba41b6b466079a7e06247f : SUCCESS in 54m 07s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/dfd6bd6fe1bc473ca52db16a02c836cb : SUCCESS in 1h 14m 36s\n- nova-next https://zuul.opendev.org/t/openstack/build/24a5ea8747c74c35930d8907b1ce0c47 : FAILURE in 1h 33m 24s\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/6d5f6648ddec4c60a9c2290cec4f9cd2 : SUCCESS in 19m 04s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/b30efd7f5af842ae88187ac5383b455b : SUCCESS in 20m 44s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/fb90ef0a7d6a4b7c98ad29f289002ac8 : SUCCESS in 1h 12m 11s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/bcf95d0e175b4cb3a40dc1917fd2304c : SUCCESS in 45m 52s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/ee23cad7278d4af3bb35389da9c12316 : SUCCESS in 46m 22s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/6e8b4481e1954d2787ac1362eb59a86b : FAILURE in 26m 19s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/8be419dc77f944be911b8461109d3d10 : FAILURE in 27m 27s (non-voting)\n\nWarning:\n  Comments left for invalid file nova/crypto.py","accounts_in_message":[],"_revision_number":3},{"id":"0623db7273b7d9cd925b37a8216283553a32378e","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2021-06-10 13:35:40.000000000","message":"Patch Set 3:\n\n\u003e I don\u0027t really get what you mean by \"disable glance\u0027s location API\", but my understanding of the code, at least for the RBD one, is: Try to guess the glance image pool by looking at the parent of the vm disk (https://github.com/openstack/nova/blob/b0cd985f0c09088098f74cc0cb1df616cc0ef12b/nova/virt/libvirt/imagebackend.py#L1110), then only after call glance API and try to look into all the image locations if any is located into the same ceph cluster.\n\nI mean disable these:\n\nhttps://docs.openstack.org/glance/rocky/configuration/glance_api.html#DEFAULT.show_image_direct_url\nhttps://docs.openstack.org/glance/rocky/configuration/glance_api.html#DEFAULT.show_multiple_locations\n\n\u003e One solution can effectively to flatten manually all the vm disk to remove every relation between the disk and the image + disable glance location API, but it is really cumbersome, and purpose of my patch is just to make that more clearer for operators who need to perform such kind of operation during life of their deployments\n\nAre you saying you aren\u0027t moving away from ceph, you just don\u0027t want the smart clone features that come with it? I think disabling the locations bit above will accomplish that, FWIW.","accounts_in_message":[],"_revision_number":3},{"id":"88f232794270533d0991e10a03ca012fa9d89677","tag":"autogenerated:cloudbase-ci","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-06-10 13:51:10.000000000","message":"Patch Set 3:\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/794837/3/iscsi : FAILURE in 2h 32m 22s","accounts_in_message":[],"_revision_number":3},{"id":"bfff152f12d2be6a9942d99d3dd9a8a0cbb8a965","author":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"date":"2021-06-10 14:15:25.000000000","message":"Patch Set 3:\n\n\u003e Patch Set 3:\n\u003e \n\u003e \u003e I don\u0027t really get what you mean by \"disable glance\u0027s location API\", but my understanding of the code, at least for the RBD one, is: Try to guess the glance image pool by looking at the parent of the vm disk (https://github.com/openstack/nova/blob/b0cd985f0c09088098f74cc0cb1df616cc0ef12b/nova/virt/libvirt/imagebackend.py#L1110), then only after call glance API and try to look into all the image locations if any is located into the same ceph cluster.\n\u003e \n\u003e I mean disable these:\n\u003e \n\u003e https://docs.openstack.org/glance/rocky/configuration/glance_api.html#DEFAULT.show_image_direct_url\n\u003e https://docs.openstack.org/glance/rocky/configuration/glance_api.html#DEFAULT.show_multiple_locations\n\u003e \n\u003e \u003e One solution can effectively to flatten manually all the vm disk to remove every relation between the disk and the image + disable glance location API, but it is really cumbersome, and purpose of my patch is just to make that more clearer for operators who need to perform such kind of operation during life of their deployments\n\u003e \n\u003e Are you saying you aren\u0027t moving away from ceph, you just don\u0027t want the smart clone features that come with it? I think disabling the locations bit above will accomplish that, FWIW.\n\nYes I am trying to move image backend from ceph to swift, but with nova disks still in ceph. Currently, you can\u0027t achieve that as direct_snapshot feature will always try to put back the snapshot to images pool in ceph. New instances from theses snapshot will perform copy on write, so they will be child of images pool, then snapshot of these new vms will also be stored on ceph in image pool, etc...\n\nChanging glance configuration make nothing to help, as nova bypass everything and firstly rely on rbd call to take his decision. Nova will only notify glance by adding a new location to the image at the end of the process, bypassing the glance import workflow.","accounts_in_message":[],"_revision_number":3},{"id":"dc87edb8cc2749e9451ffc3cecad163f3e01a9ad","author":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"date":"2021-06-22 16:07:21.000000000","message":"Patch Set 3:\n\nrecheck","accounts_in_message":[],"_revision_number":3},{"id":"e37c40e3f7da2d9d86191ac526e5849b9d377d84","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-22 16:38:38.000000000","message":"Patch Set 3:\n\nBuild succeeded (ARM64 pipeline).\n\n- openstack-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/4a5fa1e54603444a9690d97be5fc6943 : SUCCESS in 29m 07s (non-voting)\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/0a9ddbc53c554ba685926abff7721f56 : SUCCESS in 20m 52s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"c51ffe6922a7705a7f7bef60f402eb5a9b76632d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-22 18:07:20.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/6b986050fd1341ce95b0f999d8272f25 : SUCCESS in 1h 09m 03s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/17a72a145bc84ca599a204939b85a0da : SUCCESS in 19m 14s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/048596d64dda4389a51d7837842b86b0 : SUCCESS in 15m 04s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/0f6a99034cde4517a67add532e13c39a : SUCCESS in 9m 05s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/7ea0696628544b6d8fd6a999de0dfe38 : SUCCESS in 13m 04s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/222bad64a7b141a8a4b0d2df053939f3 : SUCCESS in 12m 41s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/a565bd7c8dd843a299a413f9420cac32 : SUCCESS in 16m 14s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/22d2f099e98a408dadcad80468c8994c : SUCCESS in 10m 13s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/b343c4eb9769482785371d6c9ed4ef4c : SUCCESS in 18m 59s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/0cf1c37371884c73b711c334781141fc : SUCCESS in 1h 26m 16s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/6edf43dc935f4210bbc68d764881f73f : SUCCESS in 1h 30m 16s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/079fe104e4674958aeff58900651088a : SUCCESS in 1h 07m 11s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/df195ff432cc4ce2942deda2abe57829 : SUCCESS in 53m 00s (non-voting)\n- nova-lvm https://zuul.opendev.org/t/openstack/build/335edb9d4a074b52a1b8f202d88fa25f : SUCCESS in 50m 47s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/f42c889271b84810b54f0e8149646e0d : SUCCESS in 1h 30m 39s\n- nova-next https://zuul.opendev.org/t/openstack/build/a2c8461c57674349adbd4fbbadf0f188 : SUCCESS in 1h 22m 06s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/9db5528997dc4dc79c9ce55d371d3022 : SUCCESS in 7m 17s (non-voting)\n- nova-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/1aa8c299ce2747f39249821d4df5d3a1 : SUCCESS in 34m 17s\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/02081fe1cf1946bcb16615a349743f92 : SUCCESS in 22m 38s (non-voting)\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/977ecb0f4de6473abf9fad419f409ee5 : SUCCESS in 1h 11m 49s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/c50c1e48b3b844399ac272c8bf064a01 : SUCCESS in 49m 08s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/b2ab1e74dab14752891f6f8e5f690c9d : SUCCESS in 47m 00s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/6e5ecf2667da438586dc3e12b65ef8f9 : FAILURE in 29m 44s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/2160a64f209240d68953ceb0fb6a1b7d : SUCCESS in 40m 39s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"cd9b722bc8fbcb9d52bd3f6b2aeb2601c01dc49d","author":{"_account_id":15941,"name":"DellEMC PowerFlex CI","email":"emc.scaleio.ci@emc.com","username":"emc-scaleio-ci","tags":["SERVICE_USER"]},"date":"2021-06-22 18:14:09.000000000","message":"Patch Set 3:\n\nBuild failed.  For information on how to proceed, see https://docs.openstack.org/infra/manual/developers.html\n\n- EMC_PowerFlex_NOVA http://publiclogs.emc.com/37/794837/3/check/EMC_PowerFlex_NOVA/250d766/EMC_PowerFlex_NOVA/None : NOT_REGISTERED\n\nLeave a comment with \u0027run-DellEMC PowerFlex CI\u0027 to trigger a recheck. For more information about CI, please see https://wiki.openstack.org/wiki/ThirdPartySystems/DellEMC_PowerFlex_CI","accounts_in_message":[],"_revision_number":3},{"id":"55c5ad524880f51cb83e57d8a02560b4103965c3","tag":"autogenerated:cloudbase-ci","author":{"_account_id":26515,"name":"Cloudbase Nova Hyper-V CI","email":"nova_hyperv_ci@cloudbasesolutions.com","username":"nova_hyperv_ci"},"date":"2021-06-22 18:50:38.000000000","message":"Patch Set 3:\n\nBuild succeeded. For checking Nova with Ceph, add a comment with run-Cloudbase Nova Ceph Hyper-V CI\n\n- nova http://cloudbase-ci.com/nova/794837/3/iscsi : SUCCESS in 2h 41m 18s","accounts_in_message":[],"_revision_number":3},{"id":"46053e2176a42ba7ec5e9ef85da65a3c21b1d4df","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2021-06-22 21:41:51.000000000","message":"Patch Set 3:\n\nBuild failed. 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-focal-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/37/794837/3/check/tempest-dsvm-full-focal-py3/7bf2f80/ : TIMED_OUT in 3h 11m 32s","accounts_in_message":[],"_revision_number":3},{"id":"2ae50016d12183e1ba46740437f39973d019e719","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2021-06-23 06:04:40.000000000","message":"Patch Set 3:\n\nBuild succeeded\n\n- dsvm-nova http://207.189.188.190/logs/794837/794837/3/check-not-vote/ext-nova-zuul/2bee7f2 : SUCCESS in 58m 49s","accounts_in_message":[],"_revision_number":3},{"id":"17e1381f200a2b45ca7d3f824f74ff3cc2bf3880","author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"date":"2021-06-28 09:01:28.000000000","message":"Patch Set 3:\n\n@Victor: To get the bp approved please add it to the agenda of a Nova weekly meeting[1].\n\n[1] https://wiki.openstack.org/wiki/Meetings/Nova","accounts_in_message":[],"_revision_number":3},{"id":"ae9b1f791c77405bd8a8f02fcd14ccaae894812e","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-07-08 09:47:29.000000000","message":"Patch Set 3: Code-Review-1\n\n(1 comment)\n\nApologies but I don\u0027t get the usecase for this, moving storage backends in Glance might be valid but given the way the rbd imagebackend is written within the libvirt driver it\u0027s going to take manual operator interaction to break the backing chains in RBD before you do it.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"6f7a7e16db4ff00b73ea01955669a257dad34e20","revisions":{"9fd1b225c07b1c478ea4bf955773ab501cb20939":{"kind":"REWORK","_number":1,"created":"2021-06-04 14:21:15.000000000","uploader":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"ref":"refs/changes/37/794837/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/37/794837/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/794837/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/794837/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/794837/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/37/794837/1"}}},"commit":{"parents":[{"commit":"cd084aeeb8a2110759912c1b529917a9d3aac555","subject":"Merge \"Error anti-affinity violation on migrations\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/cd084aeeb8a2110759912c1b529917a9d3aac555"}]}],"author":{"name":"Victor Coutellier","email":"victor.coutellier@ovhcloud.com","date":"2021-06-04 14:20:18.000000000","tz":120},"committer":{"name":"Victor Coutellier","email":"victor.coutellier@ovhcloud.com","date":"2021-06-04 14:20:18.000000000","tz":120},"subject":"Allow configuration of direct-snapshot feature","message":"Allow configuration of direct-snapshot feature\n\nAdd a new configuration option, ``[libvirt]/allow_direct_snapshot``.\nThis allow operator force nova to not use the direct_snapshot workflow\nwhen performing instance snapshot, even if it is implemented in the\nbacking driver. Default value is True.\n\nThis can be usefull in case of nova backend migration, to force nova\nhonor the glance configuration and do not guess it from the actual\nbackend.\n\nImplements: blueprint configure-direct-snapshot\nChange-Id: Iab55d6a1fcde763ca859d0d1ecfb11a11cf35738\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9fd1b225c07b1c478ea4bf955773ab501cb20939"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9fd1b225c07b1c478ea4bf955773ab501cb20939"}]},"branch":"refs/heads/master"},"65923f2c8bc717b60811d3c313a10e43a3205b34":{"kind":"REWORK","_number":2,"created":"2021-06-10 08:59:02.000000000","uploader":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"ref":"refs/changes/37/794837/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/37/794837/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/794837/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/794837/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/794837/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/37/794837/2"}}},"commit":{"parents":[{"commit":"cd084aeeb8a2110759912c1b529917a9d3aac555","subject":"Merge \"Error anti-affinity violation on migrations\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/cd084aeeb8a2110759912c1b529917a9d3aac555"}]}],"author":{"name":"Victor Coutellier","email":"victor.coutellier@ovhcloud.com","date":"2021-06-04 14:20:18.000000000","tz":120},"committer":{"name":"Victor Coutellier","email":"victor.coutellier@ovhcloud.com","date":"2021-06-10 08:57:49.000000000","tz":120},"subject":"Allow configuration of direct-snapshot feature","message":"Allow configuration of direct-snapshot feature\n\nAdd a new configuration option, ``[libvirt]/allow_direct_snapshot``.\nThis allow operator force nova to not use the direct_snapshot workflow\nwhen performing instance snapshot, even if it is implemented in the\nbacking driver. Default value is True.\n\nThis can be useful in case of nova backend migration, to force nova\nhonor the glance configuration and do not guess it from the actual\nbackend. Indeed, when using the nova RBD backend with image who were\noriginally stored in ceph, it will put the image snapshot in RBD too\neven if an RBD store is no longer configured in glance.\n\nImplements: blueprint configure-direct-snapshot\nChange-Id: Iab55d6a1fcde763ca859d0d1ecfb11a11cf35738\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/65923f2c8bc717b60811d3c313a10e43a3205b34"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/65923f2c8bc717b60811d3c313a10e43a3205b34"}]},"branch":"refs/heads/master"},"6f7a7e16db4ff00b73ea01955669a257dad34e20":{"kind":"REWORK","_number":3,"created":"2021-06-10 09:13:17.000000000","uploader":{"_account_id":28595,"name":"Victor Coutellier","email":"victor.coutellier@gmail.com","username":"alistarle"},"ref":"refs/changes/37/794837/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/37/794837/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/794837/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/794837/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/794837/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/37/794837/3"}}},"commit":{"parents":[{"commit":"cd084aeeb8a2110759912c1b529917a9d3aac555","subject":"Merge \"Error anti-affinity violation on migrations\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/cd084aeeb8a2110759912c1b529917a9d3aac555"}]}],"author":{"name":"Victor Coutellier","email":"victor.coutellier@ovhcloud.com","date":"2021-06-04 14:20:18.000000000","tz":120},"committer":{"name":"Victor Coutellier","email":"victor.coutellier@ovhcloud.com","date":"2021-06-10 09:13:03.000000000","tz":120},"subject":"Allow configuration of direct-snapshot feature","message":"Allow configuration of direct-snapshot feature\n\nAdd a new configuration option, ``[libvirt]/allow_direct_snapshot``.\nThis allow operator force nova to not use the direct_snapshot workflow\nwhen performing instance snapshot, even if it is implemented in the\nbacking driver. Default value is True.\n\nThis can be useful in case of nova backend migration, to force nova\nhonor the glance configuration and do not guess it from the actual\nbackend. Indeed, when using the nova RBD backend with image who were\noriginally stored in ceph, it will put the image snapshot in RBD too\neven if an RBD store is no longer configured in glance.\n\nImplements: blueprint configure-direct-snapshot\nChange-Id: Iab55d6a1fcde763ca859d0d1ecfb11a11cf35738\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/6f7a7e16db4ff00b73ea01955669a257dad34e20"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/6f7a7e16db4ff00b73ea01955669a257dad34e20"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"OK","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY","applied_by":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}},{"label":"Workflow","status":"MAY"},{"label":"Review-Priority","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"name":"Review-Priority","description":"Review Priority","status":"NOT_APPLICABLE","is_legacy":false,"applicability_expression_result":{"fulfilled":false,"status":"FAIL"},"submittability_expression_result":{"expression":"is:true","fulfilled":true,"status":"NOT_EVALUATED","passing_atoms":[],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}