)]}'
{"id":"openstack%2Fdevstack-gate~61137","triplet_id":"openstack%2Fdevstack-gate~master~Ic4122e6c91840ef8117b73add38147ae43753716","project":"openstack/devstack-gate","branch":"master","topic":"bug1254520","hashtags":[],"change_id":"Ic4122e6c91840ef8117b73add38147ae43753716","subject":"Set ovs-vsctl timeout to 10 seconds","status":"ABANDONED","created":"2013-12-10 14:45:42.000000000","updated":"2013-12-12 19:11:19.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"9caf5f90fb41f817f7df681003315e1ec30c8aca","_number":61137,"virtual_id_number":61137,"owner":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":-1,"date":"2013-12-10 15:34:12.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":{"disliked":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-12-11 11:03:40.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"value":-1,"date":"2013-12-12 02:07:03.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"value":0,"date":"2013-12-10 15:00:43.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-12-10 15:00:43.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2013-12-10 15:34:12.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2013-12-12 02:07:03.000000000","updated_by":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"reviewer":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"state":"REVIEWER"}],"messages":[{"id":"44de3718c95f2c87ee2bdd52a251503bd3fce9f6","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-12-10 15:00:43.000000000","message":"Patch Set 1:\n\nquestion: would it be better to do this in devstack as a default? is there a reason d-g is different than default?","accounts_in_message":[],"_revision_number":1},{"id":"7112fbf29b1b0db13abf2aad12573cce5a34db96","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-10 15:34:12.000000000","message":"Patch Set 1: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- check-dg-tempest-dsvm-full http://logs.openstack.org/37/61137/1/check/check-dg-tempest-dsvm-full/2eea1e2 : SUCCESS in 35m 44s\n- check-dg-tempest-dsvm-full-reexec http://logs.openstack.org/37/61137/1/check/check-dg-tempest-dsvm-full-reexec/bcb0794 : SUCCESS in 48m 25s\n- check-tempest-dsvm-full http://logs.openstack.org/37/61137/1/check/check-tempest-dsvm-full/cc810da : FAILURE in 33m 42s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/37/61137/1/check/check-tempest-dsvm-postgres-full/b919f0a : SUCCESS in 33m 33s\n- check-tempest-dsvm-neutron http://logs.openstack.org/37/61137/1/check/check-tempest-dsvm-neutron/a5a14e6 : SUCCESS in 30m 38s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/37/61137/1/check/gate-tempest-dsvm-large-ops/a366f3e : SUCCESS in 13m 03s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/37/61137/1/check/gate-tempest-dsvm-neutron-large-ops/313ba6e : SUCCESS in 16m 28s\n- check-grenade-dsvm http://logs.openstack.org/37/61137/1/check/check-grenade-dsvm/aaf106a : SUCCESS in 27m 30s\n- check-swift-dsvm-functional http://logs.openstack.org/37/61137/1/check/check-swift-dsvm-functional/ed5ce54 : SUCCESS in 13m 08s\n- check-tempest-dsvm-full-grizzly http://logs.openstack.org/37/61137/1/check/check-tempest-dsvm-full-grizzly/3c8c2c9 : SUCCESS in 35m 20s (non-voting)\n- check-tempest-dsvm-neutron-grizzly http://logs.openstack.org/37/61137/1/check/check-tempest-dsvm-neutron-grizzly/75f6cbb : FAILURE in 6m 04s (non-voting)\n- check-tempest-dsvm-postgres-full-grizzly http://logs.openstack.org/37/61137/1/check/check-tempest-dsvm-postgres-full-grizzly/14312a2 : SUCCESS in 39m 47s (non-voting)\n- check-devstack-dsvm-cells-grizzly http://logs.openstack.org/37/61137/1/check/check-devstack-dsvm-cells-grizzly/011a032 : SUCCESS in 11m 26s (non-voting)\n- check-tempest-dsvm-cells-grizzly http://logs.openstack.org/37/61137/1/check/check-tempest-dsvm-cells-grizzly/665fa5c : SUCCESS in 16m 03s (non-voting)\n- check-tempest-dsvm-cells-full-grizzly http://logs.openstack.org/37/61137/1/check/check-tempest-dsvm-cells-full-grizzly/f13f6ac : SUCCESS in 34m 54s (non-voting)\n- check-tempest-dsvm-full-havana http://logs.openstack.org/37/61137/1/check/check-tempest-dsvm-full-havana/b53f7c7 : SUCCESS in 31m 48s (non-voting)\n- check-tempest-dsvm-neutron-havana http://logs.openstack.org/37/61137/1/check/check-tempest-dsvm-neutron-havana/1a647d5 : SUCCESS in 30m 01s (non-voting)\n- check-tempest-dsvm-postgres-full-havana http://logs.openstack.org/37/61137/1/check/check-tempest-dsvm-postgres-full-havana/6c625bb : SUCCESS in 35m 09s (non-voting)\n- check-devstack-dsvm-cells-havana http://logs.openstack.org/37/61137/1/check/check-devstack-dsvm-cells-havana/a893f4f : SUCCESS in 11m 31s (non-voting)\n- check-tempest-dsvm-cells-havana http://logs.openstack.org/37/61137/1/check/check-tempest-dsvm-cells-havana/4649629 : FAILURE in 14m 14s (non-voting)\n- check-tempest-dsvm-cells-full-havana http://logs.openstack.org/37/61137/1/check/check-tempest-dsvm-cells-full-havana/c4711bf : FAILURE in 23m 57s (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"1e72f675487d6985fff5b693e80de2b17370abbf","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-12-11 11:03:40.000000000","message":"Patch Set 1:\n\nI just thought people did not want the devstack default changed; the value proposed in this patch is the lowest timeout that I see never failing on the tests I\u0027ve done on the gate.\n\nBut I have no strong reason for having the patch either in devstack-gate or devstack; if we deem ok changing the devstack default I\u0027m ok with that.","accounts_in_message":[],"_revision_number":1},{"id":"5b62d167977819f519a6e553cb95d831047ee8bd","author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"date":"2013-12-12 02:07:03.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nI feel very strongly that we should be testing defaults except when the test case itself is about testing something extraordinary.\n\n-\u003e This shouldn\u0027t be in *either* devstack or devstack-gate : it should be in Neutron.","accounts_in_message":[],"_revision_number":1},{"id":"84a20de5d2c6fe3290d0b4fba6c09ed1b64c671a","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-12-12 17:44:06.000000000","message":"Patch Set 1:\n\nRobert,\n\nIn real-life deployments we probably don\u0027t need to set the timeout to 10.\nA real-life deployment is one whether the dhcp agent, and the l3 agent are on distinct host, and the load on vswitchd and ovsdbserver is much less than what currently happens when running single-host tempest smoke tests with parallelism and full tenant isolation. This is why I initially proposed to increase the tests just for the gate tests.\n\nKeeping this in mind, we\u0027re actually splitting hairs here; if nobody from the neutron\u0027s core team objects to increasing the default to 10 I\u0027ll do that there.\n\nOn the other hand I don\u0027t see what would be the negative effect of increasing the default ovs timeout.","accounts_in_message":[],"_revision_number":1},{"id":"9bf2f166fc78453ba68034234b44abfe63df7674","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-12-12 17:44:12.000000000","message":"Patch Set 1: Abandoned","accounts_in_message":[],"_revision_number":1},{"id":"69e606d9c1e91def6e96dd829298004c52b9208d","author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"date":"2013-12-12 19:11:19.000000000","message":"Patch Set 1:\n\nCool - thanks. That said, I don\u0027t understand why the load would be a lot lower: tempest is only equivalent to 10-20 tenants, real world is *much* busier :)","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"c193f8b442d6885711bde25027ceb9dcab179b27","revisions":{"c193f8b442d6885711bde25027ceb9dcab179b27":{"kind":"REWORK","_number":1,"created":"2013-12-10 14:45:42.000000000","uploader":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"ref":"refs/changes/37/61137/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/37/61137/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/37/61137/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/37/61137/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/37/61137/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/37/61137/1"}}},"commit":{"parents":[{"commit":"365ef9093f5351fed514415aab8d98217b3ee50e","subject":"Merge \"Use select-mirror in devstack-gate\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/365ef9093f5351fed514415aab8d98217b3ee50e"}]}],"author":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-12-10 14:40:29.000000000","tz":-480},"committer":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-12-10 14:40:29.000000000","tz":-480},"subject":"Set ovs-vsctl timeout to 10 seconds","message":"Set ovs-vsctl timeout to 10 seconds\n\nThis patch applies only when neutron is enabled\n\nChange-Id: Ic4122e6c91840ef8117b73add38147ae43753716\nCloses-Bug: #1254520\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/c193f8b442d6885711bde25027ceb9dcab179b27"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/c193f8b442d6885711bde25027ceb9dcab179b27"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}