)]}'
{"nova/tests/xenapi/test_vm_utils.py":[{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"25cef63a2a96c1d1778eeab0825c8b28268902a5","unresolved":false,"context_lines":[{"line_number":10,"context_line":"import unittest"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"def get_fake_dev_params():"},{"line_number":14,"context_line":"    return {\u0027sr_uuid\u0027: \u0027falseSR\u0027,"},{"line_number":15,"context_line":"            \u0027name_label\u0027: \u0027fake_storage\u0027,"},{"line_number":16,"context_line":"            \u0027name_description\u0027: \u0027test purposes\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F30U%3D","line":13,"updated":"2012-12-11 16:01:57.000000000","message":"I started to call this structure as connection_data. This is the part, which is specific to the volume driver used. It would be good to see iscsi_connection_data, and xensm_connection_data, to communicate, that they are separate things.","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"cabe87c2e1b1ea2534443f767fc1871a9bc574fe","unresolved":false,"context_lines":[{"line_number":10,"context_line":"import unittest"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"def get_fake_dev_params():"},{"line_number":14,"context_line":"    return {\u0027sr_uuid\u0027: \u0027falseSR\u0027,"},{"line_number":15,"context_line":"            \u0027name_label\u0027: \u0027fake_storage\u0027,"},{"line_number":16,"context_line":"            \u0027name_description\u0027: \u0027test purposes\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F2FE%3D","line":13,"in_reply_to":"AAAAMX%2F%2F30U%3D","updated":"2012-12-12 22:56:47.000000000","message":"Done","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"f2e7737942f4ba74fec6878aa3b42fc4e9b2cd01","unresolved":false,"context_lines":[{"line_number":10,"context_line":"import unittest"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"def get_fake_dev_params():"},{"line_number":14,"context_line":"    return {\u0027sr_uuid\u0027: \u0027falseSR\u0027,"},{"line_number":15,"context_line":"            \u0027name_label\u0027: \u0027fake_storage\u0027,"},{"line_number":16,"context_line":"            \u0027name_description\u0027: \u0027test purposes\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3sE%3D","line":13,"in_reply_to":"AAAAMX%2F%2F30U%3D","updated":"2012-12-11 16:50:27.000000000","message":"I can split the two.","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"25cef63a2a96c1d1778eeab0825c8b28268902a5","unresolved":false,"context_lines":[{"line_number":20,"context_line":"            \u0027introduce_sr_keys\u0027: [\u0027server\u0027,"},{"line_number":21,"context_line":"                                  \u0027serverpath\u0027,"},{"line_number":22,"context_line":"                                  \u0027sr_type\u0027],"},{"line_number":23,"context_line":"            \u0027volume_id\u0027: \u0027fake_volume_id\u0027,"},{"line_number":24,"context_line":"            \u0027vdi_uuid\u0027: \u0027falseVDI\u0027}"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3zI%3D","line":23,"updated":"2012-12-11 16:01:57.000000000","message":"volume_id is new, it didn\u0027t exist in the original dict","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"cabe87c2e1b1ea2534443f767fc1871a9bc574fe","unresolved":false,"context_lines":[{"line_number":20,"context_line":"            \u0027introduce_sr_keys\u0027: [\u0027server\u0027,"},{"line_number":21,"context_line":"                                  \u0027serverpath\u0027,"},{"line_number":22,"context_line":"                                  \u0027sr_type\u0027],"},{"line_number":23,"context_line":"            \u0027volume_id\u0027: \u0027fake_volume_id\u0027,"},{"line_number":24,"context_line":"            \u0027vdi_uuid\u0027: \u0027falseVDI\u0027}"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F2IE%3D","line":23,"in_reply_to":"AAAAMX%2F%2F3zI%3D","updated":"2012-12-12 22:56:47.000000000","message":"Done","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"25cef63a2a96c1d1778eeab0825c8b28268902a5","unresolved":false,"context_lines":[{"line_number":86,"context_line":"                          vm_utils.get_vdis_for_boot_from_vol,"},{"line_number":87,"context_line":"                          driver._session, dev_params)"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def test_get_vdis_for_boot_from_vol_missing_sr_uuid(self):"},{"line_number":90,"context_line":"        dev_params \u003d get_fake_dev_params()"},{"line_number":91,"context_line":"        del dev_params[\u0027sr_uuid\u0027]"},{"line_number":92,"context_line":"        del dev_params[\u0027vdi_uuid\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3zM%3D","line":89,"updated":"2012-12-11 16:01:57.000000000","message":"I would include iscsi in the name of this function.","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"cabe87c2e1b1ea2534443f767fc1871a9bc574fe","unresolved":false,"context_lines":[{"line_number":86,"context_line":"                          vm_utils.get_vdis_for_boot_from_vol,"},{"line_number":87,"context_line":"                          driver._session, dev_params)"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def test_get_vdis_for_boot_from_vol_missing_sr_uuid(self):"},{"line_number":90,"context_line":"        dev_params \u003d get_fake_dev_params()"},{"line_number":91,"context_line":"        del dev_params[\u0027sr_uuid\u0027]"},{"line_number":92,"context_line":"        del dev_params[\u0027vdi_uuid\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F2FA%3D","line":89,"in_reply_to":"AAAAMX%2F%2F3zM%3D","updated":"2012-12-12 22:56:47.000000000","message":"Done","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"f2e7737942f4ba74fec6878aa3b42fc4e9b2cd01","unresolved":false,"context_lines":[{"line_number":86,"context_line":"                          vm_utils.get_vdis_for_boot_from_vol,"},{"line_number":87,"context_line":"                          driver._session, dev_params)"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def test_get_vdis_for_boot_from_vol_missing_sr_uuid(self):"},{"line_number":90,"context_line":"        dev_params \u003d get_fake_dev_params()"},{"line_number":91,"context_line":"        del dev_params[\u0027sr_uuid\u0027]"},{"line_number":92,"context_line":"        del dev_params[\u0027vdi_uuid\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3r8%3D","line":89,"in_reply_to":"AAAAMX%2F%2F3zM%3D","updated":"2012-12-11 16:50:27.000000000","message":"can do","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"25cef63a2a96c1d1778eeab0825c8b28268902a5","unresolved":false,"context_lines":[{"line_number":87,"context_line":"                          driver._session, dev_params)"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def test_get_vdis_for_boot_from_vol_missing_sr_uuid(self):"},{"line_number":90,"context_line":"        dev_params \u003d get_fake_dev_params()"},{"line_number":91,"context_line":"        del dev_params[\u0027sr_uuid\u0027]"},{"line_number":92,"context_line":"        del dev_params[\u0027vdi_uuid\u0027]"},{"line_number":93,"context_line":"        del dev_params[\u0027introduce_sr_keys\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3zY%3D","line":90,"updated":"2012-12-11 16:01:57.000000000","message":"As said previously, creating a separate dict for the iscsi connection_data would be better. Hacking around the xensm structure, and modding it suggests that they have something to do with each other.","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"cabe87c2e1b1ea2534443f767fc1871a9bc574fe","unresolved":false,"context_lines":[{"line_number":87,"context_line":"                          driver._session, dev_params)"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def test_get_vdis_for_boot_from_vol_missing_sr_uuid(self):"},{"line_number":90,"context_line":"        dev_params \u003d get_fake_dev_params()"},{"line_number":91,"context_line":"        del dev_params[\u0027sr_uuid\u0027]"},{"line_number":92,"context_line":"        del dev_params[\u0027vdi_uuid\u0027]"},{"line_number":93,"context_line":"        del dev_params[\u0027introduce_sr_keys\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F2E4%3D","line":90,"in_reply_to":"AAAAMX%2F%2F3zY%3D","updated":"2012-12-12 22:56:47.000000000","message":"Done","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"f2e7737942f4ba74fec6878aa3b42fc4e9b2cd01","unresolved":false,"context_lines":[{"line_number":87,"context_line":"                          driver._session, dev_params)"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def test_get_vdis_for_boot_from_vol_missing_sr_uuid(self):"},{"line_number":90,"context_line":"        dev_params \u003d get_fake_dev_params()"},{"line_number":91,"context_line":"        del dev_params[\u0027sr_uuid\u0027]"},{"line_number":92,"context_line":"        del dev_params[\u0027vdi_uuid\u0027]"},{"line_number":93,"context_line":"        del dev_params[\u0027introduce_sr_keys\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3r4%3D","line":90,"in_reply_to":"AAAAMX%2F%2F3zY%3D","updated":"2012-12-11 16:50:27.000000000","message":"can do","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"25cef63a2a96c1d1778eeab0825c8b28268902a5","unresolved":false,"context_lines":[{"line_number":95,"context_line":"                           \u0027volume_id\u0027: \u0027fake_volume_id\u0027,"},{"line_number":96,"context_line":"                           \u0027target_iqn\u0027: \u0027fake_iqn:volume-fake_volume_id\u0027,"},{"line_number":97,"context_line":"                           \u0027target_portal\u0027: u\u0027localhost:3260\u0027,"},{"line_number":98,"context_line":"                           \u0027target_discovered\u0027: False})"},{"line_number":99,"context_line":"        stubs.stubout_session(self.stubs, fake.SessionBase)"},{"line_number":100,"context_line":"        driver \u003d xenapi_conn.XenAPIDriver(False)"},{"line_number":101,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3zU%3D","line":98,"updated":"2012-12-11 16:01:57.000000000","message":"What is this target_discovered?","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"f2e7737942f4ba74fec6878aa3b42fc4e9b2cd01","unresolved":false,"context_lines":[{"line_number":95,"context_line":"                           \u0027volume_id\u0027: \u0027fake_volume_id\u0027,"},{"line_number":96,"context_line":"                           \u0027target_iqn\u0027: \u0027fake_iqn:volume-fake_volume_id\u0027,"},{"line_number":97,"context_line":"                           \u0027target_portal\u0027: u\u0027localhost:3260\u0027,"},{"line_number":98,"context_line":"                           \u0027target_discovered\u0027: False})"},{"line_number":99,"context_line":"        stubs.stubout_session(self.stubs, fake.SessionBase)"},{"line_number":100,"context_line":"        driver \u003d xenapi_conn.XenAPIDriver(False)"},{"line_number":101,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3r0%3D","line":98,"in_reply_to":"AAAAMX%2F%2F3zU%3D","updated":"2012-12-11 16:50:27.000000000","message":"these are attributes that come out from the iscsi case.","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"}],"nova/virt/xenapi/vm_utils.py":[{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"25cef63a2a96c1d1778eeab0825c8b28268902a5","unresolved":false,"context_lines":[{"line_number":455,"context_line":"    return vdi_ref"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":""},{"line_number":458,"context_line":"def get_vdis_for_boot_from_vol(session, dev_params):"},{"line_number":459,"context_line":"    vdis \u003d {}"},{"line_number":460,"context_line":"    sr_params \u003d {}"},{"line_number":461,"context_line":"    if \u0027sr_uuid\u0027 not in dev_params:"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3yI%3D","line":458,"updated":"2012-12-11 16:01:57.000000000","message":"It smells like connect_volume from ./nova/virt/xenapi/volumeops.py It would be a better step to refactor that function, and have only one function to get vdis from a given connection_data","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"cabe87c2e1b1ea2534443f767fc1871a9bc574fe","unresolved":false,"context_lines":[{"line_number":455,"context_line":"    return vdi_ref"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":""},{"line_number":458,"context_line":"def get_vdis_for_boot_from_vol(session, dev_params):"},{"line_number":459,"context_line":"    vdis \u003d {}"},{"line_number":460,"context_line":"    sr_params \u003d {}"},{"line_number":461,"context_line":"    if \u0027sr_uuid\u0027 not in dev_params:"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F2E0%3D","line":458,"in_reply_to":"AAAAMX%2F%2F3yI%3D","updated":"2012-12-12 22:56:47.000000000","message":"Done","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"f2e7737942f4ba74fec6878aa3b42fc4e9b2cd01","unresolved":false,"context_lines":[{"line_number":455,"context_line":"    return vdi_ref"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":""},{"line_number":458,"context_line":"def get_vdis_for_boot_from_vol(session, dev_params):"},{"line_number":459,"context_line":"    vdis \u003d {}"},{"line_number":460,"context_line":"    sr_params \u003d {}"},{"line_number":461,"context_line":"    if \u0027sr_uuid\u0027 not in dev_params:"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3sc%3D","line":458,"in_reply_to":"AAAAMX%2F%2F3yI%3D","updated":"2012-12-11 16:50:27.000000000","message":"sure, I can do that.","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"}],"nova/virt/xenapi/volume_utils.py":[{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"25cef63a2a96c1d1778eeab0825c8b28268902a5","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    # If the sr_type is missing, we assume we are"},{"line_number":71,"context_line":"    # using the default iscsi back-end"},{"line_number":72,"context_line":"    type \u003d params.get(\u0027sr_type\u0027) or \u0027iscsi\u0027"},{"line_number":73,"context_line":"    LOG.debug(_(\u0027type is \u003d %s\u0027) % type)"},{"line_number":74,"context_line":"    if \u0027name_description\u0027 in params:"},{"line_number":75,"context_line":"        desc \u003d params[\u0027name_description\u0027]"},{"line_number":76,"context_line":"        LOG.debug(_(\u0027name \u003d %s\u0027) % desc)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3x0%3D","line":73,"updated":"2012-12-11 16:01:57.000000000","message":"I am not sure, why del params[\u0027sr_type\u0027] was there, but you definitely removed that, so I would recommend to put that back.","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"cabe87c2e1b1ea2534443f767fc1871a9bc574fe","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    # If the sr_type is missing, we assume we are"},{"line_number":71,"context_line":"    # using the default iscsi back-end"},{"line_number":72,"context_line":"    type \u003d params.get(\u0027sr_type\u0027) or \u0027iscsi\u0027"},{"line_number":73,"context_line":"    LOG.debug(_(\u0027type is \u003d %s\u0027) % type)"},{"line_number":74,"context_line":"    if \u0027name_description\u0027 in params:"},{"line_number":75,"context_line":"        desc \u003d params[\u0027name_description\u0027]"},{"line_number":76,"context_line":"        LOG.debug(_(\u0027name \u003d %s\u0027) % desc)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F2Eg%3D","line":73,"in_reply_to":"AAAAMX%2F%2F2Ks%3D","updated":"2012-12-12 22:56:47.000000000","message":"Done","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"22543d74101754c72014f7539dc4210563371ad2","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    # If the sr_type is missing, we assume we are"},{"line_number":71,"context_line":"    # using the default iscsi back-end"},{"line_number":72,"context_line":"    type \u003d params.get(\u0027sr_type\u0027) or \u0027iscsi\u0027"},{"line_number":73,"context_line":"    LOG.debug(_(\u0027type is \u003d %s\u0027) % type)"},{"line_number":74,"context_line":"    if \u0027name_description\u0027 in params:"},{"line_number":75,"context_line":"        desc \u003d params[\u0027name_description\u0027]"},{"line_number":76,"context_line":"        LOG.debug(_(\u0027name \u003d %s\u0027) % desc)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F2JY%3D","line":73,"in_reply_to":"AAAAMX%2F%2F2Ks%3D","updated":"2012-12-12 21:29:44.000000000","message":"thanks Kevin, will do!","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"1f6c4259cbf98f7b6a1ba30bb805fc9a9e75cc98","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    # If the sr_type is missing, we assume we are"},{"line_number":71,"context_line":"    # using the default iscsi back-end"},{"line_number":72,"context_line":"    type \u003d params.get(\u0027sr_type\u0027) or \u0027iscsi\u0027"},{"line_number":73,"context_line":"    LOG.debug(_(\u0027type is \u003d %s\u0027) % type)"},{"line_number":74,"context_line":"    if \u0027name_description\u0027 in params:"},{"line_number":75,"context_line":"        desc \u003d params[\u0027name_description\u0027]"},{"line_number":76,"context_line":"        LOG.debug(_(\u0027name \u003d %s\u0027) % desc)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F2Ks%3D","line":73,"in_reply_to":"AAAAMX%2F%2F3sw%3D","updated":"2012-12-12 21:21:48.000000000","message":"I see several other things that delete values; I suspect it could confuse the SR.introduce call later on.  I suggest doing:\n\n    type \u003d params.pop(\u0027sr_type\u0027, \u0027iscsi\u0027)","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"f2e7737942f4ba74fec6878aa3b42fc4e9b2cd01","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    # If the sr_type is missing, we assume we are"},{"line_number":71,"context_line":"    # using the default iscsi back-end"},{"line_number":72,"context_line":"    type \u003d params.get(\u0027sr_type\u0027) or \u0027iscsi\u0027"},{"line_number":73,"context_line":"    LOG.debug(_(\u0027type is \u003d %s\u0027) % type)"},{"line_number":74,"context_line":"    if \u0027name_description\u0027 in params:"},{"line_number":75,"context_line":"        desc \u003d params[\u0027name_description\u0027]"},{"line_number":76,"context_line":"        LOG.debug(_(\u0027name \u003d %s\u0027) % desc)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3sw%3D","line":73,"in_reply_to":"AAAAMX%2F%2F3x0%3D","updated":"2012-12-11 16:50:27.000000000","message":"I removed because I don\u0027t see why it is needed. Can you be more specific?","commit_id":"5cabf348debb8c6e9c4ace1a19cce064d2a87b3c"}]}