)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"42e0303122e662eb0e037c80318218d9ab2c8875","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Return the Etag directly from the headers without modification."},{"line_number":10,"context_line":"This workarounmd was used for Lenovo nodes who would reject weak etags."},{"line_number":11,"context_line":"The recent firmware version fixed this issue."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Story: #2009680"},{"line_number":14,"context_line":"Task: #45815"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"50d767b1_3e257e07","line":11,"updated":"2022-07-25 11:57:50.000000000","message":"Unfortunately, we cannot rely on the users to have recent firmware.. Is there anything wrong with the workaround?","commit_id":"c11947b9d3af07fe01f69a6fbb308fde14e753a9"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"ea6ba5c4baf188e7cd110442a082541c60956b3b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"26ba1f64_f4e6cb41","updated":"2022-07-20 01:21:50.000000000","message":"Hey Jacob, thanks for the link for the BZ! I had the feeling we faced a problem in another type of HW but couldn\u0027t find the BZ.\n\nWe\u0027ll probably need to test with some different HW to see how it goes to make sure we are not doing any regression when we backport this...\nWe will also need handle the scenario where people won\u0027t update the FW in Lenovo.. .-.","commit_id":"c11947b9d3af07fe01f69a6fbb308fde14e753a9"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"2b3f6efdcec9b1bfbb9f635379ede4c25c20a7f7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9013ff06_32a58ddc","updated":"2022-07-20 00:18:06.000000000","message":"Hi Uemit,\n\nThank you for the patch.\n\nI think we need to have a further discussion in the Community about making this work cross-vendor. Not long ago I was fixing a bug which was impacting HP and was caused by lack of the code you\u0027re proposing to revert:\n\nhttps://bugzilla.redhat.com/show_bug.cgi?id\u003d2088196\n\nI\u0027m concerned this revert may fix it on Lenovo and break it on older HPs...\n\nWhat are your thoughts on this? (also CC Iury and Arne)\n\nThank you,\nJacob\n\n","commit_id":"c11947b9d3af07fe01f69a6fbb308fde14e753a9"},{"author":{"_account_id":11292,"name":"Arne Wiebalck","email":"Arne.Wiebalck@cern.ch","username":"wiebalck"},"change_message_id":"cbbf93d24365de7860db1dd244955571f03a28a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c16ab34e_1b672335","updated":"2022-07-20 08:31:35.000000000","message":"I am not sure we should remove this. While we should clearly try to remove workarounds for things which are not needed any more, we cannot be sure they are not needed any more 😊\n\nMaybe we should (next time) explicitly state the h/w and f/w versions for which such a workaround was put in place ... but then: others may be covered without us knowing ...\n\nEqually, with such workarounds we do not detect \u0026 flag \"wrong\" implementations.\n\nHow about we keep the workaround, but log a message saying: \"The Redfish endpoint seems to be using weak ETags when it should not.\" ?","commit_id":"c11947b9d3af07fe01f69a6fbb308fde14e753a9"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"3a3329118e8e3fc30ecfa1f11cffb505c2b66b3c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8759a2be_2c377944","updated":"2023-03-02 02:28:15.000000000","message":"Thank you for the patch Uemit.\n\nGiven we handled this in https://review.opendev.org/c/openstack/sushy/+/856123, do we still need this patch or is it no longer required (I believe this is the case)?","commit_id":"c11947b9d3af07fe01f69a6fbb308fde14e753a9"}]}