)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"22bcb85d1fe2ab8408a9e0148ec48bf1d1935fc5","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently ilo5 based hardware does not support"},{"line_number":10,"context_line":"redfish based firmware update. This patch fixes"},{"line_number":11,"context_line":"the issue by making the necessary changes."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I9d70fed0de1829973748c06a1342d7a7af0f93d4"},{"line_number":14,"context_line":"Story: #2008403"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f88c80e_0b483fce","line":11,"range":{"start_line":11,"start_character":20,"end_line":11,"end_character":41},"updated":"2020-12-14 03:12:30.000000000","message":"A sentence or two on what the necessary changes are would be useful for some context.","commit_id":"3cdea6c6a17b43d84a272f0f213ec89f46e75eb1"},{"author":{"_account_id":19604,"name":"ankit","email":"ankit.dhn31@gmail.com","username":"ankit"},"change_message_id":"8d089eacdbc0bc6227b2626deb3b1048788cf616","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently ilo5 based hardware does not support"},{"line_number":10,"context_line":"redfish based firmware update. This patch fixes"},{"line_number":11,"context_line":"the issue by making the necessary changes."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I9d70fed0de1829973748c06a1342d7a7af0f93d4"},{"line_number":14,"context_line":"Story: #2008403"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"11fa0b38_ed1191b4","line":11,"range":{"start_line":11,"start_character":20,"end_line":11,"end_character":41},"in_reply_to":"9f88c80e_0b483fce","updated":"2020-12-15 07:49:23.000000000","message":"Done","commit_id":"3cdea6c6a17b43d84a272f0f213ec89f46e75eb1"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"53422129178ea2f4aafca11779f9c5c6a1a1210d","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes issue of redfish firmware update"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently ilo5 based hardware does not support redfish"},{"line_number":10,"context_line":"based firmware update. This patch fixes the issue by"},{"line_number":11,"context_line":"making the change to to check whether sushy_task.messages"},{"line_number":12,"context_line":"is present."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I9d70fed0de1829973748c06a1342d7a7af0f93d4"},{"line_number":15,"context_line":"Story: #2008403"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"ba48a06a_26ca2442","line":12,"range":{"start_line":9,"start_character":0,"end_line":12,"end_character":11},"updated":"2021-01-20 16:18:21.000000000","message":"It was also not calling prepare_ramdisk() before rebooting the system to update the firmware. It has been fixed in this patch.","commit_id":"bdbbb023d598fe6b19eaaa4bc58c6392aef99b94"},{"author":{"_account_id":19604,"name":"ankit","email":"ankit.dhn31@gmail.com","username":"ankit"},"change_message_id":"11e6cb1afbc035eeb08ce5554c9ea6169b1f09c6","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes issue of redfish firmware update"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently ilo5 based hardware does not support redfish"},{"line_number":10,"context_line":"based firmware update. This patch fixes the issue by"},{"line_number":11,"context_line":"making the change to to check whether sushy_task.messages"},{"line_number":12,"context_line":"is present."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I9d70fed0de1829973748c06a1342d7a7af0f93d4"},{"line_number":15,"context_line":"Story: #2008403"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"569a2d83_d309a3a8","line":12,"range":{"start_line":9,"start_character":0,"end_line":12,"end_character":11},"in_reply_to":"ba48a06a_26ca2442","updated":"2021-01-21 06:26:37.000000000","message":"Done","commit_id":"bdbbb023d598fe6b19eaaa4bc58c6392aef99b94"}],"ironic/drivers/modules/redfish/management.py":[{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"c3723c8a60c543a18e8b721a7f9c501cc4973482","unresolved":true,"context_lines":[{"line_number":747,"context_line":"        driver_internal_info \u003d node.driver_internal_info"},{"line_number":748,"context_line":"        firmware_update[\u0027task_monitor\u0027] \u003d task_monitor.task_monitor"},{"line_number":749,"context_line":"        if task_monitor._task_uri:"},{"line_number":750,"context_line":"            task_uri \u003d firmware_update[\u0027task_monitor\u0027].replace(\u0027TaskMonitors\u0027,"},{"line_number":751,"context_line":"                                                               \u0027Tasks\u0027)"},{"line_number":752,"context_line":"            firmware_update[\u0027task_uri\u0027] \u003d task_uri"},{"line_number":753,"context_line":"        driver_internal_info[\u0027firmware_updates\u0027] \u003d firmware_updates"},{"line_number":754,"context_line":"        node.driver_internal_info \u003d driver_internal_info"}],"source_content_type":"text/x-python","patch_set":1,"id":"acf13043_041ee758","line":751,"range":{"start_line":750,"start_character":12,"end_line":751,"end_character":71},"updated":"2020-12-04 11:14:46.000000000","message":"Is this required? The task_uri is a Task URI and not TaskMonitor URI. Isn\u0027t it?","commit_id":"3cdea6c6a17b43d84a272f0f213ec89f46e75eb1"},{"author":{"_account_id":19604,"name":"ankit","email":"ankit.dhn31@gmail.com","username":"ankit"},"change_message_id":"8d089eacdbc0bc6227b2626deb3b1048788cf616","unresolved":false,"context_lines":[{"line_number":747,"context_line":"        driver_internal_info \u003d node.driver_internal_info"},{"line_number":748,"context_line":"        firmware_update[\u0027task_monitor\u0027] \u003d task_monitor.task_monitor"},{"line_number":749,"context_line":"        if task_monitor._task_uri:"},{"line_number":750,"context_line":"            task_uri \u003d firmware_update[\u0027task_monitor\u0027].replace(\u0027TaskMonitors\u0027,"},{"line_number":751,"context_line":"                                                               \u0027Tasks\u0027)"},{"line_number":752,"context_line":"            firmware_update[\u0027task_uri\u0027] \u003d task_uri"},{"line_number":753,"context_line":"        driver_internal_info[\u0027firmware_updates\u0027] \u003d firmware_updates"},{"line_number":754,"context_line":"        node.driver_internal_info \u003d driver_internal_info"}],"source_content_type":"text/x-python","patch_set":1,"id":"7978b8ed_b1a37149","line":751,"range":{"start_line":750,"start_character":12,"end_line":751,"end_character":71},"in_reply_to":"97975449_45761dcd","updated":"2020-12-15 07:49:23.000000000","message":"Done","commit_id":"3cdea6c6a17b43d84a272f0f213ec89f46e75eb1"},{"author":{"_account_id":19604,"name":"ankit","email":"ankit.dhn31@gmail.com","username":"ankit"},"change_message_id":"72f0675821f25edf3782deeb0ed9968e0ceb43d5","unresolved":true,"context_lines":[{"line_number":747,"context_line":"        driver_internal_info \u003d node.driver_internal_info"},{"line_number":748,"context_line":"        firmware_update[\u0027task_monitor\u0027] \u003d task_monitor.task_monitor"},{"line_number":749,"context_line":"        if task_monitor._task_uri:"},{"line_number":750,"context_line":"            task_uri \u003d firmware_update[\u0027task_monitor\u0027].replace(\u0027TaskMonitors\u0027,"},{"line_number":751,"context_line":"                                                               \u0027Tasks\u0027)"},{"line_number":752,"context_line":"            firmware_update[\u0027task_uri\u0027] \u003d task_uri"},{"line_number":753,"context_line":"        driver_internal_info[\u0027firmware_updates\u0027] \u003d firmware_updates"},{"line_number":754,"context_line":"        node.driver_internal_info \u003d driver_internal_info"}],"source_content_type":"text/x-python","patch_set":1,"id":"97975449_45761dcd","line":751,"range":{"start_line":750,"start_character":12,"end_line":751,"end_character":71},"in_reply_to":"acf13043_041ee758","updated":"2020-12-09 05:32:46.000000000","message":"Yes, this is required because as part of response we get only this \"@odata.id\u0027: \u0027/redfish/v1/TaskService/Tasks/17/\" for task_uri. So to make it a complete Task URI, i did this change. \n\nTo make it more clear \nTaskMonitor URI : https://172.17.1.108/redfish/v1/TaskService/TaskMonitors/17/\n\nTask URI : https://172.17.1.93/redfish/v1/TaskService/Tasks/9/","commit_id":"3cdea6c6a17b43d84a272f0f213ec89f46e75eb1"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"fa6eaf6ea76e23bee78c5b95b6056c688f619d30","unresolved":true,"context_lines":[{"line_number":953,"context_line":"            return"},{"line_number":954,"context_line":""},{"line_number":955,"context_line":"        try:"},{"line_number":956,"context_line":"            if current_update.get(\u0027task_uri\u0027):"},{"line_number":957,"context_line":"                task_monitor \u003d update_service.get_task_monitor("},{"line_number":958,"context_line":"                    current_update[\u0027task_uri\u0027])"},{"line_number":959,"context_line":"            else:"},{"line_number":960,"context_line":"                task_monitor \u003d update_service.get_task_monitor("},{"line_number":961,"context_line":"                    current_update[\u0027task_monitor\u0027])"},{"line_number":962,"context_line":"        except sushy.exceptions.ResourceNotFoundError:"},{"line_number":963,"context_line":"            # The BMC deleted the Task before we could query it"},{"line_number":964,"context_line":"            LOG.warning(\u0027Firmware update completed for node %(node)s, \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"caf4277c_04dc9e3f","line":961,"range":{"start_line":956,"start_character":0,"end_line":961,"end_character":51},"updated":"2020-12-08 16:38:28.000000000","message":"it would be better to hide/abstract these differences in sushy, otherwise this will be necessary all the time when Redfish async operations are involved in Ironic (or any other sushy consumer) and I don\u0027t think that sushy consumers should think about these differences. In this scenario all it cares about is Task status/is it still running and messages.","commit_id":"3cdea6c6a17b43d84a272f0f213ec89f46e75eb1"},{"author":{"_account_id":19604,"name":"ankit","email":"ankit.dhn31@gmail.com","username":"ankit"},"change_message_id":"8d089eacdbc0bc6227b2626deb3b1048788cf616","unresolved":false,"context_lines":[{"line_number":953,"context_line":"            return"},{"line_number":954,"context_line":""},{"line_number":955,"context_line":"        try:"},{"line_number":956,"context_line":"            if current_update.get(\u0027task_uri\u0027):"},{"line_number":957,"context_line":"                task_monitor \u003d update_service.get_task_monitor("},{"line_number":958,"context_line":"                    current_update[\u0027task_uri\u0027])"},{"line_number":959,"context_line":"            else:"},{"line_number":960,"context_line":"                task_monitor \u003d update_service.get_task_monitor("},{"line_number":961,"context_line":"                    current_update[\u0027task_monitor\u0027])"},{"line_number":962,"context_line":"        except sushy.exceptions.ResourceNotFoundError:"},{"line_number":963,"context_line":"            # The BMC deleted the Task before we could query it"},{"line_number":964,"context_line":"            LOG.warning(\u0027Firmware update completed for node %(node)s, \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aaab753_8da8d13b","line":961,"range":{"start_line":956,"start_character":0,"end_line":961,"end_character":51},"in_reply_to":"caf4277c_04dc9e3f","updated":"2020-12-15 07:49:23.000000000","message":"Done","commit_id":"3cdea6c6a17b43d84a272f0f213ec89f46e75eb1"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"fa6eaf6ea76e23bee78c5b95b6056c688f619d30","unresolved":true,"context_lines":[{"line_number":980,"context_line":"            # Only parse the messages if the BMC did not return parsed"},{"line_number":981,"context_line":"            # messages"},{"line_number":982,"context_line":"            messages \u003d []"},{"line_number":983,"context_line":"            if sushy_task.messages:"},{"line_number":984,"context_line":"                sushy_task.parse_messages()"},{"line_number":985,"context_line":""},{"line_number":986,"context_line":"            messages \u003d [m.message for m in sushy_task.messages]"}],"source_content_type":"text/x-python","patch_set":1,"id":"94999294_36fb46be","line":983,"range":{"start_line":983,"start_character":12,"end_line":983,"end_character":35},"updated":"2020-12-08 16:38:28.000000000","message":"This update is changing the logic. Previous code was checking if `.message` field was populated (assuming that it will be populated not only for the first message, but also all the others). And if it is not populated then `parse_messages`. \n\nThe current logic checks if there are any messages, and if there are, then parse. In this case the `.message` field might be already populated and will be overwritten. For iDRAC this field will be populated from BMC, by parsing\u0026overwriting this could end up in worse message/might not have enough info to parse the message.\n\nIs there a reason why this was changed? Something different in iLO? (mind the comment in ln 980-981 that is outdated by this change, if kept).","commit_id":"3cdea6c6a17b43d84a272f0f213ec89f46e75eb1"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"0f3f1b17486373cc19fc62c86d56e8ef4b670402","unresolved":true,"context_lines":[{"line_number":980,"context_line":"            # Only parse the messages if the BMC did not return parsed"},{"line_number":981,"context_line":"            # messages"},{"line_number":982,"context_line":"            messages \u003d []"},{"line_number":983,"context_line":"            if sushy_task.messages:"},{"line_number":984,"context_line":"                sushy_task.parse_messages()"},{"line_number":985,"context_line":""},{"line_number":986,"context_line":"            messages \u003d [m.message for m in sushy_task.messages]"}],"source_content_type":"text/x-python","patch_set":1,"id":"c276a072_7a3c7853","line":983,"range":{"start_line":983,"start_character":12,"end_line":983,"end_character":35},"in_reply_to":"2b5bf58b_bd336adf","updated":"2020-12-09 08:48:06.000000000","message":"then could change this to `if sushy_task.messages and not sushy_task.messages[0].message` to keep the previous check also?","commit_id":"3cdea6c6a17b43d84a272f0f213ec89f46e75eb1"},{"author":{"_account_id":19604,"name":"ankit","email":"ankit.dhn31@gmail.com","username":"ankit"},"change_message_id":"308e2a504178f39a24ad3afbda3ba53ee862939e","unresolved":true,"context_lines":[{"line_number":980,"context_line":"            # Only parse the messages if the BMC did not return parsed"},{"line_number":981,"context_line":"            # messages"},{"line_number":982,"context_line":"            messages \u003d []"},{"line_number":983,"context_line":"            if sushy_task.messages:"},{"line_number":984,"context_line":"                sushy_task.parse_messages()"},{"line_number":985,"context_line":""},{"line_number":986,"context_line":"            messages \u003d [m.message for m in sushy_task.messages]"}],"source_content_type":"text/x-python","patch_set":1,"id":"2b5bf58b_bd336adf","line":983,"range":{"start_line":983,"start_character":12,"end_line":983,"end_character":35},"in_reply_to":"94999294_36fb46be","updated":"2020-12-09 05:27:09.000000000","message":"In case of iLO, sushy_task does not contain the messages attribute and because of that we were getting the below error.\n\nAttributeError: \u0027Task\u0027 object has no attribute \u0027messages\u0027\n\nSo provided the check if task contains the messages attribute or not.","commit_id":"3cdea6c6a17b43d84a272f0f213ec89f46e75eb1"},{"author":{"_account_id":19604,"name":"ankit","email":"ankit.dhn31@gmail.com","username":"ankit"},"change_message_id":"8d089eacdbc0bc6227b2626deb3b1048788cf616","unresolved":false,"context_lines":[{"line_number":980,"context_line":"            # Only parse the messages if the BMC did not return parsed"},{"line_number":981,"context_line":"            # messages"},{"line_number":982,"context_line":"            messages \u003d []"},{"line_number":983,"context_line":"            if sushy_task.messages:"},{"line_number":984,"context_line":"                sushy_task.parse_messages()"},{"line_number":985,"context_line":""},{"line_number":986,"context_line":"            messages \u003d [m.message for m in sushy_task.messages]"}],"source_content_type":"text/x-python","patch_set":1,"id":"bea20b02_fb48d71a","line":983,"range":{"start_line":983,"start_character":12,"end_line":983,"end_character":35},"in_reply_to":"c276a072_7a3c7853","updated":"2020-12-15 07:49:23.000000000","message":"Done","commit_id":"3cdea6c6a17b43d84a272f0f213ec89f46e75eb1"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"49913b15716a09bebda0435d3d5ad8ae1011d66e","unresolved":false,"context_lines":[{"line_number":745,"context_line":"        task_monitor \u003d update_service.simple_update(firmware_url)"},{"line_number":746,"context_line":""},{"line_number":747,"context_line":"        driver_internal_info \u003d node.driver_internal_info"},{"line_number":748,"context_line":"        if task_monitor._task_uri:"},{"line_number":749,"context_line":"            firmware_update[\u0027task_monitor\u0027] \u003d task_monitor._task_uri"},{"line_number":750,"context_line":"        else:"},{"line_number":751,"context_line":"            firmware_update[\u0027task_monitor\u0027] \u003d task_monitor.task_monitor"}],"source_content_type":"text/x-python","patch_set":3,"id":"3c03049b_f4ffe36e","line":748,"updated":"2020-12-14 11:50:19.000000000","message":"Please don\u0027t access private fields of sushy objects","commit_id":"b26c49cd4f25031162a7efea359859e21fa3b15e"},{"author":{"_account_id":19604,"name":"ankit","email":"ankit.dhn31@gmail.com","username":"ankit"},"change_message_id":"8d089eacdbc0bc6227b2626deb3b1048788cf616","unresolved":false,"context_lines":[{"line_number":745,"context_line":"        task_monitor \u003d update_service.simple_update(firmware_url)"},{"line_number":746,"context_line":""},{"line_number":747,"context_line":"        driver_internal_info \u003d node.driver_internal_info"},{"line_number":748,"context_line":"        if task_monitor._task_uri:"},{"line_number":749,"context_line":"            firmware_update[\u0027task_monitor\u0027] \u003d task_monitor._task_uri"},{"line_number":750,"context_line":"        else:"},{"line_number":751,"context_line":"            firmware_update[\u0027task_monitor\u0027] \u003d task_monitor.task_monitor"}],"source_content_type":"text/x-python","patch_set":3,"id":"c2e727e3_2559af85","line":748,"in_reply_to":"3c03049b_f4ffe36e","updated":"2020-12-15 07:49:23.000000000","message":"\u003e Please don\u0027t access private fields of sushy objects\n\nDone","commit_id":"b26c49cd4f25031162a7efea359859e21fa3b15e"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"9c4b2979863e454423e4397cd23b3d1796dd8502","unresolved":true,"context_lines":[{"line_number":745,"context_line":"        task_monitor \u003d update_service.simple_update(firmware_url)"},{"line_number":746,"context_line":""},{"line_number":747,"context_line":"        driver_internal_info \u003d node.driver_internal_info"},{"line_number":748,"context_line":"        if task_monitor.task_uri:"},{"line_number":749,"context_line":"            firmware_update[\u0027task_monitor\u0027] \u003d task_monitor.task_uri"},{"line_number":750,"context_line":"        else:"},{"line_number":751,"context_line":"            firmware_update[\u0027task_monitor\u0027] \u003d task_monitor.task_monitor"},{"line_number":752,"context_line":""},{"line_number":753,"context_line":"        driver_internal_info[\u0027firmware_updates\u0027] \u003d firmware_updates"},{"line_number":754,"context_line":"        node.driver_internal_info \u003d driver_internal_info"}],"source_content_type":"text/x-python","patch_set":5,"id":"95d421fa_08ed02e0","line":751,"range":{"start_line":748,"start_character":0,"end_line":751,"end_character":71},"updated":"2020-12-29 10:49:32.000000000","message":"could go further and create a property in sushy that returns either task_uri or task_monitor - that is, this logic is part of sushy and does not need repeating in Ironic whenever Tasks are used.","commit_id":"88532d42a4b2f1184bc4042cda7d9abf6aa1c2ad"},{"author":{"_account_id":19604,"name":"ankit","email":"ankit.dhn31@gmail.com","username":"ankit"},"change_message_id":"23b453b05d356ca7cc32d414d7b820d117529a60","unresolved":false,"context_lines":[{"line_number":745,"context_line":"        task_monitor \u003d update_service.simple_update(firmware_url)"},{"line_number":746,"context_line":""},{"line_number":747,"context_line":"        driver_internal_info \u003d node.driver_internal_info"},{"line_number":748,"context_line":"        if task_monitor.task_uri:"},{"line_number":749,"context_line":"            firmware_update[\u0027task_monitor\u0027] \u003d task_monitor.task_uri"},{"line_number":750,"context_line":"        else:"},{"line_number":751,"context_line":"            firmware_update[\u0027task_monitor\u0027] \u003d task_monitor.task_monitor"},{"line_number":752,"context_line":""},{"line_number":753,"context_line":"        driver_internal_info[\u0027firmware_updates\u0027] \u003d firmware_updates"},{"line_number":754,"context_line":"        node.driver_internal_info \u003d driver_internal_info"}],"source_content_type":"text/x-python","patch_set":5,"id":"4e585358_6ab1903c","line":751,"range":{"start_line":748,"start_character":0,"end_line":751,"end_character":71},"in_reply_to":"95d421fa_08ed02e0","updated":"2021-01-12 10:04:06.000000000","message":"Done","commit_id":"88532d42a4b2f1184bc4042cda7d9abf6aa1c2ad"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"1f4733303955d7a471307631615ca8926c3b3072","unresolved":true,"context_lines":[{"line_number":746,"context_line":""},{"line_number":747,"context_line":"        driver_internal_info \u003d node.driver_internal_info"},{"line_number":748,"context_line":"        firmware_update[\u0027task_monitor\u0027] \u003d task_monitor.task_monitor"},{"line_number":749,"context_line":""},{"line_number":750,"context_line":"        driver_internal_info[\u0027firmware_updates\u0027] \u003d firmware_updates"},{"line_number":751,"context_line":"        node.driver_internal_info \u003d driver_internal_info"},{"line_number":752,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"4c21dab6_2e97aa43","line":749,"updated":"2021-01-13 10:22:58.000000000","message":"nit: unnecessary new line","commit_id":"5c39daf463214fd3a9226c782b23dffbcd0960b1"},{"author":{"_account_id":19604,"name":"ankit","email":"ankit.dhn31@gmail.com","username":"ankit"},"change_message_id":"956d9d930a65fdfa91f4d278d01baa2a2bbc22d3","unresolved":false,"context_lines":[{"line_number":746,"context_line":""},{"line_number":747,"context_line":"        driver_internal_info \u003d node.driver_internal_info"},{"line_number":748,"context_line":"        firmware_update[\u0027task_monitor\u0027] \u003d task_monitor.task_monitor"},{"line_number":749,"context_line":""},{"line_number":750,"context_line":"        driver_internal_info[\u0027firmware_updates\u0027] \u003d firmware_updates"},{"line_number":751,"context_line":"        node.driver_internal_info \u003d driver_internal_info"},{"line_number":752,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"0539a0a1_8fc78706","line":749,"in_reply_to":"4c21dab6_2e97aa43","updated":"2021-01-15 04:58:19.000000000","message":"Done","commit_id":"5c39daf463214fd3a9226c782b23dffbcd0960b1"}],"releasenotes/notes/fixes-ilo5-redfish-firmware-update-issue-c6dfcd71a2f659a5.yaml":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"55cf57e635ba5aab64ca5cdeff37203e346b1e1f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b2eeba1c_b76f77ed","line":8,"range":{"start_line":4,"start_character":4,"end_line":8,"end_character":0},"updated":"2020-12-15 15:29:35.000000000","message":"Please reformat in the style of \"Fixes xyz on abc.\" followed up by the additional details.","commit_id":"b0ab6a5178b142ffa344cdadf7bbc8473e1fc7f9"},{"author":{"_account_id":19604,"name":"ankit","email":"ankit.dhn31@gmail.com","username":"ankit"},"change_message_id":"30e275fd2f4a54c18af4ea07765f68a5c6f36ded","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d44b317f_dbfeb27e","line":8,"range":{"start_line":4,"start_character":4,"end_line":8,"end_character":0},"in_reply_to":"b2eeba1c_b76f77ed","updated":"2020-12-16 06:46:17.000000000","message":"Done","commit_id":"b0ab6a5178b142ffa344cdadf7bbc8473e1fc7f9"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"53422129178ea2f4aafca11779f9c5c6a1a1210d","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fixes redfish firmware update for ilo5 based hardware by"},{"line_number":5,"context_line":"    making necessary changes to check whether sushy_task.messages"},{"line_number":6,"context_line":"    is present, since in case of iLo task data does not contain"},{"line_number":7,"context_line":"    messages attribute."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"11554048_d70d87b0","line":5,"range":{"start_line":4,"start_character":4,"end_line":5,"end_character":65},"updated":"2021-01-20 16:18:21.000000000","message":"It was also not calling prepare_ramdisk() before rebooting the system to update the firmware. It has been fixed in this patch.","commit_id":"bdbbb023d598fe6b19eaaa4bc58c6392aef99b94"},{"author":{"_account_id":19604,"name":"ankit","email":"ankit.dhn31@gmail.com","username":"ankit"},"change_message_id":"11e6cb1afbc035eeb08ce5554c9ea6169b1f09c6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fixes redfish firmware update for ilo5 based hardware by"},{"line_number":5,"context_line":"    making necessary changes to check whether sushy_task.messages"},{"line_number":6,"context_line":"    is present, since in case of iLo task data does not contain"},{"line_number":7,"context_line":"    messages attribute."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"291daada_a128c86b","line":5,"range":{"start_line":4,"start_character":4,"end_line":5,"end_character":65},"in_reply_to":"11554048_d70d87b0","updated":"2021-01-21 06:26:37.000000000","message":"Done","commit_id":"bdbbb023d598fe6b19eaaa4bc58c6392aef99b94"}]}
