)]}'
{"id":"openstack%2Ftripleo-validations~744857","triplet_id":"openstack%2Ftripleo-validations~stable%2Ftrain~Ifacbbd6a493019e606105f45f0302bf1a88bed62","project":"openstack/tripleo-validations","branch":"stable/train","hashtags":[],"change_id":"Ifacbbd6a493019e606105f45f0302bf1a88bed62","subject":"Fix neutron_sanity_check for ML2/OVS overcloud","status":"MERGED","created":"2020-08-05 08:43:42.000000000","updated":"2020-08-07 16:19:32.000000000","submitted":"2020-08-07 16:19:32.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":"744857-1596817172954-88ec3396","meta_rev_id":"a43fff48abb013bd06745594d02882d5223ba996","_number":744857,"virtual_id_number":744857,"owner":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-08-07 16:19:32.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":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},{"value":0,"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"value":0,"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"}],"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":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2020-08-07 09:15:29.000000000","_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},{"value":0,"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"value":2,"date":"2020-08-07 13:17:48.000000000","_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"}],"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":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":0,"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},{"value":0,"date":"2020-08-07 09:18:41.000000000","_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"value":1,"date":"2020-08-07 13:17:48.000000000","_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"date":"2020-08-07 09:18:38.000000000","_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"date":"2020-08-07 09:19:03.000000000","_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-08-07 09:15:29.000000000","updated_by":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"reviewer":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"state":"REVIEWER"},{"updated":"2020-08-07 09:18:38.000000000","updated_by":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"reviewer":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"state":"REVIEWER"},{"updated":"2020-08-07 09:18:41.000000000","updated_by":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"reviewer":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"state":"REVIEWER"},{"updated":"2020-08-07 09:19:03.000000000","updated_by":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"reviewer":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"state":"REVIEWER"},{"updated":"2020-08-07 13:17:48.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":"2020-08-07 16:19:32.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":"23a2a1724271c30b12645513b9f84de31cadc716","author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"date":"2020-08-05 08:43:42.000000000","message":"Patch Set 1: Cherry Picked from branch master.","accounts_in_message":[],"_revision_number":1},{"id":"3e852ec22322a11d50316cb7ff509d6c720e1c80","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-05 12:35:49.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tripleo-ci-centos-8-containers-multinode https://zuul.opendev.org/t/openstack/build/b3c75819eb6242b8b8009c96baf086bb : SUCCESS in 2h 07m 45s\n- tripleo-ci-centos-7-containers-multinode https://zuul.opendev.org/t/openstack/build/54b702fca1994e5aa40d10142c162261 : SUCCESS in 2h 54m 14s\n- tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates https://zuul.opendev.org/t/openstack/build/efbb0f554bf24d1b853e5da528d3a542 : SUCCESS in 2h 04m 14s\n- tripleo-ci-centos-7-scenario000-multinode-oooq-container-upgrades https://zuul.opendev.org/t/openstack/build/63a9de0b2b3e4b5dab0d6306f8415a78 : SUCCESS in 2h 23m 10s (non-voting)\n- tripleo-ci-centos-8-scenario000-multinode-oooq-container-updates-train https://zuul.opendev.org/t/openstack/build/74a10f2038224985b02d690220e51613 : TIMED_OUT in 3h 36m 31s (non-voting)\n- tripleo-ci-centos-8-standalone https://zuul.opendev.org/t/openstack/build/ea9cb2862dea4d8fa4b9551868896dd7 : SUCCESS in 1h 33m 06s\n- tripleo-ci-centos-7-standalone-upgrade-train https://zuul.opendev.org/t/openstack/build/eb615c1e579c49229cab39b7ed89eac6 : SUCCESS in 2h 38m 27s (non-voting)\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/36ba32f5ca2949c1bf7521ca0c4a3c8d : SUCCESS in 4m 52s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/08b4ffd264914ff4813cfc9127108dd9 : SUCCESS in 7m 03s\n- tripleo-validations-centos-7-molecule-neutron_sanity_check https://zuul.opendev.org/t/openstack/build/9a9a96dc27504cc38a4a9c9f7bc69563 : RETRY_LIMIT in 3m 11s (non-voting)\n- openstack-tox-linters https://zuul.opendev.org/t/openstack/build/0b4eefd6022c4f4f8df9705b56941e76 : SUCCESS in 7m 12s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/ad7c69c656e9405fa181dcdfb4cc4ab8 : SUCCESS in 6m 37s","accounts_in_message":[],"_revision_number":1},{"id":"63f37f0f8a5dd65b92aceb1ea5d1713c3b8b29f4","author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"date":"2020-08-07 09:15:29.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"064a74e04914112e02d995768604d0f7a2c56acf","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-08-07 13:17:48.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"0f59ca950e62eefd4a23a412ea7a09cf86b9c9b8","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-07 13:18:05.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"7af5856a5aee5f3fc218db4d4ea63e37c3f7104f","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-07 16:19:32.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- tripleo-ci-centos-7-containers-multinode https://zuul.opendev.org/t/openstack/build/4d0b009d53cf4661bdf53906e05d4b79 : SUCCESS in 2h 45m 18s\n- tripleo-ci-centos-8-containers-multinode https://zuul.opendev.org/t/openstack/build/4cdde836f7c8421c9bf0173758e680d8 : SUCCESS in 2h 58m 59s\n- tripleo-ci-centos-8-standalone https://zuul.opendev.org/t/openstack/build/f167631a9c054433a65d1f89be50c944 : SUCCESS in 1h 44m 02s\n- tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates https://zuul.opendev.org/t/openstack/build/1073709c2d2245b4b4b3df429febe182 : SUCCESS in 2h 01m 47s\n- tripleo-ci-centos-7-standalone-upgrade-train https://zuul.opendev.org/t/openstack/build/8787ae70bf284954b56ae4aa3b10a16a : SUCCESS in 2h 13m 09s (non-voting)\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/7a4979615a054285ba770a1b8e800110 : SUCCESS in 7m 36s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/24ae194060e743359e8adfb3ed8eb18d : SUCCESS in 7m 09s\n- openstack-tox-linters https://zuul.opendev.org/t/openstack/build/ea17e0f5505d4d5f9b7d598f1676fa40 : SUCCESS in 6m 28s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/95e35750c2ad41b9b805a82cf8d6bed4 : SUCCESS in 7m 06s","accounts_in_message":[],"_revision_number":1},{"id":"d92db22bd73ff2d96aadcaa844aa68fb580256a7","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-07 16:19:32.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"f8d7882f3343fa4cae843b6c20f1606cf768a16d","revisions":{"f8d7882f3343fa4cae843b6c20f1606cf768a16d":{"kind":"REWORK","_number":1,"created":"2020-08-05 08:43:42.000000000","uploader":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"ref":"refs/changes/57/744857/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tripleo-validations","ref":"refs/changes/57/744857/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tripleo-validations refs/changes/57/744857/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tripleo-validations refs/changes/57/744857/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tripleo-validations refs/changes/57/744857/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tripleo-validations refs/changes/57/744857/1"}}},"commit":{"parents":[{"commit":"5dcaa10e894438fbc3b0cd6ba48a2407081d8613","subject":"Make Get OSD stat percentage compatible with jq \u003c 1.5","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-validations/commit/5dcaa10e894438fbc3b0cd6ba48a2407081d8613"}]}],"author":{"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","date":"2020-07-31 12:02:00.000000000","tz":120},"committer":{"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","date":"2020-08-05 08:43:42.000000000","tz":0},"subject":"Fix neutron_sanity_check for ML2/OVS overcloud","message":"Fix neutron_sanity_check for ML2/OVS overcloud\n\nPreviously validator failed running on neutron_api as the\nneutron-sanity-check binary requires additional capabilities, causing it\nto alway fail [0]. In addition, in some cases a race condition in\noslo_privsep can cause the validator to hang indefinitely [1].\n\nThis fixes it if the overcloud uses ML2/OVS, using the neutron_ovs_agent\ncontainer as we already did on the undercloud, and skipping the check if\nit cannot be found (ML2/OVN deployments typically).\n\nAdditional fixes to the validator:\n* skip gather_facts step\n* fix output parsing\n* call neutron-sanity-check only once, with all configuration files\n  passed to it\n* drop redundant default values in playbook itself\n\n[0] https://bugzilla.redhat.com/show_bug.cgi?id\u003d1783195\n[1] https://bugzilla.redhat.com/show_bug.cgi?id\u003d1862364\n\nChange-Id: Ifacbbd6a493019e606105f45f0302bf1a88bed62\nCo-Authored-By: Cédric Jeanneret \u003ccjeanner@redhat.com\u003e\n(cherry picked from commit 3561628f2aac62f27ba6f6f97f814366be7e8993)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-validations/commit/f8d7882f3343fa4cae843b6c20f1606cf768a16d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-validations/commit/f8d7882f3343fa4cae843b6c20f1606cf768a16d"}]},"branch":"refs/heads/stable/train"}},"requirements":[],"submit_records":[],"submit_requirements":[]}