)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"6ff4fb81c3de8afa7c045e7cd2ee44ec6c4f6d96","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Convert found vlan to int"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Starting with 17 the vlan id stored in guest xml is a str, e.g.:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"    \u003cinterface type\u003d\u0027hostdev\u0027 managed\u003d\u0027yes\u0027\u003e"},{"line_number":12,"context_line":"      \u003cmac address\u003d\u0027fa:16:3e:5f:c6:99\u0027/\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"45ac5e3c_f83232c8","line":9,"updated":"2023-06-07 17:04:57.000000000","message":"We really should strive to leave downstream logic downstream. This probably isn\u0027t related to OSP 17 directly - in fact I\u0027m not sure how we never hit this before, since as far as I can tell the return type of get() for an attribute has always been a a string, so comparing it to an int would always return false.","commit_id":"1a99651e8cd47f9a063c1590244bca687ea37d3e"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"a71740109dfd5d211d8729ff3354a101bdd05ee4","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Convert found vlan to int"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Starting with 17 the vlan id stored in guest xml is a str, e.g.:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"    \u003cinterface type\u003d\u0027hostdev\u0027 managed\u003d\u0027yes\u0027\u003e"},{"line_number":12,"context_line":"      \u003cmac address\u003d\u0027fa:16:3e:5f:c6:99\u0027/\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"640d9bf9_ddf7e0b3","line":9,"in_reply_to":"45ac5e3c_f83232c8","updated":"2023-06-07 19:42:58.000000000","message":"This change was applied quite a while ago downstream so I was hitting this for sometime but I think a change to the vlan config option might be a better path forward.","commit_id":"1a99651e8cd47f9a063c1590244bca687ea37d3e"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"6ff4fb81c3de8afa7c045e7cd2ee44ec6c4f6d96","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"38121331_f1ccbfd4","updated":"2023-06-07 17:04:57.000000000","message":"Just the commit message to something more generic. I think it\u0027s OK that we don\u0027t necessarily understand the root cause, but we can explain the change in terms of comparing ints to ints rather than strings to ints.","commit_id":"1a99651e8cd47f9a063c1590244bca687ea37d3e"}]}