)]}'
{"whitebox_tempest_plugin/api/compute/test_virtio_scsi_attach.py":[{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"8cd6b30ac33b5c191c3b41904987d0f45345d4a2","unresolved":true,"context_lines":[{"line_number":86,"context_line":"        :returns: A list of volume id\u0027s that are attached to the instance"},{"line_number":87,"context_line":"        \"\"\""},{"line_number":88,"context_line":"        attachments \u003d self.servers_client.list_volume_attachments(server_id)"},{"line_number":89,"context_line":"        return [a.get(\u0027volumeId\u0027) for a in attachments[\u0027volumeAttachments\u0027]]"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def get_attached_serial_ids(self, disks):"},{"line_number":92,"context_line":"        \"\"\"Create a list of serial ids from a list of disks"}],"source_content_type":"text/x-python","patch_set":2,"id":"b593cc5d_89c0ce7a","line":89,"range":{"start_line":89,"start_character":16,"end_line":89,"end_character":33},"updated":"2021-03-18 15:33:13.000000000","message":"Can we add a default here so that it doesn\u0027t raise an exception, probably None? a.get(\u0027volumeId\u0027, None)","commit_id":"c6252d4e2ad4b91ff510287354c144ffb599a99e"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"2f4fe16c4b55de36908bd2f2fa17c45ac916aeb0","unresolved":false,"context_lines":[{"line_number":86,"context_line":"        :returns: A list of volume id\u0027s that are attached to the instance"},{"line_number":87,"context_line":"        \"\"\""},{"line_number":88,"context_line":"        attachments \u003d self.servers_client.list_volume_attachments(server_id)"},{"line_number":89,"context_line":"        return [a.get(\u0027volumeId\u0027) for a in attachments[\u0027volumeAttachments\u0027]]"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def get_attached_serial_ids(self, disks):"},{"line_number":92,"context_line":"        \"\"\"Create a list of serial ids from a list of disks"}],"source_content_type":"text/x-python","patch_set":2,"id":"31111def_7d9f8ecb","line":89,"range":{"start_line":89,"start_character":16,"end_line":89,"end_character":33},"in_reply_to":"27b07783_c44d222e","updated":"2021-03-18 16:58:38.000000000","message":"Correct default is None unless you otherwise specify something else.  If you try to access a key via brackets though e.g. a[\u0027key that does not exist\u0027] that would throw a KeyError exception.","commit_id":"c6252d4e2ad4b91ff510287354c144ffb599a99e"},{"author":{"_account_id":27478,"name":"Archit Modi","email":"amodi@redhat.com","username":"amodi"},"change_message_id":"026abcc40edf93bde7e5b5f392d61e4ac636589e","unresolved":true,"context_lines":[{"line_number":86,"context_line":"        :returns: A list of volume id\u0027s that are attached to the instance"},{"line_number":87,"context_line":"        \"\"\""},{"line_number":88,"context_line":"        attachments \u003d self.servers_client.list_volume_attachments(server_id)"},{"line_number":89,"context_line":"        return [a.get(\u0027volumeId\u0027) for a in attachments[\u0027volumeAttachments\u0027]]"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def get_attached_serial_ids(self, disks):"},{"line_number":92,"context_line":"        \"\"\"Create a list of serial ids from a list of disks"}],"source_content_type":"text/x-python","patch_set":2,"id":"27b07783_c44d222e","line":89,"range":{"start_line":89,"start_character":16,"end_line":89,"end_character":33},"in_reply_to":"b593cc5d_89c0ce7a","updated":"2021-03-18 15:36:10.000000000","message":"Oh my bad, i guess the default is None, so no need to explicitly define it here","commit_id":"c6252d4e2ad4b91ff510287354c144ffb599a99e"}]}