)]}'
{"aodh/api/policies.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"5ec7725f9e89073c80e66bc739b29ea99c15a782","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        operations\u003d["},{"line_number":43,"context_line":"            {"},{"line_number":44,"context_line":"                \u0027path\u0027: \u0027/v2/alarms/(alarm_id)\u0027,"},{"line_number":45,"context_line":"                \u0027method\u0027: \u0027GET\u0027"},{"line_number":46,"context_line":"            }"},{"line_number":47,"context_line":"        ]"},{"line_number":48,"context_line":"    ),"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f4b6375_3475bcfa","line":45,"updated":"2017-10-26 17:03:40.000000000","message":"nit: The convention used by most other projects is {\n\n  \u0027path\u0027: \u0027/v2/alarms/{alarm_id}\u0027,","commit_id":"8f87ee0cb02c0898e9ffcc88f8e58ce6c30db057"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"4100c0d565c98efe6248e3efc0c704dcc7caff3c","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        check_str\u003dRULE_ADMIN_OR_OWNER"},{"line_number":37,"context_line":"    ),"},{"line_number":38,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":39,"context_line":"        name\u003d\"telemetry:get_alarm\","},{"line_number":40,"context_line":"        check_str\u003dRULE_ADMIN_OR_OWNER,"},{"line_number":41,"context_line":"        description\u003d\u0027Get an alarm.\u0027,"},{"line_number":42,"context_line":"        operations\u003d["}],"source_content_type":"text/x-python","patch_set":8,"id":"3f4b6375_eabe4aac","line":39,"range":{"start_line":39,"start_character":14,"end_line":39,"end_character":23},"updated":"2017-11-03 20:45:12.000000000","message":"This could be pulled out into it\u0027s own variable and injected in later. That should keep the service bit of the policy name consistent.","commit_id":"36b21c28d0d69f8a05e53b49e32293d477b06df1"},{"author":{"_account_id":19930,"name":"Dai Dang Van","email":"daidv@omzcloud.vn","username":"daidv"},"change_message_id":"cc32a0379cfc37920ec958e9b6ee9db2b6af3c81","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        check_str\u003dRULE_ADMIN_OR_OWNER"},{"line_number":37,"context_line":"    ),"},{"line_number":38,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":39,"context_line":"        name\u003d\"telemetry:get_alarm\","},{"line_number":40,"context_line":"        check_str\u003dRULE_ADMIN_OR_OWNER,"},{"line_number":41,"context_line":"        description\u003d\u0027Get an alarm.\u0027,"},{"line_number":42,"context_line":"        operations\u003d["}],"source_content_type":"text/x-python","patch_set":8,"id":"3f4b6375_86dfc364","line":39,"range":{"start_line":39,"start_character":14,"end_line":39,"end_character":23},"in_reply_to":"3f4b6375_eabe4aac","updated":"2017-11-06 01:36:34.000000000","message":"Sure! I will do it in a follow up patchset.","commit_id":"36b21c28d0d69f8a05e53b49e32293d477b06df1"}],"aodh/tests/open-policy.json":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"5ec7725f9e89073c80e66bc739b29ea99c15a782","unresolved":false,"context_lines":[{"line_number":3,"context_line":"    \"segregation\": \"rule:context_is_admin\","},{"line_number":4,"context_line":"    \"default\": \"\","},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"    \"telemetry:get_alarm\": \"\","},{"line_number":7,"context_line":"    \"telemetry:get_alarms\": \"\","},{"line_number":8,"context_line":"    \"telemetry:query_alarm\": \"\","},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"    \"telemetry:create_alarm\": \"\","},{"line_number":11,"context_line":"    \"telemetry:change_alarm\": \"\","},{"line_number":12,"context_line":"    \"telemetry:delete_alarm\": \"\","},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"    \"telemetry:get_alarm_state\": \"\","},{"line_number":15,"context_line":"    \"telemetry:change_alarm_state\": \"\","}],"source_content_type":"application/json","patch_set":3,"id":"3f4b6375_945d907a","line":12,"range":{"start_line":6,"start_character":4,"end_line":12,"end_character":33},"updated":"2017-10-26 17:03:40.000000000","message":"The policy enforce() method could be stubbed to return the expected outcome instead of maintaining a separate list of fake test policies.","commit_id":"8f87ee0cb02c0898e9ffcc88f8e58ce6c30db057"},{"author":{"_account_id":19930,"name":"Dai Dang Van","email":"daidv@omzcloud.vn","username":"daidv"},"change_message_id":"30d4f6a9680afc0d681b00566a4284f91fec7fcc","unresolved":false,"context_lines":[{"line_number":3,"context_line":"    \"segregation\": \"rule:context_is_admin\","},{"line_number":4,"context_line":"    \"default\": \"\","},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"    \"telemetry:get_alarm\": \"\","},{"line_number":7,"context_line":"    \"telemetry:get_alarms\": \"\","},{"line_number":8,"context_line":"    \"telemetry:query_alarm\": \"\","},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"    \"telemetry:create_alarm\": \"\","},{"line_number":11,"context_line":"    \"telemetry:change_alarm\": \"\","},{"line_number":12,"context_line":"    \"telemetry:delete_alarm\": \"\","},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"    \"telemetry:get_alarm_state\": \"\","},{"line_number":15,"context_line":"    \"telemetry:change_alarm_state\": \"\","}],"source_content_type":"application/json","patch_set":3,"id":"3f4b6375_f575d432","line":12,"range":{"start_line":6,"start_character":4,"end_line":12,"end_character":33},"in_reply_to":"3f4b6375_945d907a","updated":"2017-10-27 04:13:22.000000000","message":"Ok.","commit_id":"8f87ee0cb02c0898e9ffcc88f8e58ce6c30db057"}]}