)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4c180ffd7f15c9a5ef30f31e381af2f4185816cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e9ce8b2d_22bba421","updated":"2022-01-22 20:12:17.000000000","message":"few comments, Also please add\n- release note for the new client interface\n- add 3.27 microversion tests entry in this doc under volume section - https://github.com/openstack/tempest/blob/master/doc/source/microversion_testing.rst#microversion-tests-implemented-in-tempest","commit_id":"8dcd3ea0a6de993eca71d667beb1a41049c8c531"}],"tempest/api/volume/test_volume_attachment.py":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"e6af4a4411452249f418c7a9135dde87c9e4f6a2","unresolved":true,"context_lines":[{"line_number":36,"context_line":"        server \u003d self.create_server()"},{"line_number":37,"context_line":"        attachment \u003d self.attachments_client.create_attachment("},{"line_number":38,"context_line":"            server[\u0027id\u0027], volume[\u0027id\u0027])[\u0027attachment\u0027]"},{"line_number":39,"context_line":"        volume \u003d self.volumes_client.show_volume(volume[\u0027id\u0027])[\u0027volume\u0027]"},{"line_number":40,"context_line":"        self.assertEqual(\u0027reserved\u0027, volume[\u0027status\u0027])"},{"line_number":41,"context_line":"        self.assertEqual(server[\u0027id\u0027], attachment[\u0027instance\u0027])"},{"line_number":42,"context_line":"        self.assertEqual(volume[\u0027id\u0027], attachment[\u0027volume_id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"ee42f89c_66d62d59","line":39,"updated":"2021-05-20 12:55:54.000000000","message":"Hi Rajat, this is only a suggestion: maybe it would be cool to write some data and verify the data. Maybe something like this:\n\n```\n        # write content to volume on instance\n        timestamp \u003d self.create_timestamp(ip_server,\n                                          private_key\u003dkeypair1[\u0027private_key\u0027],\n                                          server\u003dserver)\n        # verify the data\n        timestamp2 \u003d self.get_timestamp(ip_server,\n                                        private_key\u003dprivate_key,\n                                        server\u003dserver)\n        self.assertEqual(timestamp, timestamp2) \n```","commit_id":"8dcd3ea0a6de993eca71d667beb1a41049c8c531"}],"tempest/lib/services/volume/v3/attachments_client.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4c180ffd7f15c9a5ef30f31e381af2f4185816cb","unresolved":true,"context_lines":[{"line_number":20,"context_line":"    \"\"\"Client class to send CRUD attachment V3 API requests\"\"\""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    def create_attachment(self, instance_id, volume_id, connector\u003dNone):"},{"line_number":23,"context_line":"        \"\"\"Create volume attachment.\"\"\""},{"line_number":24,"context_line":"        url \u003d \"attachments\""},{"line_number":25,"context_line":"        kwargs \u003d {"},{"line_number":26,"context_line":"            \"volume_uuid\": volume_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"ac8297d9_cce65574","line":23,"range":{"start_line":23,"start_character":35,"end_line":23,"end_character":36},"updated":"2022-01-22 20:12:17.000000000","message":"please add api-ref doc link here in doc string\n\nhttps://docs.openstack.org/api-ref//block-storage/v3/index.html#create-attachment","commit_id":"8dcd3ea0a6de993eca71d667beb1a41049c8c531"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4c180ffd7f15c9a5ef30f31e381af2f4185816cb","unresolved":true,"context_lines":[{"line_number":33,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def update_attachment(self, attachment_id, connector):"},{"line_number":36,"context_line":"        \"\"\"Create volume attachment.\"\"\""},{"line_number":37,"context_line":"        url \u003d \"attachments/%s\" % (attachment_id)"},{"line_number":38,"context_line":"        kwargs \u003d {\"connector\": connector}"},{"line_number":39,"context_line":"        post_body \u003d json.dumps({\u0027attachment\u0027: kwargs})"}],"source_content_type":"text/x-python","patch_set":2,"id":"bfb19fd1_fbb1a675","line":36,"range":{"start_line":36,"start_character":37,"end_line":36,"end_character":38},"updated":"2022-01-22 20:12:17.000000000","message":"ditto","commit_id":"8dcd3ea0a6de993eca71d667beb1a41049c8c531"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4c180ffd7f15c9a5ef30f31e381af2f4185816cb","unresolved":true,"context_lines":[{"line_number":33,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def update_attachment(self, attachment_id, connector):"},{"line_number":36,"context_line":"        \"\"\"Create volume attachment.\"\"\""},{"line_number":37,"context_line":"        url \u003d \"attachments/%s\" % (attachment_id)"},{"line_number":38,"context_line":"        kwargs \u003d {\"connector\": connector}"},{"line_number":39,"context_line":"        post_body \u003d json.dumps({\u0027attachment\u0027: kwargs})"}],"source_content_type":"text/x-python","patch_set":2,"id":"757feae0_5526814f","line":36,"range":{"start_line":36,"start_character":11,"end_line":36,"end_character":18},"updated":"2022-01-22 20:12:17.000000000","message":"update","commit_id":"8dcd3ea0a6de993eca71d667beb1a41049c8c531"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4c180ffd7f15c9a5ef30f31e381af2f4185816cb","unresolved":true,"context_lines":[{"line_number":43,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    def complete_attachment(self, attachment_id, connector):"},{"line_number":46,"context_line":"        \"\"\"Create volume attachment.\"\"\""},{"line_number":47,"context_line":"        url \u003d \"attachments/%s/action\" % (attachment_id)"},{"line_number":48,"context_line":"        resp, body \u003d self.post(url)"},{"line_number":49,"context_line":"        body \u003d json.loads(body)"}],"source_content_type":"text/x-python","patch_set":2,"id":"282cf3e2_8aaf8d41","line":46,"range":{"start_line":46,"start_character":11,"end_line":46,"end_character":18},"updated":"2022-01-22 20:12:17.000000000","message":"complete","commit_id":"8dcd3ea0a6de993eca71d667beb1a41049c8c531"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4c180ffd7f15c9a5ef30f31e381af2f4185816cb","unresolved":true,"context_lines":[{"line_number":43,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    def complete_attachment(self, attachment_id, connector):"},{"line_number":46,"context_line":"        \"\"\"Create volume attachment.\"\"\""},{"line_number":47,"context_line":"        url \u003d \"attachments/%s/action\" % (attachment_id)"},{"line_number":48,"context_line":"        resp, body \u003d self.post(url)"},{"line_number":49,"context_line":"        body \u003d json.loads(body)"}],"source_content_type":"text/x-python","patch_set":2,"id":"63484e1e_75d4038b","line":46,"range":{"start_line":46,"start_character":37,"end_line":46,"end_character":38},"updated":"2022-01-22 20:12:17.000000000","message":"ditto","commit_id":"8dcd3ea0a6de993eca71d667beb1a41049c8c531"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4c180ffd7f15c9a5ef30f31e381af2f4185816cb","unresolved":true,"context_lines":[{"line_number":50,"context_line":"        self.expected_success(200, resp.status)"},{"line_number":51,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    def get_attachment(self):"},{"line_number":54,"context_line":"        \"\"\"Show volume attachment.\"\"\""},{"line_number":55,"context_line":"        url \u003d \"attachments\""},{"line_number":56,"context_line":"        resp, body \u003d self.get(url)"}],"source_content_type":"text/x-python","patch_set":2,"id":"ae612639_b758e059","line":53,"range":{"start_line":53,"start_character":27,"end_line":53,"end_character":28},"updated":"2022-01-22 20:12:17.000000000","message":"please add **kwargs so that tests can pass the allowed request param/filters etc.","commit_id":"8dcd3ea0a6de993eca71d667beb1a41049c8c531"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4c180ffd7f15c9a5ef30f31e381af2f4185816cb","unresolved":true,"context_lines":[{"line_number":51,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    def get_attachment(self):"},{"line_number":54,"context_line":"        \"\"\"Show volume attachment.\"\"\""},{"line_number":55,"context_line":"        url \u003d \"attachments\""},{"line_number":56,"context_line":"        resp, body \u003d self.get(url)"},{"line_number":57,"context_line":"        body \u003d json.loads(body)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3b464ae4_fb6b11c9","line":54,"range":{"start_line":54,"start_character":11,"end_line":54,"end_character":16},"updated":"2022-01-22 20:12:17.000000000","message":"list","commit_id":"8dcd3ea0a6de993eca71d667beb1a41049c8c531"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4c180ffd7f15c9a5ef30f31e381af2f4185816cb","unresolved":true,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    def get_attachment(self):"},{"line_number":54,"context_line":"        \"\"\"Show volume attachment.\"\"\""},{"line_number":55,"context_line":"        url \u003d \"attachments\""},{"line_number":56,"context_line":"        resp, body \u003d self.get(url)"},{"line_number":57,"context_line":"        body \u003d json.loads(body)"},{"line_number":58,"context_line":"        self.expected_success(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":2,"id":"ddde1d81_2b55dec7","line":55,"range":{"start_line":55,"start_character":8,"end_line":55,"end_character":12},"updated":"2022-01-22 20:12:17.000000000","message":"modify the url for detail list if requested in kwargs.","commit_id":"8dcd3ea0a6de993eca71d667beb1a41049c8c531"}]}