)]}'
{"specs/ussuri/approved/evacuate-non-recovery-instances.rst":[{"author":{"_account_id":15397,"name":"Toshikazu Kazu Ichikawa","email":"t.ichikawa@ntt.com","username":"kazu"},"change_message_id":"ec08d72000deef1093fe7c5bde4fc2bf03f7f1b2","unresolved":false,"context_lines":[{"line_number":98,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |valid criteria.                             |"},{"line_number":99,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":100,"context_line":"|  HA_Enabled\u003dTrue  |                          |                                   |                                            |                                       |                          |Since instance is in error state, it has    |"},{"line_number":101,"context_line":"|  vm_state\u003dError,  |                          |    OK                             |    OK                                      |                                       |    N                     |higher precedence than the remaining valid  |"},{"line_number":102,"context_line":"|  tenant_3         |                          |                                   |                                            |                                       |                          |criteria.                                   |"},{"line_number":103,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":104,"context_line":"|  HA_Enabled\u003dFalse |                          |                                   |                                            |                                       |                          |Since instance is in error state, it has    |"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3fa7e38b_bf24d9b2","line":101,"range":{"start_line":101,"start_character":88,"end_line":101,"end_character":90},"updated":"2020-02-18 09:56:53.000000000","message":"This should not be \"OK\" because this case is \u0027HA_Enabled\u003dTrue\u0027 and \u0027evacuate_non_ha_enabled_to_shutoff_state\u0027 has nothing to do with it.","commit_id":"ffee778a7c68952cf9fb8cdbdef215d26e58863b"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"41f205ca5c7fb91bab038621419b17b975229d56","unresolved":false,"context_lines":[{"line_number":98,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |valid criteria.                             |"},{"line_number":99,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":100,"context_line":"|  HA_Enabled\u003dTrue  |                          |                                   |                                            |                                       |                          |Since instance is in error state, it has    |"},{"line_number":101,"context_line":"|  vm_state\u003dError,  |                          |    OK                             |    OK                                      |                                       |    N                     |higher precedence than the remaining valid  |"},{"line_number":102,"context_line":"|  tenant_3         |                          |                                   |                                            |                                       |                          |criteria.                                   |"},{"line_number":103,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":104,"context_line":"|  HA_Enabled\u003dFalse |                          |                                   |                                            |                                       |                          |Since instance is in error state, it has    |"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1fa4df85_fa8d4dd3","line":101,"range":{"start_line":101,"start_character":88,"end_line":101,"end_character":90},"in_reply_to":"3fa7e38b_bf24d9b2","updated":"2020-02-27 09:08:56.000000000","message":"Not necessary.\nConfig options whatever be set, what action will be taken is listed here.\n\nFor more details about \u0027evacuate_all_instances\u0027 option please refer [1]\n\n[1]: https://docs.openstack.org/masakari/latest/configuration/sample_config.html","commit_id":"ffee778a7c68952cf9fb8cdbdef215d26e58863b"},{"author":{"_account_id":15397,"name":"Toshikazu Kazu Ichikawa","email":"t.ichikawa@ntt.com","username":"kazu"},"change_message_id":"ec08d72000deef1093fe7c5bde4fc2bf03f7f1b2","unresolved":false,"context_lines":[{"line_number":106,"context_line":"|  tenant_3         |                          |                                   |                                            |                                       |                          |criteria.                                   |"},{"line_number":107,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":108,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":109,"context_line":"|  vm_state\u003dActive, |    OK                    |                                   |                                            |                                       |    Y                     |                                            |"},{"line_number":110,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":111,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":112,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3fa7e38b_3fa30900","line":109,"range":{"start_line":109,"start_character":133,"end_line":109,"end_character":135},"updated":"2020-02-18 09:56:53.000000000","message":"Additionally, this should be \"OK\" for \u0027ignore_instances_for_tenants\u0027 because this case is \"OK\" for \u0027evacuate_non_ha_enabled_to_shutoff_state\u0027 and its tenant is \u0027tenant_1.\u0027","commit_id":"ffee778a7c68952cf9fb8cdbdef215d26e58863b"},{"author":{"_account_id":15397,"name":"Toshikazu Kazu Ichikawa","email":"t.ichikawa@ntt.com","username":"kazu"},"change_message_id":"ec08d72000deef1093fe7c5bde4fc2bf03f7f1b2","unresolved":false,"context_lines":[{"line_number":106,"context_line":"|  tenant_3         |                          |                                   |                                            |                                       |                          |criteria.                                   |"},{"line_number":107,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":108,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":109,"context_line":"|  vm_state\u003dActive, |    OK                    |                                   |                                            |                                       |    Y                     |                                            |"},{"line_number":110,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":111,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":112,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3fa7e38b_9ffc7dbd","line":109,"range":{"start_line":109,"start_character":173,"end_line":109,"end_character":174},"updated":"2020-02-18 09:56:53.000000000","message":"I want a clarification here.\nWhen \u0027evacuate_all_instances\u0027 is set \u0027True,\u0027 this case is \"Y.\"\nIf \u0027evacuate_all_instances\u0027 is set \u0027false\u0027 and \u0027evacuate_non_ha_enabled_to_shutoff_state\u0027 is set \u0027True,\u0027 then this case should be \"N\" as \u0027ignore_instances_for_tenants\u0027 is matched.","commit_id":"ffee778a7c68952cf9fb8cdbdef215d26e58863b"},{"author":{"_account_id":15397,"name":"Toshikazu Kazu Ichikawa","email":"t.ichikawa@ntt.com","username":"kazu"},"change_message_id":"ec08d72000deef1093fe7c5bde4fc2bf03f7f1b2","unresolved":false,"context_lines":[{"line_number":106,"context_line":"|  tenant_3         |                          |                                   |                                            |                                       |                          |criteria.                                   |"},{"line_number":107,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":108,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":109,"context_line":"|  vm_state\u003dActive, |    OK                    |                                   |                                            |                                       |    Y                     |                                            |"},{"line_number":110,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":111,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":112,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3fa7e38b_ff3ed1b4","line":109,"range":{"start_line":109,"start_character":88,"end_line":109,"end_character":90},"updated":"2020-02-18 09:56:53.000000000","message":"When \u0027HA_Enabled\u0027 parameter is omitted, it\u0027s a part of non-recovery instance of \u0027evacuate_non_ha_enabled_to_shutoff_state.\u0027 Therefore, this case should be \"OK\" for \u0027evacuate_non_ha_enabled_to_shutoff_state\u0027 and it should be effective when \u0027evacuate_all_instances\u0027 is set \u0027false.\u0027","commit_id":"ffee778a7c68952cf9fb8cdbdef215d26e58863b"},{"author":{"_account_id":15397,"name":"Toshikazu Kazu Ichikawa","email":"t.ichikawa@ntt.com","username":"kazu"},"change_message_id":"ec08d72000deef1093fe7c5bde4fc2bf03f7f1b2","unresolved":false,"context_lines":[{"line_number":110,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":111,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":112,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":113,"context_line":"|  vm_state\u003dError,  |    OK                    |    OK                             |                                            |                                       |    N                     |Instance is in error state.                 |"},{"line_number":114,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":115,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":116,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3fa7e38b_df88b56d","line":113,"range":{"start_line":113,"start_character":133,"end_line":113,"end_character":135},"updated":"2020-02-18 09:56:53.000000000","message":"Additionally, this should be \"OK\" for \u0027ignore_instances_for_tenants\u0027 because this case is \"OK\" for \u0027evacuate_non_ha_enabled_to_shutoff_state\u0027 and its tenant is \u0027tenant_1.\u0027","commit_id":"ffee778a7c68952cf9fb8cdbdef215d26e58863b"},{"author":{"_account_id":15397,"name":"Toshikazu Kazu Ichikawa","email":"t.ichikawa@ntt.com","username":"kazu"},"change_message_id":"ec08d72000deef1093fe7c5bde4fc2bf03f7f1b2","unresolved":false,"context_lines":[{"line_number":110,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":111,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":112,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":113,"context_line":"|  vm_state\u003dError,  |    OK                    |    OK                             |                                            |                                       |    N                     |Instance is in error state.                 |"},{"line_number":114,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":115,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":116,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3fa7e38b_bf7299ae","line":113,"range":{"start_line":113,"start_character":88,"end_line":113,"end_character":90},"updated":"2020-02-18 09:56:53.000000000","message":"This case should be \"OK\" as same as above.","commit_id":"ffee778a7c68952cf9fb8cdbdef215d26e58863b"},{"author":{"_account_id":15397,"name":"Toshikazu Kazu Ichikawa","email":"t.ichikawa@ntt.com","username":"kazu"},"change_message_id":"ec08d72000deef1093fe7c5bde4fc2bf03f7f1b2","unresolved":false,"context_lines":[{"line_number":114,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":115,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":116,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":117,"context_line":"|  vm_state\u003dActive, |    OK                    |                                   |                                            |                                       |    Y                     |                                            |"},{"line_number":118,"context_line":"|  tenant_3         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":119,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":120,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3fa7e38b_1f616dd6","line":117,"range":{"start_line":117,"start_character":88,"end_line":117,"end_character":90},"updated":"2020-02-18 09:56:53.000000000","message":"This case should be \"OK\" as same as above.","commit_id":"ffee778a7c68952cf9fb8cdbdef215d26e58863b"},{"author":{"_account_id":15397,"name":"Toshikazu Kazu Ichikawa","email":"t.ichikawa@ntt.com","username":"kazu"},"change_message_id":"ec08d72000deef1093fe7c5bde4fc2bf03f7f1b2","unresolved":false,"context_lines":[{"line_number":118,"context_line":"|  tenant_3         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":119,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":120,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":121,"context_line":"|  vm_state\u003dError,  |    OK                    |    OK                             |                                            |                                       |    N                     |Instance is in error state.                 |"},{"line_number":122,"context_line":"|  tenant_3         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":123,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":124,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"3fa7e38b_3f5ea914","line":121,"range":{"start_line":121,"start_character":88,"end_line":121,"end_character":90},"updated":"2020-02-18 09:56:53.000000000","message":"This case should be \"OK\" as same as above.","commit_id":"ffee778a7c68952cf9fb8cdbdef215d26e58863b"},{"author":{"_account_id":15397,"name":"Toshikazu Kazu Ichikawa","email":"t.ichikawa@ntt.com","username":"kazu"},"change_message_id":"2798cc8c67ad7a05bde9abbacf2eb60a1477495e","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":72,"context_line":"| Instance metadata | set config option        | set config option                 | set config option                          | set                                   |                          |                                            |"},{"line_number":73,"context_line":"|                   | evacuate_all_instances   | ignore_instances_in_error_state   | evacuate_non_ha_enabled_to_shutoff_state   | ignore_instances_for_tenants \u003d        | Evacuate (Y/N)           | Notes                                      |"},{"line_number":74,"context_line":"|                   | True                     | True                              | True                                       | [tenant_1, tenant_2]                  |                          |                                            |"},{"line_number":75,"context_line":"+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+"},{"line_number":76,"context_line":"|  HA_Enabled\u003dTrue, |                          |                                   |                                            |                                       |                          |                                            |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1fa4df85_4a499c5d","line":73,"range":{"start_line":73,"start_character":170,"end_line":73,"end_character":184},"updated":"2020-03-09 10:08:34.000000000","message":"For clarification:\nDoes this \"Evacuate\" column show the behavior in case of  evacuate_all_instances \u003d True, ignore_instances_in_error_state \u003d True,  evacuate_non_ha_enabled_to_shutoff_state \u003d True and ignore_instances_for_tenants \u003d [tenant_1, tenant_2]?","commit_id":"efb7ea1403e01ac126de4276004762006fb3bbc5"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"c49da8eb825fa1a3a5f3b0ef201753f5e4ff5f11","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":72,"context_line":"| Instance metadata | set config option        | set config option                 | set config option                          | set                                   |                          |                                            |"},{"line_number":73,"context_line":"|                   | evacuate_all_instances   | ignore_instances_in_error_state   | evacuate_non_ha_enabled_to_shutoff_state   | ignore_instances_for_tenants \u003d        | Evacuate (Y/N)           | Notes                                      |"},{"line_number":74,"context_line":"|                   | True                     | True                              | True                                       | [tenant_1, tenant_2]                  |                          |                                            |"},{"line_number":75,"context_line":"+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+"},{"line_number":76,"context_line":"|  HA_Enabled\u003dTrue, |                          |                                   |                                            |                                       |                          |                                            |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"df33271e_7cc9d476","line":73,"range":{"start_line":73,"start_character":170,"end_line":73,"end_character":184},"in_reply_to":"1fa4df85_4a499c5d","updated":"2020-03-24 12:06:52.000000000","message":"For simplicity, modified the table, now mentioning only config names as column heading and column will hold value of the config.\n\nYes, \u0027Evacuate\u0027 column will list \u0027Y\u0027 if that instance will evacuate and \u0027N\u0027 if not.","commit_id":"efb7ea1403e01ac126de4276004762006fb3bbc5"},{"author":{"_account_id":15397,"name":"Toshikazu Kazu Ichikawa","email":"t.ichikawa@ntt.com","username":"kazu"},"change_message_id":"2798cc8c67ad7a05bde9abbacf2eb60a1477495e","unresolved":false,"context_lines":[{"line_number":110,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":111,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":112,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":113,"context_line":"|  vm_state\u003dActive, |    OK                    |                                   |                                            |     OK                                |    Y                     |                                            |"},{"line_number":114,"context_line":"|  tenant_3         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":115,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":116,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1fa4df85_aada9074","line":113,"range":{"start_line":113,"start_character":196,"end_line":113,"end_character":240},"updated":"2020-03-09 10:08:34.000000000","message":"\"Instance will be evacuated in shutoff state\" should be added here.","commit_id":"efb7ea1403e01ac126de4276004762006fb3bbc5"},{"author":{"_account_id":15397,"name":"Toshikazu Kazu Ichikawa","email":"t.ichikawa@ntt.com","username":"kazu"},"change_message_id":"2798cc8c67ad7a05bde9abbacf2eb60a1477495e","unresolved":false,"context_lines":[{"line_number":110,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":111,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":112,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":113,"context_line":"|  vm_state\u003dActive, |    OK                    |                                   |                                            |     OK                                |    Y                     |                                            |"},{"line_number":114,"context_line":"|  tenant_3         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":115,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":116,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1fa4df85_0a3ac497","line":113,"range":{"start_line":113,"start_character":88,"end_line":113,"end_character":90},"updated":"2020-03-09 10:08:34.000000000","message":"\"OK\" should be added here(evacuate_non_ha_enabled_to_shutoff_state column). It might be mistaken with the right column.","commit_id":"efb7ea1403e01ac126de4276004762006fb3bbc5"},{"author":{"_account_id":15397,"name":"Toshikazu Kazu Ichikawa","email":"t.ichikawa@ntt.com","username":"kazu"},"change_message_id":"2798cc8c67ad7a05bde9abbacf2eb60a1477495e","unresolved":false,"context_lines":[{"line_number":110,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":111,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":112,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":113,"context_line":"|  vm_state\u003dActive, |    OK                    |                                   |                                            |     OK                                |    Y                     |                                            |"},{"line_number":114,"context_line":"|  tenant_3         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":115,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":116,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1fa4df85_eacde8b6","line":113,"range":{"start_line":113,"start_character":134,"end_line":113,"end_character":136},"updated":"2020-03-09 10:08:34.000000000","message":"\"OK\" should be removed here(ignore_instances_for_tenants column). It might be typo from the left column.","commit_id":"efb7ea1403e01ac126de4276004762006fb3bbc5"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"c49da8eb825fa1a3a5f3b0ef201753f5e4ff5f11","unresolved":false,"context_lines":[{"line_number":110,"context_line":"|  tenant_1         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":111,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":112,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":113,"context_line":"|  vm_state\u003dActive, |    OK                    |                                   |                                            |     OK                                |    Y                     |                                            |"},{"line_number":114,"context_line":"|  tenant_3         |                          |                                   |                                            |                                       |                          |                                            |"},{"line_number":115,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+--------------------------+--------------------------------------------+"},{"line_number":116,"context_line":"|                   |                          |                                   |                                            |                                       |                          |                                            |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"df33271e_fcaa0483","line":113,"range":{"start_line":113,"start_character":134,"end_line":113,"end_character":136},"in_reply_to":"1fa4df85_eacde8b6","updated":"2020-03-24 12:06:52.000000000","message":"For this added extra row [1]\n\n[1]: https://review.opendev.org/#/c/702427/4/specs/ussuri/approved/evacuate-non-recovery-instances.rst@114","commit_id":"efb7ea1403e01ac126de4276004762006fb3bbc5"},{"author":{"_account_id":15397,"name":"Toshikazu Kazu Ichikawa","email":"t.ichikawa@ntt.com","username":"kazu"},"change_message_id":"59addeff5bbe19a5659b440b6adfc3d57be3c920","unresolved":false,"context_lines":[{"line_number":111,"context_line":"|  vm_state\u003dActive, | TRUE                     | FALSE                             | FALSE                                      | ignore_instances_for_tenants \u003d        |    Y                                |                                            |"},{"line_number":112,"context_line":"|  tenant_3         |                          |                                   |                                            | [tenant_1, tenant_2]                  |                                     |                                            |"},{"line_number":113,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+-------------------------------------+--------------------------------------------+"},{"line_number":114,"context_line":"|  HA_Enabled\u003dFalse,|                          |                                   |                                            |                                       |                                     |                                            |"},{"line_number":115,"context_line":"|  vm_state\u003dActive, | FALSE                    | FALSE                             | TRUE                                       | ignore_instances_for_tenants \u003d        |    Y                                |Instance will be evacuated in shutoff state |"},{"line_number":116,"context_line":"|  tenant_3         |                          |                                   |                                            | [tenant_1, tenant_2]                  |                                     |                                            |"},{"line_number":117,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+-------------------------------------+--------------------------------------------+"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df33271e_a304c718","line":114,"range":{"start_line":114,"start_character":3,"end_line":114,"end_character":20},"updated":"2020-03-25 08:40:43.000000000","message":"I don\u0027t believe you need to add this \"HA_Enabled\u003dFalse,.\" If you add, the second row from the bottom is almost duplicated with the third row from the top. I guess this row intends to show the case when HA_Enabled key is not present, which is equivalent to setting False by default.","commit_id":"3273c1c7fd8462d1839d97247d0cd773c2a5b8d3"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"53a3e8316be44f7b7c86a1131b3a3617d02bb23a","unresolved":false,"context_lines":[{"line_number":111,"context_line":"|  vm_state\u003dActive, | TRUE                     | FALSE                             | FALSE                                      | ignore_instances_for_tenants \u003d        |    Y                                |                                            |"},{"line_number":112,"context_line":"|  tenant_3         |                          |                                   |                                            | [tenant_1, tenant_2]                  |                                     |                                            |"},{"line_number":113,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+-------------------------------------+--------------------------------------------+"},{"line_number":114,"context_line":"|  HA_Enabled\u003dFalse,|                          |                                   |                                            |                                       |                                     |                                            |"},{"line_number":115,"context_line":"|  vm_state\u003dActive, | FALSE                    | FALSE                             | TRUE                                       | ignore_instances_for_tenants \u003d        |    Y                                |Instance will be evacuated in shutoff state |"},{"line_number":116,"context_line":"|  tenant_3         |                          |                                   |                                            | [tenant_1, tenant_2]                  |                                     |                                            |"},{"line_number":117,"context_line":"+-------------------+--------------------------+-----------------------------------+--------------------------------------------+---------------------------------------+-------------------------------------+--------------------------------------------+"}],"source_content_type":"text/x-rst","patch_set":4,"id":"df33271e_d069bcbe","line":114,"range":{"start_line":114,"start_character":3,"end_line":114,"end_character":20},"in_reply_to":"df33271e_a304c718","updated":"2020-03-26 08:13:27.000000000","message":"Thank you, removed this in next ps.","commit_id":"3273c1c7fd8462d1839d97247d0cd773c2a5b8d3"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"b8beba85e2855fb9b7eba704c192ef3a0cf6d207","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Problem description"},{"line_number":15,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Currently, Masakari doesn\u0027t evacuate non-recovery (`HA_enabled \u003d False`)"},{"line_number":18,"context_line":"instances during host failure recovery workflow execution."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"When a failed host goes back to system as a reserved host, it is required to"},{"line_number":21,"context_line":"make the host empty and be available at its full capacity. So it would be"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_bf3f1485","line":18,"range":{"start_line":17,"start_character":11,"end_line":18,"end_character":57},"updated":"2020-05-21 05:50:32.000000000","message":"This is not completely true, if evacuate_all_instances config option is enabled, then it would evacuate instances irrespective HA_enabled is True/False.","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"6968a2fda2efec5c4ef992e2269c62ebd5f50887","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Problem description"},{"line_number":15,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Currently, Masakari doesn\u0027t evacuate non-recovery (`HA_enabled \u003d False`)"},{"line_number":18,"context_line":"instances during host failure recovery workflow execution."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"When a failed host goes back to system as a reserved host, it is required to"},{"line_number":21,"context_line":"make the host empty and be available at its full capacity. So it would be"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_8612bd60","line":18,"range":{"start_line":17,"start_character":11,"end_line":18,"end_character":57},"in_reply_to":"ff570b3c_bf3f1485","updated":"2020-05-21 15:45:45.000000000","message":"elaborated as explained.","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"b8beba85e2855fb9b7eba704c192ef3a0cf6d207","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Modify Host recovery flow to"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"* Evacuate non-recovery (`HA_enabled \u003d False`) instances in shutoff state."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"* Special instances like `Canaria` instance which is meant for host health"},{"line_number":34,"context_line":"  monitoring, should not be evacuated throughout its life cycle."}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_3f2024f7","line":31,"range":{"start_line":31,"start_character":57,"end_line":31,"end_character":67},"updated":"2020-05-21 05:50:32.000000000","message":"to ``shutoff``","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"6968a2fda2efec5c4ef992e2269c62ebd5f50887","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Modify Host recovery flow to"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"* Evacuate non-recovery (`HA_enabled \u003d False`) instances in shutoff state."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"* Special instances like `Canaria` instance which is meant for host health"},{"line_number":34,"context_line":"  monitoring, should not be evacuated throughout its life cycle."}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_6615e948","line":31,"range":{"start_line":31,"start_character":57,"end_line":31,"end_character":67},"in_reply_to":"ff570b3c_3f2024f7","updated":"2020-05-21 15:45:45.000000000","message":"Done","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"b8beba85e2855fb9b7eba704c192ef3a0cf6d207","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"* Evacuate non-recovery (`HA_enabled \u003d False`) instances in shutoff state."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"* Special instances like `Canaria` instance which is meant for host health"},{"line_number":34,"context_line":"  monitoring, should not be evacuated throughout its life cycle."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Proposed change"},{"line_number":37,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_ff750c65","line":34,"range":{"start_line":33,"start_character":2,"end_line":34,"end_character":64},"updated":"2020-05-21 05:50:32.000000000","message":"Special instances like ``Canaria`` instance which is meant for monitoring health of the host should not be evacuated throughout its life cycle.","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"6968a2fda2efec5c4ef992e2269c62ebd5f50887","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"* Evacuate non-recovery (`HA_enabled \u003d False`) instances in shutoff state."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"* Special instances like `Canaria` instance which is meant for host health"},{"line_number":34,"context_line":"  monitoring, should not be evacuated throughout its life cycle."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Proposed change"},{"line_number":37,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_a60a6122","line":34,"range":{"start_line":33,"start_character":2,"end_line":34,"end_character":64},"in_reply_to":"ff570b3c_ff750c65","updated":"2020-05-21 15:45:45.000000000","message":"Done","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"b8beba85e2855fb9b7eba704c192ef3a0cf6d207","unresolved":false,"context_lines":[{"line_number":41,"context_line":"* ``evacuate_non_ha_enabled_to_shutoff_state`` to decide whether to"},{"line_number":42,"context_line":"  evacuate instances to shutoff state."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* ``ignore_instances_for_tenants`` to decide not to evacuate special"},{"line_number":45,"context_line":"  instances like `Canaria` instance. If the instance belongs to any of tenants"},{"line_number":46,"context_line":"  specified in this config option, it should not be evacuated provided"},{"line_number":47,"context_line":"  HA_Enabled\u003dFalse or this key is not present."}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_bf339461","line":44,"range":{"start_line":44,"start_character":25,"end_line":44,"end_character":32},"updated":"2020-05-21 05:50:32.000000000","message":"Should we call it as projects as tenant is outdated in OpenStack.","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"6968a2fda2efec5c4ef992e2269c62ebd5f50887","unresolved":false,"context_lines":[{"line_number":41,"context_line":"* ``evacuate_non_ha_enabled_to_shutoff_state`` to decide whether to"},{"line_number":42,"context_line":"  evacuate instances to shutoff state."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* ``ignore_instances_for_tenants`` to decide not to evacuate special"},{"line_number":45,"context_line":"  instances like `Canaria` instance. If the instance belongs to any of tenants"},{"line_number":46,"context_line":"  specified in this config option, it should not be evacuated provided"},{"line_number":47,"context_line":"  HA_Enabled\u003dFalse or this key is not present."}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_39015606","line":44,"range":{"start_line":44,"start_character":25,"end_line":44,"end_character":32},"in_reply_to":"ff570b3c_bf339461","updated":"2020-05-21 15:45:45.000000000","message":"Done","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"b8beba85e2855fb9b7eba704c192ef3a0cf6d207","unresolved":false,"context_lines":[{"line_number":46,"context_line":"  specified in this config option, it should not be evacuated provided"},{"line_number":47,"context_line":"  HA_Enabled\u003dFalse or this key is not present."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":".. note:: Higher precedence will be given to ignore_instances_for_tenants."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Setting evacuate_non_ha_enabled_to_shutoff_state"},{"line_number":52,"context_line":"------------------------------------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_5fc19853","line":49,"range":{"start_line":49,"start_character":10,"end_line":49,"end_character":73},"updated":"2020-05-21 05:50:32.000000000","message":"Higher precedence will be given to ignore_instances_for_tenants, but over what other config options. Please list those config options as well.","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"6968a2fda2efec5c4ef992e2269c62ebd5f50887","unresolved":false,"context_lines":[{"line_number":46,"context_line":"  specified in this config option, it should not be evacuated provided"},{"line_number":47,"context_line":"  HA_Enabled\u003dFalse or this key is not present."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":".. note:: Higher precedence will be given to ignore_instances_for_tenants."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Setting evacuate_non_ha_enabled_to_shutoff_state"},{"line_number":52,"context_line":"------------------------------------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_79074e19","line":49,"range":{"start_line":49,"start_character":10,"end_line":49,"end_character":73},"in_reply_to":"ff570b3c_5fc19853","updated":"2020-05-21 15:45:45.000000000","message":"Done, mentioned other config options.","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"b8beba85e2855fb9b7eba704c192ef3a0cf6d207","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Setting evacuate_non_ha_enabled_to_shutoff_state"},{"line_number":52,"context_line":"------------------------------------------------"},{"line_number":53,"context_line":"Operators can decide whether non-recovery i.e. \u0027HA_enabled \u003d False\u0027 instances"},{"line_number":54,"context_line":"are evacuated in shutoff state. When set to True, if HA_Enabled\u003dFalse or"},{"line_number":55,"context_line":"HA_Enabled key is not present, it will evacuate instance in shutoff state."},{"line_number":56,"context_line":"If HA_Enabled\u003dTrue, it will evacuate instance to it’s original vm state."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Setting ignore_instances_for_tenants"},{"line_number":59,"context_line":"------------------------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_9f85f009","line":56,"range":{"start_line":53,"start_character":0,"end_line":56,"end_character":72},"updated":"2020-05-21 05:50:32.000000000","message":"Operators can decide whether instances should be evacuated to ``shutoff`` state or not by setting config option ``evacuate_non_ha_enabled_to_shutoff_state``to True/False.\n* If True, instance will evacuated to ``shutoff`` state provided it meets one of the criteria. \na) ``HA_Enabled`` is False\nb) if  key ``HA_Enabled`` is not present.\n* If False, then it would have no impact on the existing behavior.","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"6968a2fda2efec5c4ef992e2269c62ebd5f50887","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Setting evacuate_non_ha_enabled_to_shutoff_state"},{"line_number":52,"context_line":"------------------------------------------------"},{"line_number":53,"context_line":"Operators can decide whether non-recovery i.e. \u0027HA_enabled \u003d False\u0027 instances"},{"line_number":54,"context_line":"are evacuated in shutoff state. When set to True, if HA_Enabled\u003dFalse or"},{"line_number":55,"context_line":"HA_Enabled key is not present, it will evacuate instance in shutoff state."},{"line_number":56,"context_line":"If HA_Enabled\u003dTrue, it will evacuate instance to it’s original vm state."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Setting ignore_instances_for_tenants"},{"line_number":59,"context_line":"------------------------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_990fc22c","line":56,"range":{"start_line":53,"start_character":0,"end_line":56,"end_character":72},"in_reply_to":"ff570b3c_9f85f009","updated":"2020-05-21 15:45:45.000000000","message":"Done","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"b8beba85e2855fb9b7eba704c192ef3a0cf6d207","unresolved":false,"context_lines":[{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Setting ignore_instances_for_tenants"},{"line_number":59,"context_line":"------------------------------------"},{"line_number":60,"context_line":"Operator need to specify list of tenant IDs, instances belonging to any of"},{"line_number":61,"context_line":"the tenant IDs configured, should not be evacuated in any case even though"},{"line_number":62,"context_line":"``evacuate_non_ha_enabled_to_shutoff_state`` is set to True and metadata as"},{"line_number":63,"context_line":"HA_Enabled\u003dFalse."},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"Matrix of various host failure config option (New/ Existing) combinations"},{"line_number":66,"context_line":"-------------------------------------------------------------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_5fbd18b5","line":63,"range":{"start_line":60,"start_character":0,"end_line":63,"end_character":17},"updated":"2020-05-21 05:50:32.000000000","message":"If operators are running any ``Canaria`` instances for monitoring of the host and on event of host failure, if they don\u0027t want these instances to be evacuated, then they should enable ``ignore_instances_for_tenants`` config option and add the project_id running these instances in this config option. They will also need to set  metadata ``HA_Enabled\u003dFalse`` to the ``Canaria`` instances.","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"6968a2fda2efec5c4ef992e2269c62ebd5f50887","unresolved":false,"context_lines":[{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Setting ignore_instances_for_tenants"},{"line_number":59,"context_line":"------------------------------------"},{"line_number":60,"context_line":"Operator need to specify list of tenant IDs, instances belonging to any of"},{"line_number":61,"context_line":"the tenant IDs configured, should not be evacuated in any case even though"},{"line_number":62,"context_line":"``evacuate_non_ha_enabled_to_shutoff_state`` is set to True and metadata as"},{"line_number":63,"context_line":"HA_Enabled\u003dFalse."},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"Matrix of various host failure config option (New/ Existing) combinations"},{"line_number":66,"context_line":"-------------------------------------------------------------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_7920ee9b","line":63,"range":{"start_line":60,"start_character":0,"end_line":63,"end_character":17},"in_reply_to":"ff570b3c_5fbd18b5","updated":"2020-05-21 15:45:45.000000000","message":"Done","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"b8beba85e2855fb9b7eba704c192ef3a0cf6d207","unresolved":false,"context_lines":[{"line_number":165,"context_line":"Developer impact"},{"line_number":166,"context_line":"----------------"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"Operator will specify list of tenant ids to `ignore_instances_for_tenants`"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"Implementation"},{"line_number":171,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_5f1998d4","line":168,"range":{"start_line":168,"start_character":0,"end_line":168,"end_character":74},"updated":"2020-05-21 05:50:32.000000000","message":"No need of this statement in this section.","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"6968a2fda2efec5c4ef992e2269c62ebd5f50887","unresolved":false,"context_lines":[{"line_number":165,"context_line":"Developer impact"},{"line_number":166,"context_line":"----------------"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"Operator will specify list of tenant ids to `ignore_instances_for_tenants`"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"Implementation"},{"line_number":171,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_993862d3","line":168,"range":{"start_line":168,"start_character":0,"end_line":168,"end_character":74},"in_reply_to":"ff570b3c_5f1998d4","updated":"2020-05-21 15:45:45.000000000","message":"removed.","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"b8beba85e2855fb9b7eba704c192ef3a0cf6d207","unresolved":false,"context_lines":[{"line_number":181,"context_line":"----------"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"* Add new conf options"},{"line_number":184,"context_line":"* Modify host recovery flow to consider new config and add logic accordingly"},{"line_number":185,"context_line":"* Add unit tests for code coverage"},{"line_number":186,"context_line":"* Add documentation on how to use this feature"},{"line_number":187,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_ff050c47","line":184,"range":{"start_line":184,"start_character":44,"end_line":184,"end_character":50},"updated":"2020-05-21 05:50:32.000000000","message":"config options","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"6968a2fda2efec5c4ef992e2269c62ebd5f50887","unresolved":false,"context_lines":[{"line_number":181,"context_line":"----------"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"* Add new conf options"},{"line_number":184,"context_line":"* Modify host recovery flow to consider new config and add logic accordingly"},{"line_number":185,"context_line":"* Add unit tests for code coverage"},{"line_number":186,"context_line":"* Add documentation on how to use this feature"},{"line_number":187,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_394f966b","line":184,"range":{"start_line":184,"start_character":44,"end_line":184,"end_character":50},"in_reply_to":"ff570b3c_ff050c47","updated":"2020-05-21 15:45:45.000000000","message":"Done","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"b8beba85e2855fb9b7eba704c192ef3a0cf6d207","unresolved":false,"context_lines":[{"line_number":193,"context_line":"Testing"},{"line_number":194,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"None"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"Documentation Impact"},{"line_number":199,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_bfe8b47b","line":196,"range":{"start_line":196,"start_character":0,"end_line":196,"end_character":4},"updated":"2020-05-21 05:50:32.000000000","message":"Can you specify the reason why these changes are untestable in gate?","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"6968a2fda2efec5c4ef992e2269c62ebd5f50887","unresolved":false,"context_lines":[{"line_number":193,"context_line":"Testing"},{"line_number":194,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"None"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"Documentation Impact"},{"line_number":199,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_59588a2f","line":196,"range":{"start_line":196,"start_character":0,"end_line":196,"end_character":4},"in_reply_to":"ff570b3c_bfe8b47b","updated":"2020-05-21 15:45:45.000000000","message":"Mentioned, please verify.","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"b8beba85e2855fb9b7eba704c192ef3a0cf6d207","unresolved":false,"context_lines":[{"line_number":215,"context_line":""},{"line_number":216,"context_line":"   * - Release Name"},{"line_number":217,"context_line":"     - Description"},{"line_number":218,"context_line":"   * - Ussuri"},{"line_number":219,"context_line":"     - Introduced"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_9f0fb086","line":219,"range":{"start_line":218,"start_character":7,"end_line":219,"end_character":16},"updated":"2020-05-21 05:50:32.000000000","message":"Re-propose this spec for Victoria.\n\n* - Victoria\n   - Re-proposed","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"},{"author":{"_account_id":27302,"name":"Shilpa Devharakar","email":"shilpa.devharakar@nttdata.com","username":"shilpa.devharakar"},"change_message_id":"0c63547e9276837d62f638f6e6bf502cfca948a0","unresolved":false,"context_lines":[{"line_number":215,"context_line":""},{"line_number":216,"context_line":"   * - Release Name"},{"line_number":217,"context_line":"     - Description"},{"line_number":218,"context_line":"   * - Ussuri"},{"line_number":219,"context_line":"     - Introduced"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ff570b3c_39867667","line":219,"range":{"start_line":218,"start_character":7,"end_line":219,"end_character":16},"in_reply_to":"ff570b3c_9f0fb086","updated":"2020-05-21 15:47:54.000000000","message":"Done","commit_id":"55a9156ad17b507bb39f0f65f6dbbb90ca18cd2d"}],"specs/victoria/approved/evacuate-non-recovery-instances.rst":[{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"98aa66dbeb6ee938b4227094d2c974aa0e72efcf","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Problem description"},{"line_number":15,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"As per current design, Masakari evacuate instances irrespective HA_enabled"},{"line_number":18,"context_line":"is True/False only if ``evacuate_all_instances`` config option is enabled."},{"line_number":19,"context_line":"Otherwise it doesn\u0027t evacuate non-recovery (``HA_enabled \u003d False``) instances"},{"line_number":20,"context_line":"during host failure recovery workflow execution."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"When a failed host goes back to system as a reserved host, it is required to"},{"line_number":23,"context_line":"make the host empty and be available at its full capacity. So it would be"},{"line_number":24,"context_line":"useful to clear off/evacuate non-recovery instances in shutoff status, and"},{"line_number":25,"context_line":"while doing so one should not clear instance like ``Canaria`` which is meant for"},{"line_number":26,"context_line":"monitoring the health of the host."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Use Cases"},{"line_number":29,"context_line":"---------"}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_49d1cd2f","line":26,"range":{"start_line":17,"start_character":0,"end_line":26,"end_character":33},"updated":"2020-05-22 05:35:24.000000000","message":"Presently, Masakari can evacuate instances from a failed compute host either when ``evacuate_all_instances`` is enabled or when ``HA_enabled\u003dTrue`` metadata is set to the instance. When a failed host goes back to system as a reserved host, it is required to make the compute host empty and be available at its full capacity. So it would be useful to clear off/evacuate non-recovery instances in shutoff status i.e. ``HA_enabled\u003dFalse`` , and while doing so one should not clear instances like ``Canaria`` which is meant for monitoring the health of the compute host.","commit_id":"17a725ad6060c26e920939b3dee8bd25ba4edb27"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"98aa66dbeb6ee938b4227094d2c974aa0e72efcf","unresolved":false,"context_lines":[{"line_number":59,"context_line":"state or not by setting config option ``evacuate_non_ha_enabled_to_shutoff_state``"},{"line_number":60,"context_line":"to True/False:"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"* If True, instance will evacuated to ``shutoff`` state provided it meets one"},{"line_number":63,"context_line":"  of the criteria:"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    * ``HA_Enabled`` is False"}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_89dbe54d","line":62,"range":{"start_line":62,"start_character":11,"end_line":62,"end_character":24},"updated":"2020-05-22 05:35:24.000000000","message":"instance will be","commit_id":"17a725ad6060c26e920939b3dee8bd25ba4edb27"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"98aa66dbeb6ee938b4227094d2c974aa0e72efcf","unresolved":false,"context_lines":[{"line_number":62,"context_line":"* If True, instance will evacuated to ``shutoff`` state provided it meets one"},{"line_number":63,"context_line":"  of the criteria:"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    * ``HA_Enabled`` is False"},{"line_number":66,"context_line":"    * if key ``HA_Enabled`` is not present"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"* If False, then it would have no impact on the existing behavior."},{"line_number":69,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_c9e4bd11","line":66,"range":{"start_line":65,"start_character":4,"end_line":66,"end_character":42},"updated":"2020-05-22 05:35:24.000000000","message":"Indent it correctly by moving statement on left side by two spaces.","commit_id":"17a725ad6060c26e920939b3dee8bd25ba4edb27"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"98aa66dbeb6ee938b4227094d2c974aa0e72efcf","unresolved":false,"context_lines":[{"line_number":208,"context_line":"Testing"},{"line_number":209,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"Testing this in the gate would be difficult since the hosts which run the"},{"line_number":212,"context_line":"gate tests would have to be pre-configured and one need to be forcefully"},{"line_number":213,"context_line":"shut down for evacuation."},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"Documentation Impact"},{"line_number":216,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":8,"id":"ff570b3c_a9e189ff","line":213,"range":{"start_line":211,"start_character":0,"end_line":213,"end_character":25},"updated":"2020-05-22 05:35:24.000000000","message":"Presently, pacemaker cluster is not configured in gate to detect the failure of compute hosts, so it\u0027s not possible to send notification to masakari about compute host failure which would trigger to evacuate VMs running on the failed compute hosts on to another compute host. So it\u0027s not possible to test the changes specified in this spec until provision is made in CI gate job to configure and fence compute hosts using pacemaker cluster.","commit_id":"17a725ad6060c26e920939b3dee8bd25ba4edb27"},{"author":{"_account_id":24501,"name":"sue","display_name":"suzhengwei","email":"sugar-2008@163.com","username":"suzhengwei"},"change_message_id":"877df9d860fd581671637467dd1a3b0b94385e9d","unresolved":false,"context_lines":[{"line_number":18,"context_line":"when ``evacuate_all_instances`` is enabled or when ``HA_enabled\u003dTrue``"},{"line_number":19,"context_line":"metadata is set to the instance. When a failed host goes back to system as a"},{"line_number":20,"context_line":"reserved host, it is required to make the compute host empty and be available"},{"line_number":21,"context_line":"at its full capacity. So it would be useful to clear off/evacuate non-recovery"},{"line_number":22,"context_line":"instances in shutoff status i.e. ``HA_enabled\u003dFalse`` , and while doing so one"},{"line_number":23,"context_line":"should not clear instances like ``Canaria`` which is meant for monitoring the"},{"line_number":24,"context_line":"health of the compute host."},{"line_number":25,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"ff570b3c_4796ad96","line":22,"range":{"start_line":21,"start_character":22,"end_line":22,"end_character":55},"updated":"2020-05-26 02:59:18.000000000","message":"Cloud you give more information about scene why and how it evacuate non-recovery instances?\n\nIt\u0027s my understanding that the instances\u0027 status is update by nova-compute service. When nova-compute restart, it will set the instances to \u0027shutoff\u0027. But when host failure, the instances already can\u0027t be used, and what\u0027s the meaning to set the non-recovery instances to shutoff?","commit_id":"db83c367b34f14426bc50c30e9937a823b4e661a"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"462d21fcf6e572c4b1f75b9e50dec733b54914e4","unresolved":false,"context_lines":[{"line_number":18,"context_line":"when ``evacuate_all_instances`` is enabled or when ``HA_enabled\u003dTrue``"},{"line_number":19,"context_line":"metadata is set to the instance. When a failed host goes back to system as a"},{"line_number":20,"context_line":"reserved host, it is required to make the compute host empty and be available"},{"line_number":21,"context_line":"at its full capacity. So it would be useful to clear off/evacuate non-recovery"},{"line_number":22,"context_line":"instances in shutoff status i.e. ``HA_enabled\u003dFalse`` , and while doing so one"},{"line_number":23,"context_line":"should not clear instances like ``Canaria`` which is meant for monitoring the"},{"line_number":24,"context_line":"health of the compute host."},{"line_number":25,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"ff570b3c_628e3fd1","line":22,"range":{"start_line":21,"start_character":22,"end_line":22,"end_character":55},"in_reply_to":"ff570b3c_4796ad96","updated":"2020-05-26 07:28:12.000000000","message":"This use case is for operators who wants to evacuate only those instances with metadata ``HA_enabled\u003dTrue``. If there are any instances running with metadata ``HA_enabled\u003dFalse``, then, all such instances will be evacuated but it\u0027s vm_state will be set to ``shutoff`` state.","commit_id":"db83c367b34f14426bc50c30e9937a823b4e661a"},{"author":{"_account_id":24501,"name":"sue","display_name":"suzhengwei","email":"sugar-2008@163.com","username":"suzhengwei"},"change_message_id":"e193171ca640f53e113c4efa92625d860a312bde","unresolved":false,"context_lines":[{"line_number":18,"context_line":"when ``evacuate_all_instances`` is enabled or when ``HA_enabled\u003dTrue``"},{"line_number":19,"context_line":"metadata is set to the instance. When a failed host goes back to system as a"},{"line_number":20,"context_line":"reserved host, it is required to make the compute host empty and be available"},{"line_number":21,"context_line":"at its full capacity. So it would be useful to clear off/evacuate non-recovery"},{"line_number":22,"context_line":"instances in shutoff status i.e. ``HA_enabled\u003dFalse`` , and while doing so one"},{"line_number":23,"context_line":"should not clear instances like ``Canaria`` which is meant for monitoring the"},{"line_number":24,"context_line":"health of the compute host."},{"line_number":25,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"ff570b3c_8723a205","line":22,"range":{"start_line":21,"start_character":22,"end_line":22,"end_character":55},"in_reply_to":"ff570b3c_628e3fd1","updated":"2020-05-26 09:32:58.000000000","message":"It makes me confused. Whether to evacuate \u0027HA_enabled\u003dFalse\u0027 instances?","commit_id":"db83c367b34f14426bc50c30e9937a823b4e661a"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"8ad90f59ea2d0d7a0bf4d19e8dcf97eeecfb11f7","unresolved":false,"context_lines":[{"line_number":18,"context_line":"when ``evacuate_all_instances`` is enabled or when ``HA_enabled\u003dTrue``"},{"line_number":19,"context_line":"metadata is set to the instance. When a failed host goes back to system as a"},{"line_number":20,"context_line":"reserved host, it is required to make the compute host empty and be available"},{"line_number":21,"context_line":"at its full capacity. So it would be useful to clear off/evacuate non-recovery"},{"line_number":22,"context_line":"instances in shutoff status i.e. ``HA_enabled\u003dFalse`` , and while doing so one"},{"line_number":23,"context_line":"should not clear instances like ``Canaria`` which is meant for monitoring the"},{"line_number":24,"context_line":"health of the compute host."},{"line_number":25,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"ff570b3c_36ee38b4","line":22,"range":{"start_line":21,"start_character":22,"end_line":22,"end_character":55},"in_reply_to":"ff570b3c_8723a205","updated":"2020-05-27 02:36:43.000000000","message":"I would suggest you to check \"Matrix of various host failure config option (New/ Existing) combinations\" section which explains what would happen in various config option combinations.\n\nand then you can suggest what you would want to do for a particular combination.","commit_id":"db83c367b34f14426bc50c30e9937a823b4e661a"},{"author":{"_account_id":24501,"name":"sue","display_name":"suzhengwei","email":"sugar-2008@163.com","username":"suzhengwei"},"change_message_id":"877df9d860fd581671637467dd1a3b0b94385e9d","unresolved":false,"context_lines":[{"line_number":41,"context_line":"* ``evacuate_non_ha_enabled_to_shutoff_state`` to decide whether to"},{"line_number":42,"context_line":"  evacuate instances to shutoff state."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* ``ignore_instances_for_projects`` to decide not to evacuate special"},{"line_number":45,"context_line":"  instances like ``Canaria`` instance. If the instance belongs to any of tenants"},{"line_number":46,"context_line":"  specified in this config option, it should not be evacuated provided"},{"line_number":47,"context_line":"  HA_Enabled\u003dFalse or this key is not present."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":".. note:: Higher precedence will be given to ``ignore_instances_for_projects``"},{"line_number":50,"context_line":"          over other ``evacuate_all_instances``,"},{"line_number":51,"context_line":"          ``ignore_instances_in_error_state``,"},{"line_number":52,"context_line":"          ``evacuate_non_ha_enabled_to_shutoff_state`` config options."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Setting evacuate_non_ha_enabled_to_shutoff_state"},{"line_number":55,"context_line":"------------------------------------------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"ff570b3c_27aa99d1","line":52,"range":{"start_line":44,"start_character":0,"end_line":52,"end_character":70},"updated":"2020-05-26 02:59:18.000000000","message":"We cann\u0027t know the tenant IDs to ignore when initialize the cloud platform. And the tenants may changes over time.\nSo the \u0027ignore_instances_for_projects\u0027 is not friendly to devopts.","commit_id":"db83c367b34f14426bc50c30e9937a823b4e661a"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"462d21fcf6e572c4b1f75b9e50dec733b54914e4","unresolved":false,"context_lines":[{"line_number":41,"context_line":"* ``evacuate_non_ha_enabled_to_shutoff_state`` to decide whether to"},{"line_number":42,"context_line":"  evacuate instances to shutoff state."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* ``ignore_instances_for_projects`` to decide not to evacuate special"},{"line_number":45,"context_line":"  instances like ``Canaria`` instance. If the instance belongs to any of tenants"},{"line_number":46,"context_line":"  specified in this config option, it should not be evacuated provided"},{"line_number":47,"context_line":"  HA_Enabled\u003dFalse or this key is not present."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":".. note:: Higher precedence will be given to ``ignore_instances_for_projects``"},{"line_number":50,"context_line":"          over other ``evacuate_all_instances``,"},{"line_number":51,"context_line":"          ``ignore_instances_in_error_state``,"},{"line_number":52,"context_line":"          ``evacuate_non_ha_enabled_to_shutoff_state`` config options."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Setting evacuate_non_ha_enabled_to_shutoff_state"},{"line_number":55,"context_line":"------------------------------------------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"ff570b3c_a29e97b5","line":52,"range":{"start_line":44,"start_character":0,"end_line":52,"end_character":70},"in_reply_to":"ff570b3c_27aa99d1","updated":"2020-05-26 07:28:12.000000000","message":"The config option ``ignore_instances_for_projects`` is not meant to include ``project_ids`` of normal end users. As an operator when you want to run ``Canaria`` instances for monitoring of compute host, you will know the tenant_id in advance or after it\u0027s initialized you will need to restart the masakari services after updating the masakari conf file.","commit_id":"db83c367b34f14426bc50c30e9937a823b4e661a"},{"author":{"_account_id":24501,"name":"sue","display_name":"suzhengwei","email":"sugar-2008@163.com","username":"suzhengwei"},"change_message_id":"e193171ca640f53e113c4efa92625d860a312bde","unresolved":false,"context_lines":[{"line_number":41,"context_line":"* ``evacuate_non_ha_enabled_to_shutoff_state`` to decide whether to"},{"line_number":42,"context_line":"  evacuate instances to shutoff state."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* ``ignore_instances_for_projects`` to decide not to evacuate special"},{"line_number":45,"context_line":"  instances like ``Canaria`` instance. If the instance belongs to any of tenants"},{"line_number":46,"context_line":"  specified in this config option, it should not be evacuated provided"},{"line_number":47,"context_line":"  HA_Enabled\u003dFalse or this key is not present."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":".. note:: Higher precedence will be given to ``ignore_instances_for_projects``"},{"line_number":50,"context_line":"          over other ``evacuate_all_instances``,"},{"line_number":51,"context_line":"          ``ignore_instances_in_error_state``,"},{"line_number":52,"context_line":"          ``evacuate_non_ha_enabled_to_shutoff_state`` config options."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Setting evacuate_non_ha_enabled_to_shutoff_state"},{"line_number":55,"context_line":"------------------------------------------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"ff570b3c_e7c41e43","line":52,"range":{"start_line":44,"start_character":0,"end_line":52,"end_character":70},"in_reply_to":"ff570b3c_a29e97b5","updated":"2020-05-26 09:32:58.000000000","message":"I got your point.\nIt can use \u0027ha_enable\u0027 metadata to identify such instances, and \u0027ignore_instances_for_projects\u0027 looks redundant.\nTo the product, if it can\u0027t be configurated automatically, it is not a good design. That\u0027s why I said it is not friendly to devops.","commit_id":"db83c367b34f14426bc50c30e9937a823b4e661a"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"8ad90f59ea2d0d7a0bf4d19e8dcf97eeecfb11f7","unresolved":false,"context_lines":[{"line_number":41,"context_line":"* ``evacuate_non_ha_enabled_to_shutoff_state`` to decide whether to"},{"line_number":42,"context_line":"  evacuate instances to shutoff state."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"* ``ignore_instances_for_projects`` to decide not to evacuate special"},{"line_number":45,"context_line":"  instances like ``Canaria`` instance. If the instance belongs to any of tenants"},{"line_number":46,"context_line":"  specified in this config option, it should not be evacuated provided"},{"line_number":47,"context_line":"  HA_Enabled\u003dFalse or this key is not present."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":".. note:: Higher precedence will be given to ``ignore_instances_for_projects``"},{"line_number":50,"context_line":"          over other ``evacuate_all_instances``,"},{"line_number":51,"context_line":"          ``ignore_instances_in_error_state``,"},{"line_number":52,"context_line":"          ``evacuate_non_ha_enabled_to_shutoff_state`` config options."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Setting evacuate_non_ha_enabled_to_shutoff_state"},{"line_number":55,"context_line":"------------------------------------------------"}],"source_content_type":"text/x-rst","patch_set":9,"id":"ff570b3c_f638403c","line":52,"range":{"start_line":44,"start_character":0,"end_line":52,"end_character":70},"in_reply_to":"ff570b3c_e7c41e43","updated":"2020-05-27 02:36:43.000000000","message":"I can understand it\u0027s painful for DevOps but there are many other OpenStack services where they are managing to do the same thing like Nova scheduler filters: e.g. isolated host filter.\n\nThe other possible solution would be to add metadata to failover_segment in which you can specify the project_id that you would otherwise specify in ``ignore_instances_for_projects`` config option.","commit_id":"db83c367b34f14426bc50c30e9937a823b4e661a"}]}