)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"68c5a217c708edd764431b431b2eb19a502b3394","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2d54a2c3_b9ba14c8","updated":"2023-08-15 13:44:48.000000000","message":"recheck mirror download/connectivity failures","commit_id":"4f5e08d995ca28dd3781d517be1f054a9e2285b2"},{"author":{"_account_id":11292,"name":"Arne Wiebalck","email":"Arne.Wiebalck@cern.ch","username":"wiebalck"},"change_message_id":"bb0c83ebb9af123a45c299a0fd4cab9425ee4fac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"598491f4_fd204ca4","updated":"2024-01-05 14:34:12.000000000","message":"Thanks, Julia!","commit_id":"279e57eb5c854d3ceb19af6bf2adb423beddba1c"},{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"60c7bdac99cf1ea285c544df4866eb5ecebf249d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"87079d31_2ae6eda0","updated":"2024-03-05 02:23:53.000000000","message":"recheck","commit_id":"82dbaa96687848c5d5b95fe56c9ca635a023fef2"}],"ironic/conductor/manager.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"867c2ab163ffbda4239a616a511909b60602cb65","unresolved":false,"context_lines":[{"line_number":1436,"context_line":"        METRICS.send_gauge("},{"line_number":1437,"context_line":"            \u0027ConductorManager.PowerSyncNodesCount\u0027,"},{"line_number":1438,"context_line":"            len(nodes))"},{"line_number":1439,"context_line":"        LOG.debug(\u0027Completed _sync_power_state operation, evaluated %s\u0027"},{"line_number":1440,"context_line":"                  \u0027nodes.\u0027, len(nodes))"},{"line_number":1441,"context_line":""},{"line_number":1442,"context_line":"    def _sync_power_state_nodes_task(self, context, nodes):"}],"source_content_type":"text/x-python","patch_set":2,"id":"97c2db51_569cc019","line":1439,"updated":"2023-12-07 10:21:51.000000000","message":"I\u0027d use a human text rather than a function name","commit_id":"400768d6180c01fe1dbb8d240c24d4cb049d4a6b"},{"author":{"_account_id":11292,"name":"Arne Wiebalck","email":"Arne.Wiebalck@cern.ch","username":"wiebalck"},"change_message_id":"8c6cf8ba7d548a2781eeb89ddcbd060237a86b57","unresolved":true,"context_lines":[{"line_number":1436,"context_line":"        METRICS.send_gauge("},{"line_number":1437,"context_line":"            \u0027ConductorManager.PowerSyncNodesCount\u0027,"},{"line_number":1438,"context_line":"            len(nodes))"},{"line_number":1439,"context_line":"        LOG.debug(\u0027Completed _sync_power_state operation, evaluated %s\u0027"},{"line_number":1440,"context_line":"                  \u0027nodes.\u0027, len(nodes))"},{"line_number":1441,"context_line":""},{"line_number":1442,"context_line":"    def _sync_power_state_nodes_task(self, context, nodes):"}],"source_content_type":"text/x-python","patch_set":2,"id":"365047f1_729a6681","line":1439,"range":{"start_line":1439,"start_character":29,"end_line":1439,"end_character":46},"updated":"2023-08-15 14:40:35.000000000","message":"nit: as this is the function\u0027s name, it should read \"Completed _sync_power_states operation\" (with an \u0027s\u0027)","commit_id":"400768d6180c01fe1dbb8d240c24d4cb049d4a6b"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"7ef898e09a61c8a74751fb3a5e52b94db9d245a4","unresolved":true,"context_lines":[{"line_number":1436,"context_line":"        METRICS.send_gauge("},{"line_number":1437,"context_line":"            \u0027ConductorManager.PowerSyncNodesCount\u0027,"},{"line_number":1438,"context_line":"            len(nodes))"},{"line_number":1439,"context_line":"        LOG.debug(\u0027Completed _sync_power_state operation, evaluated %s\u0027"},{"line_number":1440,"context_line":"                  \u0027nodes.\u0027, len(nodes))"},{"line_number":1441,"context_line":""},{"line_number":1442,"context_line":"    def _sync_power_state_nodes_task(self, context, nodes):"}],"source_content_type":"text/x-python","patch_set":2,"id":"e59d02ac_9dc8ae2a","line":1439,"range":{"start_line":1439,"start_character":29,"end_line":1439,"end_character":46},"in_reply_to":"365047f1_729a6681","updated":"2024-01-03 23:05:54.000000000","message":"Went with Dmitry\u0027s idea, human text. 😊","commit_id":"400768d6180c01fe1dbb8d240c24d4cb049d4a6b"},{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"2e675a2276d19f31267a5c402df1cc8783e3e58d","unresolved":true,"context_lines":[{"line_number":1439,"context_line":"        METRICS.send_gauge("},{"line_number":1440,"context_line":"            \u0027ConductorManager.PowerSyncNodesCount\u0027,"},{"line_number":1441,"context_line":"            len(nodes))"},{"line_number":1442,"context_line":"        LOG.debug(\u0027Completed power state sync operation, evaluated %s\u0027"},{"line_number":1443,"context_line":"                  \u0027nodes.\u0027, len(nodes))"},{"line_number":1444,"context_line":""},{"line_number":1445,"context_line":"    def _sync_power_state_nodes_task(self, context, nodes):"}],"source_content_type":"text/x-python","patch_set":3,"id":"e7564de5_66172704","line":1442,"updated":"2024-01-05 12:02:19.000000000","message":"it seems missing a space, otherwise lgtm","commit_id":"ff06b31a93b092f4198e4439317526c86654604b"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"5622cd8dc918be981586cb7fda35ade889d44459","unresolved":true,"context_lines":[{"line_number":1439,"context_line":"        METRICS.send_gauge("},{"line_number":1440,"context_line":"            \u0027ConductorManager.PowerSyncNodesCount\u0027,"},{"line_number":1441,"context_line":"            len(nodes))"},{"line_number":1442,"context_line":"        LOG.debug(\u0027Completed power state sync operation, evaluated %s\u0027"},{"line_number":1443,"context_line":"                  \u0027nodes.\u0027, len(nodes))"},{"line_number":1444,"context_line":""},{"line_number":1445,"context_line":"    def _sync_power_state_nodes_task(self, context, nodes):"}],"source_content_type":"text/x-python","patch_set":3,"id":"97ed2334_cde8657e","line":1442,"in_reply_to":"e7564de5_66172704","updated":"2024-01-05 14:30:25.000000000","message":"great catch, thanks!","commit_id":"ff06b31a93b092f4198e4439317526c86654604b"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"2cc1f1cab4459061ed9cb6c2bab77d98ca7d201d","unresolved":true,"context_lines":[{"line_number":1440,"context_line":"            \u0027ConductorManager.PowerSyncNodesCount\u0027,"},{"line_number":1441,"context_line":"            len(nodes))"},{"line_number":1442,"context_line":"        LOG.debug(\u0027Completed power state sync operation, evaluated %s \u0027"},{"line_number":1443,"context_line":"                  \u0027nodes.\u0027, len(nodes))"},{"line_number":1444,"context_line":""},{"line_number":1445,"context_line":"    def _sync_power_state_nodes_task(self, context, nodes):"},{"line_number":1446,"context_line":"        \"\"\"Invokes power state sync on nodes from synchronized queue."}],"source_content_type":"text/x-python","patch_set":4,"id":"9bb8d628_2a382190","line":1443,"updated":"2024-01-08 17:36:27.000000000","message":"Should probably be len(futures), otherwise you may report more nodes if NoFreeConductorWorker was hit.","commit_id":"279e57eb5c854d3ceb19af6bf2adb423beddba1c"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"205bb204c59960dd697377a4b5535e74ef60dd31","unresolved":true,"context_lines":[{"line_number":1440,"context_line":"            \u0027ConductorManager.PowerSyncNodesCount\u0027,"},{"line_number":1441,"context_line":"            len(nodes))"},{"line_number":1442,"context_line":"        LOG.debug(\u0027Completed power state sync operation, evaluated %s \u0027"},{"line_number":1443,"context_line":"                  \u0027nodes.\u0027, len(nodes))"},{"line_number":1444,"context_line":""},{"line_number":1445,"context_line":"    def _sync_power_state_nodes_task(self, context, nodes):"},{"line_number":1446,"context_line":"        \"\"\"Invokes power state sync on nodes from synchronized queue."}],"source_content_type":"text/x-python","patch_set":4,"id":"b5701307_8dd57234","line":1443,"in_reply_to":"9bb8d628_2a382190","updated":"2024-01-09 14:46:39.000000000","message":"Might be worthwhile to actually log both...","commit_id":"279e57eb5c854d3ceb19af6bf2adb423beddba1c"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"ccce9d662a7e7cd54f45904e948fd54a06d18116","unresolved":false,"context_lines":[{"line_number":1440,"context_line":"            \u0027ConductorManager.PowerSyncNodesCount\u0027,"},{"line_number":1441,"context_line":"            len(nodes))"},{"line_number":1442,"context_line":"        LOG.debug(\u0027Completed power state sync operation, evaluated %s \u0027"},{"line_number":1443,"context_line":"                  \u0027nodes.\u0027, len(nodes))"},{"line_number":1444,"context_line":""},{"line_number":1445,"context_line":"    def _sync_power_state_nodes_task(self, context, nodes):"},{"line_number":1446,"context_line":"        \"\"\"Invokes power state sync on nodes from synchronized queue."}],"source_content_type":"text/x-python","patch_set":4,"id":"30aac5f3_1f4398d6","line":1443,"in_reply_to":"b5701307_8dd57234","updated":"2024-02-13 23:28:55.000000000","message":"Done","commit_id":"279e57eb5c854d3ceb19af6bf2adb423beddba1c"}]}