)]}'
{"id":"openstack%2Fdevstack~58284","triplet_id":"openstack%2Fdevstack~master~Ie075b3c4d14a07b06c42fd29b09770dd1972aa45","project":"openstack/devstack","branch":"master","topic":"bug/1253966","hashtags":[],"change_id":"Ie075b3c4d14a07b06c42fd29b09770dd1972aa45","subject":"Default to /24 prefix for floating IP range with neutron","status":"MERGED","created":"2013-11-25 13:46:42.000000000","updated":"2013-12-17 09:09:00.000000000","submitted":"2013-12-17 09:08:59.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":5,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"b2524063ff894a557361041947b140abe474e86e","_number":58284,"virtual_id_number":58284,"owner":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-12-17 09:08:59.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},{"value":0,"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},{"value":0,"_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":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2013-12-17 07:40:26.000000000","_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},{"value":0,"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},{"value":2,"date":"2013-12-16 18:54:49.000000000","_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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-12-17 07:40:26.000000000","_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},{"value":0,"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},{"value":0,"date":"2013-12-16 18:54:49.000000000","_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},"Review-Priority":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},{"value":0,"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-11-27 12:55:40.000000000","updated_by":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"reviewer":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"state":"REVIEWER"},{"updated":"2013-12-10 00:11:39.000000000","updated_by":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"reviewer":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"state":"REVIEWER"},{"updated":"2013-12-12 22:42:54.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":"2013-12-16 18:54:49.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-17 07:40:26.000000000","updated_by":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"reviewer":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"state":"REVIEWER"},{"updated":"2013-12-17 09:08:59.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"4ef6496bc609ffa0874865ffb3c6a4d2359c1679","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-11-25 14:04:43.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"0c4430b2f06d2366e61c5136f6a1a72fdeeda6db","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-11-25 14:04:50.000000000","message":"Patch Set 2:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":2},{"id":"2f36f4fabfe0c9638041eb06d0929471a6acf879","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-25 14:36:23.000000000","message":"Patch Set 2: 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- gate-devstack-bash8 http://logs.openstack.org/84/58284/2/check/gate-devstack-bash8/8ad0cfa : SUCCESS in 4s\n- check-tempest-devstack-vm-full http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-full/a3d1693 : FAILURE in 30m 46s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-postgres-full/8a09b8a : SUCCESS in 31m 36s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-neutron/0bd46f6 : SUCCESS in 25m 39s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/84/58284/2/check/gate-tempest-devstack-vm-large-ops/52bb9f5 : SUCCESS in 15m 14s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/84/58284/2/check/gate-tempest-devstack-vm-neutron-large-ops/d7f4ea7 : SUCCESS in 17m 50s\n- check-grenade-devstack-vm http://logs.openstack.org/84/58284/2/check/check-grenade-devstack-vm/5a678a8 : SUCCESS in 23m 08s\n- check-swift-devstack-vm-functional http://logs.openstack.org/84/58284/2/check/check-swift-devstack-vm-functional/53fb403 : SUCCESS in 10m 21s\n","accounts_in_message":[],"_revision_number":2},{"id":"5a14f27cf229c07e4446d9af5d30c00ad164fdcc","author":{"_account_id":9009,"name":"Red Hat CI","email":"redhatci@redhat.com","username":"redhatci","tags":["SERVICE_USER"]},"date":"2013-11-25 14:39:23.000000000","message":"Patch Set 2:\n\nredhatci results for change 58284\n\nLogs at http://people.redhat.com/~iwienand/58284/\n\n- Patch set 1 | OSLAB RHEL65 : PASS\n- Patch set 1 | OSLAB Fedora 19 : PASS\n- Patch set 1 | OSLAB RHEL64 : PASS\n- Patch set 1 | OSLAB CentOS 6.4 : PASS\n- Patch set 2 | OSLAB Fedora 19 : PASS\n- Patch set 2 | OSLAB RHEL65 : PASS\n- Patch set 2 | OSLAB CentOS 6.4 : PASS\n- Patch set 2 | OSLAB RHEL64 : PASS\n\nend","accounts_in_message":[],"_revision_number":2},{"id":"0404c60e0cec47bd75a106f6913ea4c3a94f105a","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-11-25 18:38:27.000000000","message":"Patch Set 2:\n\nI\u0027m sorry I\u0027m not able to find a relationship between the error in devstack-vm-full and this patch.\n\nAlso I do not find this error in status.openstack.org/rechecks\n\nDon\u0027t hate me for the comment I\u0027m going to post","accounts_in_message":[],"_revision_number":2},{"id":"8d677ae214335068f629290562e21244fafc2147","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-11-25 18:38:35.000000000","message":"Patch Set 2:\n\nrecheck no bug","accounts_in_message":[],"_revision_number":2},{"id":"dfd03ef88d7b5131e4777665e27b51ba966d2bb9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-25 20:05:08.000000000","message":"Patch Set 2:\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-devstack-bash8 http://logs.openstack.org/84/58284/2/check/gate-devstack-bash8/b19302e : SUCCESS in 16s\n- check-tempest-devstack-vm-full http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-full/a2ce10a : FAILURE in 40m 56s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-postgres-full/34fcc9b : SUCCESS in 38m 50s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-neutron/bc93a61 : SUCCESS in 34m 56s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/84/58284/2/check/gate-tempest-devstack-vm-large-ops/8fa2114 : SUCCESS in 14m 15s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/84/58284/2/check/gate-tempest-devstack-vm-neutron-large-ops/507d442 : SUCCESS in 26m 04s\n- check-grenade-devstack-vm http://logs.openstack.org/84/58284/2/check/check-grenade-devstack-vm/9a09d8f : SUCCESS in 31m 39s\n- check-swift-devstack-vm-functional http://logs.openstack.org/84/58284/2/check/check-swift-devstack-vm-functional/212b801 : SUCCESS in 10m 49s\n","accounts_in_message":[],"_revision_number":2},{"id":"42375e3e35a41514ab0f15b1d709e06c7c1330e7","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-11-26 00:35:12.000000000","message":"Patch Set 2:\n\nrecheck bug1253986","accounts_in_message":[],"_revision_number":2},{"id":"c75617d33e32db285d215dffecfcf94e0399b79b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-26 01:34:00.000000000","message":"Patch Set 2:\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-devstack-bash8 http://logs.openstack.org/84/58284/2/check/gate-devstack-bash8/947c410 : SUCCESS in 13s\n- check-tempest-devstack-vm-full http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-full/1ea27eb : FAILURE in 42m 48s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-postgres-full/24d4e93 : SUCCESS in 42m 11s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-neutron/bf8051a : FAILURE in 32m 14s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/84/58284/2/check/gate-tempest-devstack-vm-large-ops/0c1da52 : SUCCESS in 16m 21s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/84/58284/2/check/gate-tempest-devstack-vm-neutron-large-ops/e1c1584 : SUCCESS in 16m 26s\n- check-grenade-devstack-vm http://logs.openstack.org/84/58284/2/check/check-grenade-devstack-vm/d46cad3 : SUCCESS in 39m 16s\n- check-swift-devstack-vm-functional http://logs.openstack.org/84/58284/2/check/check-swift-devstack-vm-functional/c59286a : SUCCESS in 14m 53s\n","accounts_in_message":[],"_revision_number":2},{"id":"df9d9cbadf769fab795845626f321869de901f65","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2013-11-26 01:34:50.000000000","message":"Patch Set 2:\n\nI noticed tempest failed, I think you hit bug(s):\n\n- https://bugs.launchpad.net/bugs/1210483\n- https://bugs.launchpad.net/bugs/1253896\n\nWe don\u0027t automatically recheck or reverify, so please consider\ndoing that manually if someone hasn\u0027t already. For a code review\nwhich is not yet approved, you can recheck by leaving a code\nreview comment with just the text:\n\n    recheck bug 1210483\n\nFor a code review which has been approved but failed to merge,\nyou can reverify by leaving a comment like this:\n\n    reverify bug 1210483","accounts_in_message":[],"_revision_number":2},{"id":"9170123ad4cb927428719573fa8412cab4820def","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-11-26 15:16:34.000000000","message":"Patch Set 2:\n\nrecheck bug 1210483","accounts_in_message":[],"_revision_number":2},{"id":"2fc31b2dd517bff46c8a8609b4b3b7c5b02e26d1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-26 15:57:49.000000000","message":"Patch Set 2:\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-devstack-bash8 http://logs.openstack.org/84/58284/2/check/gate-devstack-bash8/87793a1 : SUCCESS in 4s\n- check-tempest-devstack-vm-full http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-full/9412254 : SUCCESS in 37m 11s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-postgres-full/0388dbd : SUCCESS in 40m 48s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-neutron/3ab8163 : FAILURE in 26m 16s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/84/58284/2/check/gate-tempest-devstack-vm-large-ops/b2ac2f0 : SUCCESS in 15m 12s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/84/58284/2/check/gate-tempest-devstack-vm-neutron-large-ops/7eb264d : SUCCESS in 18m 27s\n- check-grenade-devstack-vm http://logs.openstack.org/84/58284/2/check/check-grenade-devstack-vm/6f3cae5 : SUCCESS in 31m 52s\n- check-swift-devstack-vm-functional http://logs.openstack.org/84/58284/2/check/check-swift-devstack-vm-functional/08e8100 : SUCCESS in 16m 12s\n","accounts_in_message":[],"_revision_number":2},{"id":"26ee543b6721bc227f34b1d687041a5fcd33a9e0","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2013-11-26 15:58:04.000000000","message":"Patch Set 2:\n\nI noticed tempest failed, I think you hit bug(s):\n\n- https://bugs.launchpad.net/bugs/1249065\n\nWe don\u0027t automatically recheck or reverify, so please consider\ndoing that manually if someone hasn\u0027t already. For a code review\nwhich is not yet approved, you can recheck by leaving a code\nreview comment with just the text:\n\n    recheck bug 1249065\n\nFor a code review which has been approved but failed to merge,\nyou can reverify by leaving a comment like this:\n\n    reverify bug 1249065","accounts_in_message":[],"_revision_number":2},{"id":"bae40fcd9c90d3b732427d803c360da8f3ea4d29","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-11-26 18:15:48.000000000","message":"Patch Set 2:\n\nrecheck bug 1249065","accounts_in_message":[],"_revision_number":2},{"id":"134139b0b996eb3aadd663fbcf272951f7edb7d5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-11-26 19:00:57.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/84/58284/2/check/gate-devstack-bash8/cf15da4 : SUCCESS in 6s\n- check-tempest-devstack-vm-full http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-full/46a0108 : SUCCESS in 40m 11s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-postgres-full/a2d3c5d : SUCCESS in 36m 19s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/84/58284/2/check/check-tempest-devstack-vm-neutron/4f11959 : SUCCESS in 29m 47s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/84/58284/2/check/gate-tempest-devstack-vm-large-ops/a30c0c9 : SUCCESS in 15m 23s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/84/58284/2/check/gate-tempest-devstack-vm-neutron-large-ops/d4926d4 : SUCCESS in 15m 46s\n- check-grenade-devstack-vm http://logs.openstack.org/84/58284/2/check/check-grenade-devstack-vm/9a2979b : SUCCESS in 34m 42s\n- check-swift-devstack-vm-functional http://logs.openstack.org/84/58284/2/check/check-swift-devstack-vm-functional/ae8cff8 : SUCCESS in 17m 53s\n","accounts_in_message":[],"_revision_number":2},{"id":"cd95b8daf5fb60828fb9987623bebb4f5fe1a501","author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"date":"2013-11-27 12:55:40.000000000","message":"Patch Set 2:\n\nWill /27 be enough ? Why not use /24, for instance ?","accounts_in_message":[],"_revision_number":2},{"id":"ff9b4311c8b7c23c07aae3b7c3fac0f817c3b238","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-11-27 13:02:12.000000000","message":"Patch Set 2:\n\nI think /27 will be enough as it will give 32 IP addresses.\nA further shortage might happen if we then increase parallelism.\n\nI chose /27 for laziness. This way I did not have to change the gateway IP as well.\nIf there are no arguments against using a /24 for neutron public network I am happy to use a /24","accounts_in_message":[],"_revision_number":2},{"id":"6a100b7df99139ded76402970d6098b71bfd77d6","author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"date":"2013-11-27 13:15:15.000000000","message":"Patch Set 2:\n\nThat\u0027s my concern. If we increase parallelism again, 32 addresses might not be enough, so we\u0027ll have to use /26 etc. But I\u0027m not sure whether it\u0027s likely to happen, I\u0027d like to know what other reviewers, familiar with Tempest, think about this.","accounts_in_message":[],"_revision_number":2},{"id":"de34b5701dee4cacc571824f90a5b80eaed829c0","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-12-04 17:33:23.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\nI would be OK with changing the default to /27.  Alternatively, is setting FLOATING_RANGE in Tempest configs acceptable?","accounts_in_message":[],"_revision_number":2},{"id":"1a00aabf085b1fbf724cf8b58feabed79cd57b6c","author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"date":"2013-12-06 08:11:19.000000000","message":"Patch Set 2:\n\nWhy stop at /24 :)","accounts_in_message":[],"_revision_number":2},{"id":"51ca999e20bac482702936a41202f3d0366eed6a","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-12-09 23:17:52.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"301c9b7005dc0b2900894d8a95c454ddea043df0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-09 23:49:55.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/84/58284/3/check/gate-devstack-bash8/1c02b68 : SUCCESS in 8s\n- check-tempest-dsvm-full http://logs.openstack.org/84/58284/3/check/check-tempest-dsvm-full/5bfcb5f : SUCCESS in 31m 57s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/84/58284/3/check/check-tempest-dsvm-postgres-full/5cd3464 : SUCCESS in 31m 26s\n- check-tempest-dsvm-neutron http://logs.openstack.org/84/58284/3/check/check-tempest-dsvm-neutron/907a1d8 : SUCCESS in 30m 36s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/84/58284/3/check/gate-tempest-dsvm-large-ops/27cdbea : SUCCESS in 15m 30s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/84/58284/3/check/gate-tempest-dsvm-neutron-large-ops/b62161e : SUCCESS in 13m 57s\n- check-grenade-dsvm http://logs.openstack.org/84/58284/3/check/check-grenade-dsvm/54fb894 : SUCCESS in 30m 07s\n- check-swift-dsvm-functional http://logs.openstack.org/84/58284/3/check/check-swift-dsvm-functional/4b654e1 : SUCCESS in 11m 59s\n","accounts_in_message":[],"_revision_number":3},{"id":"deae27a3118b9253feaa8177f5932f105cb8a8b9","author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"date":"2013-12-10 00:11:39.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":3},{"id":"59c22b12f892c0fc0f49d0b4511c39b943c47d45","author":{"_account_id":9009,"name":"Red Hat CI","email":"redhatci@redhat.com","username":"redhatci","tags":["SERVICE_USER"]},"date":"2013-12-10 00:12:14.000000000","message":"Patch Set 3:\n\nredhatci results for change 58284\n\nLogs at http://people.redhat.com/~iwienand/58284/\n\n- Patch set 1 | OSLAB RHEL65 : PASS\n- Patch set 1 | OSLAB Fedora 19 : PASS\n- Patch set 1 | OSLAB RHEL64 : PASS\n- Patch set 1 | OSLAB CentOS 6.4 : PASS\n- Patch set 2 | OSLAB Fedora 19 : PASS\n- Patch set 2 | OSLAB RHEL65 : PASS\n- Patch set 2 | OSLAB CentOS 6.4 : PASS\n- Patch set 2 | OSLAB RHEL64 : PASS\n- Patch set 3 | OSLAB RHEL65 : PASS\n- Patch set 3 | OSLAB Fedora 20 : PASS\n- Patch set 3 | OSLAB Fedora 19 : PASS\n- Patch set 3 | OSLAB CentOS 6.4 : PASS\n- Patch set 3 | OSLAB CentOS 6.5 : FAIL\n- Patch set 3 | OSLAB RHEL64 : PASS\n\nend","accounts_in_message":[],"_revision_number":3},{"id":"ba3d22b41664e2da0c62087ae7967c6a5ed5327c","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-12-12 01:17:46.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"532cc36dd13efbe24065443ef641ab3b725e9252","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-12-12 17:45:03.000000000","message":"Patch Set 3: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"fd03402667aacf0385232b1e3b21ec587ea87dc5","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-12-12 22:30:05.000000000","message":"Patch Set 3: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"dedb492333efd3bf96789e1ba7c7f4a1292ca392","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-12-12 22:42:54.000000000","message":"Patch Set 3: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"a76fe43201724d6e43ba2df1b319c992912bd580","author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"date":"2013-12-12 23:31:41.000000000","message":"Patch Set 3: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"283226384f2451515c69e630d8cbe26ee575cfb3","author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"date":"2013-12-15 12:39:16.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"4b4e71f87c13b71dbfa1fb9de3930b340570ff87","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-15 13:22:06.000000000","message":"Patch Set 4: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/84/58284/4/check/gate-devstack-bash8/accb434 : SUCCESS in 9s\n- check-tempest-dsvm-full http://logs.openstack.org/84/58284/4/check/check-tempest-dsvm-full/5d7e119 : SUCCESS in 42m 14s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/84/58284/4/check/check-tempest-dsvm-postgres-full/7b17464 : SUCCESS in 38m 47s\n- check-tempest-dsvm-neutron http://logs.openstack.org/84/58284/4/check/check-tempest-dsvm-neutron/36744d0 : SUCCESS in 29m 11s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/84/58284/4/check/gate-tempest-dsvm-large-ops/11816bc : SUCCESS in 15m 03s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/84/58284/4/check/gate-tempest-dsvm-neutron-large-ops/ad3d993 : SUCCESS in 19m 36s\n- check-grenade-dsvm http://logs.openstack.org/84/58284/4/check/check-grenade-dsvm/e0aae84 : SUCCESS in 31m 54s\n- check-swift-dsvm-functional http://logs.openstack.org/84/58284/4/check/check-swift-dsvm-functional/60a998a : SUCCESS in 11m 40s\n","accounts_in_message":[],"_revision_number":4},{"id":"3d59cff4d0a1690848f0ade2e2d95f29aa50e123","author":{"_account_id":9009,"name":"Red Hat CI","email":"redhatci@redhat.com","username":"redhatci","tags":["SERVICE_USER"]},"date":"2013-12-16 01:06:09.000000000","message":"Patch Set 4:\n\nredhatci results for change 58284\n\nLogs at http://people.redhat.com/~iwienand/58284/\n\n- Patch set 1 | OSLAB RHEL65 : PASS\n- Patch set 1 | OSLAB Fedora 19 : PASS\n- Patch set 1 | OSLAB RHEL64 : PASS\n- Patch set 1 | OSLAB CentOS 6.4 : PASS\n- Patch set 2 | OSLAB Fedora 19 : PASS\n- Patch set 2 | OSLAB RHEL65 : PASS\n- Patch set 2 | OSLAB CentOS 6.4 : PASS\n- Patch set 2 | OSLAB RHEL64 : PASS\n- Patch set 3 | OSLAB RHEL65 : PASS\n- Patch set 3 | OSLAB Fedora 20 : PASS\n- Patch set 3 | OSLAB Fedora 19 : PASS\n- Patch set 3 | OSLAB CentOS 6.4 : PASS\n- Patch set 3 | OSLAB CentOS 6.5 : FAIL\n- Patch set 3 | OSLAB RHEL64 : PASS\n- Patch set 4 | OSLAB RHEL64 : PASS\n- Patch set 4 | OSLAB Fedora 19 : PASS\n- Patch set 4 | OSLAB Fedora 20 : FAIL\n- Patch set 4 | OSLAB RHEL65 : PASS\n- Patch set 4 | OSLAB CentOS 6.4 : PASS\n- Patch set 4 | OSLAB CentOS 6.5 : PASS\n\nend","accounts_in_message":[],"_revision_number":4},{"id":"655d904a7f37cacc364e234a78c4abbcd521aa62","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-12-16 18:54:49.000000000","message":"Patch Set 4: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":4},{"id":"0c156006ee96659b684b8c3575302b9f35378c13","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2013-12-17 07:40:26.000000000","message":"Patch Set 4: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":4},{"id":"f88199b761c5bd37aebafea6fed9b61740c92cf7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-17 07:40:27.000000000","message":"Patch Set 4: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":4},{"id":"a98115bcd4a70d3d705942bdc4c048beb02b2952","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-17 09:08:59.000000000","message":"Patch Set 4: Verified\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/84/58284/4/gate/gate-devstack-bash8/6736d60 : SUCCESS in 10s\n- gate-tempest-dsvm-full http://logs.openstack.org/84/58284/4/gate/gate-tempest-dsvm-full/66b2ae7 : SUCCESS in 38m 49s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/84/58284/4/gate/gate-tempest-dsvm-postgres-full/18aec10 : SUCCESS in 35m 02s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/84/58284/4/gate/gate-tempest-dsvm-neutron/f18afa0 : SUCCESS in 30m 32s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/84/58284/4/gate/gate-tempest-dsvm-large-ops/b2054f0 : SUCCESS in 14m 29s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/84/58284/4/gate/gate-tempest-dsvm-neutron-large-ops/aa79637 : SUCCESS in 14m 39s\n- gate-grenade-dsvm http://logs.openstack.org/84/58284/4/gate/gate-grenade-dsvm/5378c5f : SUCCESS in 29m 03s\n- gate-swift-dsvm-functional http://logs.openstack.org/84/58284/4/gate/gate-swift-dsvm-functional/8b90866 : SUCCESS in 12m 11s\n","accounts_in_message":[],"_revision_number":4},{"id":"1b94fbf166e0b5e6f8abe9ae4290e5e22811f3e2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-12-17 09:09:00.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"90234ac4b03426d844b72d251d4cae13fa09cde5","revisions":{"efce984f560314fa5778872769212bd5ebcea76d":{"kind":"REWORK","_number":1,"created":"2013-11-25 13:46:42.000000000","uploader":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"ref":"refs/changes/84/58284/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/84/58284/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/84/58284/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/84/58284/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/84/58284/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/84/58284/1"}}},"commit":{"parents":[{"commit":"ad2966d727df21653920d95dbf40711c2b8f2873","subject":"Merge \"Add hacking rules for shell scripts\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/ad2966d727df21653920d95dbf40711c2b8f2873"}]}],"author":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-11-25 13:44:10.000000000","tz":-480},"committer":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-11-25 13:44:10.000000000","tz":-480},"subject":"Default to /27 prefix for floating IP range with neutron","message":"Default to /27 prefix for floating IP range with neutron\n\nWhen running Tempest parallel tests with neutron, several long-running\ntests might create routers and floating IPs, which will result in\nIP allocations over the public network.\n\nIncreasing the public network size should ensure tests do not fail\ndue to IP address shortage.\n\nChange-Id: Ie075b3c4d14a07b06c42fd29b09770dd1972aa45\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/efce984f560314fa5778872769212bd5ebcea76d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/efce984f560314fa5778872769212bd5ebcea76d"}]},"branch":"refs/heads/master"},"6b813520318d1724d916ac1db43b28bf651a5605":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2013-11-25 14:04:43.000000000","uploader":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"ref":"refs/changes/84/58284/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/84/58284/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/84/58284/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/84/58284/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/84/58284/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/84/58284/2"}}},"commit":{"parents":[{"commit":"ad2966d727df21653920d95dbf40711c2b8f2873","subject":"Merge \"Add hacking rules for shell scripts\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/ad2966d727df21653920d95dbf40711c2b8f2873"}]}],"author":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-11-25 13:44:10.000000000","tz":-480},"committer":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-11-25 14:04:12.000000000","tz":-480},"subject":"Default to /27 prefix for floating IP range with neutron","message":"Default to /27 prefix for floating IP range with neutron\n\nWhen running Tempest parallel tests with neutron, several long-running\ntests might create routers and floating IPs, which will result in\nIP allocations over the public network.\n\nIncreasing the public network size should ensure tests do not fail\ndue to IP address shortage.\n\nRelated-Bug: 1253966\n\nChange-Id: Ie075b3c4d14a07b06c42fd29b09770dd1972aa45\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6b813520318d1724d916ac1db43b28bf651a5605"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6b813520318d1724d916ac1db43b28bf651a5605"}]},"branch":"refs/heads/master"},"944e1e35194fe5b3d496971c4ee38c4f1519c958":{"kind":"REWORK","_number":3,"created":"2013-12-09 23:17:52.000000000","uploader":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"ref":"refs/changes/84/58284/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/84/58284/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/84/58284/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/84/58284/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/84/58284/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/84/58284/3"}}},"commit":{"parents":[{"commit":"ad2966d727df21653920d95dbf40711c2b8f2873","subject":"Merge \"Add hacking rules for shell scripts\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/ad2966d727df21653920d95dbf40711c2b8f2873"}]}],"author":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-11-25 13:44:10.000000000","tz":-480},"committer":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-12-09 23:17:19.000000000","tz":-480},"subject":"Default to /24 prefix for floating IP range with neutron","message":"Default to /24 prefix for floating IP range with neutron\n\nWhen running Tempest parallel tests with neutron, several long-running\ntests might create routers and floating IPs, which will result in\nIP allocations over the public network.\n\nIncreasing the public network size should ensure tests do not fail\ndue to IP address shortage; this patch also updates the public network\ngateway IP address.\n\nRelated-Bug: 1253966\n\nChange-Id: Ie075b3c4d14a07b06c42fd29b09770dd1972aa45\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/944e1e35194fe5b3d496971c4ee38c4f1519c958"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/944e1e35194fe5b3d496971c4ee38c4f1519c958"}]},"branch":"refs/heads/master"},"90234ac4b03426d844b72d251d4cae13fa09cde5":{"kind":"REWORK","_number":4,"created":"2013-12-15 12:39:16.000000000","uploader":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"ref":"refs/changes/84/58284/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/84/58284/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/84/58284/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/84/58284/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/84/58284/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/84/58284/4"}}},"commit":{"parents":[{"commit":"ad2966d727df21653920d95dbf40711c2b8f2873","subject":"Merge \"Add hacking rules for shell scripts\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/ad2966d727df21653920d95dbf40711c2b8f2873"}]}],"author":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-11-25 13:44:10.000000000","tz":-480},"committer":{"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","date":"2013-12-15 12:25:20.000000000","tz":-480},"subject":"Default to /24 prefix for floating IP range with neutron","message":"Default to /24 prefix for floating IP range with neutron\n\nWhen running Tempest parallel tests with neutron, several long-running\ntests might create routers and floating IPs, which will result in\nIP allocations over the public network.\n\nIncreasing the public network size should ensure tests do not fail\ndue to IP address shortage; this patch also updates the public network\ngateway IP address.\n\nRelated-Bug: 1253966\n\nChange-Id: Ie075b3c4d14a07b06c42fd29b09770dd1972aa45\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/90234ac4b03426d844b72d251d4cae13fa09cde5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/90234ac4b03426d844b72d251d4cae13fa09cde5"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}