)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"6cae27b2b7138db72f4a87b4524d9092c6cd3eaf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dede46f1_58f3b5ba","updated":"2024-02-06 20:29:31.000000000","message":"some comments in-line","commit_id":"ef16887f753133a5c2633255b857098124d09ab5"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"b54649b8bf051e71a383f5687e119cc70bbc7311","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"00554596_fd44a386","updated":"2024-02-13 18:56:01.000000000","message":"Like we discussed earlier, add the (soon-to-be) deprecated drivers to this change. It will adjust the CI results accordingly.\n\nWe saw that much of the low severity issues were also XML related, up to you if you want to include them in the report, and address them in your separate driver patches.","commit_id":"a3b64750bd6efb600805f6a68152fc3a7108a2c2"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"59557e89565266940284bccf054ba2aae57d5d85","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c9e64325_b2a3aad7","updated":"2024-03-26 18:03:03.000000000","message":"recheck","commit_id":"7b4ea08b3e5cc80803c30d964ffa649375b4957d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"1bedf66f33483c2fcda345dcd3dee4622a75efca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"5707e013_6b6f74c4","updated":"2024-04-02 17:26:14.000000000","message":"recheck for fresh logs","commit_id":"d51fcb5b5a0aed1722bafb7fe77ed6aa15b8cd54"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"84b1ea042c27dd632de21fa3a694c3a6e456226f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"7fb7f9c1_e8d110ed","updated":"2024-04-09 17:08:07.000000000","message":"recheck","commit_id":"39e4141f0a794bbd9756f6797929d143cb880e64"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"d353954be03d0a3d8aa01c89a58966346ffc848f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"39de0538_e5af2ed6","updated":"2024-04-24 18:55:51.000000000","message":"recheck\n\nZuul hasn\u0027t tested the latest patch","commit_id":"eb08b5e6a1d109f6581a31dd7ced0557a47cb72a"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"6945a0fcd690279bf00914cdd9ba5565151e0d76","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"27ff4df4_8e50d839","updated":"2024-05-22 18:53:16.000000000","message":"I like the progress here so far; lets merge this and create a new bug / several bugs for the remaining medium/high importance bugs. Thanks Skylar/Ashley/Elvis/Esha!","commit_id":"333e6aee901d661ac28eea4f5661613871382cef"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"6ad1525c2e27032eb41c45de6bcee2a400c43a29","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"13dfde45_062fc820","updated":"2024-05-23 20:24:08.000000000","message":"LGTM, thank you very much for your hard work on enabling the bandit tests and figuring out all of the issues.","commit_id":"333e6aee901d661ac28eea4f5661613871382cef"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"1de5143a7372abe7cb1949442544a713bf64ecc0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"522dbabb_bba46382","updated":"2024-05-15 21:06:40.000000000","message":"bandit returns this warning: \nWARNING nosec encountered (B###), but no failed test on line ###\n\nthis is due to a bug in bandit itself, not an issue with our nosec approach. ignore the warnings for now.","commit_id":"333e6aee901d661ac28eea4f5661613871382cef"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"1a459be0c5a1c3ae7523c045c888bc7a72f500f7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"68991425_44032474","updated":"2024-05-22 18:18:57.000000000","message":"recheck grenade job fix merged","commit_id":"333e6aee901d661ac28eea4f5661613871382cef"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"5011b0e9ba0e44e3fddf26f7e7aed85a0be5e0a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"f700d7a3_35cb1215","updated":"2024-05-15 21:06:57.000000000","message":"recheck new dependencies","commit_id":"333e6aee901d661ac28eea4f5661613871382cef"}],"manila/service.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"5b8767530c5c0c4981d6915c09dcfb45322a5139","unresolved":true,"context_lines":[{"line_number":352,"context_line":"        if not rpc.initialized():"},{"line_number":353,"context_line":"            rpc.init(CONF)"},{"line_number":354,"context_line":"        self.app \u003d self.loader.load_app(name)"},{"line_number":355,"context_line":"        self.host \u003d getattr(CONF, \u0027%s_listen\u0027 % name, \"0.0.0.0\")  # nosec"},{"line_number":356,"context_line":"        self.port \u003d getattr(CONF, \u0027%s_listen_port\u0027 % name, 0)"},{"line_number":357,"context_line":"        self.workers \u003d getattr(CONF, \u0027%s_workers\u0027 % name, None)"},{"line_number":358,"context_line":"        self.use_ssl \u003d getattr(CONF, \u0027%s_use_ssl\u0027 % name, False)"}],"source_content_type":"text/x-python","patch_set":19,"id":"e8898065_f4bc2445","line":355,"range":{"start_line":355,"start_character":66,"end_line":355,"end_character":73},"updated":"2024-04-30 17:40:52.000000000","message":"Can you use a specific bandit rule here?","commit_id":"eb08b5e6a1d109f6581a31dd7ced0557a47cb72a"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"1de5143a7372abe7cb1949442544a713bf64ecc0","unresolved":false,"context_lines":[{"line_number":352,"context_line":"        if not rpc.initialized():"},{"line_number":353,"context_line":"            rpc.init(CONF)"},{"line_number":354,"context_line":"        self.app \u003d self.loader.load_app(name)"},{"line_number":355,"context_line":"        self.host \u003d getattr(CONF, \u0027%s_listen\u0027 % name, \"0.0.0.0\")  # nosec"},{"line_number":356,"context_line":"        self.port \u003d getattr(CONF, \u0027%s_listen_port\u0027 % name, 0)"},{"line_number":357,"context_line":"        self.workers \u003d getattr(CONF, \u0027%s_workers\u0027 % name, None)"},{"line_number":358,"context_line":"        self.use_ssl \u003d getattr(CONF, \u0027%s_use_ssl\u0027 % name, False)"}],"source_content_type":"text/x-python","patch_set":19,"id":"39ccd9da_5a724ca9","line":355,"range":{"start_line":355,"start_character":66,"end_line":355,"end_character":73},"in_reply_to":"e8898065_f4bc2445","updated":"2024-05-15 21:06:40.000000000","message":"Done","commit_id":"eb08b5e6a1d109f6581a31dd7ced0557a47cb72a"}],"tox.ini":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2e7c1a04e7147fcc0cdaf7b6d1986655d404a170","unresolved":true,"context_lines":[{"line_number":65,"context_line":"commands \u003d {posargs}"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"[testenv:bandit]"},{"line_number":68,"context_line":"deps \u003d bandit"},{"line_number":69,"context_line":"commands \u003d bandit -r manila --ini tox.ini -n5 -ll"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"[bandit]"}],"source_content_type":"text/x-properties","patch_set":19,"id":"fdb55159_16d8bdf6","line":68,"range":{"start_line":68,"start_character":0,"end_line":68,"end_character":13},"updated":"2024-04-22 21:26:05.000000000","message":"I really like the pre-commit approach as opposed to this: https://review.opendev.org/c/openstack/oslo.middleware/+/904216","commit_id":"eb08b5e6a1d109f6581a31dd7ced0557a47cb72a"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"6ad1525c2e27032eb41c45de6bcee2a400c43a29","unresolved":true,"context_lines":[{"line_number":65,"context_line":"commands \u003d {posargs}"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"[testenv:bandit]"},{"line_number":68,"context_line":"deps \u003d bandit"},{"line_number":69,"context_line":"commands \u003d bandit -r manila --ini tox.ini -n5 -ll"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"[bandit]"}],"source_content_type":"text/x-properties","patch_set":19,"id":"6c46c92c_f374b191","line":68,"range":{"start_line":68,"start_character":0,"end_line":68,"end_character":13},"in_reply_to":"e6bc6d76_437cf10c","updated":"2024-05-23 20:24:08.000000000","message":"++ on the pre-commit approach","commit_id":"eb08b5e6a1d109f6581a31dd7ced0557a47cb72a"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"1de5143a7372abe7cb1949442544a713bf64ecc0","unresolved":true,"context_lines":[{"line_number":65,"context_line":"commands \u003d {posargs}"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"[testenv:bandit]"},{"line_number":68,"context_line":"deps \u003d bandit"},{"line_number":69,"context_line":"commands \u003d bandit -r manila --ini tox.ini -n5 -ll"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"[bandit]"}],"source_content_type":"text/x-properties","patch_set":19,"id":"e6bc6d76_437cf10c","line":68,"range":{"start_line":68,"start_character":0,"end_line":68,"end_character":13},"in_reply_to":"fdb55159_16d8bdf6","updated":"2024-05-15 21:06:40.000000000","message":"Let\u0027s resolve the bandit issues first and get a clean run, before including it in pep8. That\u0027s a voting job, it would continously fail as it stands atm.","commit_id":"eb08b5e6a1d109f6581a31dd7ced0557a47cb72a"}],"zuul.d/project.yaml":[{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"6cae27b2b7138db72f4a87b4524d9092c6cd3eaf","unresolved":true,"context_lines":[{"line_number":10,"context_line":"    queue: manila"},{"line_number":11,"context_line":"    check:"},{"line_number":12,"context_line":"      jobs:"},{"line_number":13,"context_line":"        - manila-tox-genconfig"},{"line_number":14,"context_line":"        - openstack-tox-pylint:"},{"line_number":15,"context_line":"            voting: false"},{"line_number":16,"context_line":"            timeout: 5400"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"8ef9d97d_69396b58","line":13,"updated":"2024-02-06 20:29:31.000000000","message":"you\u0027ll have to add manila-tox-bandit to the list of jobs to run here.","commit_id":"ef16887f753133a5c2633255b857098124d09ab5"},{"author":{"_account_id":36757,"name":"Skylar Markegard","display_name":"skylarmarkegard","email":"skylar.markegard@ndsu.edu","username":"skylarmarkegard"},"change_message_id":"1fabbc2e5d3d85184c9785dd00daad43d87598e4","unresolved":false,"context_lines":[{"line_number":10,"context_line":"    queue: manila"},{"line_number":11,"context_line":"    check:"},{"line_number":12,"context_line":"      jobs:"},{"line_number":13,"context_line":"        - manila-tox-genconfig"},{"line_number":14,"context_line":"        - openstack-tox-pylint:"},{"line_number":15,"context_line":"            voting: false"},{"line_number":16,"context_line":"            timeout: 5400"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"17fa0087_d86b6966","line":13,"in_reply_to":"8ef9d97d_69396b58","updated":"2024-04-16 16:51:37.000000000","message":"Acknowledged","commit_id":"ef16887f753133a5c2633255b857098124d09ab5"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"6cae27b2b7138db72f4a87b4524d9092c6cd3eaf","unresolved":true,"context_lines":[{"line_number":92,"context_line":"  # Security testing for known issues"},{"line_number":93,"context_line":"    name: manila-tox-bandit"},{"line_number":94,"context_line":"    parent: openstack-tox"},{"line_number":95,"context_line":"    timeout: 2400"},{"line_number":96,"context_line":"    vars:"},{"line_number":97,"context_line":"      tox_envlist: bandit"},{"line_number":98,"context_line":"    irrelevant-files: *irrelevant-files"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"ab3d1530_21c84c53","line":95,"updated":"2024-02-06 20:29:31.000000000","message":"add voting: false","commit_id":"ef16887f753133a5c2633255b857098124d09ab5"},{"author":{"_account_id":36757,"name":"Skylar Markegard","display_name":"skylarmarkegard","email":"skylar.markegard@ndsu.edu","username":"skylarmarkegard"},"change_message_id":"1fabbc2e5d3d85184c9785dd00daad43d87598e4","unresolved":false,"context_lines":[{"line_number":92,"context_line":"  # Security testing for known issues"},{"line_number":93,"context_line":"    name: manila-tox-bandit"},{"line_number":94,"context_line":"    parent: openstack-tox"},{"line_number":95,"context_line":"    timeout: 2400"},{"line_number":96,"context_line":"    vars:"},{"line_number":97,"context_line":"      tox_envlist: bandit"},{"line_number":98,"context_line":"    irrelevant-files: *irrelevant-files"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"e079f946_966f754b","line":95,"in_reply_to":"ab3d1530_21c84c53","updated":"2024-04-16 16:51:37.000000000","message":"Acknowledged","commit_id":"ef16887f753133a5c2633255b857098124d09ab5"}]}
