)]}'
{"id":"openstack%2Fpython-openstackclient~420418","triplet_id":"openstack%2Fpython-openstackclient~master~Id9f11661365e2787c65454ba2148d10cf85bbe53","project":"openstack/python-openstackclient","branch":"master","topic":"test-subnet-create","hashtags":[],"change_id":"Id9f11661365e2787c65454ba2148d10cf85bbe53","subject":"Specify invariable subnet range for fuc test of subnet","status":"ABANDONED","created":"2017-01-15 14:27:11.000000000","updated":"2017-01-18 01:28:01.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"5287f6566e16e7dbec4ef331ccf59e3856680a10","_number":420418,"virtual_id_number":420418,"owner":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":-1,"date":"2017-01-15 15:47:46.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2017-01-15 15:48:38.000000000","_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"date":"2017-01-16 14:18:21.000000000","_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},{"date":"2017-01-15 14:41:48.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"}],"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":{"all":[{"date":"2017-01-15 21:55:07.000000000","_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-01-15 14:41:48.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2017-01-15 15:47:46.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2017-01-15 15:48:38.000000000","updated_by":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"reviewer":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"state":"REVIEWER"},{"updated":"2017-01-15 21:55:07.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"}],"messages":[{"id":"ec5b10373130c0b57ece1d7f2f18ce96ba40556b","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2017-01-15 14:27:11.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"4969236b33269f11aa430281bc908b705d53cc06","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2017-01-15 14:41:48.000000000","message":"Patch Set 1:\n\nI think a random range is good, it might even be good to do random netmasks too.  We can not hard-code as parallel testing would have a collision.\n\nSkipping certain networks is probably required, doing just the multicast isn\u0027t enough.  I think we have to also check and make sure there are no overlaps with existing usage, i.e. 127/8 is bad not because it is \u0027magic\u0027 ( it isn\u0027t) but because it is already in use, so that failed test overlaps an existing configured network on the test system.\n\nWhat I think we need to do is just set a retry here for 4 or so attempts at a random network.  I am guessing that getting the existing list of subnets to avoid conflict is not going to be quite good enough (127/8 is not going to be on that list), so we may as well just try a couple of times and record that.","accounts_in_message":[],"_revision_number":1},{"id":"3ebe94c166a7e18b5750a202ef9f6c37380e7e61","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-01-15 15:47:46.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-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/18/420418/1/check/gate-python-openstackclient-docs-ubuntu-xenial/eeacd11//doc/build/html/ : SUCCESS in 5m 15s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/18/420418/1/check/gate-python-openstackclient-pep8-ubuntu-xenial/923eee5/ : FAILURE in 4m 52s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/18/420418/1/check/gate-python-openstackclient-python27-ubuntu-xenial/087638f/ : SUCCESS in 4m 29s\n- gate-python-openstackclient-python35 http://logs.openstack.org/18/420418/1/check/gate-python-openstackclient-python35/62908c8/ : SUCCESS in 4m 29s\n- gate-tempest-dsvm-neutron-src-python-openstackclient-ubuntu-xenial http://logs.openstack.org/18/420418/1/check/gate-tempest-dsvm-neutron-src-python-openstackclient-ubuntu-xenial/e4cdc96/ : FAILURE in 1h 20m 23s\n- check-osc-plugins http://logs.openstack.org/18/420418/1/check/check-osc-plugins/b366190/ : SUCCESS in 6m 32s (non-voting)\n- gate-osc-dsvm-functional-ubuntu-xenial http://logs.openstack.org/18/420418/1/check/gate-osc-dsvm-functional-ubuntu-xenial/c87cd45/ : SUCCESS in 48m 00s\n- gate-osc-dsvm-functional-tips-ubuntu-xenial-nv http://logs.openstack.org/18/420418/1/check/gate-osc-dsvm-functional-tips-ubuntu-xenial-nv/71e3ddb/ : FAILURE in 52m 14s (non-voting)\n- gate-tripleo-ci-centos-7-nonha-multinode-nv http://logs.openstack.org/18/420418/1/check/gate-tripleo-ci-centos-7-nonha-multinode-nv/ccb766d/ : SUCCESS in 1h 16m 11s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"4eb0050eab73497084f9ce97aba449a3e102a6bd","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2017-01-15 15:48:38.000000000","message":"Patch Set 1:\n\nI noticed jenkins failed, I think you hit bug(s):\n\n- gate-python-openstackclient-pep8-ubuntu-xenial: unrecognized error\n- gate-tempest-dsvm-neutron-src-python-openstackclient-ubuntu-xenial: https://bugs.launchpad.net/bugs/1323658 https://bugs.launchpad.net/bugs/1349617\n\nSome of the tests failed in a way that we did not understand. Please help us classify these issues so that they can be part of Elastic Recheck http://status.openstack.org/elastic-recheck/\nFor more details on this and other bugs, please see http://status.openstack.org/elastic-recheck/","accounts_in_message":[],"_revision_number":1},{"id":"b0ab86dda99606803812ba43896a40c216c85e5c","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2017-01-16 14:18:21.000000000","message":"Patch Set 1:\n\n\u003e I think a random range is good, it might even be good to do random\n \u003e netmasks too.  We can not hard-code as parallel testing would have\n \u003e a collision.\n \u003e \n \u003e Skipping certain networks is probably required, doing just the\n \u003e multicast isn\u0027t enough.  I think we have to also check and make\n \u003e sure there are no overlaps with existing usage, i.e. 127/8 is bad\n \u003e not because it is \u0027magic\u0027 ( it isn\u0027t) but because it is already in\n \u003e use, so that failed test overlaps an existing configured network on\n \u003e the test system.\n \u003e \n \u003e What I think we need to do is just set a retry here for 4 or so\n \u003e attempts at a random network.  I am guessing that getting the\n \u003e existing list of subnets to avoid conflict is not going to be quite\n \u003e good enough (127/8 is not going to be on that list), so we may as\n \u003e well just try a couple of times and record that.\n\nHi, Dean, I agree with your point that keep a random range, but I think it is not easy to make sure what is good range and what is not, so do you mean that if a error raise we retry here with new range?","accounts_in_message":[],"_revision_number":1},{"id":"3de117b247bbd068f57d27eee44f1973b79dfd56","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2017-01-16 14:52:35.000000000","message":"Patch Set 1:\n\nYes, retry a couple of times on an error.  If we knew for sue that we could know in advance what the \u0027bad\u0027 network are we could just check that list, but I do not think that list is available to clients.","accounts_in_message":[],"_revision_number":1},{"id":"51f23c63d56b6eb4e22796ffa81983dcd7f201c6","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2017-01-18 01:28:01.000000000","message":"Abandoned\n\nhttps://review.openstack.org/#/c/421310/ instead","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"1db69396ce21f9f8ed5fa51723482c834673027a","revisions":{"1db69396ce21f9f8ed5fa51723482c834673027a":{"kind":"REWORK","_number":1,"created":"2017-01-15 14:27:11.000000000","uploader":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"ref":"refs/changes/18/420418/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/18/420418/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/18/420418/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/18/420418/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/18/420418/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/18/420418/1"}}},"commit":{"parents":[{"commit":"368666194bb11e1543fb683e8b42729620a69aac","subject":"Merge \"Fix quota show --default command\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/368666194bb11e1543fb683e8b42729620a69aac"}]}],"author":{"name":"Huanxuan Ao","email":"huanxuan.ao@easystack.cn","date":"2017-01-15 14:16:06.000000000","tz":480},"committer":{"name":"Huanxuan Ao","email":"huanxuan.ao@easystack.cn","date":"2017-01-15 14:16:06.000000000","tz":480},"subject":"Specify invariable subnet range for fuc test of subnet","message":"Specify invariable subnet range for fuc test of subnet\n\nNoticed functional test failed again by the subnet\ncreating since the first address of subnet range\nis 127, log see[1]\nShould we specify a invariable subnet range instead of\nrandom range to avoid unexception error or conflict\nand make the test safer\n\n[1]: http://logs.openstack.org/99/420399/1/gate/gate-osc-dsvm-functional-ubuntu-xenial/17835cb/console.html#_2017-01-15_11_53_39_708708\n\nChange-Id: Id9f11661365e2787c65454ba2148d10cf85bbe53\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/1db69396ce21f9f8ed5fa51723482c834673027a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/1db69396ce21f9f8ed5fa51723482c834673027a"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}