)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"e5cabb4018bc875be932a91d1402ba2990cfcaee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9f560f44_5a8c8838","line":21,"updated":"2020-10-05 23:19:02.000000000","message":"add\nCloses-Bug: #1898526","commit_id":"7f1185a96a5d2ef72b7b433a05a962e99a2b7058"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"c46e430f9c72a3d0d32c152cd5b69d9e47c792cf","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Vu Tuong \u003ctuongvx@dcn.ssu.ac.kr\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-10-06 01:51:14 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix python3 utf-8 decode issue while creating app_monitor"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"From the Python3 documentation for ast.literal_eval()"},{"line_number":10,"context_line":"Safely evaluate an expression node or a string containing"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9f560f44_fd48f68a","line":7,"updated":"2020-10-06 02:10:55.000000000","message":"Your updates includes two fixes, but not mentioned both in this title. So, could you revise?","commit_id":"14e72508e1790bfb3a988ffa46997de6578f3470"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"c46e430f9c72a3d0d32c152cd5b69d9e47c792cf","unresolved":false,"context_lines":[{"line_number":12,"context_line":"only consist of the following Python literal structures:"},{"line_number":13,"context_line":"strings, numbers, tuples, lists, dicts, booleans, and None."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This change corrects the input for ast.literal_eval()"},{"line_number":16,"context_line":"and also corrects the variable (zapi.URL) called in"},{"line_number":17,"context_line":"zabbix_api.py and zabbix.py"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9f560f44_3d438e6e","line":15,"updated":"2020-10-06 02:10:55.000000000","message":"I think it is better to move this 2nd sentence to the top because the 1st sentence says only one topic of two. You\u0027d better to explain whole of things of your update first.","commit_id":"14e72508e1790bfb3a988ffa46997de6578f3470"}],"tacker/vnfm/monitor.py":[{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"e5cabb4018bc875be932a91d1402ba2990cfcaee","unresolved":false,"context_lines":[{"line_number":233,"context_line":"    def _create_app_monitoring_dict(self, dev_attrs, mgmt_ip_address):"},{"line_number":234,"context_line":"        app_policy \u003d \u0027app_monitoring_policy\u0027"},{"line_number":235,"context_line":"        appmonitoring_dict \u003d\\"},{"line_number":236,"context_line":"            ast.literal_eval(dev_attrs[app_policy].decode(\u0027utf-8\u0027))"},{"line_number":237,"context_line":"        vdulist \u003d appmonitoring_dict[\u0027vdus\u0027].keys()"},{"line_number":238,"context_line":""},{"line_number":239,"context_line":"        for vduname in vdulist:"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_1aab3088","line":236,"updated":"2020-10-05 23:19:02.000000000","message":"It is preferred to wrap long lines in parentheses and not a backslash for line continuation. see https://docs.openstack.org/hacking/latest/user/hacking.html#general\n\nIt can be;\nappmonitoring_dict \u003d ast.literal_eval(\n    dev_attrs[app_policy].decode(\u0027utf-8\u0027))","commit_id":"7f1185a96a5d2ef72b7b433a05a962e99a2b7058"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"6fca1955da8cb4693106fda178bf19ee276b6acf","unresolved":false,"context_lines":[{"line_number":232,"context_line":""},{"line_number":233,"context_line":"    def _create_app_monitoring_dict(self, dev_attrs, mgmt_ip_address):"},{"line_number":234,"context_line":"        app_policy \u003d \u0027app_monitoring_policy\u0027"},{"line_number":235,"context_line":"        appmonitoring_dict \u003dast.literal_eval("},{"line_number":236,"context_line":"            dev_attrs[app_policy].decode(\u0027utf-8\u0027))"},{"line_number":237,"context_line":"        vdulist \u003d appmonitoring_dict[\u0027vdus\u0027].keys()"},{"line_number":238,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_5306331e","line":235,"updated":"2020-10-06 05:10:14.000000000","message":"pep8: E225 missing whitespace around operator","commit_id":"900ff65f6d27ae992d601692fffd19f55c6b68a2"}]}