)]}'
{"id":"openstack%2Fpython-novaclient~418190","triplet_id":"openstack%2Fpython-novaclient~master~I22ee148581a94b153cf7e733563cfafaa56b1ffd","project":"openstack/python-novaclient","branch":"master","topic":"bug/1654806","hashtags":[],"change_id":"I22ee148581a94b153cf7e733563cfafaa56b1ffd","subject":"Fix functional tests to deal with multiple networks","status":"MERGED","created":"2017-01-10 03:14:40.000000000","updated":"2017-01-25 15:00:10.000000000","submitted":"2017-01-25 15:00:10.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"4a3e22cd6dc8498b92ebfe6fcf0f8ddaa8c237a0","_number":418190,"virtual_id_number":418190,"owner":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":2,"date":"2017-01-25 15:00:10.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},{"value":0,"_account_id":11034,"name":"Yanyan Hu","email":"huyanyan84@gmail.com","username":"yanyanhu"},{"value":0,"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},{"value":0,"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},{"value":0,"_account_id":13404,"name":"Alberto Planas","email":"aplanas@gmail.com","username":"aplanas"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"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":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"recommended":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"all":[{"value":1,"date":"2017-01-25 02:45:01.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":2,"date":"2017-01-24 23:52:11.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},{"value":1,"date":"2017-01-25 10:19:38.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11034,"name":"Yanyan Hu","email":"huyanyan84@gmail.com","username":"yanyanhu"},{"value":0,"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},{"value":0,"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},{"value":1,"date":"2017-01-25 09:21:52.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":13404,"name":"Alberto Planas","email":"aplanas@gmail.com","username":"aplanas"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":2,"date":"2017-01-25 14:01:23.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"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":{"approved":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"all":[{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},{"value":0,"_account_id":11034,"name":"Yanyan Hu","email":"huyanyan84@gmail.com","username":"yanyanhu"},{"value":0,"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},{"value":0,"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},{"value":0,"_account_id":13404,"name":"Alberto Planas","email":"aplanas@gmail.com","username":"aplanas"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":1,"date":"2017-01-25 14:01:23.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},{"value":0,"_account_id":11034,"name":"Yanyan Hu","email":"huyanyan84@gmail.com","username":"yanyanhu"},{"value":0,"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},{"value":0,"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},{"value":0,"_account_id":13404,"name":"Alberto Planas","email":"aplanas@gmail.com","username":"aplanas"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},{"_account_id":11034,"name":"Yanyan Hu","email":"huyanyan84@gmail.com","username":"yanyanhu"},{"_account_id":13404,"name":"Alberto Planas","email":"aplanas@gmail.com","username":"aplanas"},{"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-01-10 13:07:50.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":"2017-01-10 13:12:31.000000000","updated_by":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"reviewer":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"state":"REVIEWER"},{"updated":"2017-01-10 14:26:35.000000000","updated_by":{"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},"reviewer":{"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},"state":"REVIEWER"},{"updated":"2017-01-10 16:57:54.000000000","updated_by":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"reviewer":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"state":"REVIEWER"},{"updated":"2017-01-24 23:52:11.000000000","updated_by":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"reviewer":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"state":"REVIEWER"},{"updated":"2017-01-25 02:45:01.000000000","updated_by":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"reviewer":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"state":"REVIEWER"},{"updated":"2017-01-25 09:21:52.000000000","updated_by":{"_account_id":13404,"name":"Alberto Planas","email":"aplanas@gmail.com","username":"aplanas"},"reviewer":{"_account_id":13404,"name":"Alberto Planas","email":"aplanas@gmail.com","username":"aplanas"},"state":"REVIEWER"},{"updated":"2017-01-25 10:19:38.000000000","updated_by":{"_account_id":11034,"name":"Yanyan Hu","email":"huyanyan84@gmail.com","username":"yanyanhu"},"reviewer":{"_account_id":11034,"name":"Yanyan Hu","email":"huyanyan84@gmail.com","username":"yanyanhu"},"state":"REVIEWER"},{"updated":"2017-01-25 14:01:23.000000000","updated_by":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"reviewer":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"state":"REVIEWER"},{"updated":"2017-01-25 15:00:10.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"34d00906d1f956b26a0bad2d588fb9f4b2dd750a","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2017-01-10 03:14:40.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"7feeb77ecbe6e945bf1a350b6d6256b14494b54d","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2017-01-10 03:58:11.000000000","message":"Patch Set 1: Workflow-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"06f08fa910c4527db6ba7c56c8952112dadf23ba","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-01-10 04:28:01.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-python-novaclient-docs-ubuntu-xenial http://docs-draft.openstack.org/90/418190/1/check/gate-python-novaclient-docs-ubuntu-xenial/c629209//doc/build/html/ : SUCCESS in 2m 51s\n- gate-python-novaclient-pep8-ubuntu-xenial http://logs.openstack.org/90/418190/1/check/gate-python-novaclient-pep8-ubuntu-xenial/2153a3f/ : SUCCESS in 3m 06s\n- gate-python-novaclient-python27-ubuntu-xenial http://logs.openstack.org/90/418190/1/check/gate-python-novaclient-python27-ubuntu-xenial/d088ec5/ : SUCCESS in 1m 54s\n- gate-python-novaclient-python35 http://logs.openstack.org/90/418190/1/check/gate-python-novaclient-python35/7c53388/ : SUCCESS in 2m 16s\n- gate-tempest-dsvm-neutron-src-python-novaclient-ubuntu-xenial http://logs.openstack.org/90/418190/1/check/gate-tempest-dsvm-neutron-src-python-novaclient-ubuntu-xenial/1d9fa48/ : SUCCESS in 1h 12m 04s\n- gate-novaclient-dsvm-functional-mitaka-ubuntu-trusty-nv http://logs.openstack.org/90/418190/1/check/gate-novaclient-dsvm-functional-mitaka-ubuntu-trusty-nv/3a29049/ : FAILURE in 27m 23s (non-voting)\n- gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/90/418190/1/check/gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/ddf8812/ : FAILURE in 39m 43s (non-voting)\n- gate-novaclient-dsvm-functional-neutron-ubuntu-xenial http://logs.openstack.org/90/418190/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/2b844b8/ : FAILURE in 40m 23s","accounts_in_message":[],"_revision_number":1},{"id":"d68c155d6c39a71ca9cf094e9577e0a69ce34824","author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"date":"2017-01-10 08:17:00.000000000","message":"Patch Set 1: Code-Review-1\n\nSomething is really messed up if that change impacted you because that is for the pecan framework that we aren\u0027t even using yet. :)\n\nHit me up on IRC when you get a chance.","accounts_in_message":[],"_revision_number":1},{"id":"706680c6737bf601ece6660eef0fbf1e534d6a66","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2017-01-10 08:31:58.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"46e68bfd66f2fa79a08e563dc0883fba79e59e41","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-01-10 10:05:35.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-novaclient-docs-ubuntu-xenial http://docs-draft.openstack.org/90/418190/2/check/gate-python-novaclient-docs-ubuntu-xenial/0aeb8fa//doc/build/html/ : SUCCESS in 2m 34s\n- gate-python-novaclient-pep8-ubuntu-xenial http://logs.openstack.org/90/418190/2/check/gate-python-novaclient-pep8-ubuntu-xenial/1644df9/ : SUCCESS in 1m 48s\n- gate-python-novaclient-python27-ubuntu-xenial http://logs.openstack.org/90/418190/2/check/gate-python-novaclient-python27-ubuntu-xenial/1834fe1/ : SUCCESS in 1m 43s\n- gate-python-novaclient-python35 http://logs.openstack.org/90/418190/2/check/gate-python-novaclient-python35/f395f18/ : SUCCESS in 2m 22s\n- gate-tempest-dsvm-neutron-src-python-novaclient-ubuntu-xenial http://logs.openstack.org/90/418190/2/check/gate-tempest-dsvm-neutron-src-python-novaclient-ubuntu-xenial/89b414c/ : SUCCESS in 1h 30m 36s\n- gate-novaclient-dsvm-functional-mitaka-ubuntu-trusty-nv http://logs.openstack.org/90/418190/2/check/gate-novaclient-dsvm-functional-mitaka-ubuntu-trusty-nv/e31e27a/ : SUCCESS in 29m 12s (non-voting)\n- gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/90/418190/2/check/gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/881fb87/ : SUCCESS in 1h 11m 02s (non-voting)\n- gate-novaclient-dsvm-functional-neutron-ubuntu-xenial http://logs.openstack.org/90/418190/2/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/df21dac/ : SUCCESS in 42m 28s","accounts_in_message":[],"_revision_number":2},{"id":"057ef10f445c1d43d9a0ae1fc50d845929e6939a","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2017-01-10 13:07:50.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"d507d5d21e4a2edd5c7aff6b04f6f1a16f1d6373","author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"date":"2017-01-10 13:12:31.000000000","message":"Patch Set 2: Code-Review-1\n\nI really think this commit message should be updated. That patch changed code that isn\u0027t even active right now (pecan isn\u0027t switched on in Neutron).","accounts_in_message":[],"_revision_number":2},{"id":"ba314243cccc47fb056495e0140ca12490fc7391","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2017-01-10 13:15:47.000000000","message":"Patch Set 2:\n\nKevin, I don\u0027t know what else it is then, but https://review.openstack.org/#/c/417645/ didn\u0027t pass until I made it depend on https://review.openstack.org/#/c/418178/.","accounts_in_message":[],"_revision_number":2},{"id":"ae1507a0a7a8da4401088b4c395d96582cfe5679","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2017-01-10 13:17:37.000000000","message":"Patch Set 2:\n\nThis https://review.openstack.org/#/c/417645/ will test just the revert of the neutron pecan policy change now.","accounts_in_message":[],"_revision_number":2},{"id":"b5a035fc77f6274eb464ee255536cf70647993b3","author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"date":"2017-01-10 13:50:07.000000000","message":"Patch Set 2:\n\nHere is my postmortem. Something is janky in the openstackclient. It\u0027s not including the tenant_id in the network body in a bad run so the admin is getting an extra network.\n\n\n\u003ckevinbenton\u003e sc68cal, mriedem: found it!\n\u003ckevinbenton\u003e the openstack client is sickly\n\u003ckevinbenton\u003e claims it\u0027s creating network here: http://logs.openstack.org/45/417645/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/cc21d78/logs/devstacklog.txt.gz#_2017-01-08_03_27_50_751\n\u003ckevinbenton\u003e for project 708c451330e3412f85db275646fa3cd4\n\u003ckevinbenton\u003e but then it doesn\u0027t put the project in the body to neutron\n\u003ckevinbenton\u003e http://logs.openstack.org/45/417645/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/cc21d78/logs/screen-q-svc.txt.gz#_2017-01-08_03_27_51_963\n\u003ckevinbenton\u003e That\u0027s from a bad run\n\u003ckevinbenton\u003e here is what i looks like in a good run\n\u003ckevinbenton\u003e same command: http://logs.openstack.org/45/417645/4/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/642060d/logs/devstacklog.txt.gz#_2017-01-10_02_43_16_279\n\u003ckevinbenton\u003e but it actually populates it in the body: http://logs.openstack.org/45/417645/4/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/642060d/logs/screen-q-svc.txt.gz#_2017-01-10_02_43_18_351\n\u003ckevinbenton\u003e So in the broken run, the admin is getting an extra private network under it\u0027s project\n\u003ckevinbenton\u003e hence multiple networks\n\u003ckevinbenton\u003e Therefore, pistols at dawn can be called off","accounts_in_message":[],"_revision_number":2},{"id":"6eae71c0a184751e1020ba97a03801206e91b466","author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"date":"2017-01-10 14:17:13.000000000","message":"Patch Set 2:\n\npython-openstacksdk versions differed between the two.\n\nwas fixed in 0.9.12. Issue was checking for \u0027tenant_id\u0027 vs \u0027project_id\u0027 on the CLI. See L83 of https://review.openstack.org/#/c/364052/4/openstack/network/v2/network.py","accounts_in_message":[],"_revision_number":2},{"id":"46fd51d1b07ceb0845bbad6ce725b513252c67f2","author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"date":"2017-01-10 14:18:58.000000000","message":"Patch Set 2:\n\nsorry, meant 0.9.12 broke it. it has been banished\n\nhttps://github.com/openstack/requirements/commit/6da1fafee9ec30d3d9653c64cd27f61bbfb21862","accounts_in_message":[],"_revision_number":2},{"id":"807cc0e03285463b8693bb354e07eb90db0cf50c","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2017-01-10 14:43:43.000000000","message":"Patch Set 2: Workflow-1\n\n6da1fafee9ec30d3d9653c64cd27f61bbfb21862 fixes this it looks like.","accounts_in_message":[],"_revision_number":2},{"id":"81e6ca7379d81a6b75c67ea646762cde3743841f","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2017-01-11 00:50:19.000000000","message":"Patch Set 2:\n\nSo we could clean up the commit message and just still make this change, which might not be a bad idea in case we have some new issue in the future where these tests are running against multiple networks and would fail but for different root causes, or I can just abandon and try to forget about this chapter of my life.","accounts_in_message":[],"_revision_number":2},{"id":"a39e34a068af0b6fff070aa10d6fcb26f4077964","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2017-01-24 23:47:06.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"6be2c8b039f19d14914e9d1d08ea29122c6f45e9","author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"date":"2017-01-24 23:52:11.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"5f393385b6a83e80b8b524f529db75deaed5bf4e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-01-25 01:07:39.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-novaclient-docs-ubuntu-xenial http://docs-draft.openstack.org/90/418190/3/check/gate-python-novaclient-docs-ubuntu-xenial/3fba251//doc/build/html/ : SUCCESS in 2m 52s\n- gate-python-novaclient-pep8-ubuntu-xenial http://logs.openstack.org/90/418190/3/check/gate-python-novaclient-pep8-ubuntu-xenial/b79ebef/ : SUCCESS in 3m 18s\n- gate-python-novaclient-python27-ubuntu-xenial http://logs.openstack.org/90/418190/3/check/gate-python-novaclient-python27-ubuntu-xenial/0646fe2/ : SUCCESS in 3m 18s\n- gate-python-novaclient-python35 http://logs.openstack.org/90/418190/3/check/gate-python-novaclient-python35/1cd0f3d/ : SUCCESS in 2m 52s\n- gate-tempest-dsvm-neutron-src-python-novaclient-ubuntu-xenial http://logs.openstack.org/90/418190/3/check/gate-tempest-dsvm-neutron-src-python-novaclient-ubuntu-xenial/2a67581/ : SUCCESS in 1h 18m 38s\n- gate-novaclient-dsvm-functional-mitaka-ubuntu-trusty-nv http://logs.openstack.org/90/418190/3/check/gate-novaclient-dsvm-functional-mitaka-ubuntu-trusty-nv/5340c45/ : SUCCESS in 29m 45s (non-voting)\n- gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/90/418190/3/check/gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/cebd364/ : SUCCESS in 1h 03m 12s (non-voting)\n- gate-novaclient-dsvm-functional-neutron-ubuntu-xenial http://logs.openstack.org/90/418190/3/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/d29a50f/ : SUCCESS in 44m 14s","accounts_in_message":[],"_revision_number":3},{"id":"d01a7ff8dce029885936b31f016096af37b66e65","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2017-01-25 02:45:01.000000000","message":"Patch Set 3: Code-Review+1","accounts_in_message":[],"_revision_number":3},{"id":"927bfe82583f61c6918463d26eed339892ebd4a5","author":{"_account_id":13404,"name":"Alberto Planas","email":"aplanas@gmail.com","username":"aplanas"},"date":"2017-01-25 09:21:52.000000000","message":"Patch Set 3: Code-Review+1\n\nMakes sense to me","accounts_in_message":[],"_revision_number":3},{"id":"40d222d976b6ecbb429b38a9c93f024e7c53c018","author":{"_account_id":11034,"name":"Yanyan Hu","email":"huyanyan84@gmail.com","username":"yanyanhu"},"date":"2017-01-25 10:19:38.000000000","message":"Patch Set 3: Code-Review+1","accounts_in_message":[],"_revision_number":3},{"id":"d8dea6797ce06828b2e150486034ce551560a3b7","author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"date":"2017-01-25 14:01:23.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1\n\nJenkins is happy with that so I\u0027m happy (Droopy voice)","accounts_in_message":[],"_revision_number":3},{"id":"dadd74e00f28c75a9cbba1b5bfc2680891cdde66","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-01-25 14:01:34.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"ce8fe8b9dd05165f8f58a21f0bcfd166a2fa39f6","date":"2017-01-25 15:00:10.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":3},{"id":"1e85295e50e4aa6b37a600fa69f34f4d5e5b39e0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-01-25 15:00:10.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-novaclient-docs-ubuntu-xenial http://docs-draft.openstack.org/90/418190/3/gate/gate-python-novaclient-docs-ubuntu-xenial/a0a433f//doc/build/html/ : SUCCESS in 1m 29s\n- gate-python-novaclient-pep8-ubuntu-xenial http://logs.openstack.org/90/418190/3/gate/gate-python-novaclient-pep8-ubuntu-xenial/44d6be4/ : SUCCESS in 2m 03s\n- gate-python-novaclient-python27-ubuntu-xenial http://logs.openstack.org/90/418190/3/gate/gate-python-novaclient-python27-ubuntu-xenial/c072ea8/ : SUCCESS in 1m 46s\n- gate-python-novaclient-python35 http://logs.openstack.org/90/418190/3/gate/gate-python-novaclient-python35/f642b45/ : SUCCESS in 1m 46s\n- gate-tempest-dsvm-neutron-src-python-novaclient-ubuntu-xenial http://logs.openstack.org/90/418190/3/gate/gate-tempest-dsvm-neutron-src-python-novaclient-ubuntu-xenial/eeb4e37/ : SUCCESS in 58m 14s\n- gate-novaclient-dsvm-functional-neutron-ubuntu-xenial http://logs.openstack.org/90/418190/3/gate/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/3c1d55d/ : SUCCESS in 43m 17s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"ae699768023bb0a8d60798eca7f0cc8aebfe3355","revisions":{"d38fb9c117cd9c93bab6b374ebe0134e81c74631":{"kind":"REWORK","_number":1,"created":"2017-01-10 03:14:40.000000000","uploader":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"ref":"refs/changes/90/418190/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/90/418190/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/90/418190/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/90/418190/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/90/418190/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/90/418190/1"}}},"commit":{"parents":[{"commit":"5e5835de67aa3262b2ba51b7baec564be223166c","subject":"Merge \"Blacklist rather than whitelist autodoc modules\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/5e5835de67aa3262b2ba51b7baec564be223166c"}]}],"author":{"name":"Matt Riedemann","email":"mriedem@us.ibm.com","date":"2017-01-10 03:09:22.000000000","tz":-300},"committer":{"name":"Matt Riedemann","email":"mriedem@us.ibm.com","date":"2017-01-10 03:09:22.000000000","tz":-300},"subject":"Fix functional tests to deal with multiple networks","message":"Fix functional tests to deal with multiple networks\n\nNeutron fixed a bug in it\u0027s policy handling code in\nchange aacd8e0d043367d84ff9e50bad83c5c361852980 which\nexposed a bug in the novaclient functional testing where\nthe admin user is attempting to create a server but not\nspecify a specific network to use, but multiple networks\nare available to the admin (it can list multiple networks).\nIn this case the networks are the standard public and private\nnetworks that are created in devstack. Since a network isn\u0027t\nspecified when creating the server, the nova API fails with a\n409 error because it can\u0027t determine which network to use.\n\nThis patch fixes the testing in novaclient by checking to see\nif there are multiple networks available and if so, specifies\none for the legacy BDM tests that weren\u0027t specifying a network\nID (those tests don\u0027t really care about the networking).\n\nThe auto-network test is skipped if there are multiple networks\navailable because passing in a specific network would defeat\nthe purpose of that test.\n\nChange-Id: I22ee148581a94b153cf7e733563cfafaa56b1ffd\nCloses-Bug: #1654806\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/d38fb9c117cd9c93bab6b374ebe0134e81c74631"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/d38fb9c117cd9c93bab6b374ebe0134e81c74631"}]},"branch":"refs/heads/master"},"e2745b97884d62ef45bad251284f227c5334882d":{"kind":"REWORK","_number":2,"created":"2017-01-10 08:31:58.000000000","uploader":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"ref":"refs/changes/90/418190/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/90/418190/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/90/418190/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/90/418190/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/90/418190/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/90/418190/2"}}},"commit":{"parents":[{"commit":"5e5835de67aa3262b2ba51b7baec564be223166c","subject":"Merge \"Blacklist rather than whitelist autodoc modules\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/5e5835de67aa3262b2ba51b7baec564be223166c"}]}],"author":{"name":"Matt Riedemann","email":"mriedem@us.ibm.com","date":"2017-01-10 03:09:22.000000000","tz":-300},"committer":{"name":"Matt Riedemann","email":"mriedem@us.ibm.com","date":"2017-01-10 08:31:53.000000000","tz":-300},"subject":"Fix functional tests to deal with multiple networks","message":"Fix functional tests to deal with multiple networks\n\nNeutron fixed a bug in it\u0027s policy handling code in\nchange aacd8e0d043367d84ff9e50bad83c5c361852980 which\nexposed a bug in the novaclient functional testing where\nthe admin user is attempting to create a server but not\nspecify a specific network to use, but multiple networks\nare available to the admin (it can list multiple networks).\nIn this case the networks are the standard public and private\nnetworks that are created in devstack. Since a network isn\u0027t\nspecified when creating the server, the nova API fails with a\n409 error because it can\u0027t determine which network to use.\n\nThis patch fixes the testing in novaclient by checking to see\nif there are multiple networks available and if so, specifies\none for the legacy BDM tests that weren\u0027t specifying a network\nID (those tests don\u0027t really care about the networking).\n\nThe auto-network test is skipped if there are multiple networks\navailable because passing in a specific network would defeat\nthe purpose of that test.\n\nChange-Id: I22ee148581a94b153cf7e733563cfafaa56b1ffd\nCloses-Bug: #1654806\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/e2745b97884d62ef45bad251284f227c5334882d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/e2745b97884d62ef45bad251284f227c5334882d"}]},"branch":"refs/heads/master"},"ae699768023bb0a8d60798eca7f0cc8aebfe3355":{"kind":"TRIVIAL_REBASE_WITH_MESSAGE_UPDATE","_number":3,"created":"2017-01-24 23:47:06.000000000","uploader":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"ref":"refs/changes/90/418190/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/90/418190/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/90/418190/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/90/418190/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/90/418190/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/90/418190/3"}}},"commit":{"parents":[{"commit":"16b5bd09e93bc2dc0b6e4955e8508c0299efd2e0","subject":"Merge \"ListExtResource given in place of ListExtManager\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/16b5bd09e93bc2dc0b6e4955e8508c0299efd2e0"}]}],"author":{"name":"Matt Riedemann","email":"mriedem@us.ibm.com","date":"2017-01-10 03:09:22.000000000","tz":-300},"committer":{"name":"Matt Riedemann","email":"mriedem@us.ibm.com","date":"2017-01-24 23:44:37.000000000","tz":-300},"subject":"Fix functional tests to deal with multiple networks","message":"Fix functional tests to deal with multiple networks\n\nThere was a change in the openstacksdk 0.9.11 which\npython-openstackclient uses to create networks in a devstack\nrun with Neutron. Because of this change, the admin tenant\nhas access to both the \u0027public\u0027 and \u0027private\u0027 network setup\nin devstack which before used to just be the \u0027public\u0027 network,\nwhich exposed a bug in the novaclient functional testing where\nthe admin user is attempting to create a server but not\nspecify a specific network to use, but multiple networks\nare available to the admin (it can list multiple networks).\nIn this case the networks are the standard public and private\nnetworks that are created in devstack. Since a network isn\u0027t\nspecified when creating the server, the nova API fails with a\n409 error because it can\u0027t determine which network to use.\n\nThis patch fixes the testing in novaclient by checking to see\nif there are multiple networks available and if so, specifies\none for the legacy BDM tests that weren\u0027t specifying a network\nID (those tests don\u0027t really care about the networking).\n\nThe auto-network test is skipped if there are multiple networks\navailable because passing in a specific network would defeat\nthe purpose of that test.\n\nChange-Id: I22ee148581a94b153cf7e733563cfafaa56b1ffd\nCloses-Bug: #1654806\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/ae699768023bb0a8d60798eca7f0cc8aebfe3355"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/ae699768023bb0a8d60798eca7f0cc8aebfe3355"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}