)]}'
{"cinder/zonemanager/fc_san_lookup_service.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"962ee21eab7b30f808b7250959c223cefc9a2332","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        super(FCSanLookupService, self).__init__(**kwargs)"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        opts \u003d fc_zone_manager.zone_manager_opts"},{"line_number":53,"context_line":"        self.configuration \u003d config.Configuration(opts, \u0027fc-zone-manager\u0027)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    def get_device_mapping_from_network(self, initiator_list, target_list):"},{"line_number":56,"context_line":"        \"\"\"Get device mapping from FC network."}],"source_content_type":"text/x-python","patch_set":1,"id":"ba7be1f8_0ab9e2cb","line":53,"updated":"2015-02-24 21:30:15.000000000","message":"Not following why we init here like this?  Shouldn\u0027t we just be getting this via the inheritance structure?","commit_id":"ef63d75db7181a79933b207fbb1ba8b0e7a1c0a7"}],"cinder/zonemanager/fc_zone_manager.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"962ee21eab7b30f808b7250959c223cefc9a2332","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        super(ZoneManager, self).__init__(**kwargs)"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"        self.configuration \u003d config.Configuration(zone_manager_opts,"},{"line_number":92,"context_line":"                                                  \u0027fc-zone-manager\u0027)"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        self._build_driver()"},{"line_number":95,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ba7be1f8_2abea6b4","line":92,"updated":"2015-02-24 21:30:15.000000000","message":"Same question as above kinda... shouldn\u0027t this just be appending the fc-zone-manager to self.config (we should have that via the inherittance no?)","commit_id":"ef63d75db7181a79933b207fbb1ba8b0e7a1c0a7"}]}