)]}'
{"tempest/api/image/v2/test_images_metadefs_namespaces.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"fd7fbc4461c48a078c45489d0962690d16f7c1ee","unresolved":false,"context_lines":[{"line_number":36,"context_line":"                                             description\u003d\u0027Tempest\u0027,"},{"line_number":37,"context_line":"                                             display_name\u003dnamespace_name,"},{"line_number":38,"context_line":"                                             resource_type_associations\u003dname,"},{"line_number":39,"context_line":"                                             protected\u003dTrue)"},{"line_number":40,"context_line":"        # get namespaces details"},{"line_number":41,"context_line":"        body \u003d self.client.get_namespaces_detail(namespace_name)"},{"line_number":42,"context_line":"        self.assertEqual(namespace_name, body[\u0027namespace\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"da20952f_1158a055","line":39,"updated":"2015-09-10 09:42:28.000000000","message":"needs to add cleanup here, How about below cleanup and add it to addcleanUp after L39\n\n_cleanup_namespace()-\n try \n    get_namespaces_detail\n    update namespace with protected\u003dFalse\n    delete namespace\n expect NotFound\n    pass","commit_id":"94aa4c55d80c2b3fc0d3b72cef9635fb3c2a975e"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"c03db918dd7939391264501032e41880e0d204e0","unresolved":false,"context_lines":[{"line_number":24,"context_line":"    Here we will test the Metadata definition Namespaces basic functionality."},{"line_number":25,"context_line":"    \"\"\""},{"line_number":26,"context_line":"    @test.idempotent_id(\u0027319b765e-7f3d-4b3d-8b37-3ca3876ee768\u0027)"},{"line_number":27,"context_line":"    def test_basic_metadata_defination_namespaces(self):"},{"line_number":28,"context_line":"        # get the available resource types and use one resource_type"},{"line_number":29,"context_line":"        body \u003d self.client.list_resource_types()"},{"line_number":30,"context_line":"        resource_name \u003d body[\u0027resource_types\u0027][0][\u0027name\u0027]"}],"source_content_type":"text/x-python","patch_set":6,"id":"ba15a1d1_d4a50947","line":27,"updated":"2015-09-16 15:36:26.000000000","message":"typo here.\n\nSorry to be picky but I want to avoid a future \"fix typo in test name\" patch.","commit_id":"8e31fef5165f2b1491d2890c34bbe3498a621df6"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"c03db918dd7939391264501032e41880e0d204e0","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        # now able to delete the non-protected namespace"},{"line_number":57,"context_line":"        self.client.delete_namespaces(namespace_name)"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    def _Cleanup_namespace(self, namespace_name):"},{"line_number":60,"context_line":"        # this is used to cleanup the resources"},{"line_number":61,"context_line":"        try:"},{"line_number":62,"context_line":"            body \u003d self.client.show_namespaces(namespace_name)"}],"source_content_type":"text/x-python","patch_set":6,"id":"ba15a1d1_5447b976","line":59,"updated":"2015-09-16 15:36:26.000000000","message":"it should be: def _cleanup_namespace","commit_id":"8e31fef5165f2b1491d2890c34bbe3498a621df6"}],"tempest/services/image/v2/json/image_client.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"fd7fbc4461c48a078c45489d0962690d16f7c1ee","unresolved":false,"context_lines":[{"line_number":212,"context_line":"        body \u003d json.loads(body)"},{"line_number":213,"context_line":"        return service_client.ResponseBody(resp, body)"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"    def get_resource_types(self):"},{"line_number":216,"context_line":"        url \u003d \u0027/v2/metadefs/resource_types\u0027"},{"line_number":217,"context_line":"        resp, body \u003d self.get(url)"},{"line_number":218,"context_line":"        self.expected_success(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":4,"id":"da20952f_06906494","line":215,"updated":"2015-09-10 09:42:28.000000000","message":"As these going to be in lib soon, for consistent interface please rename this method to list_resource_types","commit_id":"94aa4c55d80c2b3fc0d3b72cef9635fb3c2a975e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"fd7fbc4461c48a078c45489d0962690d16f7c1ee","unresolved":false,"context_lines":[{"line_number":219,"context_line":"        body \u003d json.loads(body)"},{"line_number":220,"context_line":"        return service_client.ResponseBody(resp, body)"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    def create_namespaces(self, namespace, visibility, **kwargs):"},{"line_number":223,"context_line":"        params \u003d {"},{"line_number":224,"context_line":"            \"namespace\": namespace,"},{"line_number":225,"context_line":"            \"visibility\": visibility,"}],"source_content_type":"text/x-python","patch_set":4,"id":"da20952f_119f4060","line":222,"updated":"2015-09-10 09:42:28.000000000","message":"visibility is optional as you can take in **kwargs","commit_id":"94aa4c55d80c2b3fc0d3b72cef9635fb3c2a975e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"fd7fbc4461c48a078c45489d0962690d16f7c1ee","unresolved":false,"context_lines":[{"line_number":240,"context_line":"        body \u003d json.loads(body)"},{"line_number":241,"context_line":"        return service_client.ResponseBody(resp, body)"},{"line_number":242,"context_line":""},{"line_number":243,"context_line":"    def get_namespaces_detail(self, namespace):"},{"line_number":244,"context_line":"        url \u003d \u0027/v2/metadefs/namespaces/%s\u0027 % namespace"},{"line_number":245,"context_line":"        resp, body \u003d self.get(url)"},{"line_number":246,"context_line":"        self.expected_success(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":4,"id":"da20952f_268ec8d0","line":243,"updated":"2015-09-10 09:42:28.000000000","message":"ditto. get_namespaces_detail -\u003e show_namespaces","commit_id":"94aa4c55d80c2b3fc0d3b72cef9635fb3c2a975e"}]}