)]}'
{"patrole_tempest_plugin/tests/api/network/test_agents_rbac.py":[{"author":{"_account_id":28719,"name":"Phil Sphicas","email":"phil.sphicas@att.com","username":"ps3910"},"change_message_id":"6c759477e6154be75c7315b12fbfb4a546efc5b5","unresolved":false,"context_lines":[{"line_number":268,"context_line":"        cls.router \u003d cls.ntp_client.create_router(name)[\u0027router\u0027]"},{"line_number":269,"context_line":"        cls.addClassResourceCleanup("},{"line_number":270,"context_line":"            test_utils.call_and_ignore_notfound_exc,"},{"line_number":271,"context_line":"            cls.routers_client.delete_router,"},{"line_number":272,"context_line":"            cls.router[\u0027id\u0027])"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"    @decorators.idempotent_id(\u00275d2bbdbc-40a5-43d2-828a-84dc93bcd321\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_76b655fd","line":271,"range":{"start_line":271,"start_character":16,"end_line":271,"end_character":30},"updated":"2019-09-19 19:36:48.000000000","message":"Why is the router created with `cls.ntp_client` but deleted with `cls.routers_client`? Should one of them be changed?","commit_id":"43944b0285aeead134b317ce8f7f105b2b2a8064"},{"author":{"_account_id":17896,"name":"Rick Bartra","email":"rickbartra@microsoft.com","username":"rb560u"},"change_message_id":"844455e5bda5d428855c140eebbdeec2c9bdcef5","unresolved":false,"context_lines":[{"line_number":268,"context_line":"        cls.router \u003d cls.ntp_client.create_router(name)[\u0027router\u0027]"},{"line_number":269,"context_line":"        cls.addClassResourceCleanup("},{"line_number":270,"context_line":"            test_utils.call_and_ignore_notfound_exc,"},{"line_number":271,"context_line":"            cls.routers_client.delete_router,"},{"line_number":272,"context_line":"            cls.router[\u0027id\u0027])"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"    @decorators.idempotent_id(\u00275d2bbdbc-40a5-43d2-828a-84dc93bcd321\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_36a3fda0","line":271,"range":{"start_line":271,"start_character":16,"end_line":271,"end_character":30},"in_reply_to":"3fa7e38b_76b655fd","updated":"2019-09-19 19:44:39.000000000","message":"Thanks for pointing that out, I didn\u0027t realize that the router was created with the ntp_client. For consistency sake, I will do the cleanup with the ntp_client as well. Thank you.","commit_id":"43944b0285aeead134b317ce8f7f105b2b2a8064"}]}