)]}'
{"cinder/volume/drivers/dell/dell_storagecenter_api.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"77ef8ff320ee03ed26c19563168574096498fa1d","unresolved":false,"context_lines":[{"line_number":1690,"context_line":"                    if portal \u003d\u003d newportal and iqns[idx] \u003d\u003d iqn:"},{"line_number":1691,"context_line":"                        LOG.debug(\u0027Skipping duplicate portal %(ptrl)s and\u0027"},{"line_number":1692,"context_line":"                                  \u0027iqn %(iqn)s.\u0027, {\u0027ptrl\u0027: portal, \u0027iqn\u0027: iqn})"},{"line_number":1693,"context_line":"                        return"},{"line_number":1694,"context_line":"                # It isn\u0027t in the list so process it."},{"line_number":1695,"context_line":"                portals.append(newportal)"},{"line_number":1696,"context_line":"                iqns.append(iqn)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8ec9b2_bf19d37b","line":1693,"range":{"start_line":1693,"start_character":24,"end_line":1693,"end_character":30},"updated":"2016-09-14 16:39:40.000000000","message":"Couldn\u0027t we just do:\n\nnewportal \u003d \"%s:%s\" % (address, port)\nif newportal not in portals:\n    portals.append(newportal)","commit_id":"4708aa16a7fd43a8fba95b51bf26021498776868"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"8ad0b78383ee1e299de3ae2140f32a1ca7226fb4","unresolved":false,"context_lines":[{"line_number":1690,"context_line":"                    if portal \u003d\u003d newportal and iqns[idx] \u003d\u003d iqn:"},{"line_number":1691,"context_line":"                        LOG.debug(\u0027Skipping duplicate portal %(ptrl)s and\u0027"},{"line_number":1692,"context_line":"                                  \u0027iqn %(iqn)s.\u0027, {\u0027ptrl\u0027: portal, \u0027iqn\u0027: iqn})"},{"line_number":1693,"context_line":"                        return"},{"line_number":1694,"context_line":"                # It isn\u0027t in the list so process it."},{"line_number":1695,"context_line":"                portals.append(newportal)"},{"line_number":1696,"context_line":"                iqns.append(iqn)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8ec9b2_8aca136d","line":1693,"range":{"start_line":1693,"start_character":24,"end_line":1693,"end_character":30},"in_reply_to":"7a8ec9b2_bf19d37b","updated":"2016-09-14 16:44:49.000000000","message":"Are you referring to just the newportal\u003d line or the whole kit and kaboodle? Cause it has to be both iqn and portal.","commit_id":"4708aa16a7fd43a8fba95b51bf26021498776868"}]}