)]}'
{"nova/privsep/fs.py":[{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"32641cd56620bf94ebf4210bda49731cf459ca66","unresolved":true,"context_lines":[{"line_number":67,"context_line":"    )"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"@nova.privsep.sys_admin_pctxt.entrypoint"},{"line_number":71,"context_line":"def lvlist(vg):"},{"line_number":72,"context_line":"    return processutils.execute(\u0027lvs\u0027, \u0027--noheadings\u0027, \u0027-o\u0027, \u0027lv_name\u0027, vg)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"@nova.privsep.sys_admin_pctxt.entrypoint"},{"line_number":76,"context_line":"def lvinfo(path):"}],"source_content_type":"text/x-python","patch_set":1,"id":"86072535_a3347337","line":73,"range":{"start_line":70,"start_character":0,"end_line":73,"end_character":0},"updated":"2021-06-14 12:30:02.000000000","message":"Bug #1931710 initially detailed a failure of this call FWIW. Can you also add the retry here?","commit_id":"dfbe2a8a8eaf1227969151f3bf5009e88ea183e7"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"526cc5ede994cb2eb02a1b6ad825d56c46d3417e","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    )"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"@nova.privsep.sys_admin_pctxt.entrypoint"},{"line_number":71,"context_line":"def lvlist(vg):"},{"line_number":72,"context_line":"    return processutils.execute(\u0027lvs\u0027, \u0027--noheadings\u0027, \u0027-o\u0027, \u0027lv_name\u0027, vg)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"@nova.privsep.sys_admin_pctxt.entrypoint"},{"line_number":76,"context_line":"def lvinfo(path):"}],"source_content_type":"text/x-python","patch_set":1,"id":"47f18588_51cc6d9f","line":73,"range":{"start_line":70,"start_character":0,"end_line":73,"end_character":0},"in_reply_to":"86072535_a3347337","updated":"2021-06-14 12:41:49.000000000","message":"Done. I also checked the available logs of the nova-lvm job runs and we had two hits on lvs and one on vgs and no other hits for ERROR code -11. So now we are covered :)","commit_id":"dfbe2a8a8eaf1227969151f3bf5009e88ea183e7"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"04348b74bd9b7d1b97864d05de72447a5ec5a4c0","unresolved":true,"context_lines":[{"line_number":70,"context_line":"@nova.privsep.sys_admin_pctxt.entrypoint"},{"line_number":71,"context_line":"def lvlist(vg):"},{"line_number":72,"context_line":"    return processutils.execute("},{"line_number":73,"context_line":"        \u0027lvs\u0027, \u0027--noheadings\u0027, \u0027-o\u0027, \u0027lv_name\u0027, vg, attempts\u003d3,"},{"line_number":74,"context_line":"        delay_on_retry\u003dTrue)"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"6a112960_4481bfb1","line":73,"range":{"start_line":73,"start_character":52,"end_line":73,"end_character":63},"updated":"2021-06-15 10:29:52.000000000","message":"style nit: would be nice to drag this down like you\u0027ve done above to visually separate the actual command from the other kwargs","commit_id":"7bff51f7ac2f998a6e361310e63ed5f14ef4ff45"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"2de07c993837332d1fb549647b8b8cb5ff90307d","unresolved":false,"context_lines":[{"line_number":70,"context_line":"@nova.privsep.sys_admin_pctxt.entrypoint"},{"line_number":71,"context_line":"def lvlist(vg):"},{"line_number":72,"context_line":"    return processutils.execute("},{"line_number":73,"context_line":"        \u0027lvs\u0027, \u0027--noheadings\u0027, \u0027-o\u0027, \u0027lv_name\u0027, vg, attempts\u003d3,"},{"line_number":74,"context_line":"        delay_on_retry\u003dTrue)"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"93063a0a_88d326bf","line":73,"range":{"start_line":73,"start_character":52,"end_line":73,"end_character":63},"in_reply_to":"6a112960_4481bfb1","updated":"2021-06-15 13:41:43.000000000","message":"Done","commit_id":"7bff51f7ac2f998a6e361310e63ed5f14ef4ff45"}]}
