)]}'
{"nova/templates/bin/_cell-setup.sh.tpl":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"109a04d1155448362ca22f46a0be681d9e1b6d09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"5f7c97a3_83199d89","updated":"2018-05-03 15:35:21.000000000","message":"we could do this via scheduler config - but it seems neater to explicitly provide this via a cron job.","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"},{"author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"change_message_id":"114b1dbe2106e8ed1cf283d9e683f0b57a069d29","unresolved":false,"context_lines":[{"line_number":23,"context_line":"# NOTE(portdirect): check if nova fully supports cells v2, and manage"},{"line_number":24,"context_line":"# accordingly. Support was complete in ocata (V14.x.x)."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"if [ \"${NOVA_VERSION%%.*}\" -gt \"14\" ]; then"},{"line_number":27,"context_line":"  nova-manage cell_v2 discover_hosts --verbose"},{"line_number":28,"context_line":"fi"}],"source_content_type":"text/x-smarty","patch_set":13,"id":"5f7c97a3_ace84516","line":26,"range":{"start_line":26,"start_character":31,"end_line":26,"end_character":35},"updated":"2018-05-03 18:30:32.000000000","message":"don\u0027t actually need quotes ...","commit_id":"18b622f4fe357a485b4f3eb15f9629d3e4fd2e77"}],"nova/templates/bin/_db-sync.sh.tpl":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"06583fb924741c4cfea6a5350ea57375e0cadc6f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"5f7c97a3_03602cce","updated":"2018-05-03 15:49:08.000000000","message":"Cell management isn\u0027t *really* related to db_sync so I\u0027m curious why the manage_cells() function wasn\u0027t put in the _cell-setup.sh script?","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"95c55cb892da3ef1a4a0f1999f113e91bd12f129","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"5f7c97a3_a3193889","in_reply_to":"5f7c97a3_03602cce","updated":"2018-05-03 16:16:39.000000000","message":"simply to replicate the setup described in steps 3-6 here: https://docs.openstack.org/nova/queens/install/controller-install-rdo.html#install-and-configure-components","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"e10140090ca2ab316af20590b448fef49279da7c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"5f7c97a3_c6bc82bd","in_reply_to":"5f7c97a3_a3193889","updated":"2018-05-03 16:45:50.000000000","message":"Sure, understood. I was just thinking that the cell manage stuff would go in the cell setup script.","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"06583fb924741c4cfea6a5350ea57375e0cadc6f","unresolved":false,"context_lines":[{"line_number":30,"context_line":"  fi"},{"line_number":31,"context_line":"}"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"# NOTE(portdirect): if the db has been populated we should setup cells if"},{"line_number":34,"context_line":"# required, otherwise we should poulate the api db, and then setup cells."},{"line_number":35,"context_line":"if [ \"$(nova-manage api_db version)\" -gt \"0\" ]; then"},{"line_number":36,"context_line":"  manage_cells"},{"line_number":37,"context_line":"  nova-manage api_db sync"},{"line_number":38,"context_line":"else"},{"line_number":39,"context_line":"  nova-manage api_db sync"},{"line_number":40,"context_line":"  manage_cells"},{"line_number":41,"context_line":"fi"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"nova-manage db sync"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-smarty","patch_set":12,"id":"5f7c97a3_23976893","line":41,"range":{"start_line":33,"start_character":0,"end_line":41,"end_character":2},"updated":"2018-05-03 15:49:08.000000000","message":"Pretty sure you should be above to always do this:\n\n nova-manage api_db sync\n manage_cells\n\nIn other words, I don\u0027t believe that the syncing the schema of the API database is going to adversely impact listing or mapping cells.","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"2eca6be37c2c2532169e0b23d40ef185f67b06bb","unresolved":false,"context_lines":[{"line_number":30,"context_line":"  fi"},{"line_number":31,"context_line":"}"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"# NOTE(portdirect): if the db has been populated we should setup cells if"},{"line_number":34,"context_line":"# required, otherwise we should poulate the api db, and then setup cells."},{"line_number":35,"context_line":"if [ \"$(nova-manage api_db version)\" -gt \"0\" ]; then"},{"line_number":36,"context_line":"  manage_cells"},{"line_number":37,"context_line":"  nova-manage api_db sync"},{"line_number":38,"context_line":"else"},{"line_number":39,"context_line":"  nova-manage api_db sync"},{"line_number":40,"context_line":"  manage_cells"},{"line_number":41,"context_line":"fi"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"nova-manage db sync"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-smarty","patch_set":12,"id":"5f7c97a3_83ee7c86","line":41,"range":{"start_line":33,"start_character":0,"end_line":41,"end_character":2},"in_reply_to":"5f7c97a3_23976893","updated":"2018-05-03 16:08:28.000000000","message":"will validate this - I took my lead from RDO upgrade documentation but will run this through my lab without the conditional.","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"527762a9cf15432d3bc23cc429c14f9e27ce40b6","unresolved":false,"context_lines":[{"line_number":30,"context_line":"  fi"},{"line_number":31,"context_line":"}"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"# NOTE(portdirect): if the db has been populated we should setup cells if"},{"line_number":34,"context_line":"# required, otherwise we should poulate the api db, and then setup cells."},{"line_number":35,"context_line":"if [ \"$(nova-manage api_db version)\" -gt \"0\" ]; then"},{"line_number":36,"context_line":"  manage_cells"},{"line_number":37,"context_line":"  nova-manage api_db sync"},{"line_number":38,"context_line":"else"},{"line_number":39,"context_line":"  nova-manage api_db sync"},{"line_number":40,"context_line":"  manage_cells"},{"line_number":41,"context_line":"fi"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"nova-manage db sync"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-smarty","patch_set":12,"id":"5f7c97a3_e6765e3f","line":41,"range":{"start_line":33,"start_character":0,"end_line":41,"end_character":2},"in_reply_to":"5f7c97a3_83ee7c86","updated":"2018-05-03 16:41:07.000000000","message":"This footwork is required, unfortunately: http://paste.openstack.org/raw/720304/\n\nAs it stands this script lets us upgrade from Newton to Queens (tested with running VM\u0027s and a stopover at each release), without the ordering behind a conditional, it blows up between Newton and Ocata.","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"227ba18fcc5b0b2b433d353bda98b867d61888a8","unresolved":false,"context_lines":[{"line_number":30,"context_line":"  fi"},{"line_number":31,"context_line":"}"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"# NOTE(portdirect): if the db has been populated we should setup cells if"},{"line_number":34,"context_line":"# required, otherwise we should poulate the api db, and then setup cells."},{"line_number":35,"context_line":"if [ \"$(nova-manage api_db version)\" -gt \"0\" ]; then"},{"line_number":36,"context_line":"  manage_cells"},{"line_number":37,"context_line":"  nova-manage api_db sync"},{"line_number":38,"context_line":"else"},{"line_number":39,"context_line":"  nova-manage api_db sync"},{"line_number":40,"context_line":"  manage_cells"},{"line_number":41,"context_line":"fi"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"nova-manage db sync"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-smarty","patch_set":12,"id":"5f7c97a3_06735a17","line":41,"range":{"start_line":33,"start_character":0,"end_line":41,"end_character":2},"in_reply_to":"5f7c97a3_a6dce618","updated":"2018-05-03 16:52:08.000000000","message":"This fails - as you cannot create cells in later releases without 1st populating the api db :(","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"e10140090ca2ab316af20590b448fef49279da7c","unresolved":false,"context_lines":[{"line_number":30,"context_line":"  fi"},{"line_number":31,"context_line":"}"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"# NOTE(portdirect): if the db has been populated we should setup cells if"},{"line_number":34,"context_line":"# required, otherwise we should poulate the api db, and then setup cells."},{"line_number":35,"context_line":"if [ \"$(nova-manage api_db version)\" -gt \"0\" ]; then"},{"line_number":36,"context_line":"  manage_cells"},{"line_number":37,"context_line":"  nova-manage api_db sync"},{"line_number":38,"context_line":"else"},{"line_number":39,"context_line":"  nova-manage api_db sync"},{"line_number":40,"context_line":"  manage_cells"},{"line_number":41,"context_line":"fi"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"nova-manage db sync"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-smarty","patch_set":12,"id":"5f7c97a3_a6dce618","line":41,"range":{"start_line":33,"start_character":0,"end_line":41,"end_character":2},"in_reply_to":"5f7c97a3_e6765e3f","updated":"2018-05-03 16:45:50.000000000","message":"Interesting.\n\nIf it\u0027s not a pain, would you mind testing to see if the following works for all cases?\n\n manage_cells\n nova-manage api_db sync\n manage-cells","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"}],"nova/values.yaml":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"06583fb924741c4cfea6a5350ea57375e0cadc6f","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"jobs:"},{"line_number":89,"context_line":"  cell_setup:"},{"line_number":90,"context_line":"    cron: \"*/5 * * * *\""},{"line_number":91,"context_line":"    history:"},{"line_number":92,"context_line":"      success: 3"},{"line_number":93,"context_line":"      failed: 1"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"5f7c97a3_6359e0fc","line":90,"range":{"start_line":90,"start_character":4,"end_line":90,"end_character":23},"updated":"2018-05-03 15:49:08.000000000","message":"I know that cell setup is idempotent, but running it every 5 minutes seems a bit much :)","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"2eca6be37c2c2532169e0b23d40ef185f67b06bb","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"jobs:"},{"line_number":89,"context_line":"  cell_setup:"},{"line_number":90,"context_line":"    cron: \"*/5 * * * *\""},{"line_number":91,"context_line":"    history:"},{"line_number":92,"context_line":"      success: 3"},{"line_number":93,"context_line":"      failed: 1"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"5f7c97a3_83875cd8","line":90,"range":{"start_line":90,"start_character":4,"end_line":90,"end_character":23},"in_reply_to":"5f7c97a3_6359e0fc","updated":"2018-05-03 16:08:28.000000000","message":"I wondered if anyone would spot that ;) I\u0027ll put it to a more reasonable one hour. And add a note to counter cry\u0027s of \"where are my nodez\".","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"},{"author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"change_message_id":"d4aa94ef561bca958d667e94c55997d6d659c995","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"jobs:"},{"line_number":89,"context_line":"  cell_setup:"},{"line_number":90,"context_line":"    cron: \"*/5 * * * *\""},{"line_number":91,"context_line":"    history:"},{"line_number":92,"context_line":"      success: 3"},{"line_number":93,"context_line":"      failed: 1"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"5f7c97a3_c9501336","line":90,"range":{"start_line":90,"start_character":4,"end_line":90,"end_character":23},"in_reply_to":"5f7c97a3_666c8e08","updated":"2018-05-03 18:15:36.000000000","message":"To be clear, in a cells world does this mean you would need to wait an hour before a new node would be online for scheduling?","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"d0600ac32be7677cd2b02ac23664c3c2ba67f4c3","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"jobs:"},{"line_number":89,"context_line":"  cell_setup:"},{"line_number":90,"context_line":"    cron: \"*/5 * * * *\""},{"line_number":91,"context_line":"    history:"},{"line_number":92,"context_line":"      success: 3"},{"line_number":93,"context_line":"      failed: 1"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"5f7c97a3_666c8e08","line":90,"range":{"start_line":90,"start_character":4,"end_line":90,"end_character":23},"in_reply_to":"5f7c97a3_83875cd8","updated":"2018-05-03 17:01:17.000000000","message":"Updated to one hour, in a future PS I\u0027ll make this cleaner by getting compute pods to register with a cell (and AZ/Aggregate) as a post start action.","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"02ee584f7db05d01d1c0deaa59ff46797b9daf50","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"jobs:"},{"line_number":89,"context_line":"  cell_setup:"},{"line_number":90,"context_line":"    cron: \"*/5 * * * *\""},{"line_number":91,"context_line":"    history:"},{"line_number":92,"context_line":"      success: 3"},{"line_number":93,"context_line":"      failed: 1"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"5f7c97a3_490ac326","line":90,"range":{"start_line":90,"start_character":4,"end_line":90,"end_character":23},"in_reply_to":"5f7c97a3_c9501336","updated":"2018-05-03 18:19:58.000000000","message":"up to, yes - as part of my work to add compute node registration I\u0027ll get this down to 30-40s from coming online.","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"88fd6e912a2662e621b2c22290d199a7e140b056","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"jobs:"},{"line_number":89,"context_line":"  cell_setup:"},{"line_number":90,"context_line":"    cron: \"*/5 * * * *\""},{"line_number":91,"context_line":"    history:"},{"line_number":92,"context_line":"      success: 3"},{"line_number":93,"context_line":"      failed: 1"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"5f7c97a3_6913ff67","line":90,"range":{"start_line":90,"start_character":4,"end_line":90,"end_character":23},"in_reply_to":"5f7c97a3_c9501336","updated":"2018-05-03 18:19:34.000000000","message":"yes, alan. That\u0027s why the discover_hosts call should probably be moved into its own cron\u0027d thing, separate from the \"setup\" work, which really is a one-time thing.","commit_id":"60cbbd13edace03c3b1826777ddfb508786ffc6a"}]}
