)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6456,"name":"Sahdev Zala","email":"spzala@us.ibm.com","username":"spzala"},"change_message_id":"a274ea876ca8149057a3e9633e72d141bf0d0123","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Adds ability to configure various clients used by the Heat"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This commit adds config sections [clients_nova], [clients_swift],"},{"line_number":10,"context_line":"[clients_neutron], [clients_cinder] and [clients_ceilometer]."},{"line_number":11,"context_line":"These sections contain additional configuration"},{"line_number":12,"context_line":"options for corresponding OpenStack clients."},{"line_number":13,"context_line":"Currently those are only SSL-related setting ca_file, cert_file,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"AAAATn%2F%2BuE8%3D","line":10,"updated":"2013-11-21 04:36:02.000000000","message":"Seems like you have missed adding [clients_keystone] here.","commit_id":"c14046d3044214ff1542b2734b4ee73da2248752"},{"author":{"_account_id":6456,"name":"Sahdev Zala","email":"spzala@us.ibm.com","username":"spzala"},"change_message_id":"d86c3b3456eaf9ce67ccbd0a32924b0eeaea2228","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Adds ability to configure various clients used by the Heat"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This commit adds config sections [clients_nova], [clients_swift],"},{"line_number":10,"context_line":"[clients_neutron], [clients_cinder] and [clients_ceilometer]."},{"line_number":11,"context_line":"These sections contain additional configuration"},{"line_number":12,"context_line":"options for corresponding OpenStack clients."},{"line_number":13,"context_line":"Currently those are only SSL-related setting ca_file, cert_file,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"AAAATn%2F%2BrnU%3D","line":10,"in_reply_to":"AAAATn%2F%2Btjk%3D","updated":"2013-11-21 16:06:35.000000000","message":"I would say No but if another patch is required for any major change then it\u0027s something you can consider updating. Thanks!","commit_id":"c14046d3044214ff1542b2734b4ee73da2248752"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"319be286d3fb5cf46b22e022edeb5370b2a8946a","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Adds ability to configure various clients used by the Heat"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This commit adds config sections [clients_nova], [clients_swift],"},{"line_number":10,"context_line":"[clients_neutron], [clients_cinder] and [clients_ceilometer]."},{"line_number":11,"context_line":"These sections contain additional configuration"},{"line_number":12,"context_line":"options for corresponding OpenStack clients."},{"line_number":13,"context_line":"Currently those are only SSL-related setting ca_file, cert_file,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"AAAATn%2F%2Bqvs%3D","line":10,"in_reply_to":"AAAATn%2F%2Btjk%3D","updated":"2013-11-21 19:50:04.000000000","message":"yes please fix - changelogs are important to a variety of folks and incorrect changelogs make tracking changes more difficult","commit_id":"c14046d3044214ff1542b2734b4ee73da2248752"},{"author":{"_account_id":7226,"name":"Stan Lagun","email":"stan.lagun@gmail.com","username":"istalker2"},"change_message_id":"63f7644de384bf32645ea707ca2c89a0fbf660e7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Adds ability to configure various clients used by the Heat"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This commit adds config sections [clients_nova], [clients_swift],"},{"line_number":10,"context_line":"[clients_neutron], [clients_cinder] and [clients_ceilometer]."},{"line_number":11,"context_line":"These sections contain additional configuration"},{"line_number":12,"context_line":"options for corresponding OpenStack clients."},{"line_number":13,"context_line":"Currently those are only SSL-related setting ca_file, cert_file,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"AAAATn%2F%2Btjk%3D","line":10,"in_reply_to":"AAAATn%2F%2BuE8%3D","updated":"2013-11-21 07:55:19.000000000","message":"Yes, you\u0027re right. Should I fix it considering that it would require another round of code review?","commit_id":"c14046d3044214ff1542b2734b4ee73da2248752"}],"heat/common/config.py":[{"author":{"_account_id":7230,"name":"Vijendar Komalla","email":"vijendar.komalla@rackspace.com","username":"vijendar"},"change_message_id":"d3e028696a480105b7aece3b37310be33311a2f7","unresolved":false,"context_lines":[{"line_number":126,"context_line":"                       \u0027to be specified.\u0027))]"},{"line_number":127,"context_line":"clients_opts \u003d ["},{"line_number":128,"context_line":"    cfg.StrOpt(\u0027ca_file\u0027,"},{"line_number":129,"context_line":"               help\u003d\u0027Optional CA cert file to use in SSL connections\u0027),"},{"line_number":130,"context_line":"    cfg.StrOpt(\u0027cert_file\u0027,"},{"line_number":131,"context_line":"               help\u003d\u0027Optional PEM-formatted certificate chain file\u0027),"},{"line_number":132,"context_line":"    cfg.StrOpt(\u0027key_file\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2Fmzs%3D","line":129,"updated":"2013-10-31 14:38:59.000000000","message":"include help text in _(\u0027 \u0027)","commit_id":"35e1781587b33587fcf41bcaee0526deb0ea0d2c"},{"author":{"_account_id":7230,"name":"Vijendar Komalla","email":"vijendar.komalla@rackspace.com","username":"vijendar"},"change_message_id":"d3e028696a480105b7aece3b37310be33311a2f7","unresolved":false,"context_lines":[{"line_number":128,"context_line":"    cfg.StrOpt(\u0027ca_file\u0027,"},{"line_number":129,"context_line":"               help\u003d\u0027Optional CA cert file to use in SSL connections\u0027),"},{"line_number":130,"context_line":"    cfg.StrOpt(\u0027cert_file\u0027,"},{"line_number":131,"context_line":"               help\u003d\u0027Optional PEM-formatted certificate chain file\u0027),"},{"line_number":132,"context_line":"    cfg.StrOpt(\u0027key_file\u0027,"},{"line_number":133,"context_line":"               help\u003d\u0027Optional PEM-formatted file that contains the \u0027"},{"line_number":134,"context_line":"                    \u0027private key\u0027),"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2Fmzo%3D","line":131,"updated":"2013-10-31 14:38:59.000000000","message":"same here and below!","commit_id":"35e1781587b33587fcf41bcaee0526deb0ea0d2c"}],"heat/engine/clients.py":[{"author":{"_account_id":7395,"name":"Andrew Plunk","email":"andrew.plunk@rackspace.com","username":"aplunk"},"change_message_id":"e46fc409badef9f678201148b800e3dc9e5ba543","unresolved":false,"context_lines":[{"line_number":103,"context_line":"            \u0027service_type\u0027: service_type,"},{"line_number":104,"context_line":"            \u0027username\u0027: None,"},{"line_number":105,"context_line":"            \u0027api_key\u0027: None,"},{"line_number":106,"context_line":"            \u0027extensions\u0027: extensions,"},{"line_number":107,"context_line":"            \u0027cacert\u0027: self._get_client_option(\u0027nova\u0027, \u0027ca_file\u0027),"},{"line_number":108,"context_line":"            \u0027insecure\u0027: self._get_client_option(\u0027nova\u0027, \u0027insecure\u0027)"},{"line_number":109,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FlYk%3D","line":106,"updated":"2013-10-31 21:54:04.000000000","message":"Why are cert_file and key_file not added here?","commit_id":"adb67448727207fb887873b444d37cfd0709b976"},{"author":{"_account_id":7226,"name":"Stan Lagun","email":"stan.lagun@gmail.com","username":"istalker2"},"change_message_id":"f7386b0148a154b20996fd4706507912a809b3b4","unresolved":false,"context_lines":[{"line_number":103,"context_line":"            \u0027service_type\u0027: service_type,"},{"line_number":104,"context_line":"            \u0027username\u0027: None,"},{"line_number":105,"context_line":"            \u0027api_key\u0027: None,"},{"line_number":106,"context_line":"            \u0027extensions\u0027: extensions,"},{"line_number":107,"context_line":"            \u0027cacert\u0027: self._get_client_option(\u0027nova\u0027, \u0027ca_file\u0027),"},{"line_number":108,"context_line":"            \u0027insecure\u0027: self._get_client_option(\u0027nova\u0027, \u0027insecure\u0027)"},{"line_number":109,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FEAM%3D","line":106,"in_reply_to":"AAAATn%2F%2FlYk%3D","updated":"2013-11-15 00:11:18.000000000","message":"Because nova client doesn\u0027t support those parameters. Take a look at nova-client\u0027s constructor","commit_id":"adb67448727207fb887873b444d37cfd0709b976"},{"author":{"_account_id":7253,"name":"Jason Dunsmore","email":"jasondunsmore@gmail.com","username":"jbd"},"change_message_id":"104e9e47b80bcdbb8ba4133354a96d90658d9d04","unresolved":false,"context_lines":[{"line_number":104,"context_line":"            \u0027username\u0027: None,"},{"line_number":105,"context_line":"            \u0027api_key\u0027: None,"},{"line_number":106,"context_line":"            \u0027extensions\u0027: extensions,"},{"line_number":107,"context_line":"            \u0027cacert\u0027: self._get_client_option(\u0027nova\u0027, \u0027ca_file\u0027),"},{"line_number":108,"context_line":"            \u0027insecure\u0027: self._get_client_option(\u0027nova\u0027, \u0027insecure\u0027)"},{"line_number":109,"context_line":"        }"},{"line_number":110,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FiSI%3D","line":107,"updated":"2013-11-01 20:19:36.000000000","message":"Shouldn\u0027t this be:\n\n \u0027ca_file\u0027: self._get_client_option(\u0027nova\u0027, \u0027ca_file\u0027),\n\n?","commit_id":"adb67448727207fb887873b444d37cfd0709b976"},{"author":{"_account_id":7226,"name":"Stan Lagun","email":"stan.lagun@gmail.com","username":"istalker2"},"change_message_id":"f7386b0148a154b20996fd4706507912a809b3b4","unresolved":false,"context_lines":[{"line_number":104,"context_line":"            \u0027username\u0027: None,"},{"line_number":105,"context_line":"            \u0027api_key\u0027: None,"},{"line_number":106,"context_line":"            \u0027extensions\u0027: extensions,"},{"line_number":107,"context_line":"            \u0027cacert\u0027: self._get_client_option(\u0027nova\u0027, \u0027ca_file\u0027),"},{"line_number":108,"context_line":"            \u0027insecure\u0027: self._get_client_option(\u0027nova\u0027, \u0027insecure\u0027)"},{"line_number":109,"context_line":"        }"},{"line_number":110,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FEAA%3D","line":107,"in_reply_to":"AAAATn%2F%2FiSI%3D","updated":"2013-11-15 00:11:18.000000000","message":"nova-client\u0027s constructor parameter is called cacert, not ca_file","commit_id":"adb67448727207fb887873b444d37cfd0709b976"},{"author":{"_account_id":7395,"name":"Andrew Plunk","email":"andrew.plunk@rackspace.com","username":"aplunk"},"change_message_id":"e46fc409badef9f678201148b800e3dc9e5ba543","unresolved":false,"context_lines":[{"line_number":135,"context_line":"            \u0027key\u0027: None,"},{"line_number":136,"context_line":"            \u0027authurl\u0027: None,"},{"line_number":137,"context_line":"            \u0027preauthtoken\u0027: self.auth_token,"},{"line_number":138,"context_line":"            \u0027preauthurl\u0027: self.url_for(service_type\u003d\u0027object-store\u0027),"},{"line_number":139,"context_line":"            \u0027cacert\u0027: self._get_client_option(\u0027swift\u0027, \u0027ca_file\u0027),"},{"line_number":140,"context_line":"            \u0027insecure\u0027: self._get_client_option(\u0027swift\u0027, \u0027insecure\u0027)"},{"line_number":141,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FlYg%3D","line":138,"updated":"2013-10-31 21:54:04.000000000","message":"ditto","commit_id":"adb67448727207fb887873b444d37cfd0709b976"},{"author":{"_account_id":7253,"name":"Jason Dunsmore","email":"jasondunsmore@gmail.com","username":"jbd"},"change_message_id":"104e9e47b80bcdbb8ba4133354a96d90658d9d04","unresolved":false,"context_lines":[{"line_number":136,"context_line":"            \u0027authurl\u0027: None,"},{"line_number":137,"context_line":"            \u0027preauthtoken\u0027: self.auth_token,"},{"line_number":138,"context_line":"            \u0027preauthurl\u0027: self.url_for(service_type\u003d\u0027object-store\u0027),"},{"line_number":139,"context_line":"            \u0027cacert\u0027: self._get_client_option(\u0027swift\u0027, \u0027ca_file\u0027),"},{"line_number":140,"context_line":"            \u0027insecure\u0027: self._get_client_option(\u0027swift\u0027, \u0027insecure\u0027)"},{"line_number":141,"context_line":"        }"},{"line_number":142,"context_line":"        self._swift \u003d swiftclient.Connection(**args)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FiNg%3D","line":139,"updated":"2013-11-01 20:19:36.000000000","message":"I haven\u0027t checked the swift option, but assuming it\u0027s \u0027ca_file\u0027, shouldn\u0027t this be:\n\n \u0027ca_file\u0027: self._get_client_option(\u0027swift\u0027, \u0027ca_file\u0027),\n\n?","commit_id":"adb67448727207fb887873b444d37cfd0709b976"},{"author":{"_account_id":7226,"name":"Stan Lagun","email":"stan.lagun@gmail.com","username":"istalker2"},"change_message_id":"f7386b0148a154b20996fd4706507912a809b3b4","unresolved":false,"context_lines":[{"line_number":136,"context_line":"            \u0027authurl\u0027: None,"},{"line_number":137,"context_line":"            \u0027preauthtoken\u0027: self.auth_token,"},{"line_number":138,"context_line":"            \u0027preauthurl\u0027: self.url_for(service_type\u003d\u0027object-store\u0027),"},{"line_number":139,"context_line":"            \u0027cacert\u0027: self._get_client_option(\u0027swift\u0027, \u0027ca_file\u0027),"},{"line_number":140,"context_line":"            \u0027insecure\u0027: self._get_client_option(\u0027swift\u0027, \u0027insecure\u0027)"},{"line_number":141,"context_line":"        }"},{"line_number":142,"context_line":"        self._swift \u003d swiftclient.Connection(**args)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FD%2FQ%3D","line":139,"in_reply_to":"AAAATn%2F%2FiNg%3D","updated":"2013-11-15 00:11:18.000000000","message":"it is cacert: \nhttps://github.com/openstack/python-swiftclient/blob/master/swiftclient/client.py#L1028","commit_id":"adb67448727207fb887873b444d37cfd0709b976"},{"author":{"_account_id":7395,"name":"Andrew Plunk","email":"andrew.plunk@rackspace.com","username":"aplunk"},"change_message_id":"e46fc409badef9f678201148b800e3dc9e5ba543","unresolved":false,"context_lines":[{"line_number":157,"context_line":"            \u0027auth_url\u0027: con.auth_url,"},{"line_number":158,"context_line":"            \u0027service_type\u0027: \u0027network\u0027,"},{"line_number":159,"context_line":"            \u0027token\u0027: self.auth_token,"},{"line_number":160,"context_line":"            \u0027endpoint_url\u0027: self.url_for(service_type\u003d\u0027network\u0027),"},{"line_number":161,"context_line":"            \u0027ca_cert\u0027: self._get_client_option(\u0027neutron\u0027, \u0027ca_file\u0027),"},{"line_number":162,"context_line":"            \u0027insecure\u0027: self._get_client_option(\u0027neutron\u0027, \u0027insecure\u0027)"},{"line_number":163,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FlYc%3D","line":160,"updated":"2013-10-31 21:54:04.000000000","message":"\u0027ca_cert\u0027 is different than line 139 \u0027cacert\u0027, was that intended?","commit_id":"adb67448727207fb887873b444d37cfd0709b976"},{"author":{"_account_id":7226,"name":"Stan Lagun","email":"stan.lagun@gmail.com","username":"istalker2"},"change_message_id":"f7386b0148a154b20996fd4706507912a809b3b4","unresolved":false,"context_lines":[{"line_number":157,"context_line":"            \u0027auth_url\u0027: con.auth_url,"},{"line_number":158,"context_line":"            \u0027service_type\u0027: \u0027network\u0027,"},{"line_number":159,"context_line":"            \u0027token\u0027: self.auth_token,"},{"line_number":160,"context_line":"            \u0027endpoint_url\u0027: self.url_for(service_type\u003d\u0027network\u0027),"},{"line_number":161,"context_line":"            \u0027ca_cert\u0027: self._get_client_option(\u0027neutron\u0027, \u0027ca_file\u0027),"},{"line_number":162,"context_line":"            \u0027insecure\u0027: self._get_client_option(\u0027neutron\u0027, \u0027insecure\u0027)"},{"line_number":163,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FD%2F4%3D","line":160,"in_reply_to":"AAAATn%2F%2FlYc%3D","updated":"2013-11-15 00:11:18.000000000","message":"Yes, neutron\u0027s client calls this ca_cert, not cacert","commit_id":"adb67448727207fb887873b444d37cfd0709b976"},{"author":{"_account_id":7253,"name":"Jason Dunsmore","email":"jasondunsmore@gmail.com","username":"jbd"},"change_message_id":"104e9e47b80bcdbb8ba4133354a96d90658d9d04","unresolved":false,"context_lines":[{"line_number":158,"context_line":"            \u0027service_type\u0027: \u0027network\u0027,"},{"line_number":159,"context_line":"            \u0027token\u0027: self.auth_token,"},{"line_number":160,"context_line":"            \u0027endpoint_url\u0027: self.url_for(service_type\u003d\u0027network\u0027),"},{"line_number":161,"context_line":"            \u0027ca_cert\u0027: self._get_client_option(\u0027neutron\u0027, \u0027ca_file\u0027),"},{"line_number":162,"context_line":"            \u0027insecure\u0027: self._get_client_option(\u0027neutron\u0027, \u0027insecure\u0027)"},{"line_number":163,"context_line":"        }"},{"line_number":164,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FiNM%3D","line":161,"updated":"2013-11-01 20:19:36.000000000","message":"I haven\u0027t checked the neutron option, but assuming it\u0027s \u0027ca_file\u0027, shouldn\u0027t this be:\n\n \u0027ca_file\u0027: self._get_client_option(\u0027neutron\u0027, \u0027ca_file\u0027),\n\n?","commit_id":"adb67448727207fb887873b444d37cfd0709b976"},{"author":{"_account_id":7253,"name":"Jason Dunsmore","email":"jasondunsmore@gmail.com","username":"jbd"},"change_message_id":"104e9e47b80bcdbb8ba4133354a96d90658d9d04","unresolved":false,"context_lines":[{"line_number":183,"context_line":"            \u0027project_id\u0027: con.tenant,"},{"line_number":184,"context_line":"            \u0027username\u0027: None,"},{"line_number":185,"context_line":"            \u0027api_key\u0027: None,"},{"line_number":186,"context_line":"            \u0027cacert\u0027: self._get_client_option(\u0027cinder\u0027, \u0027ca_file\u0027),"},{"line_number":187,"context_line":"            \u0027insecure\u0027: self._get_client_option(\u0027cinder\u0027, \u0027insecure\u0027)"},{"line_number":188,"context_line":"        }"},{"line_number":189,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FiTY%3D","line":186,"updated":"2013-11-01 20:19:36.000000000","message":"The cinder option is \u0027ssl_ca_file\u0027.  So shouldn\u0027t this be:\n\n \u0027ca_file\u0027: self._get_client_option(\u0027cinder\u0027, \u0027ssl_ca_file\u0027)\n\n?\n\nhttps://github.com/openstack/cinder/blob/master/etc/cinder/cinder.conf.sample#L109","commit_id":"adb67448727207fb887873b444d37cfd0709b976"},{"author":{"_account_id":7226,"name":"Stan Lagun","email":"stan.lagun@gmail.com","username":"istalker2"},"change_message_id":"f7386b0148a154b20996fd4706507912a809b3b4","unresolved":false,"context_lines":[{"line_number":183,"context_line":"            \u0027project_id\u0027: con.tenant,"},{"line_number":184,"context_line":"            \u0027username\u0027: None,"},{"line_number":185,"context_line":"            \u0027api_key\u0027: None,"},{"line_number":186,"context_line":"            \u0027cacert\u0027: self._get_client_option(\u0027cinder\u0027, \u0027ca_file\u0027),"},{"line_number":187,"context_line":"            \u0027insecure\u0027: self._get_client_option(\u0027cinder\u0027, \u0027insecure\u0027)"},{"line_number":188,"context_line":"        }"},{"line_number":189,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FD%2Fk%3D","line":186,"in_reply_to":"AAAATn%2F%2FiTY%3D","updated":"2013-11-15 00:11:18.000000000","message":"This is a parameter for cinder-client, not cinder itself.\nhttps://github.com/openstack/python-cinderclient/blob/master/cinderclient/v2/client.py#L98","commit_id":"adb67448727207fb887873b444d37cfd0709b976"},{"author":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"change_message_id":"a5e92a8c449809a02cd172a8b3e218829c8ac7f2","unresolved":false,"context_lines":[{"line_number":229,"context_line":"            return getattr(getattr(cfg.CONF, group_name), option)"},{"line_number":230,"context_line":"        except (cfg.NoSuchGroupError, cfg.NoSuchOptError):"},{"line_number":231,"context_line":"            cfg.CONF.import_opt(option, \u0027heat.common.config\u0027, group\u003d\u0027clients\u0027)"},{"line_number":232,"context_line":"            return getattr(cfg.CONF.clients, option)"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":""},{"line_number":235,"context_line":"class ClientBackend(object):"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2Bu4A%3D","line":232,"updated":"2013-11-20 23:25:17.000000000","message":"isn\u0027t this basically the same method as the one in heat_keystoneclient but with just a little tweak?","commit_id":"c14046d3044214ff1542b2734b4ee73da2248752"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"319be286d3fb5cf46b22e022edeb5370b2a8946a","unresolved":false,"context_lines":[{"line_number":229,"context_line":"            return getattr(getattr(cfg.CONF, group_name), option)"},{"line_number":230,"context_line":"        except (cfg.NoSuchGroupError, cfg.NoSuchOptError):"},{"line_number":231,"context_line":"            cfg.CONF.import_opt(option, \u0027heat.common.config\u0027, group\u003d\u0027clients\u0027)"},{"line_number":232,"context_line":"            return getattr(cfg.CONF.clients, option)"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":""},{"line_number":235,"context_line":"class ClientBackend(object):"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2Bqt0%3D","line":232,"in_reply_to":"AAAATn%2F%2BtkA%3D","updated":"2013-11-21 19:50:04.000000000","message":"Clint,\n\nLooking around in the code base, i don\u0027t see a good place to consolidate this method.","commit_id":"c14046d3044214ff1542b2734b4ee73da2248752"},{"author":{"_account_id":7226,"name":"Stan Lagun","email":"stan.lagun@gmail.com","username":"istalker2"},"change_message_id":"f67c9c9fccfeca6f695fb337f5fd61625982b70b","unresolved":false,"context_lines":[{"line_number":229,"context_line":"            return getattr(getattr(cfg.CONF, group_name), option)"},{"line_number":230,"context_line":"        except (cfg.NoSuchGroupError, cfg.NoSuchOptError):"},{"line_number":231,"context_line":"            cfg.CONF.import_opt(option, \u0027heat.common.config\u0027, group\u003d\u0027clients\u0027)"},{"line_number":232,"context_line":"            return getattr(cfg.CONF.clients, option)"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":""},{"line_number":235,"context_line":"class ClientBackend(object):"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2BtkA%3D","line":232,"in_reply_to":"AAAATn%2F%2Bu4A%3D","updated":"2013-11-21 07:51:01.000000000","message":"Yes. I\u0027d say that heat_keystoneclient has a simplified version of this as it deals with only one client. Probably this can be moved to some common code but not sure where exactly","commit_id":"c14046d3044214ff1542b2734b4ee73da2248752"}]}