)]}'
{"cinder/volume/drivers/dell/dell_storagecenter_api.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"0ed6e9da1e794604ed6c464d9d08271cf35c6f4c","unresolved":false,"context_lines":[{"line_number":657,"context_line":"        be seen by the server"},{"line_number":658,"context_line":"        \u0027\u0027\u0027"},{"line_number":659,"context_line":"        domains \u003d self._find_domains(cportid)"},{"line_number":660,"context_line":"        if domains is not None:"},{"line_number":661,"context_line":"            # Wiffle through the domains looking for our"},{"line_number":662,"context_line":"            # configured ip."},{"line_number":663,"context_line":"            for domain in domains:"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_9b8fedcc","line":660,"updated":"2015-02-18 05:56:41.000000000","message":"It\u0027s better to check \"if domains:\" so it covers both None and an empty list.","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"7e2f85039381c2369642b56febb8b243f246dd9e","unresolved":false,"context_lines":[{"line_number":657,"context_line":"        be seen by the server"},{"line_number":658,"context_line":"        \u0027\u0027\u0027"},{"line_number":659,"context_line":"        domains \u003d self._find_domains(cportid)"},{"line_number":660,"context_line":"        if domains is not None:"},{"line_number":661,"context_line":"            # Wiffle through the domains looking for our"},{"line_number":662,"context_line":"            # configured ip."},{"line_number":663,"context_line":"            for domain in domains:"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_c3916153","line":660,"in_reply_to":"da86d52c_9b8fedcc","updated":"2015-02-18 17:20:14.000000000","message":"Done","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"0ed6e9da1e794604ed6c464d9d08271cf35c6f4c","unresolved":false,"context_lines":[{"line_number":795,"context_line":"                cport \u003d mapping.get(\u0027controllerPort\u0027)"},{"line_number":796,"context_line":"                cportid \u003d self._get_id(cport)"},{"line_number":797,"context_line":"                domains \u003d self._find_domains(cportid)"},{"line_number":798,"context_line":"                if domains is not None:"},{"line_number":799,"context_line":"                    controllerport \u003d self._find_controller_port(cportid)"},{"line_number":800,"context_line":"                    if controllerport is not None:"},{"line_number":801,"context_line":"                        appendproperties \u003d False"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_db632518","line":798,"updated":"2015-02-18 05:56:41.000000000","message":"if domains:","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"7e2f85039381c2369642b56febb8b243f246dd9e","unresolved":false,"context_lines":[{"line_number":795,"context_line":"                cport \u003d mapping.get(\u0027controllerPort\u0027)"},{"line_number":796,"context_line":"                cportid \u003d self._get_id(cport)"},{"line_number":797,"context_line":"                domains \u003d self._find_domains(cportid)"},{"line_number":798,"context_line":"                if domains is not None:"},{"line_number":799,"context_line":"                    controllerport \u003d self._find_controller_port(cportid)"},{"line_number":800,"context_line":"                    if controllerport is not None:"},{"line_number":801,"context_line":"                        appendproperties \u003d False"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_e39ae56d","line":798,"in_reply_to":"da86d52c_db632518","updated":"2015-02-18 17:20:14.000000000","message":"Done","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"0ed6e9da1e794604ed6c464d9d08271cf35c6f4c","unresolved":false,"context_lines":[{"line_number":803,"context_line":"                            ipaddress \u003d d.get(\u0027targetIpv4Address\u0027,"},{"line_number":804,"context_line":"                                              d.get(\u0027wellKnownIpAddress\u0027))"},{"line_number":805,"context_line":"                            portnumber \u003d d.get(\u0027portNumber\u0027)"},{"line_number":806,"context_line":"                            if (ip is None or ip \u003d\u003d ipaddress) and \\"},{"line_number":807,"context_line":"                               (port is None or port \u003d\u003d portnumber):"},{"line_number":808,"context_line":"                                portal \u003d ipaddress + \u0027:\u0027 + str(portnumber)"},{"line_number":809,"context_line":"                                # I\u0027m not sure when we can have more than"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_1b6a1ded","line":806,"updated":"2015-02-18 05:56:41.000000000","message":"()","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"7e2f85039381c2369642b56febb8b243f246dd9e","unresolved":false,"context_lines":[{"line_number":803,"context_line":"                            ipaddress \u003d d.get(\u0027targetIpv4Address\u0027,"},{"line_number":804,"context_line":"                                              d.get(\u0027wellKnownIpAddress\u0027))"},{"line_number":805,"context_line":"                            portnumber \u003d d.get(\u0027portNumber\u0027)"},{"line_number":806,"context_line":"                            if (ip is None or ip \u003d\u003d ipaddress) and \\"},{"line_number":807,"context_line":"                               (port is None or port \u003d\u003d portnumber):"},{"line_number":808,"context_line":"                                portal \u003d ipaddress + \u0027:\u0027 + str(portnumber)"},{"line_number":809,"context_line":"                                # I\u0027m not sure when we can have more than"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_23887d21","line":806,"in_reply_to":"da86d52c_1b6a1ded","updated":"2015-02-18 17:20:14.000000000","message":"Done","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"0ed6e9da1e794604ed6c464d9d08271cf35c6f4c","unresolved":false,"context_lines":[{"line_number":804,"context_line":"                                              d.get(\u0027wellKnownIpAddress\u0027))"},{"line_number":805,"context_line":"                            portnumber \u003d d.get(\u0027portNumber\u0027)"},{"line_number":806,"context_line":"                            if (ip is None or ip \u003d\u003d ipaddress) and \\"},{"line_number":807,"context_line":"                               (port is None or port \u003d\u003d portnumber):"},{"line_number":808,"context_line":"                                portal \u003d ipaddress + \u0027:\u0027 + str(portnumber)"},{"line_number":809,"context_line":"                                # I\u0027m not sure when we can have more than"},{"line_number":810,"context_line":"                                # one portal for a domain but since it is an"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_7b2239ba","line":807,"updated":"2015-02-18 05:56:41.000000000","message":"Are you sure portnumber won\u0027t be None?","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"7e2f85039381c2369642b56febb8b243f246dd9e","unresolved":false,"context_lines":[{"line_number":804,"context_line":"                                              d.get(\u0027wellKnownIpAddress\u0027))"},{"line_number":805,"context_line":"                            portnumber \u003d d.get(\u0027portNumber\u0027)"},{"line_number":806,"context_line":"                            if (ip is None or ip \u003d\u003d ipaddress) and \\"},{"line_number":807,"context_line":"                               (port is None or port \u003d\u003d portnumber):"},{"line_number":808,"context_line":"                                portal \u003d ipaddress + \u0027:\u0027 + str(portnumber)"},{"line_number":809,"context_line":"                                # I\u0027m not sure when we can have more than"},{"line_number":810,"context_line":"                                # one portal for a domain but since it is an"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_a3a2ad97","line":807,"in_reply_to":"da86d52c_7b2239ba","updated":"2015-02-18 17:20:14.000000000","message":"Yes. portNumber is always populated in the Domain object.","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"0ed6e9da1e794604ed6c464d9d08271cf35c6f4c","unresolved":false,"context_lines":[{"line_number":805,"context_line":"                            portnumber \u003d d.get(\u0027portNumber\u0027)"},{"line_number":806,"context_line":"                            if (ip is None or ip \u003d\u003d ipaddress) and \\"},{"line_number":807,"context_line":"                               (port is None or port \u003d\u003d portnumber):"},{"line_number":808,"context_line":"                                portal \u003d ipaddress + \u0027:\u0027 + str(portnumber)"},{"line_number":809,"context_line":"                                # I\u0027m not sure when we can have more than"},{"line_number":810,"context_line":"                                # one portal for a domain but since it is an"},{"line_number":811,"context_line":"                                # array being returned it is best to check."}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_bb7af1b7","line":808,"updated":"2015-02-18 05:56:41.000000000","message":"Use six.text_type() instead of str().","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"7e2f85039381c2369642b56febb8b243f246dd9e","unresolved":false,"context_lines":[{"line_number":805,"context_line":"                            portnumber \u003d d.get(\u0027portNumber\u0027)"},{"line_number":806,"context_line":"                            if (ip is None or ip \u003d\u003d ipaddress) and \\"},{"line_number":807,"context_line":"                               (port is None or port \u003d\u003d portnumber):"},{"line_number":808,"context_line":"                                portal \u003d ipaddress + \u0027:\u0027 + str(portnumber)"},{"line_number":809,"context_line":"                                # I\u0027m not sure when we can have more than"},{"line_number":810,"context_line":"                                # one portal for a domain but since it is an"},{"line_number":811,"context_line":"                                # array being returned it is best to check."}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_e3a8257a","line":808,"in_reply_to":"da86d52c_bb7af1b7","updated":"2015-02-18 17:20:14.000000000","message":"Done","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"0ed6e9da1e794604ed6c464d9d08271cf35c6f4c","unresolved":false,"context_lines":[{"line_number":815,"context_line":"                        # We do not report lun and iqn info unless it is for"},{"line_number":816,"context_line":"                        # the configured port OR the user has not enabled"},{"line_number":817,"context_line":"                        # multipath.  (In which case ip and port sent in"},{"line_number":818,"context_line":"                        # will be None."},{"line_number":819,"context_line":"                        if appendproperties is True:"},{"line_number":820,"context_line":"                            iqns.append(controllerport.get(\u0027iscsiName\u0027))"},{"line_number":821,"context_line":"                            luns.append(mapping.get(\u0027lun\u0027))"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_9b028d0a","line":818,"updated":"2015-02-18 05:56:41.000000000","message":"Missing a closing )","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"7e2f85039381c2369642b56febb8b243f246dd9e","unresolved":false,"context_lines":[{"line_number":815,"context_line":"                        # We do not report lun and iqn info unless it is for"},{"line_number":816,"context_line":"                        # the configured port OR the user has not enabled"},{"line_number":817,"context_line":"                        # multipath.  (In which case ip and port sent in"},{"line_number":818,"context_line":"                        # will be None."},{"line_number":819,"context_line":"                        if appendproperties is True:"},{"line_number":820,"context_line":"                            iqns.append(controllerport.get(\u0027iscsiName\u0027))"},{"line_number":821,"context_line":"                            luns.append(mapping.get(\u0027lun\u0027))"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_23b6bddc","line":818,"in_reply_to":"da86d52c_9b028d0a","updated":"2015-02-18 17:20:14.000000000","message":"Done","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"}],"cinder/volume/drivers/dell/dell_storagecenter_iscsi.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"0ed6e9da1e794604ed6c464d9d08271cf35c6f4c","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"                        if multipath:"},{"line_number":81,"context_line":"                            # Just return our properties with all the mappings"},{"line_number":82,"context_line":"                            iscsiproperties \u003d \\"},{"line_number":83,"context_line":"                                api.find_iscsi_properties(scvolume,"},{"line_number":84,"context_line":"                                                          None,"},{"line_number":85,"context_line":"                                                          None)"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_3b37e14e","line":82,"updated":"2015-02-18 05:56:41.000000000","message":"Can you use () instead of \\?","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"7e2f85039381c2369642b56febb8b243f246dd9e","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"                        if multipath:"},{"line_number":81,"context_line":"                            # Just return our properties with all the mappings"},{"line_number":82,"context_line":"                            iscsiproperties \u003d \\"},{"line_number":83,"context_line":"                                api.find_iscsi_properties(scvolume,"},{"line_number":84,"context_line":"                                                          None,"},{"line_number":85,"context_line":"                                                          None)"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_e3bf45a9","line":82,"in_reply_to":"da86d52c_3b37e14e","updated":"2015-02-18 17:20:14.000000000","message":"Done","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"9ee7df87946c5bc76aec42a9dee7f74cc5db3d3f","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"                        if multipath:"},{"line_number":81,"context_line":"                            # Just return our properties with all the mappings"},{"line_number":82,"context_line":"                            iscsiproperties \u003d \\"},{"line_number":83,"context_line":"                                api.find_iscsi_properties(scvolume,"},{"line_number":84,"context_line":"                                                          None,"},{"line_number":85,"context_line":"                                                          None)"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_c35400c8","line":82,"in_reply_to":"da86d52c_3b37e14e","updated":"2015-02-18 17:28:36.000000000","message":"We\u0027ve changed it but the line ends up looking like ...\n\niscsiproperties \u003d (\n    api.find_iscsi_properties(scvolume,\n\t\t                            None,\n\t\t                            None))\n\n... which I find aesthetically displeasing.  That said I can get over it.","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"0ed6e9da1e794604ed6c464d9d08271cf35c6f4c","unresolved":false,"context_lines":[{"line_number":89,"context_line":"                            # Only return the iqn for the user specified port."},{"line_number":90,"context_line":"                            ip \u003d self.configuration.iscsi_ip_address"},{"line_number":91,"context_line":"                            port \u003d self.configuration.iscsi_port"},{"line_number":92,"context_line":"                            iscsiproperties \u003d \\"},{"line_number":93,"context_line":"                                api.find_iscsi_properties(scvolume,"},{"line_number":94,"context_line":"                                                          ip,"},{"line_number":95,"context_line":"                                                          port)"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_5b3f3525","line":92,"updated":"2015-02-18 05:56:41.000000000","message":"()","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"7e2f85039381c2369642b56febb8b243f246dd9e","unresolved":false,"context_lines":[{"line_number":89,"context_line":"                            # Only return the iqn for the user specified port."},{"line_number":90,"context_line":"                            ip \u003d self.configuration.iscsi_ip_address"},{"line_number":91,"context_line":"                            port \u003d self.configuration.iscsi_port"},{"line_number":92,"context_line":"                            iscsiproperties \u003d \\"},{"line_number":93,"context_line":"                                api.find_iscsi_properties(scvolume,"},{"line_number":94,"context_line":"                                                          ip,"},{"line_number":95,"context_line":"                                                          port)"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_23dddd14","line":92,"in_reply_to":"da86d52c_5b3f3525","updated":"2015-02-18 17:20:14.000000000","message":"Done","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"0ed6e9da1e794604ed6c464d9d08271cf35c6f4c","unresolved":false,"context_lines":[{"line_number":101,"context_line":"                            # we are looking for.  Otherwise error."},{"line_number":102,"context_line":"                            if len(portals) \u003e 0:"},{"line_number":103,"context_line":"                                properties[\u0027target_portal\u0027] \u003d portals[0]"},{"line_number":104,"context_line":"                                properties[\u0027target_iqn\u0027] \u003d \\"},{"line_number":105,"context_line":"                                    iscsiproperties[\u0027target_iqns\u0027][0]"},{"line_number":106,"context_line":"                                properties[\u0027target_lun\u0027] \u003d \\"},{"line_number":107,"context_line":"                                    iscsiproperties[\u0027target_luns\u0027][0]"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_fb1709a1","line":104,"updated":"2015-02-18 05:56:41.000000000","message":"()","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"7e2f85039381c2369642b56febb8b243f246dd9e","unresolved":false,"context_lines":[{"line_number":101,"context_line":"                            # we are looking for.  Otherwise error."},{"line_number":102,"context_line":"                            if len(portals) \u003e 0:"},{"line_number":103,"context_line":"                                properties[\u0027target_portal\u0027] \u003d portals[0]"},{"line_number":104,"context_line":"                                properties[\u0027target_iqn\u0027] \u003d \\"},{"line_number":105,"context_line":"                                    iscsiproperties[\u0027target_iqns\u0027][0]"},{"line_number":106,"context_line":"                                properties[\u0027target_lun\u0027] \u003d \\"},{"line_number":107,"context_line":"                                    iscsiproperties[\u0027target_luns\u0027][0]"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_43e2d1d5","line":104,"in_reply_to":"da86d52c_fb1709a1","updated":"2015-02-18 17:20:14.000000000","message":"Done","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"0ed6e9da1e794604ed6c464d9d08271cf35c6f4c","unresolved":false,"context_lines":[{"line_number":103,"context_line":"                                properties[\u0027target_portal\u0027] \u003d portals[0]"},{"line_number":104,"context_line":"                                properties[\u0027target_iqn\u0027] \u003d \\"},{"line_number":105,"context_line":"                                    iscsiproperties[\u0027target_iqns\u0027][0]"},{"line_number":106,"context_line":"                                properties[\u0027target_lun\u0027] \u003d \\"},{"line_number":107,"context_line":"                                    iscsiproperties[\u0027target_luns\u0027][0]"},{"line_number":108,"context_line":"                                properties[\u0027access_mode\u0027] \u003d \\"},{"line_number":109,"context_line":"                                    iscsiproperties[\u0027access_mode\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_1b093d84","line":106,"updated":"2015-02-18 05:56:41.000000000","message":"()","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"7e2f85039381c2369642b56febb8b243f246dd9e","unresolved":false,"context_lines":[{"line_number":103,"context_line":"                                properties[\u0027target_portal\u0027] \u003d portals[0]"},{"line_number":104,"context_line":"                                properties[\u0027target_iqn\u0027] \u003d \\"},{"line_number":105,"context_line":"                                    iscsiproperties[\u0027target_iqns\u0027][0]"},{"line_number":106,"context_line":"                                properties[\u0027target_lun\u0027] \u003d \\"},{"line_number":107,"context_line":"                                    iscsiproperties[\u0027target_luns\u0027][0]"},{"line_number":108,"context_line":"                                properties[\u0027access_mode\u0027] \u003d \\"},{"line_number":109,"context_line":"                                    iscsiproperties[\u0027access_mode\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_83e849b4","line":106,"in_reply_to":"da86d52c_1b093d84","updated":"2015-02-18 17:20:14.000000000","message":"Done","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"0ed6e9da1e794604ed6c464d9d08271cf35c6f4c","unresolved":false,"context_lines":[{"line_number":105,"context_line":"                                    iscsiproperties[\u0027target_iqns\u0027][0]"},{"line_number":106,"context_line":"                                properties[\u0027target_lun\u0027] \u003d \\"},{"line_number":107,"context_line":"                                    iscsiproperties[\u0027target_luns\u0027][0]"},{"line_number":108,"context_line":"                                properties[\u0027access_mode\u0027] \u003d \\"},{"line_number":109,"context_line":"                                    iscsiproperties[\u0027access_mode\u0027]"},{"line_number":110,"context_line":"                                LOG.debug(properties)"},{"line_number":111,"context_line":"                                return {\u0027driver_volume_type\u0027: \u0027iscsi\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_db1245b3","line":108,"updated":"2015-02-18 05:56:41.000000000","message":"()","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"},{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"7e2f85039381c2369642b56febb8b243f246dd9e","unresolved":false,"context_lines":[{"line_number":105,"context_line":"                                    iscsiproperties[\u0027target_iqns\u0027][0]"},{"line_number":106,"context_line":"                                properties[\u0027target_lun\u0027] \u003d \\"},{"line_number":107,"context_line":"                                    iscsiproperties[\u0027target_luns\u0027][0]"},{"line_number":108,"context_line":"                                properties[\u0027access_mode\u0027] \u003d \\"},{"line_number":109,"context_line":"                                    iscsiproperties[\u0027access_mode\u0027]"},{"line_number":110,"context_line":"                                LOG.debug(properties)"},{"line_number":111,"context_line":"                                return {\u0027driver_volume_type\u0027: \u0027iscsi\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_a3d0edf7","line":108,"in_reply_to":"da86d52c_db1245b3","updated":"2015-02-18 17:20:14.000000000","message":"Done","commit_id":"2ababac57f353c470975624480e1ff13a7e64179"}]}