)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"d0f7cb62b849fc108d43065fc568febae6ac8fc9","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP] [CI] Test Ironic upgrade"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"TODO: Currently, it checks for available states to check if"},{"line_number":10,"context_line":"error signaling works."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4a5e446b2344f15749113d2e295217844489252e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":20,"id":"d076727a_b0db85c3","line":9,"updated":"2022-03-22 19:14:27.000000000","message":"this is obviously wrong atm","commit_id":"1dc71e731b403d8e328d4a97d39b63fb21514d70"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"5a320a1f07a59582c12221a4fa9b5ef54a6c988d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"0e53c738_e39d3dfa","updated":"2022-03-21 15:01:51.000000000","message":"recheck","commit_id":"0981752703023832c88dd3d26b69be69d175c744"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"cecb8e5851ea5635feaa70496351c1d980b71805","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"397ba3cd_a39389b9","updated":"2022-03-21 12:54:59.000000000","message":"recheck","commit_id":"0981752703023832c88dd3d26b69be69d175c744"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"414722e92550e36b31161a3775067807a05f46f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"247d5985_8d625b6a","updated":"2022-03-21 20:40:09.000000000","message":"recheck","commit_id":"9f3a5a03fb4ebfc741983630235ffef8844b414a"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"44126d7fcffffc36a9516b1969637ca2c3b11b0f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"764115bb_49d06dcf","updated":"2022-03-22 14:26:19.000000000","message":"needs more clients in kolla-toolbox","commit_id":"7dde8ec6ee777a303d38492fe53262778d3ec2bb"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"7d69178b94029448b0b5a70aa466df33fdc60ab5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"db382a58_e59bb1a2","updated":"2022-03-22 19:10:47.000000000","message":"Mark, please have a look here (upgrade.yml) - it is what I want to propose in https://review.opendev.org/c/openstack/kolla-ansible/+/832159","commit_id":"1dc71e731b403d8e328d4a97d39b63fb21514d70"}],"ansible/roles/ironic/tasks/upgrade.yml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"89d921d71861e21b5ba0e23d6effc1fa8317ca93","unresolved":true,"context_lines":[{"line_number":24,"context_line":"       map(attribute\u003d\u0027Provisioning State\u0027) |"},{"line_number":25,"context_line":"       select(\u0027search\u0027, \u0027\\\\bwait\\\\b\u0027) |"},{"line_number":26,"context_line":"       length) \u003d\u003d 0"},{"line_number":27,"context_line":"  run_once: true"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"# TODO(yoctozepto): Remove this task in Zed."},{"line_number":30,"context_line":"- name: Remove old Ironic containers"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"be5f67c5_4a38ca38","line":27,"updated":"2022-03-30 15:12:10.000000000","message":"This seems like a reasonable check to me. I assume you\u0027ve tested it with a waiting node?\n\nIt might be sensible to add a check to bypass the task.","commit_id":"1dc71e731b403d8e328d4a97d39b63fb21514d70"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"8f41da982b5b67f65d8be088d1e39c3f466db34e","unresolved":false,"context_lines":[{"line_number":24,"context_line":"       map(attribute\u003d\u0027Provisioning State\u0027) |"},{"line_number":25,"context_line":"       select(\u0027search\u0027, \u0027\\\\bwait\\\\b\u0027) |"},{"line_number":26,"context_line":"       length) \u003d\u003d 0"},{"line_number":27,"context_line":"  run_once: true"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"# TODO(yoctozepto): Remove this task in Zed."},{"line_number":30,"context_line":"- name: Remove old Ironic containers"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"46fee58b_c4dafa01","line":27,"in_reply_to":"4aacabcd_b9538efb","updated":"2022-03-30 16:24:31.000000000","message":"Done","commit_id":"1dc71e731b403d8e328d4a97d39b63fb21514d70"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"ffab0cfc5989c90c5eb6022c445d8b84846e03f2","unresolved":true,"context_lines":[{"line_number":24,"context_line":"       map(attribute\u003d\u0027Provisioning State\u0027) |"},{"line_number":25,"context_line":"       select(\u0027search\u0027, \u0027\\\\bwait\\\\b\u0027) |"},{"line_number":26,"context_line":"       length) \u003d\u003d 0"},{"line_number":27,"context_line":"  run_once: true"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"# TODO(yoctozepto): Remove this task in Zed."},{"line_number":30,"context_line":"- name: Remove old Ironic containers"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"4aacabcd_b9538efb","line":27,"in_reply_to":"be5f67c5_4a38ca38","updated":"2022-03-30 15:38:06.000000000","message":"Yes.\n\nOk, \"add a check\", checked!\n\nI will then patch this in the proper change.","commit_id":"1dc71e731b403d8e328d4a97d39b63fb21514d70"}]}