)]}'
{"id":"openstack%2Fdevstack~608703","triplet_id":"openstack%2Fdevstack~master~I75f41d2633b751977b12eda6fccae9521dabd0be","project":"openstack/devstack","branch":"master","topic":"configure-discover_hosts_in_cells_interval","hashtags":[],"change_id":"I75f41d2633b751977b12eda6fccae9521dabd0be","subject":"nova: Configure discover_hosts_in_cells_interval for multi-host","status":"ABANDONED","created":"2018-10-08 15:22:47.000000000","updated":"2018-10-09 15:58:02.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"ef96361335b63d75bed723292763e988372b409e","_number":608703,"virtual_id_number":608703,"owner":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"date":"2018-10-08 18:24:45.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"date":"2018-10-08 17:00:45.000000000","_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"date":"2018-10-08 16:51:36.000000000","_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"date":"2018-10-08 16:32:04.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":1,"date":"2018-10-08 18:16:00.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"value":-1,"date":"2018-10-08 16:15:35.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-10-08 16:15:35.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":"2018-10-08 16:32:04.000000000","updated_by":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2018-10-08 17:00:45.000000000","updated_by":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2018-10-08 18:16:00.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2018-10-08 18:24:45.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"08bfff32357a275996a1115830432a9acda40343","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2018-10-08 15:22:47.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"4c8025108540b1d012a219be86402f8ce6c91404","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2018-10-08 16:15:35.000000000","message":"Patch Set 1: Code-Review-1\n\nWhy exactly?\n\nThis interval was basically a hack for ironic deployments that didn\u0027t have discoverable nodes until after ironic resources were enrolled and exposed. I think that now that we can discover by service this is largely unnecessary in most cases. For devstack, it would only be a thing for multinode setups, which are very much the exception.\n\nIs there some real situation that brought this up? Barring one, I think I\u0027m -1 on just enabling this \"because.\"","accounts_in_message":[],"_revision_number":1},{"id":"4ba8a274c32ca026c5287da88947797243f2c42a","author":{"_account_id":10118,"name":"IBM PowerKVM CI","email":"kvmpower@linux.vnet.ibm.com","username":"powerkvm","tags":["SERVICE_USER"]},"date":"2018-10-08 16:32:04.000000000","message":"Patch Set 1:\n\nBuild succeeded. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm: recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\n- tempest-dsvm-full-xenial https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/devstack/03/608703/1/check/tempest-dsvm-full-xenial/ba57413/ : SUCCESS in 1h 09m 03s","accounts_in_message":[],"_revision_number":1},{"id":"8ad8743d749dbc73c2b7ba2b0b40ccceafb85a05","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2018-10-08 16:51:36.000000000","message":"Patch Set 1:\n\n\u003e Why exactly?\n \u003e \n \u003e This interval was basically a hack for ironic deployments that\n \u003e didn\u0027t have discoverable nodes until after ironic resources were\n \u003e enrolled and exposed. I think that now that we can discover by\n \u003e service this is largely unnecessary in most cases. For devstack, it\n \u003e would only be a thing for multinode setups, which are very much the\n \u003e exception.\n \u003e \n \u003e Is there some real situation that brought this up? Barring one, I\n \u003e think I\u0027m -1 on just enabling this \"because.\"\n\nAs noted in the commit message\n\n\u003e Deploy a multi-host configuration and try to boot an instance and\n\u003e you\u0027ll be greeted with the following:\n\u003e\n\u003e   Host \u0027foo\u0027 is not mapped to any cell.\n\u003e\n\u003e The solution, as noted in the nova docs [1], is clear: either run\n\u003e \u0027nova-manage cell_v2 discover_hosts\u0027 on the controller or configure\n\u003e \u0027[scheduler] discover_hosts_in_cells_interval\u0027 so that nova will do this\n\u003e for us. For the sanity of our users, let\u0027s do the latter.\n\nSo it\u0027s not \"because\" but rather because it seemed like the more automatic of the two methods of the the docs tell me I can and should use to resolve my issue. If this option is a hack then I can call out the need to run nova-manage in the multi-host doc instead, but the option should really be marked as advanced in nova, it should have it\u0027s hackiness mentioned as part of the option help text and that document should not tell people to use it. I can fix all of these but only if it\u0027s a hack (I don\u0027t know)","accounts_in_message":[],"_revision_number":1},{"id":"afe4f0ffa41072da60988375f7ac53f8b2a87287","author":{"_account_id":14595,"name":"z Systems KVM","email":"zkvm-ci@linux.vnet.ibm.com","username":"ibm-zkvm-ci","tags":["SERVICE_USER"]},"date":"2018-10-08 17:00:45.000000000","message":"Patch Set 1:\n\nTesting succeeded on ubuntu-xenial-s390x. For rechecking only on the ubuntu-xenail-s390x CI, add a review comment with recheck-zkvm. Contact info: zkvm-ci@linux.vnet.ibm.com. For more information, see https://wiki.openstack.org/wiki/ThirdPartySystems/IBM_zKVM_CI\n\n- check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x http://sng01.objectstorage.softlayer.net/v1/AUTH_1940ea10-6e82-4501-b2f9-eb236510e575/ibmzkvmci/production/608703/1/check-tempest-dsvm-neutron-full-ubuntu-xenial-s390x/5538807 : SUCCESS in 1h 37m 38s","accounts_in_message":[],"_revision_number":1},{"id":"1da5ad2502fd05a2cefda272947c235596033f95","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2018-10-08 17:08:56.000000000","message":"Patch Set 1:\n\nIt\u0027s definitely a hack, IMHO. We added it for ironic because they had a bunch of sequencing problems without it. Then with it, they still had problems because of the inherent raciness of needing to basically wait 2*interval before trying anything that assumed the hosts were up. After adding the by-service approach, they were finally able to have a repeatable process, and IIRC, tripleo removed the use of the periodic as a result.\n\nEspecially given the \"why doesn\u0027t my ... oh, 30s later it does, hmm\" behavior that people would experience, I\u0027d much rather have it in the multinode docs explicitly.","accounts_in_message":[],"_revision_number":1},{"id":"56f67bbe220d5327844390579600db4480d8413a","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2018-10-08 17:13:12.000000000","message":"Patch Set 1:\n\nSome context: https://bugs.launchpad.net/nova/+bug/1755602","accounts_in_message":[],"_revision_number":1},{"id":"d90b6abed0cca81c547b41a3702e799111220c76","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-10-08 18:16:00.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- legacy-devstack-dsvm-updown http://logs.openstack.org/03/608703/1/check/legacy-devstack-dsvm-updown/0c1ce12/ : SUCCESS in 30m 24s\n- tempest-full http://logs.openstack.org/03/608703/1/check/tempest-full/b9f0d3c/ : SUCCESS in 1h 29m 18s\n- neutron-grenade http://logs.openstack.org/03/608703/1/check/neutron-grenade/d8b5891/ : SUCCESS in 52m 43s\n- tempest-full-py3 http://logs.openstack.org/03/608703/1/check/tempest-full-py3/9dad85b/ : SUCCESS in 1h 26m 06s\n- openstack-tox-docs http://logs.openstack.org/03/608703/1/check/openstack-tox-docs/ff05944/html/ : SUCCESS in 4m 20s\n- devstack http://logs.openstack.org/03/608703/1/check/devstack/550242b/ : SUCCESS in 22m 53s\n- devstack-platform-centos-7 http://logs.openstack.org/03/608703/1/check/devstack-platform-centos-7/7a41fe3/ : FAILURE in 1h 38m 12s (non-voting)\n- devstack-platform-opensuse-423 http://logs.openstack.org/03/608703/1/check/devstack-platform-opensuse-423/c141f3f/ : SUCCESS in 1h 32m 04s (non-voting)\n- devstack-platform-opensuse-tumbleweed http://logs.openstack.org/03/608703/1/check/devstack-platform-opensuse-tumbleweed/db609f5/ : FAILURE in 1h 23m 46s (non-voting)\n- devstack-platform-fedora-latest http://logs.openstack.org/03/608703/1/check/devstack-platform-fedora-latest/743bc2f/ : FAILURE in 1h 29m 15s (non-voting)\n- devstack-multinode http://logs.openstack.org/03/608703/1/check/devstack-multinode/7c3347b/ : SUCCESS in 31m 40s\n- devstack-unit-tests http://logs.openstack.org/03/608703/1/check/devstack-unit-tests/06c243f/ : SUCCESS in 2m 54s\n- openstack-tox-bashate http://logs.openstack.org/03/608703/1/check/openstack-tox-bashate/8fb4702/ : SUCCESS in 3m 50s\n- ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa http://logs.openstack.org/03/608703/1/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/5b975b7/ : SUCCESS in 48m 49s (non-voting)\n- swift-dsvm-functional http://logs.openstack.org/03/608703/1/check/swift-dsvm-functional/e7c8d18/ : SUCCESS in 23m 00s (non-voting)\n- neutron-grenade-multinode http://logs.openstack.org/03/608703/1/check/neutron-grenade-multinode/24f6833/ : SUCCESS in 1h 13m 52s\n- neutron-tempest-linuxbridge http://logs.openstack.org/03/608703/1/check/neutron-tempest-linuxbridge/6d46254/ : SUCCESS in 1h 24m 26s\n- tempest-multinode-full http://logs.openstack.org/03/608703/1/check/tempest-multinode-full/2af6da3/ : SUCCESS in 2h 03m 32s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"6589f1bd5470913037b9f04f0149fd5168e648fc","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2018-10-08 18:24:45.000000000","message":"Patch Set 1:\n\nBuild succeeded.\n\n- dsvm-tempest-neutron-network http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/03/608703/1/check/dsvm-tempest-neutron-network/efa40b3 : SUCCESS in 2h 44m 44s","accounts_in_message":[],"_revision_number":1},{"id":"69a41d7be1caa5701ed09afdbae3e53ed8247417","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2018-10-09 15:58:02.000000000","message":"Abandoned\n\nTurns out it\u0027s already called out and I  simply ignored it\n\n\u003e Once the compute node services shows up, run the ./tools/discover_hosts.sh script from the control node to map compute hosts to the single cell.\n\nMoving onto the nova docs work.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"a2a13edbf061e6723d773668d585f2eeb13fa7d5","revisions":{"a2a13edbf061e6723d773668d585f2eeb13fa7d5":{"kind":"REWORK","_number":1,"created":"2018-10-08 15:22:47.000000000","uploader":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"ref":"refs/changes/03/608703/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/03/608703/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/03/608703/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/03/608703/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/03/608703/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/03/608703/1"}}},"commit":{"parents":[{"commit":"e1276ba32d65f14a297fc33a5d1d3e0ef1ceb11d","subject":"Merge \"Update horizon wsgi wrapper to the recommended one\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e1276ba32d65f14a297fc33a5d1d3e0ef1ceb11d"}]}],"author":{"name":"Stephen Finucane","email":"sfinucan@redhat.com","date":"2018-10-08 15:18:25.000000000","tz":60},"committer":{"name":"Stephen Finucane","email":"sfinucan@redhat.com","date":"2018-10-08 15:18:25.000000000","tz":60},"subject":"nova: Configure discover_hosts_in_cells_interval for multi-host","message":"nova: Configure discover_hosts_in_cells_interval for multi-host\n\nDeploy a multi-host configuration and try to boot an instance and you\u0027ll\nbe greeted with the following:\n\n  Host \u0027foo\u0027 is not mapped to any cell.\n\nThe solution, as noted in the nova docs [1], is clear: either run\n\u0027nova-manage cell_v2 discover_hosts\u0027 on the controller or configure\n\u0027[scheduler] discover_hosts_in_cells_interval\u0027 so that nova will do this\nfor us. For the sanity of our users, let\u0027s do the latter.\n\n[1] https://docs.openstack.org/nova/rocky/user/cells.html#fresh-install\n\nChange-Id: I75f41d2633b751977b12eda6fccae9521dabd0be\nSigned-off-by: Stephen Finucane \u003csfinucan@redhat.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a2a13edbf061e6723d773668d585f2eeb13fa7d5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a2a13edbf061e6723d773668d585f2eeb13fa7d5"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}