)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"2fbeaab57b56ef7d71009d9289d252763b0c3e58","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"41727a4b_e0f830c9","updated":"2022-05-20 09:31:14.000000000","message":"LGTM but please add a reno, we will want to backport this (up to Wallaby if I am not mistaken).","commit_id":"b98af343a96e283ad3d504b36c6a6cba3d156277"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"2fb62878c8844d903388b6f33dce0719bd589a47","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"879423e3_f8479d41","updated":"2022-05-20 08:52:41.000000000","message":"Should these images be built implicitly when masakari is enabled?\n\n@yoctozepto: Is there a reason why enable_hacluster is not automatically true when enable_masakari is true in kolla-ansible?","commit_id":"b98af343a96e283ad3d504b36c6a6cba3d156277"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"edbb6fc869a143b7cae74f93f00e43116234b199","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7413a20c_295ac0e5","in_reply_to":"18fe8c91_1bd54ae1","updated":"2022-05-20 11:08:37.000000000","message":"Pierre, I\u0027ve proposed to enable hacluster when masakari is enabled, but this is not accepted by Radosław. BTW hostmonitors are deployed always with Masakari, which depends on pacemaker from hacluster ALWAYS! https://github.com/openstack/kolla-ansible/blob/master/ansible/roles/masakari/templates/masakari-monitors.conf.j2#L36-L41.","commit_id":"b98af343a96e283ad3d504b36c6a6cba3d156277"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"c45fadc085d2dd05a6fecd4ff47f440034285bc9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"68cf1530_f5fb8243","in_reply_to":"3bfa27d3_63e2d2e0","updated":"2022-05-20 12:11:38.000000000","message":"ok, will do","commit_id":"b98af343a96e283ad3d504b36c6a6cba3d156277"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"e772f3e1fd34a1a2b68671f6aca57bafc0a4c01c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c140c036_edfb4613","in_reply_to":"41727a4b_e0f830c9","updated":"2022-05-20 11:09:58.000000000","message":"done","commit_id":"b98af343a96e283ad3d504b36c6a6cba3d156277"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"fc64e04fb2965edc8a1e9be391f07cd63b4870bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"aaea4f18_475d986b","in_reply_to":"7413a20c_295ac0e5","updated":"2022-05-20 11:11:57.000000000","message":"Radosław, JFYI, we don\u0027t have any mechanism to deploy only instancemonitors without hostmonitors right now.","commit_id":"b98af343a96e283ad3d504b36c6a6cba3d156277"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"0b12344970b1012c27550ea2052ec69824daa3d7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"57e5ab4b_05758dd3","in_reply_to":"7413a20c_295ac0e5","updated":"2022-05-20 11:10:05.000000000","message":"Well then it makes sense to not deploy hostmonitors sometimes. This can be behind a separate flag. Thanks for digging.","commit_id":"b98af343a96e283ad3d504b36c6a6cba3d156277"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"7038471ebc70b93c728ec0bc8d021ad63d1b86b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e5661e6c_05e59f60","in_reply_to":"879423e3_f8479d41","updated":"2022-05-20 09:03:44.000000000","message":"There is a valid use case to have only instance monitors which do not rely on hacluster which is bulky.","commit_id":"b98af343a96e283ad3d504b36c6a6cba3d156277"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"9421f8070a046fb9eb79efdb20961e2cf31f2f85","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3bfa27d3_63e2d2e0","in_reply_to":"aaea4f18_475d986b","updated":"2022-05-20 11:49:21.000000000","message":"Yes, I am aware. I was meaning we should have one. Please propose if you have time.","commit_id":"b98af343a96e283ad3d504b36c6a6cba3d156277"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"2fbeaab57b56ef7d71009d9289d252763b0c3e58","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"18fe8c91_1bd54ae1","in_reply_to":"e5661e6c_05e59f60","updated":"2022-05-20 09:31:14.000000000","message":"Thanks.","commit_id":"b98af343a96e283ad3d504b36c6a6cba3d156277"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"7e5e2ca1d2dde98a673cc7da322679580d565132","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d36aee61_ccf2a83f","updated":"2022-05-20 14:42:47.000000000","message":"Let\u0027s merge it, I will prepare the backports till Wallaby.","commit_id":"ac337a8f955af5b50dad032b6f1340c0be2ef991"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"591af961acb10d34cf33c0ca1bf88cba4240b61b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0815c60b_77b0989b","updated":"2022-05-20 13:54:44.000000000","message":"Thanks! I made a slight rewording to match a previous reno for the same kind of issue.","commit_id":"ac337a8f955af5b50dad032b6f1340c0be2ef991"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"a9908249742cdaae116484ec0d20a6304c0d6cce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"11ea2005_0f0d9905","updated":"2022-05-20 18:38:14.000000000","message":"recheck","commit_id":"ac337a8f955af5b50dad032b6f1340c0be2ef991"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"beeaed76c43852c28ee809e8df354a531a5eaac7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1b29c9eb_f056ba78","in_reply_to":"d36aee61_ccf2a83f","updated":"2022-05-20 15:45:07.000000000","message":"Just needs another core to approve.","commit_id":"ac337a8f955af5b50dad032b6f1340c0be2ef991"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"05a26e4ec818482e5aab740ac0c9707824694dfd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a8216bb4_a1a4d5ad","updated":"2022-05-24 07:42:54.000000000","message":"@Maksim: Please stop rechecking, it won\u0027t resolve the issue.\n\nWe are broken following the release of python-novaclient 18.0.0 which now requires Python 3.8.","commit_id":"a3ef9ccb240f6dacea3bdae1e6cc37ffc2047ca1"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"8329d1f8f58113d2f4ddb6cbdb4491ad76913f5b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3121ff91_8b8bf474","updated":"2022-05-23 17:12:19.000000000","message":"recheck","commit_id":"a3ef9ccb240f6dacea3bdae1e6cc37ffc2047ca1"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"86d80137d914384c6ce0c2014fe6f3a9465a4184","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"34a81968_325bcd41","updated":"2022-05-23 22:10:43.000000000","message":"recheck","commit_id":"a3ef9ccb240f6dacea3bdae1e6cc37ffc2047ca1"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"8e28b7431ed44f06655cd3489b8deb807c1462c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c81f0983_e71fc69a","updated":"2022-05-23 15:26:14.000000000","message":"recheck","commit_id":"a3ef9ccb240f6dacea3bdae1e6cc37ffc2047ca1"}]}
