)]}'
{"id":"openstack%2Fdevstack~53468","triplet_id":"openstack%2Fdevstack~master~I633344c8784c154f61e751cd0a408196e61525b3","project":"openstack/devstack","branch":"master","topic":"bug/1243961","hashtags":[],"change_id":"I633344c8784c154f61e751cd0a408196e61525b3","subject":"Start nova-compute with child cell conf","status":"MERGED","created":"2013-10-23 22:31:20.000000000","updated":"2013-10-24 21:03:34.000000000","submitted":"2013-10-24 21:03:34.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"dd2268c0550af2ccf460d2ca0732cd8f20005402","_number":53468,"virtual_id_number":53468,"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":2,"date":"2013-10-24 21:03:34.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":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"recommended":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-10-24 16:27:14.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":1,"date":"2013-10-24 19:09:31.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":2,"date":"2013-10-24 20:01:48.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2013-10-24 19:33:19.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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"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":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":1,"date":"2013-10-24 20:01:48.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2013-10-24 19:33:19.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":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-10-24 19:09:31.000000000","updated_by":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"reviewer":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"state":"REVIEWER"},{"updated":"2013-10-24 19:33:19.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-10-24 20:01: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":"2013-10-24 21:03:34.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"3bb771551c415df3e25d4892e65945ba8956a3a2","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2013-10-23 22:47:05.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"7dc8e6982e3e6153261b3fa30a15dfaa41bfde15","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-10-23 22:50:42.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"24f053931ebb5168bf03b7e860c5042d17137225","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-10-23 22:50:49.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":"7626ab68a9c175e9c0e55484a8a47bcd12751958","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-23 23:34:01.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/68/53468/2/check/gate-devstack-bash8/b58f5b7 : SUCCESS in 4s\n- check-tempest-devstack-vm-full http://logs.openstack.org/68/53468/2/check/check-tempest-devstack-vm-full/46dd31c : SUCCESS in 27m 43s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/68/53468/2/check/check-tempest-devstack-vm-postgres-full/60c220a : SUCCESS in 43m 01s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/68/53468/2/check/check-tempest-devstack-vm-neutron/8e5a504 : SUCCESS in 29m 30s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/68/53468/2/check/gate-tempest-devstack-vm-large-ops/16c6878 : SUCCESS in 15m 54s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/68/53468/2/check/gate-tempest-devstack-vm-neutron-large-ops/f5160fa : SUCCESS in 17m 25s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/68/53468/2/check/check-grenade-devstack-vm/021e6a5 : SUCCESS in 35m 35s\n- check-swift-devstack-vm-functional http://logs.openstack.org/68/53468/2/check/check-swift-devstack-vm-functional/1a20531 : SUCCESS in 10m 59s\n","accounts_in_message":[],"_revision_number":2},{"id":"6345ceb5153c83a84562e30d305d0d459fbaa36b","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-10-24 11:42:46.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\ngiven this is shell, all the variables are in a global namespace by default, so I actually think we should either explicitly use \u0027local\u0027 to prevent the two functions from changing the same globals, or pull out the NOVA_CONF_BOTTOM logic from the functions and put it inline above them. Because  today the interaction in changing that variable would work as written, but might be a little surprising to people what\u0027s changing what and when.","accounts_in_message":[],"_revision_number":2},{"id":"faf13bde7f57a32ac1af93eb547315c888cc146a","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-10-24 15:59:10.000000000","message":"Patch Set 2:\n\nYeah, I was going to clean all of that up in a follow up patch I have\n\nI\u0027ll just include it all in here.","accounts_in_message":[],"_revision_number":2},{"id":"28ffe95faa434dc4692fc1c78a42fd133d3ba7d0","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-10-24 16:26:31.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"44e24128c52142a74052c3415c062e5f223ab1d1","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-10-24 16:27:14.000000000","message":"Patch Set 3:\n\ncleaning up the unrelated method here:\n\nhttps://review.openstack.org/#/c/53681/","accounts_in_message":[],"_revision_number":3},{"id":"03e194a5170dd53d5170a254686b880b99cf5b07","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-24 17:28:04.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/68/53468/3/check/gate-devstack-bash8/70110f8 : SUCCESS in 4s\n- check-tempest-devstack-vm-full http://logs.openstack.org/68/53468/3/check/check-tempest-devstack-vm-full/3fe38f1 : SUCCESS in 30m 41s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/68/53468/3/check/check-tempest-devstack-vm-postgres-full/c06194c : SUCCESS in 1h 01m 17s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/68/53468/3/check/check-tempest-devstack-vm-neutron/d4f4dc6 : SUCCESS in 18m 53s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/68/53468/3/check/gate-tempest-devstack-vm-large-ops/920a7b3 : SUCCESS in 15m 25s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/68/53468/3/check/gate-tempest-devstack-vm-neutron-large-ops/0115296 : SUCCESS in 17m 25s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/68/53468/3/check/check-grenade-devstack-vm/6007948 : SUCCESS in 25m 01s\n- check-swift-devstack-vm-functional http://logs.openstack.org/68/53468/3/check/check-swift-devstack-vm-functional/1fe2e46 : SUCCESS in 11m 42s\n","accounts_in_message":[],"_revision_number":3},{"id":"ef368034a0eb4ac66b12d2e961c348eacb13986e","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2013-10-24 19:09:31.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":3},{"id":"0ff59c957dc5cff2c15bb3a38b38a9fe0ea50811","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-10-24 19:33:19.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"ba04eaf8fa044fe208f3539cfec6f0a20d1a9072","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-24 20:01:48.000000000","message":"Patch Set 3: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"b42ab8c5e7d9d5c2f061ad118c0e71754933175e","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-10-24 20:01:48.000000000","message":"Patch Set 3: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":3},{"id":"f17f2138538759830c41db3a55651981c2ce369c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-24 21:03:34.000000000","message":"Patch Set 3: Verified\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/68/53468/3/gate/gate-devstack-bash8/e51d850 : SUCCESS in 5s\n- gate-tempest-devstack-vm-full http://logs.openstack.org/68/53468/3/gate/gate-tempest-devstack-vm-full/2a00477 : SUCCESS in 29m 36s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/68/53468/3/gate/gate-tempest-devstack-vm-postgres-full/78feb3f : SUCCESS in 31m 08s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/68/53468/3/gate/gate-tempest-devstack-vm-neutron/fd5dc08 : SUCCESS in 19m 09s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/68/53468/3/gate/gate-tempest-devstack-vm-large-ops/2ba2a7b : SUCCESS in 14m 44s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/68/53468/3/gate/gate-tempest-devstack-vm-neutron-large-ops/98fdef3 : SUCCESS in 18m 13s (non-voting)\n- gate-grenade-devstack-vm http://logs.openstack.org/68/53468/3/gate/gate-grenade-devstack-vm/26af5ca : SUCCESS in 28m 00s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/68/53468/3/gate/gate-swift-devstack-vm-functional/d2809fa : SUCCESS in 11m 33s\n","accounts_in_message":[],"_revision_number":3},{"id":"a510f9ca197601cfd9bec899b93163fd1f6e197e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-24 21:03:34.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"6db29904df63ae26a0f4a4b2e4e0c2e6f2cef669","revisions":{"65862ec714f1327698d90a6538b0eb1cb3157419":{"kind":"REWORK","_number":1,"created":"2013-10-23 22:31:20.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/68/53468/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/68/53468/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/53468/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/53468/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/53468/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/68/53468/1"}}},"commit":{"parents":[{"commit":"897f48a897f9404a6825275c97ae386f0994c0e6","subject":"Merge \"Add the doc build tools\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/897f48a897f9404a6825275c97ae386f0994c0e6"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-10-22 16:22:36.000000000","tz":-420},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-10-22 16:22:36.000000000","tz":-420},"subject":"Start nova-compute with child cell conf","message":"Start nova-compute with child cell conf\n\nA recent commit broke the cells support and switched nova-compute to always start with the API cell .conf.  This corrects the regression.\n\nChange-Id: I633344c8784c154f61e751cd0a408196e61525b3\nCloses-bug: 1243961\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/65862ec714f1327698d90a6538b0eb1cb3157419"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/65862ec714f1327698d90a6538b0eb1cb3157419"}]},"branch":"refs/heads/master"},"1711257a136546c6309be2147f8fed0fbcbf35d0":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2013-10-23 22:50:42.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/68/53468/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/68/53468/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/53468/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/53468/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/53468/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/68/53468/2"}}},"commit":{"parents":[{"commit":"897f48a897f9404a6825275c97ae386f0994c0e6","subject":"Merge \"Add the doc build tools\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/897f48a897f9404a6825275c97ae386f0994c0e6"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-10-22 16:22:36.000000000","tz":-420},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-10-22 16:42:43.000000000","tz":-420},"subject":"Start nova-compute with child cell conf","message":"Start nova-compute with child cell conf\n\nA recent commit broke the cells support and switched nova-compute to always\nstart with the API cell .conf.  This corrects the regression.\n\nChange-Id: I633344c8784c154f61e751cd0a408196e61525b3\nCloses-bug: 1243961\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/1711257a136546c6309be2147f8fed0fbcbf35d0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/1711257a136546c6309be2147f8fed0fbcbf35d0"}]},"branch":"refs/heads/master"},"6db29904df63ae26a0f4a4b2e4e0c2e6f2cef669":{"kind":"REWORK","_number":3,"created":"2013-10-24 16:26:31.000000000","uploader":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"ref":"refs/changes/68/53468/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/68/53468/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/53468/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/53468/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/53468/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/68/53468/3"}}},"commit":{"parents":[{"commit":"897f48a897f9404a6825275c97ae386f0994c0e6","subject":"Merge \"Add the doc build tools\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/897f48a897f9404a6825275c97ae386f0994c0e6"}]}],"author":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-10-22 16:22:36.000000000","tz":-420},"committer":{"name":"Chris Behrens","email":"cbehrens@codestud.com","date":"2013-10-23 09:56:58.000000000","tz":-420},"subject":"Start nova-compute with child cell conf","message":"Start nova-compute with child cell conf\n\nA recent commit broke the cells support and switched nova-compute to always\nstart with the API cell .conf.  This corrects the regression.\n\nChange-Id: I633344c8784c154f61e751cd0a408196e61525b3\nCloses-bug: 1243961\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6db29904df63ae26a0f4a4b2e4e0c2e6f2cef669"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6db29904df63ae26a0f4a4b2e4e0c2e6f2cef669"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}