)]}'
{"id":"openstack%2Fnova~924399","triplet_id":"openstack%2Fnova~master~Iee4e06a3917564f8c19d18bca54c6fc4c0a8f103","project":"openstack/nova","branch":"master","topic":"do-not-hide-neutron-error","attention_set":{"11604":{"account":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"last_update":"2024-07-25 13:43:10.000000000","reason":"\u003cGERRIT_ACCOUNT_11604\u003e replied on the change","reason_account":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"}},"9545":{"account":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"last_update":"2024-07-25 12:05:45.000000000","reason":"A robot voted negatively on a label"}},"removed_from_attention_set":{},"hashtags":[],"change_id":"Iee4e06a3917564f8c19d18bca54c6fc4c0a8f103","subject":"Propagate neutron\u0027s error during removeSecurityGroup action","status":"NEW","created":"2024-07-18 11:51:26.000000000","updated":"2024-07-25 13:43:10.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":true,"submittable":false,"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"117061a83d0bd7ec1d3c5ebcddc440cd1da131a3","_number":924399,"virtual_id_number":924399,"owner":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"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":"2024-07-18 13:54:02.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":"2024-07-18 12:36:22.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":"2024-07-18 13:54:02.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":"2024-07-25 13:43:10.000000000","updated_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"}],"messages":[{"id":"5edeedfc18a0cc1013445648c781b9891966a81b","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"date":"2024-07-18 11:51:26.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6583d275eae94705ca490905f9ae92534279764a","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2024-07-18 12:36:22.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/9456b4ae7c4148ec800e8a9feb5f7489\n\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/abfe5084e4c44959bcd730b2f27992e1 : SUCCESS in 42m 49s (non-voting)\n- openstack-tox-py311-arm64 https://zuul.opendev.org/t/openstack/build/49fb17eab54d413aa8483fd44b593c45 : SUCCESS in 27m 59s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"3b2e1f7676e7f515d00bcaf8f83da7f29b5bc346","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2024-07-18 13:54:02.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/04439bc28c9c46039e3219a4f627af77\n\n- grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/3c6784695b6344cc841e7fbb2ef9f6b0 : SUCCESS in 1h 00m 17s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/0c18e999e67a4c39afd1e824a72ab557 : SUCCESS in 1h 33m 49s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/3543f09b9ca348c88d180279b93adc43 : SUCCESS in 1h 00m 28s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/ad2648a28fe843a9bdd9d50da063e9ca : SUCCESS in 21m 25s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/627f9deea6624e1d90f61c959e3289c7 : SUCCESS in 7m 16s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/c22c4d3667744930a2525d88ae246b9f : FAILURE in 12m 08s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/bb4a9a2e8ddd46dfbd76d41ddc58fc7c : SUCCESS in 14m 35s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/4d35b1d176ef48b18fe436814c5d0f23 : SUCCESS in 18m 20s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/52f7fd3d8f5c498abe2408a4661c0089 : SUCCESS in 9m 02s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/4eeb7d5f7f064ccc8c338f6665e4d8a5 : SUCCESS in 1h 00m 33s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/b13bd401870f43c38b2c7df1925aebb1 : SUCCESS in 1h 42m 17s\n- neutron-linuxbridge-tempest https://zuul.opendev.org/t/openstack/build/438907cf9a304a1c93644017e3635f6e : SUCCESS in 1h 16m 01s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/829db9f86dca433088f3d30a3aef761c : SUCCESS in 42m 38s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/f6508fb14eb6404e984dd365c6fe2843 : SUCCESS in 47m 26s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/40c0d8b93c5a4c019df4f08b0b8505f2 : SUCCESS in 1h 42m 27s\n- nova-next https://zuul.opendev.org/t/openstack/build/7cb5669f42f544f2a6debda8b7ffc78d : SUCCESS in 1h 29m 58s\n- nova-ovs-hybrid-plug https://zuul.opendev.org/t/openstack/build/1c488989cb694931bfafea48316cc1bc : SUCCESS in 51m 08s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/2f59fd2cec9b4a09864742319f568fba : SUCCESS in 5m 42s (non-voting)\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/3066bd8f84d54e0db057572e4bc0ec50 : SUCCESS in 25m 27s\n- nova-tox-functional-py310 https://zuul.opendev.org/t/openstack/build/6687dc089b5a41099abad9ee885b844b : SUCCESS in 24m 44s\n- nova-tox-functional-py312 https://zuul.opendev.org/t/openstack/build/ad84d0e5ef5b47ea9eac3f4611ff93f9 : SUCCESS in 34m 31s\n- tempest-integrated-compute-enforce-scope-new-defaults https://zuul.opendev.org/t/openstack/build/d4a7ee0521c744b0a7ed70ce66938711 : SUCCESS in 1h 58m 47s\n- tempest-integrated-compute-rbac-old-defaults https://zuul.opendev.org/t/openstack/build/3aa0d539cf89447499e7b3e3d3a7f4b3 : SUCCESS in 1h 29m 30s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/94c4e3d76b8441c38d7959b8e8fa30a1 : SUCCESS in 1h 33m 22s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/544dde0e993c403097aed763630e7a2e : SUCCESS in 53m 51s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/af1d94946a544b03b8d46adfce85b692 : SUCCESS in 22m 21s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/e0153cc647654b5f813c62e162972334 : SUCCESS in 22m 43s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"fafbc78c183e847f9e6cdbae2e0b253f0033646a","author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"date":"2024-07-25 10:00:23.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"ae721783f365f5506ff506c25bcaaf9617f4018f","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2024-07-25 10:36:42.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/19a624f2a1b5443ab691397aac73cefd\n\n- openstack-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/58e1951ccf3c4549bb181c6bfafb9c27 : SUCCESS in 13m 56s (non-voting)\n- openstack-tox-py311-arm64 https://zuul.opendev.org/t/openstack/build/4b19633e539f4aa9b1159059ea8e1f4d : FAILURE in 34m 27s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"6e07c81118ada8c8bf47e8732f2e822d52f14870","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2024-07-25 12:05:45.000000000","message":"Patch Set 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\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/94858698c02f458ea42eb790f9d3a142\n\n- grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/4c961a038ccb4168a5c2600782942186 : SUCCESS in 52m 22s\n- tempest-integrated-compute https://zuul.opendev.org/t/openstack/build/609a961bb6a34c60866ec4ae2b4287a3 : SUCCESS in 1h 34m 50s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/eb0f77bd7375475fb4bdb68d13d6267d : SUCCESS in 51m 09s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/5732cadac429474d809d6007ce72e592 : SUCCESS in 23m 33s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/baef993212a14c5e825ed6cccd93815a : SUCCESS in 7m 52s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/1fbed36712ae4718841cd7d1970ab48b : SUCCESS in 13m 31s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/5880787ca00849a8a07b5b6269882214 : SUCCESS in 10m 57s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/7698c176c6464cb9a14abf5c14f9adca : SUCCESS in 17m 29s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/5ee8917de9b648c3b91a3fdd902e79f8 : SUCCESS in 10m 30s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/db0c297df56e441d84022c3b1a9f8103 : FAILURE in 26m 38s (non-voting)\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/50418d707e2f43408ea91fbe9aa74ce2 : SUCCESS in 1h 47m 29s\n- neutron-linuxbridge-tempest https://zuul.opendev.org/t/openstack/build/448957f8166242c38b4d884667083f66 : FAILURE in 1h 04m 50s\n- nova-live-migration https://zuul.opendev.org/t/openstack/build/39444fcb8cc642ec9cf6e88018cba938 : SUCCESS in 36m 09s\n- nova-live-migration-ceph https://zuul.opendev.org/t/openstack/build/576936762c88454093276714fe0561a3 : SUCCESS in 47m 45s\n- nova-multi-cell https://zuul.opendev.org/t/openstack/build/2e623d31a3ff47bd849d12c0aed65cea : SUCCESS in 2h 03m 55s\n- nova-next https://zuul.opendev.org/t/openstack/build/e319b0996d614979bead42478e705eff : SUCCESS in 1h 40m 21s\n- nova-ovs-hybrid-plug https://zuul.opendev.org/t/openstack/build/84ffb34f11b54c668273931582c77e1f : SUCCESS in 48m 31s\n- nova-tox-validate-backport https://zuul.opendev.org/t/openstack/build/dcedd9fb25474dae88ee27f696af24be : SUCCESS in 4m 24s (non-voting)\n- nova-tox-functional-py39 https://zuul.opendev.org/t/openstack/build/3d3de655c9ff4dafb781b05022f8fd2b : SUCCESS in 24m 25s\n- nova-tox-functional-py310 https://zuul.opendev.org/t/openstack/build/d809ec5fbcd3410bb0a106f868de6446 : SUCCESS in 19m 54s\n- nova-tox-functional-py312 https://zuul.opendev.org/t/openstack/build/89bd5cd46b494e638b381cd065a63975 : SUCCESS in 32m 38s\n- tempest-integrated-compute-enforce-scope-new-defaults https://zuul.opendev.org/t/openstack/build/6a1702663bd84c6e81de099236cd9f9c : SUCCESS in 1h 42m 31s\n- tempest-integrated-compute-rbac-old-defaults https://zuul.opendev.org/t/openstack/build/291f9714cbc84fecb9c425bbe46613b0 : SUCCESS in 1h 58m 45s\n- nova-grenade-multinode https://zuul.opendev.org/t/openstack/build/671dc8b6ffb14aff96b4f2b492957883 : SUCCESS in 1h 17m 38s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/136234fc42e24a06804115c2828eac39 : SUCCESS in 46m 42s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/db755a6deef94bd2afec3b9b952c1e45 : SUCCESS in 28m 15s (non-voting)\n- barbican-tempest-plugin-simple-crypto https://zuul.opendev.org/t/openstack/build/98157fbc58264c9d9d9eea8637d31135 : SUCCESS in 33m 20s (non-voting)","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"c47bbc6bc8be35bdbe9360624505818b43b4bb7e","revisions":{"c47bbc6bc8be35bdbe9360624505818b43b4bb7e":{"kind":"REWORK","_number":1,"created":"2024-07-18 11:51:26.000000000","uploader":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"ref":"refs/changes/99/924399/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/99/924399/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/99/924399/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/99/924399/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/99/924399/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/99/924399/1"}}},"commit":{"parents":[{"commit":"e82854dc8c514e457528b52834d79176fe5a2135","subject":"Merge \"hardware: Fix image_meta.id within get_mem_encryption_constraint\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/e82854dc8c514e457528b52834d79176fe5a2135"}]}],"author":{"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","date":"2024-07-18 11:51:01.000000000","tz":120},"committer":{"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","date":"2024-07-18 11:51:22.000000000","tz":120},"subject":"Propagate neutron\u0027s error during removeSecurityGroup action","message":"Propagate neutron\u0027s error during removeSecurityGroup action\n\nThe logic of handling neutron\u0027s error is different in addSecurityGroup\nand removeSecurityGroup server actions. \u0027addSecurityGroup\u0027 propagates\nneutron\u0027s 400 error versus \u0027removeSecurityGroup\u0027 which hides all the details\nfrom end-users.\nNeutron\u0027s errors include nothing sensitive, so propagating them to\nend-users improves UX in comparison to seeing scaring InternalServerError\nthat may result in calling cloud-admin to check the issue.\n\nChange-Id: Iee4e06a3917564f8c19d18bca54c6fc4c0a8f103\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c47bbc6bc8be35bdbe9360624505818b43b4bb7e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/c47bbc6bc8be35bdbe9360624505818b43b4bb7e"}]},"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"},{"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":""}}}]}