)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":35099,"name":"Jakub Jelinek","email":"jakub.jelinek@cern.ch","username":"kubajj"},"change_message_id":"a44953bdb22a0757935a00547682fa8e96c79ce7","unresolved":true,"context_lines":[{"line_number":13,"context_line":"* Add \u0027health\u0027 to node resource field definitions"},{"line_number":14,"context_line":"* Update node list command to display health field by default"},{"line_number":15,"context_line":"* Fix unit tests to expect health field in node listings"},{"line_number":16,"context_line":"* Update tox.ini to use hacking\u003e\u003d7.0.0 for Python 3.14 compatibility"},{"line_number":17,"context_line":"* Add comprehensive release note documenting the new functionality"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"The health field is now displayed in both standard and long listing"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"e324b127_f51fc95d","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":68},"updated":"2026-01-09 10:39:44.000000000","message":"nit: I think this is not done in this change... tox.ini is not listed","commit_id":"5b4a032d34705e6cf505cd03b23fce8ddccf6601"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"b791fe69c2171b2aefb1df96926862465e3be201","unresolved":false,"context_lines":[{"line_number":13,"context_line":"* Add \u0027health\u0027 to node resource field definitions"},{"line_number":14,"context_line":"* Update node list command to display health field by default"},{"line_number":15,"context_line":"* Fix unit tests to expect health field in node listings"},{"line_number":16,"context_line":"* Update tox.ini to use hacking\u003e\u003d7.0.0 for Python 3.14 compatibility"},{"line_number":17,"context_line":"* Add comprehensive release note documenting the new functionality"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"The health field is now displayed in both standard and long listing"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"e0409221_76e45490","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":68},"in_reply_to":"e324b127_f51fc95d","updated":"2026-01-09 13:49:44.000000000","message":"Good pickup, thank you Jakub","commit_id":"5b4a032d34705e6cf505cd03b23fce8ddccf6601"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"73d7b88061e3c99ad992e2db4bb334b0a0d5914d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"16222333_3bed02d5","updated":"2025-11-27 16:59:49.000000000","message":"Re-spinning the patch as a separate change-id and dependency.","commit_id":"d1a31a454d40e039f873fa7723e5c5a17e965d19"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"99e6fc0841f9b46daa22e6e2e598e0811a80354d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9d8ed4e8_6032bf87","updated":"2025-12-01 14:49:20.000000000","message":"(reason: suspect issue with the tempest job)","commit_id":"c479990faac65b7284936c1a746c8a8a554f7921"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"892a84121d7b2806b0fa5980e404fb646267f473","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0e7c1bec_cde2b036","updated":"2025-12-01 14:48:27.000000000","message":"recheck","commit_id":"c479990faac65b7284936c1a746c8a8a554f7921"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"d64b165d4678b623484615c784a666df6a39c5df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"be04a342_08657e48","updated":"2025-11-27 20:17:27.000000000","message":"recheck\ntrivial rebase, suspecting intermittent CI issue","commit_id":"c479990faac65b7284936c1a746c8a8a554f7921"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"93a811121e37fa87e28e50c8e9b1303f8c0e443a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"af7bf4f5_561e3534","updated":"2026-01-05 17:20:48.000000000","message":"rebase complete, CI pass, ready for re-reviews","commit_id":"5b4a032d34705e6cf505cd03b23fce8ddccf6601"},{"author":{"_account_id":35099,"name":"Jakub Jelinek","email":"jakub.jelinek@cern.ch","username":"kubajj"},"change_message_id":"e05b9d74ed39b4a784a52a18c4317a50de0a2f17","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"e335bdfd_ad96e9a7","updated":"2026-01-13 07:57:03.000000000","message":"Still happy with the change, thanks for the work","commit_id":"e83e4988eeae187486b4ad11c999cff530b3b6bb"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"e894d5e4e90eabc4e056de6e7d2de9590e9366d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"bc91ea0c_68c1362a","updated":"2026-01-13 09:51:09.000000000","message":"Thank you for your review, Jakub.","commit_id":"e83e4988eeae187486b4ad11c999cff530b3b6bb"},{"author":{"_account_id":35099,"name":"Jakub Jelinek","email":"jakub.jelinek@cern.ch","username":"kubajj"},"change_message_id":"93d3a9a4beb41d4e7c54562402d876e06f84e195","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"db730906_d1075507","updated":"2026-01-09 14:24:23.000000000","message":"lgtm","commit_id":"e83e4988eeae187486b4ad11c999cff530b3b6bb"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"bb2d7aef39759b85a78021b8cfe8dc664ef9a8cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"d9bef4ba_45965a75","in_reply_to":"e335bdfd_ad96e9a7","updated":"2026-01-13 15:00:53.000000000","message":"Thank you for your continuing reviews!:)","commit_id":"e83e4988eeae187486b4ad11c999cff530b3b6bb"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"75ea47d0b8266da1052796f6a45665ea50b1ca6b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":12,"id":"14e4c93e_f89992a3","updated":"2026-01-15 21:55:36.000000000","message":"LGTM other than my one comment","commit_id":"5e78a91f9d920282ffce4756aa884c0c7d8e4183"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"2170cc19e4e4f2cf9895d0ed96da022eee24f6b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"90167266_a2c16b99","in_reply_to":"14e4c93e_f89992a3","updated":"2026-01-19 16:31:48.000000000","message":"Thank you for your review, Jay.","commit_id":"5e78a91f9d920282ffce4756aa884c0c7d8e4183"},{"author":{"_account_id":35099,"name":"Jakub Jelinek","email":"jakub.jelinek@cern.ch","username":"kubajj"},"change_message_id":"c579ee5cbdeece4b1b3a819ea85644c7aa2a9bff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"9e67618f_8a443937","updated":"2026-01-19 14:56:55.000000000","message":"LGTM after the removal of unused code","commit_id":"da8eb081c08958ccec6efeeea436f60c6613e659"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"2170cc19e4e4f2cf9895d0ed96da022eee24f6b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"22f7ef2d_e759deda","updated":"2026-01-19 16:31:48.000000000","message":"Rebased the client to suit API. Removed the file as requested.","commit_id":"c1514baa7cf7e36a3a0f0c50eea1de823a9e65a7"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"df24f73ff59b36acc16c5f4f1add82d6ef662846","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":16,"id":"c1b505d8_b1ccd4bd","updated":"2026-01-21 15:14:10.000000000","message":"The change in test_http.py is wrong and there\u0027s a patch that\u0027ll land to fix it. Rebase this on top of that.","commit_id":"b17b4cfb5b4a763b9ee186529ba29d41173fda74"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"b18f6872c6f034f547156ae906fb5c3de880086b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"1d187d04_af341780","updated":"2026-01-21 09:21:59.000000000","message":"recheck","commit_id":"b17b4cfb5b4a763b9ee186529ba29d41173fda74"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"b879250941b76d4488a9be20d16fb091584f95f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"3d0d49b1_43a86cfd","in_reply_to":"c1b505d8_b1ccd4bd","updated":"2026-01-22 17:05:37.000000000","message":"Done","commit_id":"b17b4cfb5b4a763b9ee186529ba29d41173fda74"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"b879250941b76d4488a9be20d16fb091584f95f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"430edf3b_14d00313","updated":"2026-01-22 17:05:37.000000000","message":"Thanks Doug","commit_id":"81e881bc36e89e054c74a146b04b3c9b64a669c2"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"869f8ac5fe6bd088df6948fcff974f3744d1c0ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"48aac51f_1ec9b7a5","updated":"2026-01-23 09:12:32.000000000","message":"recheck","commit_id":"81e881bc36e89e054c74a146b04b3c9b64a669c2"}],"ironicclient/common/health_states.py":[{"author":{"_account_id":35099,"name":"Jakub Jelinek","email":"jakub.jelinek@cern.ch","username":"kubajj"},"change_message_id":"e05b9d74ed39b4a784a52a18c4317a50de0a2f17","unresolved":true,"context_lines":[{"line_number":21,"context_line":"comparison of health values."},{"line_number":22,"context_line":"\"\"\""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"OK \u003d \u0027OK\u0027"},{"line_number":25,"context_line":"\"\"\"Hardware is functioning normally with no issues detected.\"\"\""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"WARNING \u003d \u0027Warning\u0027"},{"line_number":28,"context_line":"\"\"\"Hardware has non-critical issues that may require attention.\"\"\""},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"CRITICAL \u003d \u0027Critical\u0027"},{"line_number":31,"context_line":"\"\"\"Hardware has critical issues requiring immediate attention.\"\"\""}],"source_content_type":"text/x-python","patch_set":10,"id":"e47119c6_46172c13","line":31,"range":{"start_line":24,"start_character":0,"end_line":31,"end_character":65},"updated":"2026-01-13 07:57:03.000000000","message":"```suggestion\nimport enum\n\n\nclass HealthState(enum.Enum):\n    \"\"\"Hardware health states reported by BMC.\"\"\"\n\n    OK \u003d \u0027OK\u0027\n    \"\"\"Hardware is functioning normally with no issues detected.\"\"\"\n\n    WARNING \u003d \u0027Warning\u0027\n    \"\"\"Hardware has non-critical issues that may require attention.\"\"\"\n\n    CRITICAL \u003d \u0027Critical\u0027\n    \"\"\"Hardware has critical issues requiring immediate attention.\"\"\"\n\n```\nI would say, if you are changing these to an enum in 966946: Add hardware health monitoring via management interface, I would probably keep it consistent","commit_id":"e83e4988eeae187486b4ad11c999cff530b3b6bb"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"e894d5e4e90eabc4e056de6e7d2de9590e9366d4","unresolved":false,"context_lines":[{"line_number":21,"context_line":"comparison of health values."},{"line_number":22,"context_line":"\"\"\""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"OK \u003d \u0027OK\u0027"},{"line_number":25,"context_line":"\"\"\"Hardware is functioning normally with no issues detected.\"\"\""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"WARNING \u003d \u0027Warning\u0027"},{"line_number":28,"context_line":"\"\"\"Hardware has non-critical issues that may require attention.\"\"\""},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"CRITICAL \u003d \u0027Critical\u0027"},{"line_number":31,"context_line":"\"\"\"Hardware has critical issues requiring immediate attention.\"\"\""}],"source_content_type":"text/x-python","patch_set":10,"id":"1efc8397_c75b642c","line":31,"range":{"start_line":24,"start_character":0,"end_line":31,"end_character":65},"in_reply_to":"e47119c6_46172c13","updated":"2026-01-13 09:51:09.000000000","message":"Good pickup. I missed this after reworking 966946. Fixed now.","commit_id":"e83e4988eeae187486b4ad11c999cff530b3b6bb"},{"author":{"_account_id":35099,"name":"Jakub Jelinek","email":"jakub.jelinek@cern.ch","username":"kubajj"},"change_message_id":"efc4a1fe32816ead0445295e1b1e0ffcf41c9273","unresolved":true,"context_lines":[{"line_number":16,"context_line":"import enum"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"class HealthState(enum.Enum):"},{"line_number":20,"context_line":"    \"\"\"Hardware health states reported by BMC.\"\"\""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    OK \u003d \u0027OK\u0027"}],"source_content_type":"text/x-python","patch_set":12,"id":"a6663c8f_e6b96218","line":19,"range":{"start_line":19,"start_character":6,"end_line":19,"end_character":17},"updated":"2026-01-13 10:56:01.000000000","message":"My current, maybe stupid, question is: where is this used?","commit_id":"5e78a91f9d920282ffce4756aa884c0c7d8e4183"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"2170cc19e4e4f2cf9895d0ed96da022eee24f6b8","unresolved":false,"context_lines":[{"line_number":16,"context_line":"import enum"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"class HealthState(enum.Enum):"},{"line_number":20,"context_line":"    \"\"\"Hardware health states reported by BMC.\"\"\""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    OK \u003d \u0027OK\u0027"}],"source_content_type":"text/x-python","patch_set":12,"id":"bc906456_aa9a079c","line":19,"range":{"start_line":19,"start_character":6,"end_line":19,"end_character":17},"in_reply_to":"05ddeafb_378bcf80","updated":"2026-01-19 16:31:48.000000000","message":"OK, removed the file.","commit_id":"5e78a91f9d920282ffce4756aa884c0c7d8e4183"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"75ea47d0b8266da1052796f6a45665ea50b1ca6b","unresolved":true,"context_lines":[{"line_number":16,"context_line":"import enum"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"class HealthState(enum.Enum):"},{"line_number":20,"context_line":"    \"\"\"Hardware health states reported by BMC.\"\"\""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    OK \u003d \u0027OK\u0027"}],"source_content_type":"text/x-python","patch_set":12,"id":"05ddeafb_378bcf80","line":19,"range":{"start_line":19,"start_character":6,"end_line":19,"end_character":17},"in_reply_to":"463a0ecc_33d8c28b","updated":"2026-01-15 21:55:36.000000000","message":"If this isn\u0027t used, it shouldn\u0027t be in this change.","commit_id":"5e78a91f9d920282ffce4756aa884c0c7d8e4183"},{"author":{"_account_id":35099,"name":"Jakub Jelinek","email":"jakub.jelinek@cern.ch","username":"kubajj"},"change_message_id":"4123a1de57eb15875fff90fff487f87e345c5ccd","unresolved":true,"context_lines":[{"line_number":16,"context_line":"import enum"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"class HealthState(enum.Enum):"},{"line_number":20,"context_line":"    \"\"\"Hardware health states reported by BMC.\"\"\""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    OK \u003d \u0027OK\u0027"}],"source_content_type":"text/x-python","patch_set":12,"id":"463a0ecc_33d8c28b","line":19,"range":{"start_line":19,"start_character":6,"end_line":19,"end_character":17},"in_reply_to":"9550101e_f464cd87","updated":"2026-01-13 16:12:16.000000000","message":"This makes sense. I am slightly more inclined to removing it though to prevent future confusion similar to mine 😄\nWill probably let somebody else to decide though","commit_id":"5e78a91f9d920282ffce4756aa884c0c7d8e4183"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"bb2d7aef39759b85a78021b8cfe8dc664ef9a8cc","unresolved":true,"context_lines":[{"line_number":16,"context_line":"import enum"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"class HealthState(enum.Enum):"},{"line_number":20,"context_line":"    \"\"\"Hardware health states reported by BMC.\"\"\""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    OK \u003d \u0027OK\u0027"}],"source_content_type":"text/x-python","patch_set":12,"id":"9550101e_f464cd87","line":19,"range":{"start_line":19,"start_character":6,"end_line":19,"end_character":17},"in_reply_to":"a6663c8f_e6b96218","updated":"2026-01-13 15:00:53.000000000","message":"I don\u0027t think this is stupid. Currently nowhere, I thought it may be used for future work. Claude suggested optionally adding it and I said yes. Would you like me to remove it?","commit_id":"5e78a91f9d920282ffce4756aa884c0c7d8e4183"}],"ironicclient/tests/unit/osc/v1/test_baremetal_node.py":[{"author":{"_account_id":35099,"name":"Jakub Jelinek","email":"jakub.jelinek@cern.ch","username":"kubajj"},"change_message_id":"d9a534e8d2f0974013a5c1e76501d5545a5542cf","unresolved":false,"context_lines":[{"line_number":1117,"context_line":"            baremetal_fakes.baremetal_power_state,"},{"line_number":1118,"context_line":"            baremetal_fakes.baremetal_provision_state,"},{"line_number":1119,"context_line":"            baremetal_fakes.baremetal_maintenance,"},{"line_number":1120,"context_line":"            \u0027\u0027,  # health"},{"line_number":1121,"context_line":"        ), )"},{"line_number":1122,"context_line":"        self.assertEqual(datalist, tuple(data))"},{"line_number":1123,"context_line":""}],"source_content_type":"text/x-python","patch_set":17,"id":"acaa893b_2436ba8d","line":1120,"updated":"2026-01-23 09:45:55.000000000","message":"probably does not make sense to also make it a baremetal_fake if it is used just in one test case","commit_id":"81e881bc36e89e054c74a146b04b3c9b64a669c2"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"2b079f84c29794eac2a046149eb7acdca0b17614","unresolved":false,"context_lines":[{"line_number":1117,"context_line":"            baremetal_fakes.baremetal_power_state,"},{"line_number":1118,"context_line":"            baremetal_fakes.baremetal_provision_state,"},{"line_number":1119,"context_line":"            baremetal_fakes.baremetal_maintenance,"},{"line_number":1120,"context_line":"            \u0027\u0027,  # health"},{"line_number":1121,"context_line":"        ), )"},{"line_number":1122,"context_line":"        self.assertEqual(datalist, tuple(data))"},{"line_number":1123,"context_line":""}],"source_content_type":"text/x-python","patch_set":17,"id":"cf92703c_ec552de8","line":1120,"in_reply_to":"acaa893b_2436ba8d","updated":"2026-01-26 14:55:14.000000000","message":"ugh, yeah.","commit_id":"81e881bc36e89e054c74a146b04b3c9b64a669c2"}]}