)]}'
{"cinder/volume/drivers/dell/dell_storagecenter_api.py":[{"author":{"_account_id":15011,"name":"Jeff Seiler","email":"jeff_seiler@dell.com","username":"jseiler"},"change_message_id":"78902af3b5fdbda86a5f1f6635458b61eb9381e0","unresolved":false,"context_lines":[{"line_number":345,"context_line":"                    self.consisgroups \u003d False"},{"line_number":346,"context_line":"                    self.legacypayloadfilters \u003d True"},{"line_number":347,"context_line":""},{"line_number":348,"context_line":"                elif version.startswith(\u00272.1\u0027):"},{"line_number":349,"context_line":"                    self.legacypayloadfilters \u003d True"},{"line_number":350,"context_line":"                return"},{"line_number":351,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_4ace56b5","line":348,"updated":"2015-07-21 19:46:41.000000000","message":"If the API version was 2.11, would this check have the desired results?  Would it make sense to do a split then look at the major and minor values instead?","commit_id":"61fcce6c5505ba5200ea0d2d671338b4c34bdba0"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"2e3e5cd3f704465ec5a61134c78a20853e3e4fe3","unresolved":false,"context_lines":[{"line_number":345,"context_line":"                    self.consisgroups \u003d False"},{"line_number":346,"context_line":"                    self.legacypayloadfilters \u003d True"},{"line_number":347,"context_line":""},{"line_number":348,"context_line":"                elif version.startswith(\u00272.1\u0027):"},{"line_number":349,"context_line":"                    self.legacypayloadfilters \u003d True"},{"line_number":350,"context_line":"                return"},{"line_number":351,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_e573d37e","line":348,"in_reply_to":"3a50d1a3_4ace56b5","updated":"2015-07-21 20:00:30.000000000","message":"Nope.  Yep.  Fixed.","commit_id":"61fcce6c5505ba5200ea0d2d671338b4c34bdba0"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"b2595e633432ce80311caed75bd55002a0e00ff8","unresolved":false,"context_lines":[{"line_number":1215,"context_line":"        if r.status_code \u003d\u003d 200:"},{"line_number":1216,"context_line":"            scconfig \u003d self._get_json(r)"},{"line_number":1217,"context_line":"            if scconfig:"},{"line_number":1218,"context_line":"                isvpmode \u003d True if (scconfig[\u0027iscsiTransportMode\u0027] \u003d\u003d"},{"line_number":1219,"context_line":"                                    \u0027VirtualPort\u0027) else False"},{"line_number":1220,"context_line":"        return isvpmode"},{"line_number":1221,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_83c7f294","line":1218,"updated":"2015-07-23 15:53:27.000000000","message":"Could just be:\n\n    isvpmode \u003d (scconfig[\u0027iscsiTransportMode\u0027] \u003d\u003d \u0027VirtualPort\u0027)","commit_id":"6806ea491f76be8e6717a7ec4238ef672df9b85a"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"cdaa3c39468256aca8084df9219ee2ea7c143466","unresolved":false,"context_lines":[{"line_number":1215,"context_line":"        if r.status_code \u003d\u003d 200:"},{"line_number":1216,"context_line":"            scconfig \u003d self._get_json(r)"},{"line_number":1217,"context_line":"            if scconfig:"},{"line_number":1218,"context_line":"                isvpmode \u003d True if (scconfig[\u0027iscsiTransportMode\u0027] \u003d\u003d"},{"line_number":1219,"context_line":"                                    \u0027VirtualPort\u0027) else False"},{"line_number":1220,"context_line":"        return isvpmode"},{"line_number":1221,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_0905430f","line":1218,"in_reply_to":"3a50d1a3_83c7f294","updated":"2015-07-23 16:03:50.000000000","message":"But I love the look of that kind of if!  will change if new patch required else leave.","commit_id":"6806ea491f76be8e6717a7ec4238ef672df9b85a"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"b2595e633432ce80311caed75bd55002a0e00ff8","unresolved":false,"context_lines":[{"line_number":1326,"context_line":"                # Get our IQN from our mapping."},{"line_number":1327,"context_line":"                iqn \u003d self._get_iqn(mapping)"},{"line_number":1328,"context_line":"                # Check if our controller ID matches our active controller ID."},{"line_number":1329,"context_line":"                isactive \u003d True if (self._get_controller_id(mapping) \u003d\u003d"},{"line_number":1330,"context_line":"                                    actvctrl) else False"},{"line_number":1331,"context_line":"                # If we have an IQN and are in virtual port mode."},{"line_number":1332,"context_line":"                if isvpmode and iqn:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_e35916fe","line":1329,"updated":"2015-07-23 15:53:27.000000000","message":"Similar to comment above.","commit_id":"6806ea491f76be8e6717a7ec4238ef672df9b85a"}]}