)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"916cbb37294b0a1d566f7b293561a97f3c2939ed","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[Docs] Update the compute node installation steps"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"backport: queens pike"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Idf66b0eb81098560d58adf1f19d61ee72947f54f"},{"line_number":12,"context_line":"Closes-Bug: #1785007"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_dd12cd58","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":21},"updated":"2018-08-08 16:32:49.000000000","message":"You can remove this, it\u0027s tracked in the launchpad bug.\n\nAlso, commit messages should explain what the problem is that you\u0027re trying to solve and how your change solves it. The problem is described in the bug:\n\n\"\"\"\nIn below section \"Add the compute node to the cell database\", number 1 is printing nova service for compute node, then number 2 is discovering compute node (and include it in nova service element). This is confusing; if we do like this, the result of number 1 will not show new compute. *This is inaccurate.*\n\nSuggestion:\nIt should be, number 2 (perform discovering compute node), after that do number 1 (shown nova service compute node).\n\"\"\"","commit_id":"a92cb9d6d2187e654bf5e203aa49cc5af314ab12"}],"doc/source/install/compute-install-obs.rst":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"916cbb37294b0a1d566f7b293561a97f3c2939ed","unresolved":false,"context_lines":[{"line_number":251,"context_line":""},{"line_number":252,"context_line":"   Run the following commands on the **controller** node."},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"#. Source the admin credentials to enable admin-only CLI commands, then confirm"},{"line_number":255,"context_line":"   there are compute hosts in the database:"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"   .. code-block:: console"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3f79a3b5_3d81814b","side":"PARENT","line":254,"updated":"2018-08-08 16:32:49.000000000","message":"I think moving this is wrong. When you start a nova-compute service, the service record is created in the cell database and the service list operation (GET /os-services) is iterating all cells building the list of compute services. We have to wait to make sure the nova-compute service on a given host is registered before you can discover it, since that\u0027s how discover_hosts works - it looks up the compute node record in the cell database to see if it\u0027s \"mapped\". This is also what we tell people to do in the cells-specific install guide:\n\nhttps://docs.openstack.org/nova/latest/user/cells.html#fresh-install\n\nSee steps 6 and 7 in that section.","commit_id":"8688b25ca7379391cba28fab30b5a628957e673e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"916cbb37294b0a1d566f7b293561a97f3c2939ed","unresolved":false,"context_lines":[{"line_number":283,"context_line":""},{"line_number":284,"context_line":"      $ . admin-openrc"},{"line_number":285,"context_line":""},{"line_number":286,"context_line":"      $ openstack compute service list --service nova-compute"},{"line_number":287,"context_line":"      +----+-------+--------------+------+-------+---------+----------------------------+"},{"line_number":288,"context_line":"      | ID | Host  | Binary       | Zone | State | Status  | Updated At                 |"},{"line_number":289,"context_line":"      +----+-------+--------------+------+-------+---------+----------------------------+"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3f79a3b5_fd92299f","line":286,"updated":"2018-08-08 16:32:49.000000000","message":"This doesn\u0027t tell you that they are discovered/mapped, as noted above.","commit_id":"a92cb9d6d2187e654bf5e203aa49cc5af314ab12"}]}