)]}'
{"id":"openstack%2Fdevstack-gate~46403","triplet_id":"openstack%2Fdevstack-gate~master~I8f7359261461621c39235a1eae8891776bd2664e","project":"openstack/devstack-gate","branch":"master","topic":"extend_neutron_floating_range","hashtags":[],"change_id":"I8f7359261461621c39235a1eae8891776bd2664e","subject":"extend the floating ips range (for neutron tests to pass)","status":"ABANDONED","created":"2013-09-13 01:48:40.000000000","updated":"2013-09-16 17:38:49.000000000","total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"77dc7d688b7f5d79521df899b850713e2a746415","_number":46403,"virtual_id_number":46403,"owner":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":1,"date":"2013-09-13 02:45:29.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}],"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":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":-1,"date":"2013-09-13 05:17:40.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-09-13 17:52:35.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}],"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":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"date":"2013-09-13 05:17:40.000000000","_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-09-13 01:51:19.000000000","updated_by":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"reviewer":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"state":"REVIEWER"},{"updated":"2013-09-13 01:51:29.000000000","updated_by":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"reviewer":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"state":"REVIEWER"},{"updated":"2013-09-13 01:51:41.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-09-13 01:52:11.000000000","updated_by":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"reviewer":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"state":"REVIEWER"},{"updated":"2013-09-13 02:02:44.000000000","updated_by":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"reviewer":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"state":"REVIEWER"},{"updated":"2013-09-13 02:45:29.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2013-09-13 05:17:40.000000000","updated_by":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"reviewer":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"state":"REVIEWER"}],"messages":[{"id":"82e0daf9e0b7d46427ed34f16747605847da13b7","author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"date":"2013-09-13 01:55:47.000000000","message":"Patch Set 1:\n\nthis is expected to make the following to succeed: https://review.openstack.org/#/c/46330/\n\nand as such to merge:\nhttps://review.openstack.org/#/c/45578/","accounts_in_message":[],"_revision_number":1},{"id":"ba7657345fdcb14c5d484eadc64af1dd84228a74","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2013-09-13 02:02:44.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nThis is a soft -1 particularly if this passes tests as is.","accounts_in_message":[],"_revision_number":1},{"id":"009a9d202b7ee78e5f38eb4e806de881106b2045","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2013-09-13 02:03:47.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"a22a33599f94730af047964f5e95674e0a7856ac","author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"date":"2013-09-13 02:14:17.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"fd4c1d2b7706ae4242055296b213471f49427c3a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-13 02:45:29.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/03/46403/2/check/gate-tempest-devstack-vm-full/f27688a : SUCCESS in 30m 35s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/03/46403/2/check/gate-tempest-devstack-vm-postgres-full/a885e12 : SUCCESS in 28m 04s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/03/46403/2/check/gate-tempest-devstack-vm-neutron/f81e5b2 : SUCCESS in 21m 29s\n- gate-grenade-devstack-vm http://logs.openstack.org/03/46403/2/check/gate-grenade-devstack-vm/9ca27e0 : SUCCESS in 26m 19s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/03/46403/2/check/gate-swift-devstack-vm-functional/f13878d : SUCCESS in 13m 39s\n","accounts_in_message":[],"_revision_number":2},{"id":"a477550e933c7dd177c694c9ea5b264a0db20bdc","author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"date":"2013-09-13 05:17:40.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\nWhy not make the change to the default in devstack?","accounts_in_message":[],"_revision_number":2},{"id":"73664fe5803482fb2a32150601401b048acbfbd4","author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"date":"2013-09-13 17:52:35.000000000","message":"Patch Set 2:\n\nhi there, thanks for helping with the review\n\nIt looks like a leakage has also been spotted in the tests so while /28 still seems very tight for parallel tests, it may be sufficient after the leakage is fixed\n\nshall I postpone (and eventually move to devstack) only if needed after the leakage is fixed or would you still apply something like this?","accounts_in_message":[],"_revision_number":2},{"id":"dbbebcf4c0adb5a0d9afe40ace272d637a2ae55b","author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"date":"2013-09-16 17:38:49.000000000","message":"Patch Set 2: Abandoned\n\nidentical change proposed for devstack, see: https://review.openstack.org/#/c/36627","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"b25d2f650a949fe1c45cbf1bd542f26786a2edae","revisions":{"21779e47dd54dfea0bcf2acbca0fe9c17b9fce16":{"kind":"REWORK","_number":1,"created":"2013-09-13 01:48:40.000000000","uploader":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"ref":"refs/changes/03/46403/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/03/46403/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/03/46403/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/03/46403/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/03/46403/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/03/46403/1"}}},"commit":{"parents":[{"commit":"1834013e81e02316ec9d83f3778553a1b927e473","subject":"add c-bak to MY_ENABLED_SERVICES","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/1834013e81e02316ec9d83f3778553a1b927e473"}]}],"author":{"name":"Giulio Fidente","email":"gfidente@redhat.com","date":"2013-09-13 01:41:26.000000000","tz":120},"committer":{"name":"Giulio Fidente","email":"gfidente@redhat.com","date":"2013-09-13 01:41:26.000000000","tz":120},"subject":"extend the floating ips range for neutron tests","message":"extend the floating ips range for neutron tests\n\nthe devstack default for the neutron external network floating\nips range is /28, see devstack/stack.sh#L271\n\nby running tests in parallel tempest is consuming more than those so\nthis change increases FLOATING_RANGE to /24 on the neutron VM\n\nChange-Id: I8f7359261461621c39235a1eae8891776bd2664e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/21779e47dd54dfea0bcf2acbca0fe9c17b9fce16"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/21779e47dd54dfea0bcf2acbca0fe9c17b9fce16"}]},"branch":"refs/heads/master"},"b25d2f650a949fe1c45cbf1bd542f26786a2edae":{"kind":"REWORK","_number":2,"created":"2013-09-13 02:14:17.000000000","uploader":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"ref":"refs/changes/03/46403/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/03/46403/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/03/46403/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/03/46403/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/03/46403/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/03/46403/2"}}},"commit":{"parents":[{"commit":"1834013e81e02316ec9d83f3778553a1b927e473","subject":"add c-bak to MY_ENABLED_SERVICES","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/1834013e81e02316ec9d83f3778553a1b927e473"}]}],"author":{"name":"Giulio Fidente","email":"gfidente@redhat.com","date":"2013-09-13 01:41:26.000000000","tz":120},"committer":{"name":"Giulio Fidente","email":"gfidente@redhat.com","date":"2013-09-13 02:16:31.000000000","tz":120},"subject":"extend the floating ips range (for neutron tests to pass)","message":"extend the floating ips range (for neutron tests to pass)\n\nthe devstack default for the external network floating ips range\nis /28, see devstack/stack.sh#L271\n\nby running tests in parallel tempest is consuming more than those so\nthis change increases FLOATING_RANGE to /24 (currently only needed\nfor the neutron tests but changed globally to remain consistent)\n\nChange-Id: I8f7359261461621c39235a1eae8891776bd2664e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/b25d2f650a949fe1c45cbf1bd542f26786a2edae"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/b25d2f650a949fe1c45cbf1bd542f26786a2edae"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}