)]}'
{"id":"openstack%2Fpuppet-vswitch~854998","triplet_id":"openstack%2Fpuppet-vswitch~stable%2Fxena~I56f85ca35f8e9ee4e5e8d07c714b65f655b1e9ae","project":"openstack/puppet-vswitch","branch":"stable/xena","topic":"bug/1987419-stable/xena","attention_set":{},"removed_from_attention_set":{"9816":{"account":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"last_update":"2022-09-01 10:31:27.000000000","reason":"Change was submitted"}},"hashtags":[],"change_id":"I56f85ca35f8e9ee4e5e8d07c714b65f655b1e9ae","subject":"vs_config: Fix broken idempotency with nil/empty value","status":"MERGED","created":"2022-08-29 16:36:27.000000000","updated":"2022-09-01 10:31:27.000000000","submitted":"2022-09-01 10:31:27.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":"854998-bug/1987419-stable/xena","cherry_pick_of_change":854893,"cherry_pick_of_patch_set":1,"meta_rev_id":"cdfeceabe8f630fb5c1aae31ebb41f73231d48ca","_number":854998,"virtual_id_number":854998,"owner":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2022-09-01 10:31:26.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":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"all":[{"value":2,"date":"2022-08-30 16:21:12.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":2,"date":"2022-09-01 09:13:40.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},{"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":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"all":[{"value":0,"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":1,"date":"2022-09-01 09:13:40.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},{"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}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2022-08-29 18:26:37.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":"2022-08-30 16:21:12.000000000","updated_by":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"reviewer":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"state":"REVIEWER"},{"updated":"2022-09-01 09:13:40.000000000","updated_by":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"reviewer":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"state":"REVIEWER"}],"messages":[{"id":"494b917a691d3ccb645cc2ca8ed61cf97ecfbe8b","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2022-08-29 16:36:27.000000000","message":"Patch Set 1: Cherry Picked from branch stable/yoga.","accounts_in_message":[],"_revision_number":1},{"id":"336de04a169d0837ccac593a4be68f4f156c94c1","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2022-08-29 16:36:35.000000000","message":"Topic bug/1987419-stable/yoga-stable/xena removed","accounts_in_message":[],"_revision_number":1},{"id":"cfaa79055bc3d4041e866d88f9a223bc8d4943b9","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2022-08-29 16:36:39.000000000","message":"Topic set to bug/1987419-stable/xena","accounts_in_message":[],"_revision_number":1},{"id":"b283e8d2418765350e4f650a463a8d73fe1bf8b1","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-08-29 18:26:37.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/c6d49b07257947c9aa5ad5d3900fc8a4\n\n- puppet-openstack-lint-ubuntu-focal https://zuul.opendev.org/t/openstack/build/b1fe1545ae0f442fb15951c2e04f70a9 : SUCCESS in 3m 23s\n- puppet-openstack-syntax-6-ubuntu-focal https://zuul.opendev.org/t/openstack/build/c6567a4bf7004a798bd446d250a2b9c1 : SUCCESS in 3m 23s\n- puppet-openstack-syntax-7-ubuntu-focal https://zuul.opendev.org/t/openstack/build/bd0d7739e13d48e6b23c529b9c47c692 : SUCCESS in 3m 17s\n- puppet-openstack-unit-6.21-centos-8-stream https://zuul.opendev.org/t/openstack/build/b25e91b54d96443187f7a60facda86f0 : SUCCESS in 12m 50s\n- puppet-openstack-unit-7.8-centos-8-stream https://zuul.opendev.org/t/openstack/build/dc891f93ae1144eba3c2ec8fc412f1c9 : SUCCESS in 13m 48s\n- puppet-openstack-unit-6.21-ubuntu-focal https://zuul.opendev.org/t/openstack/build/e31c74c424fa4961be8fff8fa439d641 : SUCCESS in 9m 39s\n- puppet-openstack-unit-7.8-ubuntu-focal https://zuul.opendev.org/t/openstack/build/1e95a29d61654b4f9a419df0ab190408 : SUCCESS in 9m 37s\n- puppet-openstack-unit-latest-ubuntu-focal https://zuul.opendev.org/t/openstack/build/fab0c026ef5845228cbd8478a6f1ba58 : SUCCESS in 9m 21s (non-voting)\n- puppet-openstack-integration-6-scenario001-tempest-ubuntu-focal https://zuul.opendev.org/t/openstack/build/243a19b6ec1d4ba59d80ca0d0c98986e : FAILURE in 49m 55s (non-voting)\n- puppet-openstack-integration-6-scenario002-tempest-ubuntu-focal https://zuul.opendev.org/t/openstack/build/316b68e52c294411949f845b166ae097 : FAILURE in 46m 43s (non-voting)\n- puppet-openstack-integration-6-scenario003-tempest-ubuntu-focal https://zuul.opendev.org/t/openstack/build/8c9682851cc342bf891eec8c3b46b43f : FAILURE in 51m 31s (non-voting)\n- puppet-openstack-integration-6-scenario004-tempest-ubuntu-focal https://zuul.opendev.org/t/openstack/build/1baefd048c3c4533be3ddf72ca199715 : FAILURE in 40m 34s (non-voting)\n- puppet-openstack-integration-7-scenario000-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/8969a05695f74b2c891d202c93f65a69 : SUCCESS in 25m 55s (non-voting)\n- puppet-openstack-integration-7-scenario001-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/6019a52e56844adbb5d97c605800ea0c : SUCCESS in 1h 25m 56s\n- puppet-openstack-integration-7-scenario002-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/0f91671354c047509f6072fda91c80c7 : SUCCESS in 1h 43m 03s\n- puppet-openstack-integration-7-scenario003-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/704f969796774187bed7a118be8f454d : SUCCESS in 1h 24m 39s\n- puppet-openstack-integration-7-scenario004-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/6203c2fff5044190903b9ff0d242fb02 : SUCCESS in 1h 39m 30s\n- puppet-openstack-litmus-6.21-ubuntu-focal https://zuul.opendev.org/t/openstack/build/33614f7475934a7f9c57a2044b0a2a96 : SUCCESS in 9m 58s (non-voting)\n- puppet-openstack-litmus-7.8-centos-8-stream https://zuul.opendev.org/t/openstack/build/9fe02b37fbbb48d29a8aba2bc4b8997e : SUCCESS in 18m 53s\n- puppet-openstack-litmus-7.8-ubuntu-focal https://zuul.opendev.org/t/openstack/build/2c021dcd936841bf810cef0cd39c319a : SUCCESS in 7m 20s (non-voting)\n- puppet-openstack-litmus-latest-ubuntu-focal https://zuul.opendev.org/t/openstack/build/7934b247672747f595e0e02b796de9cf : SUCCESS in 9m 48s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"a56f71dbfb69f9220e4f58eaee91a68c82f32787","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2022-08-30 16:21:12.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"429e7da054853bd251118bb2de1843241bd1a571","author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"date":"2022-09-01 09:13:40.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"4a609566b2bda1f45b6bf8035fb267d9e15a890a","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-09-01 09:14:45.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"f4ced392c759af2863e83018c0bd3a7396c9604d","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-09-01 10:31:26.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/a0815133779a49a5af1476ac7003e1ad\n\n- puppet-openstack-lint-ubuntu-focal https://zuul.opendev.org/t/openstack/build/f651b222c61f4c90bb89310e91035a11 : SUCCESS in 3m 34s\n- puppet-openstack-syntax-6-ubuntu-focal https://zuul.opendev.org/t/openstack/build/89acf4142b034555ad783862c258c7e6 : SUCCESS in 3m 30s\n- puppet-openstack-syntax-7-ubuntu-focal https://zuul.opendev.org/t/openstack/build/839ebcdf072648a6ad3ccda9dcb282f5 : SUCCESS in 4m 01s\n- puppet-openstack-unit-6.21-centos-8-stream https://zuul.opendev.org/t/openstack/build/86a3786d55cf42aab1ab250b23b7233c : SUCCESS in 13m 07s\n- puppet-openstack-unit-7.8-centos-8-stream https://zuul.opendev.org/t/openstack/build/5f68eec759c44b548b734d272b2521b3 : SUCCESS in 15m 00s\n- puppet-openstack-unit-6.21-ubuntu-focal https://zuul.opendev.org/t/openstack/build/85bb49fbd88149cba68e7d3949acf743 : SUCCESS in 9m 13s\n- puppet-openstack-unit-7.8-ubuntu-focal https://zuul.opendev.org/t/openstack/build/94692d966af344ae8579a21f218968ac : SUCCESS in 9m 22s\n- puppet-openstack-integration-7-scenario001-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/70e95749847040e4aa7714e6ce0a0833 : SUCCESS in 1h 05m 32s\n- puppet-openstack-integration-7-scenario002-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/a8a19c2d2bc8423fa0938e6036c0c231 : SUCCESS in 1h 02m 27s\n- puppet-openstack-integration-7-scenario003-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/d03f18079b7549c9a834db638d4d5d01 : SUCCESS in 1h 06m 52s\n- puppet-openstack-integration-7-scenario004-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/80b4722af5c04b97a84cb29c17f3136e : SUCCESS in 1h 00m 41s\n- puppet-openstack-litmus-7.8-centos-8-stream https://zuul.opendev.org/t/openstack/build/29dca1e5d3d446d6969cab7e2199a646 : SUCCESS in 15m 52s\n- puppet-openstack-litmus-6.21-ubuntu-focal https://zuul.opendev.org/t/openstack/build/58d3b38a5bce400a912cd5a2ca009111 : SUCCESS in 7m 07s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"cdfeceabe8f630fb5c1aae31ebb41f73231d48ca","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-09-01 10:31:27.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"d8d7339ff122f947a176db2b41160edbfdb65bf0","revisions":{"d8d7339ff122f947a176db2b41160edbfdb65bf0":{"kind":"REWORK","_number":1,"created":"2022-08-29 16:36:27.000000000","uploader":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"ref":"refs/changes/98/854998/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-vswitch","ref":"refs/changes/98/854998/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-vswitch refs/changes/98/854998/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-vswitch refs/changes/98/854998/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-vswitch refs/changes/98/854998/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-vswitch refs/changes/98/854998/1"}}},"commit":{"parents":[{"commit":"f5ee885fffca3d079a014fb19aa96b5579375a6b","subject":"Do not use service resource to restart openvswitch service","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-vswitch/commit/f5ee885fffca3d079a014fb19aa96b5579375a6b"}]}],"author":{"name":"Takashi Kajinami","email":"tkajinam@redhat.com","date":"2022-08-23 16:14:17.000000000","tz":540},"committer":{"name":"Takashi Kajinami","email":"tkajinam@redhat.com","date":"2022-08-29 16:36:27.000000000","tz":0},"subject":"vs_config: Fix broken idempotency with nil/empty value","message":"vs_config: Fix broken idempotency with nil/empty value\n\nIf the value is nil or empty then the option should NOT exist. This\nchange fixes the logic to determine whether the option already exists\nwhich is causing broken idempotency.\n\nCloses-Bug: #1987419\nChange-Id: I56f85ca35f8e9ee4e5e8d07c714b65f655b1e9ae\n(cherry picked from commit 10202ad22f70cbdf00401ff065f857edb500cc57)\n(cherry picked from commit f659ea896e330462ab880b73a0aefea7cb5ded52)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-vswitch/commit/d8d7339ff122f947a176db2b41160edbfdb65bf0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-vswitch/commit/d8d7339ff122f947a176db2b41160edbfdb65bf0"}]},"branch":"refs/heads/stable/xena"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"}}]}],"submit_requirements":[]}