)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"83b9c142d258388f49b480a3f9e455b596850d56","unresolved":true,"context_lines":[{"line_number":10,"context_line":"this is the sole remaining user of ironicclient which means users would"},{"line_number":11,"context_line":"continue needing to install that just for this little API. Better to"},{"line_number":12,"context_line":"switch this holdout over and finally delete all the things."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I880523935d73ca94c83e618f10c2e587362c53be"},{"line_number":15,"context_line":"Signed-off-by: Stephen Finucane \u003csfinucan@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"e7f87b8c_bb3c15af","line":13,"updated":"2023-12-07 11:26:32.000000000","message":"we really should condier deleteing this but not this cycle. \n\nnext cycle we shouod consider if its finally time to remove the proxy apis.","commit_id":"259b646afa1e1641ade84169512840cc75151274"}],"/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":"0cb163f2f3581c8e875feb0ca6c0a99152cb9edb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c0eac484_681c8407","updated":"2023-12-06 20:26:49.000000000","message":"I did not even realize this exists. Wow.","commit_id":"259b646afa1e1641ade84169512840cc75151274"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8a875f2547a3392f84fcb634e46e988cbd054841","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"97257d5a_11c941e7","updated":"2023-12-11 11:56:26.000000000","message":"recheck","commit_id":"259b646afa1e1641ade84169512840cc75151274"},{"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":"7a0695fa25d22ab9a525efb86e34ef366fbc5e3b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ab6eac84_4ff57f9c","updated":"2023-12-13 20:12:52.000000000","message":"recheck failures unrelated to these changes","commit_id":"259b646afa1e1641ade84169512840cc75151274"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"050af14ddad9120bb783a344b424867fbac6c162","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e1a68ba2_5435d77d","updated":"2023-12-07 11:00:19.000000000","message":"recheck timeout","commit_id":"259b646afa1e1641ade84169512840cc75151274"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8b86359631df6e1ed150e7d179475767e6d04279","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"47aa4d79_222d25fb","updated":"2023-12-08 10:57:40.000000000","message":"recheck unrelated timeout","commit_id":"259b646afa1e1641ade84169512840cc75151274"}],"nova/tests/functional/api_sample_tests/test_baremetal_nodes.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"83b9c142d258388f49b480a3f9e455b596850d56","unresolved":true,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"def ports(*args, **kwargs):"},{"line_number":50,"context_line":"    # return an empty generator"},{"line_number":51,"context_line":"    return"},{"line_number":52,"context_line":"    yield"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"ab543886_84194c04","line":51,"updated":"2023-12-07 11:26:32.000000000","message":"without the comment i would think your high\nbut ok i guess this makes sense event if its very odd.","commit_id":"259b646afa1e1641ade84169512840cc75151274"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"ae6dcfbf3cf135d0016df9fd7650fc65c88af607","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"def ports(*args, **kwargs):"},{"line_number":50,"context_line":"    # return an empty generator"},{"line_number":51,"context_line":"    return"},{"line_number":52,"context_line":"    yield"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"2b2e034f_19685137","line":51,"in_reply_to":"ab543886_84194c04","updated":"2023-12-07 11:49:07.000000000","message":"Yeah, it\u0027s a weird one. If you reversed the order of this, you\u0027d get a generator that returns a single element, `None`. My guess is that I read [this SO post](https://stackoverflow.com/questions/13243766/how-to-define-an-empty-generator-function) to figure this out.\n\nLater: reading through it more, it seems this would also work:\n\n```\ndef ports(*args, **kwargs):\n    yield from ()\n```\n\nand it\u0027s more readable. Will push a FUP.","commit_id":"259b646afa1e1641ade84169512840cc75151274"}]}