)]}'
{"cinder/volume/drivers/solidfire.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"a5f0fd0b8d5edcc6cbeac09be312492b8ec2a2e0","unresolved":false,"context_lines":[{"line_number":351,"context_line":"                id_string \u003d self._create_provider_id_string("},{"line_number":352,"context_line":"                    sfvol[\u0027volumeID\u0027],"},{"line_number":353,"context_line":"                    sfvol[\u0027accountID\u0027])"},{"line_number":354,"context_line":"                if v.get(\u0027provider_id\u0027, \u0027nil\u0027) !\u003d id_string:"},{"line_number":355,"context_line":"                    updates.append("},{"line_number":356,"context_line":"                        {\u0027id\u0027: v[\u0027id\u0027],"},{"line_number":357,"context_line":"                         \u0027provider_id\u0027: id_string})"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_5349898d","line":354,"range":{"start_line":354,"start_character":0,"end_line":354,"end_character":60},"updated":"2016-05-04 17:36:28.000000000","message":"This is good, but you need to address the case of cluster_uuid being set here.  Since you\u0027re doing the create_provider_id you\u0027ll need to ignore the 3\u0027rd item in the tuple for the comparison below.  So if we just chopped off the last field in the v.get(\u0027provider_id\u0027) that would keep us from overwriting cluster_uuid with None here.","commit_id":"dc2457c48645218651eef5094cbd351efaabd08e"}]}