)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"88435de6dedd779d179126c19d921f1768c85ae0","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Made changes to use default policies in our unit and functional tests"},{"line_number":10,"context_line":"rather than referring it from glance/tests/etc/policy.yaml file. Existing"},{"line_number":11,"context_line":"function \u0027set_policy_rules\u0027 can be used to test custom policy rules in"},{"line_number":12,"context_line":"functional testing."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Improvements needed in followup patch:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"277bf89e_49228173","line":11,"updated":"2021-06-29 11:33:02.000000000","message":"nit: doesn\u0027t look indented","commit_id":"03f62d086d9d1612295a39e4d2a2606359c69208"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"a9106383ce80554b0a507217cf5ccc10b494c8c1","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Made changes to use default policies in our unit and functional tests"},{"line_number":10,"context_line":"rather than referring it from glance/tests/etc/policy.yaml file. Existing"},{"line_number":11,"context_line":"function \u0027set_policy_rules\u0027 can be used to test custom policy rules in"},{"line_number":12,"context_line":"functional testing."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Improvements needed in followup patch:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"89f41e40_747a3d32","line":11,"in_reply_to":"277bf89e_49228173","updated":"2021-06-30 18:28:09.000000000","message":"Done","commit_id":"03f62d086d9d1612295a39e4d2a2606359c69208"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"88435de6dedd779d179126c19d921f1768c85ae0","unresolved":true,"context_lines":[{"line_number":12,"context_line":"functional testing."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Improvements needed in followup patch:"},{"line_number":15,"context_line":"1. Property protection related unit tests still reads the specail policy"},{"line_number":16,"context_line":"from the policy.yaml file, need to make provision to override it instead."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"2. Need to remove \u0027set_policy\u0027 function which actually responsible for"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"e80fb6cf_91030632","line":15,"updated":"2021-06-29 11:33:02.000000000","message":"s/specail/special","commit_id":"03f62d086d9d1612295a39e4d2a2606359c69208"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"a9106383ce80554b0a507217cf5ccc10b494c8c1","unresolved":false,"context_lines":[{"line_number":12,"context_line":"functional testing."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Improvements needed in followup patch:"},{"line_number":15,"context_line":"1. Property protection related unit tests still reads the specail policy"},{"line_number":16,"context_line":"from the policy.yaml file, need to make provision to override it instead."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"2. Need to remove \u0027set_policy\u0027 function which actually responsible for"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"eaa21345_cbce3fc2","line":15,"in_reply_to":"e80fb6cf_91030632","updated":"2021-06-30 18:28:09.000000000","message":"Done","commit_id":"03f62d086d9d1612295a39e4d2a2606359c69208"}],"glance/tests/etc/policy.yaml":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"82809157e1283aa6734a04da97089f2f4f64c024","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# FIXME (abhishekk): This specail rule is required in unit tests"},{"line_number":2,"context_line":"# to test property protection using policies. Need to make provision"},{"line_number":3,"context_line":"# to set such rules on the fly."},{"line_number":4,"context_line":"\"glance_creator\": \"role:admin or role:spl_role\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d43f0f9e_382d6c26","line":1,"range":{"start_line":1,"start_character":26,"end_line":1,"end_character":33},"updated":"2021-06-30 14:38:18.000000000","message":"special","commit_id":"03f62d086d9d1612295a39e4d2a2606359c69208"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"a9106383ce80554b0a507217cf5ccc10b494c8c1","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# FIXME (abhishekk): This specail rule is required in unit tests"},{"line_number":2,"context_line":"# to test property protection using policies. Need to make provision"},{"line_number":3,"context_line":"# to set such rules on the fly."},{"line_number":4,"context_line":"\"glance_creator\": \"role:admin or role:spl_role\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"70580d5d_e623c015","line":1,"range":{"start_line":1,"start_character":26,"end_line":1,"end_character":33},"in_reply_to":"d43f0f9e_382d6c26","updated":"2021-06-30 18:28:09.000000000","message":"Done","commit_id":"03f62d086d9d1612295a39e4d2a2606359c69208"}]}