)]}'
{"cinder/tests/unit/test_storwize_svc.py":[{"author":{"_account_id":16917,"name":"Alex O\u0027Rourke","email":"alex.orourke@hpe.com","username":"aorourke"},"change_message_id":"179b4f76ddfdd4974708afeadd7fda5fa1904fd7","unresolved":false,"context_lines":[{"line_number":2404,"context_line":"                    expected_fc_npiv \u003d {"},{"line_number":2405,"context_line":"                        \u0027driver_volume_type\u0027: \u0027fibre_channel\u0027,"},{"line_number":2406,"context_line":"                        \u0027data\u0027: {\u0027target_lun\u0027: 1,"},{"line_number":2407,"context_line":"                                 \u0027target_wwn\u0027: \u00275005076802132ADE\u0027,"},{"line_number":2408,"context_line":"                                 \u0027target_discovered\u0027: False}}"},{"line_number":2409,"context_line":"                    ret \u003d self.driver.initialize_connection(volume2,"},{"line_number":2410,"context_line":"                                                            self._connector)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_77936dfb","line":2407,"updated":"2015-11-09 03:07:46.000000000","message":"It looks like this is causing the py27 and py34 tests to fail. Was this changed by accident?","commit_id":"f0e8d3831ade2005515d2df695e874b1c11abf5e"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"0687cb8cfb64d739cc74b18c387aab77be6ea303","unresolved":false,"context_lines":[{"line_number":2404,"context_line":"                    expected_fc_npiv \u003d {"},{"line_number":2405,"context_line":"                        \u0027driver_volume_type\u0027: \u0027fibre_channel\u0027,"},{"line_number":2406,"context_line":"                        \u0027data\u0027: {\u0027target_lun\u0027: 1,"},{"line_number":2407,"context_line":"                                 \u0027target_wwn\u0027: \u00275005076802132ADE\u0027,"},{"line_number":2408,"context_line":"                                 \u0027target_discovered\u0027: False}}"},{"line_number":2409,"context_line":"                    ret \u003d self.driver.initialize_connection(volume2,"},{"line_number":2410,"context_line":"                                                            self._connector)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_2a41ea0b","line":2407,"in_reply_to":"da85f559_77936dfb","updated":"2015-11-09 11:12:19.000000000","message":"This change is deliberate, it\u0027s explained in the commit message.\n\ntest_storwize_initiator_target_map_npiv() failed, not this test. My change in test_storwize_initiator_target_map_npiv() was not enough to support randomized hash function.\n\ntest_storwize_initiator_target_map_npiv() should now pass with my new change set.","commit_id":"f0e8d3831ade2005515d2df695e874b1c11abf5e"},{"author":{"_account_id":8912,"name":"Ryan McNair","email":"rdmcnair@us.ibm.com","username":"rdmcnair"},"change_message_id":"3ccd2471157c844fe10ccc66f9238942a370f25f","unresolved":false,"context_lines":[{"line_number":3540,"context_line":"                          \u00275005086802532ADE\u0027))"},{"line_number":3541,"context_line":""},{"line_number":3542,"context_line":"        target_map \u003d init_ret[\u0027data\u0027].pop(\u0027initiator_target_map\u0027)"},{"line_number":3543,"context_line":"        self.assertSetEqual(set(target_map[\u0027ff00000000000000\u0027]),"},{"line_number":3544,"context_line":"                            set(conn_wwpns))"},{"line_number":3545,"context_line":"        self.assertSetEqual(set(target_map[\u0027ff00000000000001\u0027]),"},{"line_number":3546,"context_line":"                            set(conn_wwpns))"}],"source_content_type":"text/x-python","patch_set":3,"id":"da85f559_51cb9e53","line":3543,"updated":"2015-11-10 01:54:33.000000000","message":"Isn\u0027t conn_wwpns already a set?","commit_id":"9fec79dc982df80c417604ad9ea44c3b59048090"},{"author":{"_account_id":8912,"name":"Ryan McNair","email":"rdmcnair@us.ibm.com","username":"rdmcnair"},"change_message_id":"d613073f27500e3a603464038e2bbe8942b354d5","unresolved":false,"context_lines":[{"line_number":3540,"context_line":"                          \u00275005086802532ADE\u0027))"},{"line_number":3541,"context_line":""},{"line_number":3542,"context_line":"        target_map \u003d init_ret[\u0027data\u0027].pop(\u0027initiator_target_map\u0027)"},{"line_number":3543,"context_line":"        self.assertSetEqual(set(target_map[\u0027ff00000000000000\u0027]),"},{"line_number":3544,"context_line":"                            set(conn_wwpns))"},{"line_number":3545,"context_line":"        self.assertSetEqual(set(target_map[\u0027ff00000000000001\u0027]),"},{"line_number":3546,"context_line":"                            set(conn_wwpns))"}],"source_content_type":"text/x-python","patch_set":3,"id":"da85f559_d4f0a593","line":3543,"in_reply_to":"da85f559_2e4d473e","updated":"2015-11-11 00:04:04.000000000","message":"Not a big deal, but was referring to conn_wwpns which gets defined as a set  on line 3531","commit_id":"9fec79dc982df80c417604ad9ea44c3b59048090"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"aafdffad4c2e7af61a46ab1edfa3ed3fede3f879","unresolved":false,"context_lines":[{"line_number":3540,"context_line":"                          \u00275005086802532ADE\u0027))"},{"line_number":3541,"context_line":""},{"line_number":3542,"context_line":"        target_map \u003d init_ret[\u0027data\u0027].pop(\u0027initiator_target_map\u0027)"},{"line_number":3543,"context_line":"        self.assertSetEqual(set(target_map[\u0027ff00000000000000\u0027]),"},{"line_number":3544,"context_line":"                            set(conn_wwpns))"},{"line_number":3545,"context_line":"        self.assertSetEqual(set(target_map[\u0027ff00000000000001\u0027]),"},{"line_number":3546,"context_line":"                            set(conn_wwpns))"}],"source_content_type":"text/x-python","patch_set":3,"id":"da85f559_2e4d473e","line":3543,"in_reply_to":"da85f559_51cb9e53","updated":"2015-11-10 09:43:12.000000000","message":"No, it\u0027s a list. See initialize_connection() \u0026 add_fc_wwpns() functions, both functions create lists.","commit_id":"9fec79dc982df80c417604ad9ea44c3b59048090"}]}