)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"57b860da49aec7df9ce20f11ee16528fb3d9ae2c","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add get_router high availaibility test policy"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add missing RBAC test case for the neutron router in policy file [1]"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"[1]https://github.com/openstack/neutron/blob/master/etc/policy.json"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I8f2623a4f1eecce359ffb6f66e9e9ef9365690e2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff82abbf_7cb74088","line":11,"range":{"start_line":10,"start_character":0,"end_line":11,"end_character":67},"updated":"2017-11-21 03:25:24.000000000","message":"exact link to policy and code where policy is being enforced is helpful to do fast review.\n\nalways include these 2 links in commit msg\n\npolicy doc - policy sample link or https://github.com/openstack/neutron/blob/bf84f308d71ad770b1173487391d23f2afd29efa/etc/policy.json#L100\n\nPolicy enforcement  - link to code file where this policy is actually enforced.","commit_id":"1d21e17a3042d28a161996136b1b47c15abf2b65"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"feea8fd3549007d557e2f5d2b374b6d14930ca27","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Add missing RBAC test case for the neutron router in policy file [1]"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"policy doc : [1] https://github.com/openstack/neutron/blob/bf84f308d71ad770b1173487391d23f2afd29efa/etc/policy.json#L100"},{"line_number":12,"context_line":"policy enforcement :https://github.com/openstack/neutron/blob/75c1cf0abb50906a2ba34d00052a43ed5364569d/neutron/tests/functional/services/l3_router/test_l3_dvr_ha_router_plugin.py#L36"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I8f2623a4f1eecce359ffb6f66e9e9ef9365690e2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ff82abbf_d42e3cf9","line":12,"range":{"start_line":12,"start_character":20,"end_line":12,"end_character":181},"updated":"2017-11-22 07:03:27.000000000","message":"This link isn\u0027t helpful.","commit_id":"7abfd62ca607ab4ac06fba0ca6016e6bf03fd280"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f02d615d1aa4e1c36c00a61b150b8ca2d45d0ea1","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Add missing RBAC test case for the neutron router in policy file [1]"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"policy doc : [1] https://github.com/openstack/neutron/blob/bf84f308d71ad770b1173487391d23f2afd29efa/etc/policy.json#L100"},{"line_number":12,"context_line":"policy enforcement :https://github.com/openstack/neutron/blob/75c1cf0abb50906a2ba34d00052a43ed5364569d/neutron/tests/functional/services/l3_router/test_l3_dvr_ha_router_plugin.py#L36"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I8f2623a4f1eecce359ffb6f66e9e9ef9365690e2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ff82abbf_1a78ff8d","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":182},"updated":"2017-11-25 16:09:53.000000000","message":"this is wrong link, if no actual code link then we should remove it.","commit_id":"3c57264e4f3b1a519fd9b0afd249f7c24fd62748"}],"patrole_tempest_plugin/tests/api/network/test_routers_rbac.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"002e350376fe4d89dcf95c24310ee6b8cd919a1e","unresolved":false,"context_lines":[{"line_number":183,"context_line":"    def test_show_high_availability_router(self):"},{"line_number":184,"context_line":"        \"\"\"GET high-availability router"},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"        RBAC test for the neutron create_router:ha policy"},{"line_number":187,"context_line":"        \"\"\""},{"line_number":188,"context_line":"        router \u003d self.routers_client.create_router(ha\u003dTrue)[\u0027router\u0027]"},{"line_number":189,"context_line":"        self.addCleanup(self.routers_client.delete_router, router[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"ff82abbf_4a513cd8","line":186,"range":{"start_line":186,"start_character":34,"end_line":186,"end_character":40},"updated":"2017-11-21 14:49:50.000000000","message":"get","commit_id":"8e3a00a3b5c7371fbc5e8835033ada27272f6923"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"1b9118e3ebedfb156f159d88e3608bdaa07c0b5d","unresolved":false,"context_lines":[{"line_number":177,"context_line":"                attribute\u003d\u0027distributed\u0027)"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"    @decorators.idempotent_id(\u0027defc502c-4159-4824-b4d9-3cdcc39015b2\u0027)"},{"line_number":180,"context_line":"    @utils.requires_ext(extension\u003d\u0027dvr\u0027, service\u003d\u0027network\u0027)"},{"line_number":181,"context_line":"    @rbac_rule_validation.action(service\u003d\"neutron\","},{"line_number":182,"context_line":"                                 rule\u003d\"get_router:ha\")"},{"line_number":183,"context_line":"    def test_show_high_availability_router(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff82abbf_2379572e","line":180,"range":{"start_line":180,"start_character":35,"end_line":180,"end_character":38},"updated":"2017-11-22 20:02:20.000000000","message":"This should be \u0027l3-ha\u0027 [0]\n\n[0] See \"ha\" field under https://developer.openstack.org/api-ref/network/v2/index.html#show-router-details","commit_id":"7abfd62ca607ab4ac06fba0ca6016e6bf03fd280"}]}
