)]}'
{"id":"openstack%2Fovsdbapp~718166","triplet_id":"openstack%2Fovsdbapp~master~I24c2f1adc17aa02fe1042834d08fd23719a50fd8","project":"openstack/ovsdbapp","branch":"master","hashtags":[],"change_id":"I24c2f1adc17aa02fe1042834d08fd23719a50fd8","subject":"Stop iterating on first matching QoS rule","status":"MERGED","created":"2020-04-07 16:28:44.000000000","updated":"2020-04-08 20:20:35.000000000","submitted":"2020-04-08 20:17:20.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"718166-1586377040657-7fd7f7c7","meta_rev_id":"f142472baa83aa3d300272746f84a411c8589e01","_number":718166,"virtual_id_number":718166,"owner":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-04-08 20:17:20.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"}],"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":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"all":[{"value":0,"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2020-04-08 16:35:48.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"}],"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":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"all":[{"value":1,"date":"2020-04-08 17:19:02.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-04-08 16:35:48.000000000","updated_by":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"reviewer":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"state":"REVIEWER"},{"updated":"2020-04-08 20:17:20.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"90cd95599cae897151e7a95e76d1642975fcdd06","author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"date":"2020-04-07 16:28:44.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"9e828ac492df039dcdde3e4d2f7f1fc66ac14af5","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2020-04-07 17:36:13.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"a95004535042b5e85b4ec3ca8f0235ddec0530d3","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-07 18:41:22.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/284892b6898e4de2811b8b41c7f37d02 : SUCCESS in 5m 52s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/66191bb3591f40efb548c51b81d706d4 : SUCCESS in 7m 11s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/64f561eb362e4462af27223315370754 : SUCCESS in 6m 11s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/6dbdf3415ef94b4d9b0162e24ec495f9 : SUCCESS in 9m 01s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/adb71c2ea3904d0aaba2607a68fd96fa : SUCCESS in 6m 05s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/9cab8403682744e9a312b8802aae5c0c : SUCCESS in 8m 04s\n- openstack-tox-functional https://zuul.opendev.org/t/openstack/build/b315718f5b754962a61520a91eb27c83 : SUCCESS in 9m 55s\n- ovsdbapp-tempest-dsvm-networking-ovn-ovs-release https://zuul.opendev.org/t/openstack/build/2d8056802538406ca1922a17a9c48327 : SUCCESS in 2h 06m 45s\n\nWarning:\n  Comments left for invalid file ovsdbapp/backend/ovs_idl/idlutils.py","accounts_in_message":[],"_revision_number":1},{"id":"451cfb6465494344f12c023b0be61c10d83b4f2c","author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"date":"2020-04-07 19:07:36.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"5e8117b2f0aca7bbf25ee09c52014eaf617320e9","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2020-04-07 19:34:35.000000000","message":"Patch Set 1: Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"a3114baf02f5a867542932e4baef4a63e98fe341","author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"date":"2020-04-07 19:37:08.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"8e11f66c8cc2b1f873abaed341311404f435cb97","author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"date":"2020-04-07 19:40:29.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"c5053bad6ebbe35d9a5d5d6ede0b70e6298b53df","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-07 23:30:36.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/68cf4cbcfd89492f9a6c39345ef8c66c : SUCCESS in 5m 57s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c248e867d29d401e89d6f785a7572aea : SUCCESS in 5m 59s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/081df87355874c71a3c53370babcadf1 : SUCCESS in 6m 28s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/ff21d656d324499a86868e3d999b6332 : SUCCESS in 5m 31s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/65dd20ece943497a82ff0303dfa748b8 : SUCCESS in 7m 08s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2b3ce51ef3754928a75c45626199830e : SUCCESS in 7m 45s\n- openstack-tox-functional https://zuul.opendev.org/t/openstack/build/b070b5e50f634634a27eef968b7278e7 : SUCCESS in 9m 35s\n- ovsdbapp-tempest-dsvm-networking-ovn-ovs-release https://zuul.opendev.org/t/openstack/build/6d9e364f86f54388a8277bbf27cb3555 : SUCCESS in 3h 07m 19s\n\nWarning:\n  Comments left for invalid file ovsdbapp/backend/ovs_idl/idlutils.py","accounts_in_message":[],"_revision_number":2},{"id":"10e4615d78191d499a949f316f91b9d452c1da54","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2020-04-08 16:35:48.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"2f708a5398d89fc7d18e4aeb7f1e83e42cab19ee","author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"date":"2020-04-08 17:19:02.000000000","message":"Patch Set 2: Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"cf1b5d6d52c20022d8d3e7bde5afc0049b01ea89","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-08 17:19:19.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"0eab513a7478e5eb774ec5b11c861f652d1ac203","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-08 20:17:20.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2},{"id":"00e815450a88e11e051a721c0c26e3740548315f","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-08 20:17:20.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/ce9cfd73a79a4e89908836de1e441d8f : SUCCESS in 5m 46s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/8dafdaadda3146f6882e27eb5898f421 : SUCCESS in 4m 38s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/0baff88215194197bfbd2b769f32b056 : SUCCESS in 5m 35s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/74030935577e4be9af53d6757a0cd220 : SUCCESS in 6m 49s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/594926220dd640b09973605707f1d947 : SUCCESS in 6m 52s\n- openstack-tox-functional https://zuul.opendev.org/t/openstack/build/8200011dc5cc4064984efe2004a2ebfd : SUCCESS in 9m 04s\n- ovsdbapp-tempest-dsvm-networking-ovn-ovs-release https://zuul.opendev.org/t/openstack/build/9af8723750e24c59bdf3979e5b093285 : SUCCESS in 2h 54m 45s\n\nWarning:\n  Comments left for invalid file ovsdbapp/backend/ovs_idl/idlutils.py","accounts_in_message":[],"_revision_number":2},{"id":"f142472baa83aa3d300272746f84a411c8589e01","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-08 20:20:35.000000000","message":"Patch Set 2:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/bc8ca116a44e482fbe04bc8b4fcc0fab : SUCCESS in 2m 36s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"c5163fcbea7cb07e0a9272c3d7b3f7e0ff48b9ac","revisions":{"81396283ac81cb03dd0961f1131a6af070fd5d08":{"kind":"REWORK","_number":1,"created":"2020-04-07 16:28:44.000000000","uploader":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"ref":"refs/changes/66/718166/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ovsdbapp","ref":"refs/changes/66/718166/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/66/718166/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/66/718166/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/66/718166/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ovsdbapp refs/changes/66/718166/1"}}},"commit":{"parents":[{"commit":"19d11f66aa5b9ccae9eb01a0e395f12fc613acf7","subject":"Merge \"Remove the dependency on the \"mock\" package\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/19d11f66aa5b9ccae9eb01a0e395f12fc613acf7"}]}],"author":{"name":"Terry Wilson","email":"twilson@redhat.com","date":"2020-04-07 16:08:30.000000000","tz":-300},"committer":{"name":"Terry Wilson","email":"twilson@redhat.com","date":"2020-04-07 16:27:58.000000000","tz":-300},"subject":"Stop iterating on first matching QoS rule","message":"Stop iterating on first matching QoS rule\n\nInstead of creating a list of all matching rules and returning the\nfirst one, iterate only until we find the first matching rule.\n\nRelated-Bug: 1871104\nChange-Id: I24c2f1adc17aa02fe1042834d08fd23719a50fd8\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/81396283ac81cb03dd0961f1131a6af070fd5d08"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/81396283ac81cb03dd0961f1131a6af070fd5d08"}]},"branch":"refs/heads/master"},"c5163fcbea7cb07e0a9272c3d7b3f7e0ff48b9ac":{"kind":"REWORK","_number":2,"created":"2020-04-07 19:40:29.000000000","uploader":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"ref":"refs/changes/66/718166/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ovsdbapp","ref":"refs/changes/66/718166/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/66/718166/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/66/718166/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ovsdbapp refs/changes/66/718166/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ovsdbapp refs/changes/66/718166/2"}}},"commit":{"parents":[{"commit":"19d11f66aa5b9ccae9eb01a0e395f12fc613acf7","subject":"Merge \"Remove the dependency on the \"mock\" package\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/19d11f66aa5b9ccae9eb01a0e395f12fc613acf7"}]}],"author":{"name":"Terry Wilson","email":"twilson@redhat.com","date":"2020-04-07 16:08:30.000000000","tz":-300},"committer":{"name":"Terry Wilson","email":"twilson@redhat.com","date":"2020-04-07 19:40:24.000000000","tz":-300},"subject":"Stop iterating on first matching QoS rule","message":"Stop iterating on first matching QoS rule\n\nInstead of creating a list of all matching rules and returning the\nfirst one, iterate only until we find the first matching rule.\n\nRelated-Bug: 1871104\nChange-Id: I24c2f1adc17aa02fe1042834d08fd23719a50fd8\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/c5163fcbea7cb07e0a9272c3d7b3f7e0ff48b9ac"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ovsdbapp/commit/c5163fcbea7cb07e0a9272c3d7b3f7e0ff48b9ac"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}