)]}'
{"/PATCHSET_LEVEL":[{"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":"2678e4dd84eab30a12eb721f89543fe2e553332f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fc3e55b9_2eb77e0a","updated":"2022-09-07 21:18:20.000000000","message":"Is there any quantity of this bugfix that could be applied without bumping the requirement? If not, I guess it\u0027ll just have to be unpackportable :(. ","commit_id":"6b64e7814e1e90e2ce0548c27f3892ffced95b92"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"85070980b5b11226a22489cac1b40d4b36da9bb3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5222740e_f3a21424","in_reply_to":"415f0b5b_b70a5a6b","updated":"2022-09-14 12:30:22.000000000","message":"nevermind, I mixed up 2 things and driver-requirements.txt returns to this patch and removed dependency. The 2nd patch (very small) needs to get to Zed, if this one does not make to it.","commit_id":"6b64e7814e1e90e2ce0548c27f3892ffced95b92"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"87039515129bd041e511c1d8e6d0ae9b10c7d5e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"415f0b5b_b70a5a6b","in_reply_to":"e78c7198_b4159045","updated":"2022-09-14 09:24:41.000000000","message":"anyway, I split driver-requirements.txt in a separate patch, as this needs to go into Zed release irrespective of this change. Please review that first, https://review.opendev.org/c/openstack/ironic/+/857597 . Thanks.","commit_id":"6b64e7814e1e90e2ce0548c27f3892ffced95b92"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"7bfe5f02bd27d0f2ee875f598feff3b5a3948c63","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e78c7198_b4159045","in_reply_to":"fc3e55b9_2eb77e0a","updated":"2022-09-08 07:17:12.000000000","message":"see the previous thread about this - https://review.opendev.org/c/openstack/ironic/+/855872/1/driver-requirements.txt#20 \n\nWould it help if driver-requirements.txt change is separate patch as that version needs to be bumped anyway for Zed with or without this change.","commit_id":"6b64e7814e1e90e2ce0548c27f3892ffced95b92"},{"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":"287494cd052f1b0c3dd609085a063bd6f60c7606","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3a68e21c_231277d7","updated":"2022-09-14 20:55:54.000000000","message":"I\u0027m confused about the state of this change given we also have a sushy-oem-idrac bump package now. It\u0027s my understanding this needs further changes before it\u0027s ready to go? ","commit_id":"78eb482f62e780ed53abbda4c10e4d69e15da00b"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"76ffcd0ffb3b4283c4752f96d35567a29249282a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"de64d86a_e3ddb7e6","in_reply_to":"3a68e21c_231277d7","updated":"2022-09-15 07:47:36.000000000","message":"sorry for confusion, this patch is and has been ready to go. However, previously it did 2 things:\n\n1) Bumped sushy-oem-idrac upper limits for Zed\n2) Fixed a bug that requires bumping sushy-oem-idrac lower limits\n\nI separated that into 2 patches to ensure that 1) is done for Zed if this patch does not make it to Zed because this is bigger and maybe there will not be enough time to review it.\n\nI would still appreciate that this patch is reviewed and makes it to Zed, especially, as it is deemed to be not back-portable.\n\n(Rebased to resolve the merge conflict.)","commit_id":"78eb482f62e780ed53abbda4c10e4d69e15da00b"}],"driver-requirements.txt":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"e6b6b1f2fd86134168041ac36ab946962cc7af3b","unresolved":true,"context_lines":[{"line_number":17,"context_line":"python-ibmcclient\u003e\u003d0.2.2,\u003c0.3.0"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# Dell EMC iDRAC sushy OEM extension"},{"line_number":20,"context_line":"sushy-oem-idrac\u003e\u003d5.0.0,\u003c6.0.0"}],"source_content_type":"text/plain","patch_set":1,"id":"c45943c9_afe70ce3","line":20,"updated":"2022-09-05 12:39:20.000000000","message":"This patch is a bugfix, but this change makes it not backportable. Are you sure it cannot be worked around without unconditionally bumping requirements?","commit_id":"fa060047e3504b031e8c7e2fc07bd428ae7d24a6"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"14639d3ab3b5b74a69cc2eb94a5f5fe0e046393c","unresolved":true,"context_lines":[{"line_number":17,"context_line":"python-ibmcclient\u003e\u003d0.2.2,\u003c0.3.0"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# Dell EMC iDRAC sushy OEM extension"},{"line_number":20,"context_line":"sushy-oem-idrac\u003e\u003d5.0.0,\u003c6.0.0"}],"source_content_type":"text/plain","patch_set":1,"id":"ee45487f_236409ed","line":20,"in_reply_to":"c45943c9_afe70ce3","updated":"2022-09-05 12:59:36.000000000","message":"I could backport sushy-oem-idrac[1] change, but as I understand backporting sushy[2] change is not really possible. Though, could think that everything that is missing is sushy is a bug, otherwise it is not possible to fix bugs that require fields/resources in sushy that haven\u0027t been added yet.\n\nFor now, I have added error handling for missing attributes in both sushy-oem-idrac  and sushy to emit a warning for those with older libraries and don\u0027t plan to backport driver-requirements.txt change.\n\nIf there are other options, I\u0027m ready to make adjustments.\n\n[1] https://review.opendev.org/c/openstack/sushy-oem-idrac/+/851893\n[2] https://review.opendev.org/c/openstack/sushy/+/850899","commit_id":"fa060047e3504b031e8c7e2fc07bd428ae7d24a6"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"86f20de5d015a7c9cc8c8b72143ccf57012e4f52","unresolved":false,"context_lines":[{"line_number":17,"context_line":"python-ibmcclient\u003e\u003d0.2.2,\u003c0.3.0"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# Dell EMC iDRAC sushy OEM extension"},{"line_number":20,"context_line":"sushy-oem-idrac\u003e\u003d5.0.0,\u003c6.0.0"}],"source_content_type":"text/plain","patch_set":1,"id":"f1cfb49f_837d4570","line":20,"in_reply_to":"ee45487f_236409ed","updated":"2022-09-19 15:47:03.000000000","message":"Resolving this thread - no other options available for now.","commit_id":"fa060047e3504b031e8c7e2fc07bd428ae7d24a6"}]}
