)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"5468d5532ca5a7b57a6c6ec266938e482e7e6ce5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"11b93899_e7522eb7","updated":"2026-01-22 17:44:47.000000000","message":"Rebase this on top of https://review.opendev.org/c/openstack/ironic/+/955484 and remove the log line like Julia mentioned.","commit_id":"ebdfdb4689b1385c9bc5b976438ceff71f783eda"},{"author":{"_account_id":38165,"name":"Nahian Pathan","display_name":"Nahian","email":"npathan@redhat.com","username":"pixelsoccupied"},"change_message_id":"d06cedf7855c97c6bcc68387f82dc9644c949790","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e1444e9f_d1be5e91","updated":"2026-01-05 14:34:58.000000000","message":"need this before https://review.opendev.org/c/openstack/ironic/+/955484","commit_id":"ebdfdb4689b1385c9bc5b976438ceff71f783eda"}],"ironic/drivers/modules/redfish/management.py":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"ba32e657a7aadd74be25aec33058441810e75e31","unresolved":true,"context_lines":[{"line_number":715,"context_line":"                    if not sensor.get(\u0027state\u0027) and sensor.get(\u0027health\u0027):"},{"line_number":716,"context_line":"                        # If drive is reporting health status, assume enabled"},{"line_number":717,"context_line":"                        sensor[\u0027state\u0027] \u003d \u0027Enabled\u0027"},{"line_number":718,"context_line":"                        LOG.debug(\u0027Drive %(drive)s on node %(node)s has no \u0027"},{"line_number":719,"context_line":"                                  \u0027state but has health status, assuming \u0027"},{"line_number":720,"context_line":"                                  \u0027Enabled\u0027,"},{"line_number":721,"context_line":"                                  {\u0027drive\u0027: drive.name,"},{"line_number":722,"context_line":"                                   \u0027node\u0027: system.identity})"},{"line_number":723,"context_line":"                    unique_name \u003d \u0027%s:%s@%s\u0027 % ("},{"line_number":724,"context_line":"                        drive.name, storage.identity, system.identity)"},{"line_number":725,"context_line":"                    sensors[unique_name] \u003d sensor"}],"source_content_type":"text/x-python","patch_set":2,"id":"4b5babe1_3be71b26","line":722,"range":{"start_line":718,"start_character":0,"end_line":722,"end_character":60},"updated":"2026-01-06 17:55:22.000000000","message":"So, this could generate a lot of logs for just one node with sensor data collection engaged. I\u0027m worried it would flood logs with, for example, a hundred nodes with this getting logged each time. As such, I think it doesn\u0027t make sense to log that this has been done on each collection.","commit_id":"ebdfdb4689b1385c9bc5b976438ceff71f783eda"}]}
