)]}'
{"id":"openstack%2Fpuppet-vswitch~855002","triplet_id":"openstack%2Fpuppet-vswitch~stable%2Fwallaby~I56f85ca35f8e9ee4e5e8d07c714b65f655b1e9ae","project":"openstack/puppet-vswitch","branch":"stable/wallaby","topic":"bug/1987419-stable/wallaby","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:47:05.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 23:57:30.000000000","updated":"2022-09-01 10:47:05.000000000","submitted":"2022-09-01 10:47: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":"855002-bug/1987419-stable/wallaby","cherry_pick_of_change":854998,"cherry_pick_of_patch_set":1,"meta_rev_id":"90f180a150fdce92eab2529f449ac9401e6e1bcc","_number":855002,"virtual_id_number":855002,"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":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2022-09-01 10:47:05.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"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"}],"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":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2022-08-30 16:21:14.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:44.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"}],"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":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":1,"date":"2022-09-01 09:13:44.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"}],"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-30 01:29:53.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:14.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:44.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":"fa0676afc417e3da33bd5cb61a5f290f3d277e2e","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2022-08-29 23:57:30.000000000","message":"Patch Set 1: Cherry Picked from branch stable/xena.","accounts_in_message":[],"_revision_number":1},{"id":"50571be87cfcd1e1ff5b62aec4126ee5155365cb","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2022-08-29 23:57:37.000000000","message":"Topic bug/1987419-stable/xena-stable/wallaby removed","accounts_in_message":[],"_revision_number":1},{"id":"54da735a63b1b043c62c43809608a2e91b943a49","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2022-08-29 23:57:42.000000000","message":"Topic set to bug/1987419-stable/wallaby","accounts_in_message":[],"_revision_number":1},{"id":"48606f7fa32d4e4c988be8384dda4a37639310d9","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2022-08-29 23:58:22.000000000","message":"Patch Set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"2b9d650b4236eb141265b261fb3a6594632d8c50","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-08-30 01:29:53.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/360731765b124b278fbbe79988b3b501\n\n- puppet-openstack-lint-ubuntu-focal https://zuul.opendev.org/t/openstack/build/1a7346a4b6124160a965fbbccba45f5b : SUCCESS in 3m 30s\n- puppet-openstack-syntax-6-ubuntu-focal https://zuul.opendev.org/t/openstack/build/decbb719dbef4b3a8e06bacbc4c72153 : SUCCESS in 3m 13s\n- puppet-openstack-unit-6.21-centos-8-stream https://zuul.opendev.org/t/openstack/build/0b026dd93db545bb9ac02ec32feed505 : SUCCESS in 12m 25s\n- puppet-openstack-unit-6.21-ubuntu-focal https://zuul.opendev.org/t/openstack/build/6a19b7b08d9c450996701dea9f25fa48 : SUCCESS in 8m 55s\n- puppet-openstack-unit-latest-ubuntu-focal https://zuul.opendev.org/t/openstack/build/29fb9018f79b4f69b5639ad3a31acf3a : SUCCESS in 9m 31s (non-voting)\n- puppet-openstack-integration-6-scenario001-tempest-ubuntu-focal https://zuul.opendev.org/t/openstack/build/68491995799f4d6188ca723af00b6caf : FAILURE in 30m 23s (non-voting)\n- puppet-openstack-integration-6-scenario002-tempest-ubuntu-focal https://zuul.opendev.org/t/openstack/build/4defaaacd0764edc813ad6fffec06815 : FAILURE in 1h 01m 15s (non-voting)\n- puppet-openstack-integration-6-scenario003-tempest-ubuntu-focal https://zuul.opendev.org/t/openstack/build/0bac6152c0ce4ad5a714a780d66643e4 : SUCCESS in 53m 21s (non-voting)\n- puppet-openstack-integration-6-scenario004-tempest-ubuntu-focal https://zuul.opendev.org/t/openstack/build/1bb3bdb351574545a131c1b828053883 : SUCCESS in 41m 19s (non-voting)\n- puppet-openstack-integration-6-scenario001-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/24bc6c2799244184b0004a9a468f8ca4 : SUCCESS in 1h 26m 43s\n- puppet-openstack-integration-6-scenario002-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/8a2fdd77a5d848c3a9c02c35d38379eb : SUCCESS in 1h 24m 44s\n- puppet-openstack-integration-6-scenario003-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/d4ea088af3b74e5ea2df588de5a0aed3 : SUCCESS in 1h 24m 30s\n- puppet-openstack-integration-6-scenario004-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/4e75255378274512b8a1e4766dda7e25 : SUCCESS in 1h 03m 41s\n- puppet-openstack-litmus-centos-8-stream https://zuul.opendev.org/t/openstack/build/32c672948e1b4aa38a941538b96c551d : SUCCESS in 18m 57s\n- puppet-openstack-litmus-ubuntu-focal https://zuul.opendev.org/t/openstack/build/8beebb337bdc4dfe882b3e2e63056598 : SUCCESS in 9m 45s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"e7b0819fc8e01f41c80e049432143d3ba6410ea9","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2022-08-30 16:21:14.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"6b550dea5a483a7c7c34c343daa12d5d79eaf218","author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"date":"2022-09-01 09:13:44.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"bbf3ceaa1d964b58f5315a423e7b0a3f846f03fb","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-09-01 09:14:52.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"5283259e9ea66af058cd2d189f856423600195e1","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-09-01 10:47:05.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/10dce082ffee461fa784ec5de145dfa7\n\n- puppet-openstack-lint-ubuntu-focal https://zuul.opendev.org/t/openstack/build/56a270ff1fd043f48155583138ff39f8 : SUCCESS in 3m 56s\n- puppet-openstack-syntax-6-ubuntu-focal https://zuul.opendev.org/t/openstack/build/09da8597a3644246bab2979e97472cc2 : SUCCESS in 6m 04s\n- puppet-openstack-unit-6.21-centos-8-stream https://zuul.opendev.org/t/openstack/build/60aae9d6bf324e9581c81e0e05df7606 : SUCCESS in 22m 05s\n- puppet-openstack-unit-6.21-ubuntu-focal https://zuul.opendev.org/t/openstack/build/697155efe54448349e41db63ae01e6a9 : SUCCESS in 18m 21s\n- puppet-openstack-integration-6-scenario001-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/17d8c6e73b6f4ef2a271ffa637a0c103 : SUCCESS in 1h 21m 41s\n- puppet-openstack-integration-6-scenario002-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/435a5c7093f24846b81654b5219b57a3 : SUCCESS in 1h 22m 49s\n- puppet-openstack-integration-6-scenario003-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/e2cf3e8daa7d4c488564894ce06340ab : SUCCESS in 1h 24m 38s\n- puppet-openstack-integration-6-scenario004-tempest-centos-8-stream https://zuul.opendev.org/t/openstack/build/e2b8b516d4f14c2290093fff1709bcb4 : SUCCESS in 1h 06m 13s\n- puppet-openstack-litmus-centos-8-stream https://zuul.opendev.org/t/openstack/build/2f81d8c64dbe4b2cbeb142e1df8988af : SUCCESS in 29m 44s\n- puppet-openstack-litmus-ubuntu-focal https://zuul.opendev.org/t/openstack/build/ff4a79cb98af4c0fa332322a74d0c25d : SUCCESS in 19m 07s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"90f180a150fdce92eab2529f449ac9401e6e1bcc","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-09-01 10:47:05.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"33df2d222b7c9725eb49a195db606a765964d305","revisions":{"633e3cee18d491539984161f122e51a0da4f4644":{"kind":"REWORK","_number":1,"created":"2022-08-29 23:57:30.000000000","uploader":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"ref":"refs/changes/02/855002/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-vswitch","ref":"refs/changes/02/855002/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-vswitch refs/changes/02/855002/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-vswitch refs/changes/02/855002/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-vswitch refs/changes/02/855002/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-vswitch refs/changes/02/855002/1"}}},"commit":{"parents":[{"commit":"ee7c31bf5f3f8e5188df0ec2b201b31d47c8d923","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/ee7c31bf5f3f8e5188df0ec2b201b31d47c8d923"}]}],"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 23:57:30.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/633e3cee18d491539984161f122e51a0da4f4644"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-vswitch/commit/633e3cee18d491539984161f122e51a0da4f4644"}]},"branch":"refs/heads/stable/wallaby"},"33df2d222b7c9725eb49a195db606a765964d305":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2022-08-29 23:58:22.000000000","uploader":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"ref":"refs/changes/02/855002/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-vswitch","ref":"refs/changes/02/855002/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-vswitch refs/changes/02/855002/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-vswitch refs/changes/02/855002/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-vswitch refs/changes/02/855002/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-vswitch refs/changes/02/855002/2"}}},"commit":{"parents":[{"commit":"ee7c31bf5f3f8e5188df0ec2b201b31d47c8d923","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/ee7c31bf5f3f8e5188df0ec2b201b31d47c8d923"}]}],"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 23:58:22.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(cherry picked from commit d8d7339ff122f947a176db2b41160edbfdb65bf0)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-vswitch/commit/33df2d222b7c9725eb49a195db606a765964d305"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-vswitch/commit/33df2d222b7c9725eb49a195db606a765964d305"}]},"branch":"refs/heads/stable/wallaby","description":"Edit commit message"}},"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":[]}