)]}'
{"id":"openstack%2Fdevstack~38897","triplet_id":"openstack%2Fdevstack~master~I9782d1e3cda3c9dd3daefa15c043f5b06473cb87","project":"openstack/devstack","branch":"master","topic":"cells_fixes","hashtags":[],"change_id":"I9782d1e3cda3c9dd3daefa15c043f5b06473cb87","subject":"Fix various things when n-cell is enabled","status":"MERGED","created":"2013-07-27 01:15:19.000000000","updated":"2013-07-30 01:14:31.000000000","submitted":"2013-07-30 01:14:30.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":6,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d1d140044ba053d8c588cb4a467e8e5e6ff385d2","_number":38897,"virtual_id_number":38897,"owner":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},{"value":2,"date":"2013-07-30 01:14:30.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":2,"date":"2013-07-29 19:20:36.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},{"value":2,"date":"2013-07-30 00:20:06.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":1,"date":"2013-07-29 23:48:35.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"}],"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"date":"2013-07-29 19:20:36.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},{"value":1,"date":"2013-07-30 00:20:06.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},{"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-07-27 02:06:55.000000000","updated_by":{"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},"reviewer":{"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},"state":"REVIEWER"},{"updated":"2013-07-27 18:05:54.000000000","updated_by":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"reviewer":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"state":"REVIEWER"},{"updated":"2013-07-29 19:20:36.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-07-29 23:48:35.000000000","updated_by":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"reviewer":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"state":"REVIEWER"},{"updated":"2013-07-30 00:20:06.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-07-30 01:14:30.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"6f5bee350e8b09c5d980aa83bddb2670b880f26b","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-07-27 01:16:33.000000000","message":"Patch Set 1:\n\nThis patch allows exercise.sh to pass with n-cell enabled: http://paste.openstack.org/show/42074/","accounts_in_message":[],"_revision_number":1},{"id":"f480c82fb2c0d1a71eb0df2f8e1ff16a5f6a5d3c","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-07-27 01:20:17.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"0dc19d53002efbf89e5d56509bf8631108d12027","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-07-27 01:20:22.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":"ef84517596c0a5bfeee03a81012ecd5402191ffb","author":{"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},"date":"2013-07-27 02:06:55.000000000","message":"Patch Set 2: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"b71f18da813e8a789234e7d22b7a6972309fe50d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-27 02:20:59.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/97/38897/2/check/gate-tempest-devstack-vm-full/36296 : SUCCESS in 53m 25s\n- http://logs.openstack.org/97/38897/2/check/gate-tempest-devstack-vm-cells-full/3357 : FAILURE in 36m 28s (non-voting)\n- http://logs.openstack.org/97/38897/2/check/gate-tempest-devstack-vm-testr-full/2181 : FAILURE in 31m 53s (non-voting)\n- http://logs.openstack.org/97/38897/2/check/gate-tempest-devstack-vm-postgres-full/35976 : SUCCESS in 53m 08s\n- http://logs.openstack.org/97/38897/2/check/gate-tempest-devstack-vm-neutron/8334 : FAILURE in 25m 24s (non-voting)\n- http://logs.openstack.org/97/38897/2/check/gate-grenade-devstack-vm/21794 : SUCCESS in 24m 17s\n","accounts_in_message":[],"_revision_number":2},{"id":"395bf2b55c63bb3e4b24af107034f947ca0687a5","author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"date":"2013-07-27 03:48:46.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"8f0db3d1f221f7605f077fc0f1e827741d4779a5","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-07-27 12:33:25.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nlets fix up the tabs, then I\u0027m ok with it. Though part of me wants to just skip the exercises that do sec group things instead of fixing them for cells instead, as we\u0027ll probably turn that off soon regardless.\n\nWhat\u0027s the plan to get floating ips and sec groups in cells anyway?","accounts_in_message":[],"_revision_number":2},{"id":"87b67bfebe9f71b8823028b821892289dddeec98","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-07-27 14:08:01.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\n(2 inline comments)\n\nThe easy fix for screen_it() is to add an exception to is_service_enabled() for \"n-cell-\" similar to all of the others.","accounts_in_message":[],"_revision_number":2},{"id":"dda16ccc8f98ad63437f5463695c434ad5d690be","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-07-27 17:34:32.000000000","message":"Patch Set 2: (1 inline comment)\n\n@Sean:  Unfortunately mostly everything is using security groups one way or another despite sec_groups having its own exercise.  I\u0027d rather not completely disable euca.sh, because it is a valuable test, otherwise.  Same with boot_from_volume, etc.  Unfortunately this leads to a bit of a mess, however, checking for cells for those things.\n\nI\u0027ll reply on the mailing list about future plans.\n\n@Dean: Thanks, I didn\u0027t think to modify is_service_available.  I hated to modify screen_it, also. :)\n\nNew patch set coming...","accounts_in_message":[],"_revision_number":2},{"id":"7ee4b01bc7800e1b5344885c557dfa1e680e47ba","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-07-27 17:47:01.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"892ba03e9307771b204d4fa4a9552a0184a37dc7","author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"date":"2013-07-27 18:05:54.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\nConfirmed working for me.  Except volumes, but that appears to be an issue with my environment.","accounts_in_message":[],"_revision_number":3},{"id":"5f388baafd27e420a8b0e260047ea446575d47d4","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-07-27 18:12:41.000000000","message":"Patch Set 3:\n\nYa, I had to start tgtd manually on my system...","accounts_in_message":[],"_revision_number":3},{"id":"743118d96325782c16546e88d2bb64f0a2493a3f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-27 18:43:41.000000000","message":"Patch Set 3: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/97/38897/3/check/gate-tempest-devstack-vm-full/36644 : SUCCESS in 42m 54s\n- http://logs.openstack.org/97/38897/3/check/gate-tempest-devstack-vm-cells-full/3456 : FAILURE in 32m 40s (non-voting)\n- http://logs.openstack.org/97/38897/3/check/gate-tempest-devstack-vm-testr-full/2280 : FAILURE in 41m 52s (non-voting)\n- http://logs.openstack.org/97/38897/3/check/gate-tempest-devstack-vm-postgres-full/36321 : SUCCESS in 56m 35s\n- http://logs.openstack.org/97/38897/3/check/gate-tempest-devstack-vm-neutron/8686 : FAILURE in 15m 31s (non-voting)\n- http://logs.openstack.org/97/38897/3/check/gate-grenade-devstack-vm/22096 : FAILURE in 18m 23s\n","accounts_in_message":[],"_revision_number":3},{"id":"28a051bbb2c887de78668539f07d07385f781005","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-07-27 18:55:06.000000000","message":"Patch Set 3:\n\nrecheck no bug\n\ncool, the cells gate made it to the tempest suite...","accounts_in_message":[],"_revision_number":3},{"id":"ef988d5aa53459b9b5d03e64433074537d653d5e","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-07-28 15:10:06.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nLooking good.\n\nI\u0027m only concerned about the corner case mentioned in the comment because of the number of times patches are submitted that just copy an existing pattern without fully understanding it.","accounts_in_message":[],"_revision_number":3},{"id":"10598f00f7706e8aff4ffd41d8c816e7231f830d","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-07-29 05:17:54.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"1036accd2c4f4dd2b091ede2987659373e5e2d77","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-29 06:13:25.000000000","message":"Patch Set 4: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/97/38897/4/check/gate-tempest-devstack-vm-full/36855 : SUCCESS in 45m 31s\n- http://logs.openstack.org/97/38897/4/check/gate-tempest-devstack-vm-cells-full/3622 : FAILURE in 24m 54s (non-voting)\n- http://logs.openstack.org/97/38897/4/check/gate-tempest-devstack-vm-testr-full/2446 : SUCCESS in 29m 57s (non-voting)\n- http://logs.openstack.org/97/38897/4/check/gate-tempest-devstack-vm-postgres-full/36532 : SUCCESS in 55m 26s\n- http://logs.openstack.org/97/38897/4/check/gate-tempest-devstack-vm-neutron/8903 : FAILURE in 16m 14s (non-voting)\n- http://logs.openstack.org/97/38897/4/check/gate-grenade-devstack-vm/22278 : SUCCESS in 22m 18s\n","accounts_in_message":[],"_revision_number":4},{"id":"1e97ee76193d4ec565879c56ff334ee263c77dd8","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-07-29 19:20:36.000000000","message":"Patch Set 4: Looks good to me (core reviewer)\n\nThanks Chris","accounts_in_message":[],"_revision_number":4},{"id":"d1192493f5990ea6a76907a60e51221641a12cba","author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"date":"2013-07-29 23:48:35.000000000","message":"Patch Set 4: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":4},{"id":"a6931eca90ba818b376e72eed8439cff22dcb2e7","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-07-30 00:20:06.000000000","message":"Patch Set 4: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":4},{"id":"b06b212794d65701e1ecfd854e69c5efc423e93a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-30 00:20:10.000000000","message":"Patch Set 4: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":4},{"id":"ee5632add35ceae9259bb6ef528d186c82d94dc0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-30 01:14:30.000000000","message":"Patch Set 4: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/97/38897/4/gate/gate-tempest-devstack-vm-full/37173 : SUCCESS in 45m 57s\n- http://logs.openstack.org/97/38897/4/gate/gate-tempest-devstack-vm-postgres-full/36849 : SUCCESS in 54m 17s\n- http://logs.openstack.org/97/38897/4/gate/gate-tempest-devstack-vm-neutron/9253 : FAILURE in 17m 36s (non-voting)\n- http://logs.openstack.org/97/38897/4/gate/gate-grenade-devstack-vm/22564 : SUCCESS in 22m 07s\n","accounts_in_message":[],"_revision_number":4},{"id":"2b09c664088952a24183a348653c144523f9a0bd","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-30 01:14:31.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"c62c2b9b6617c731c7979d490a1d2e7a048c1cf7","revisions":{"9248084669e7d16df3d392d9c1aefd5da38d5136":{"kind":"REWORK","_number":1,"created":"2013-07-27 01:15:19.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/97/38897/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/97/38897/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/97/38897/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/97/38897/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/97/38897/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/97/38897/1"}}},"commit":{"parents":[{"commit":"c41a0598646aad3520861c185c5df370f95301a7","subject":"Merge \"Use vmware section\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c41a0598646aad3520861c185c5df370f95301a7"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-07-24 10:56:13.000000000","tz":-420},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-07-24 14:09:05.000000000","tz":-420},"subject":"Fix various things when n-cell is enabled","message":"Fix various things when n-cell is enabled\n\n1) Use the newer cell_type configuration option instead of overriding\nthe compute_api_class.\n\n2) A nova-cells service is started for both the API cell (region) and\nthe compute cell (child).  The screen names were both \u0027n-cell\u0027 which\ncaused both services to log to the same log file.  The screen names have\nbeen changed to n-cell-api and n-cell-compute.\n\n3) Security groups are not supported with cells.  Multiple exercises are\npatched to use the \u0027default\u0027 security group instead of creating a new\none.  Additionally, if one uses the \u0027default\u0027 security group, do not try\nto delete it, as one can never delete the default.\n\n4) Disable the floating_ips and aggregates exercises when n-cell is\nenabled, as they are not supported by cells.\n\n5) Related to #4, disable the floating_ips tests within euca.sh\n\n6) Update the README.md.  No services need to be disabled, and one only\nneeds to enable the n-cell service.\n\nChange-Id: I9782d1e3cda3c9dd3daefa15c043f5b06473cb87\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/9248084669e7d16df3d392d9c1aefd5da38d5136"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/9248084669e7d16df3d392d9c1aefd5da38d5136"}]},"branch":"refs/heads/master"},"9e0db4bdc46132ef7a98b0f9a86e51e1be8647fd":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2013-07-27 01:20:17.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/97/38897/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/97/38897/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/97/38897/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/97/38897/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/97/38897/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/97/38897/2"}}},"commit":{"parents":[{"commit":"c41a0598646aad3520861c185c5df370f95301a7","subject":"Merge \"Use vmware section\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c41a0598646aad3520861c185c5df370f95301a7"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-07-24 10:56:13.000000000","tz":-420},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-07-24 14:13:53.000000000","tz":-420},"subject":"Fix various things when n-cell is enabled","message":"Fix various things when n-cell is enabled\n\n1) Use the newer cell_type configuration option instead of overriding\nthe compute_api_class.\n\n2) A nova-cells service is started for both the API cell (region) and\nthe compute cell (child).  The screen names were both \u0027n-cell\u0027 which\ncaused both services to log to the same log file.  The screen names have\nbeen changed to n-cell-region and n-cell-child.\n\n3) Security groups are not supported with cells.  Multiple exercises are\npatched to use the \u0027default\u0027 security group instead of creating a new\none.  Additionally, if one uses the \u0027default\u0027 security group, do not try\nto delete it, as one can never delete the default.\n\n4) Disable the floating_ips and aggregates exercises when n-cell is\nenabled, as they are not supported by cells.\n\n5) Related to #4, disable the floating_ips tests within euca.sh\n\n6) Update the README.md.  No services need to be disabled, and one only\nneeds to enable the n-cell service.\n\nChange-Id: I9782d1e3cda3c9dd3daefa15c043f5b06473cb87\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/9e0db4bdc46132ef7a98b0f9a86e51e1be8647fd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/9e0db4bdc46132ef7a98b0f9a86e51e1be8647fd"}]},"branch":"refs/heads/master"},"6f4f1574e218b16d1e0ac4b10ba25f77fa883977":{"kind":"REWORK","_number":3,"created":"2013-07-27 17:47:01.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/97/38897/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/97/38897/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/97/38897/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/97/38897/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/97/38897/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/97/38897/3"}}},"commit":{"parents":[{"commit":"c41a0598646aad3520861c185c5df370f95301a7","subject":"Merge \"Use vmware section\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c41a0598646aad3520861c185c5df370f95301a7"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-07-24 10:56:13.000000000","tz":-420},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-07-25 06:28:49.000000000","tz":-420},"subject":"Fix various things when n-cell is enabled","message":"Fix various things when n-cell is enabled\n\n1) Use the newer cell_type configuration option instead of overriding\nthe compute_api_class.\n\n2) A nova-cells service is started for both the API cell (region) and\nthe compute cell (child).  The screen names were both \u0027n-cell\u0027 which\ncaused both services to log to the same log file.  The screen names have\nbeen changed to n-cell-region and n-cell-child.\n\n3) Security groups are not supported with cells.  Multiple exercises are\npatched to use the \u0027default\u0027 security group instead of creating a new\none.  Additionally, if one uses the \u0027default\u0027 security group, do not try\nto delete it, as one can never delete the default.\n\n4) Disable the floating_ips and aggregates exercises when n-cell is\nenabled, as they are not supported by cells.\n\n5) Related to #4, disable the floating_ips tests within euca.sh\n\n6) Update the README.md.  No services need to be disabled, and one only\nneeds to enable the n-cell service.\n\nChange-Id: I9782d1e3cda3c9dd3daefa15c043f5b06473cb87\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6f4f1574e218b16d1e0ac4b10ba25f77fa883977"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6f4f1574e218b16d1e0ac4b10ba25f77fa883977"}]},"branch":"refs/heads/master"},"c62c2b9b6617c731c7979d490a1d2e7a048c1cf7":{"kind":"REWORK","_number":4,"created":"2013-07-29 05:17:54.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/97/38897/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/97/38897/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/97/38897/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/97/38897/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/97/38897/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/97/38897/4"}}},"commit":{"parents":[{"commit":"c41a0598646aad3520861c185c5df370f95301a7","subject":"Merge \"Use vmware section\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c41a0598646aad3520861c185c5df370f95301a7"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-07-24 10:56:13.000000000","tz":-420},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-07-26 18:11:25.000000000","tz":-420},"subject":"Fix various things when n-cell is enabled","message":"Fix various things when n-cell is enabled\n\n1) Use the newer cell_type configuration option instead of overriding\nthe compute_api_class.\n\n2) A nova-cells service is started for both the API cell (region) and\nthe compute cell (child).  The screen names were both \u0027n-cell\u0027 which\ncaused both services to log to the same log file.  The screen names have\nbeen changed to n-cell-region and n-cell-child.\n\n3) Security groups are not supported with cells.  Multiple exercises are\npatched to use the \u0027default\u0027 security group instead of creating a new\none.  Additionally, if one uses the \u0027default\u0027 security group, do not try\nto delete it, as one can never delete the default.\n\n4) Disable the floating_ips and aggregates exercises when n-cell is\nenabled, as they are not supported by cells.\n\n5) Related to #4, disable the floating_ips tests within euca.sh\n\n6) Update the README.md.  No services need to be disabled, and one only\nneeds to enable the n-cell service.\n\nChange-Id: I9782d1e3cda3c9dd3daefa15c043f5b06473cb87\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c62c2b9b6617c731c7979d490a1d2e7a048c1cf7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c62c2b9b6617c731c7979d490a1d2e7a048c1cf7"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}