)]}'
{"contrib/rackspace/heat/engine/plugins/clients.py":[{"author":{"_account_id":7230,"name":"Vijendar Komalla","email":"vijendar.komalla@rackspace.com","username":"vijendar"},"change_message_id":"ae29b9f130751be58fc17c88b0a6acfdce7e036a","unresolved":false,"context_lines":[{"line_number":38,"context_line":"cloud_opts \u003d ["},{"line_number":39,"context_line":"    cfg.StrOpt(\u0027region_name\u0027,"},{"line_number":40,"context_line":"               default\u003dNone,"},{"line_number":41,"context_line":"               help\u003d\u0027Region for connecting to services\u0027)"},{"line_number":42,"context_line":"]"},{"line_number":43,"context_line":"cfg.CONF.register_opts(cloud_opts)"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FbE4%3D","line":41,"updated":"2013-11-06 15:34:02.000000000","message":"include help message in _(\u0027 \u0027) for translation/localization.","commit_id":"8635e9d9c56fdbde162e6d0d3f5d1c4b75b6d936"},{"author":{"_account_id":7395,"name":"Andrew Plunk","email":"andrew.plunk@rackspace.com","username":"aplunk"},"change_message_id":"2f007211915665a68fa24118052eb1d790b10577","unresolved":false,"context_lines":[{"line_number":38,"context_line":"cloud_opts \u003d ["},{"line_number":39,"context_line":"    cfg.StrOpt(\u0027region_name\u0027,"},{"line_number":40,"context_line":"               default\u003dNone,"},{"line_number":41,"context_line":"               help\u003d\u0027Region for connecting to services\u0027)"},{"line_number":42,"context_line":"]"},{"line_number":43,"context_line":"cfg.CONF.register_opts(cloud_opts)"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2Fa14%3D","line":41,"in_reply_to":"AAAATn%2F%2FbE4%3D","updated":"2013-11-06 17:07:41.000000000","message":"Done","commit_id":"8635e9d9c56fdbde162e6d0d3f5d1c4b75b6d936"},{"author":{"_account_id":7230,"name":"Vijendar Komalla","email":"vijendar.komalla@rackspace.com","username":"vijendar"},"change_message_id":"ae29b9f130751be58fc17c88b0a6acfdce7e036a","unresolved":false,"context_lines":[{"line_number":75,"context_line":"    def neutron(self):"},{"line_number":76,"context_line":"        \u0027\u0027\u0027Rackspace neutron client.\u0027\u0027\u0027"},{"line_number":77,"context_line":"        return self._get_client(\"network\")"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"    def __authenticate(self):"},{"line_number":80,"context_line":"        # current implemenation shown below authenticates using"},{"line_number":81,"context_line":"        # username and password. Need make it work with auth-token"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FbCg%3D","line":78,"updated":"2013-11-06 15:34:02.000000000","message":"Here I think we need to override cinder() client as well?!","commit_id":"8635e9d9c56fdbde162e6d0d3f5d1c4b75b6d936"},{"author":{"_account_id":7395,"name":"Andrew Plunk","email":"andrew.plunk@rackspace.com","username":"aplunk"},"change_message_id":"2f007211915665a68fa24118052eb1d790b10577","unresolved":false,"context_lines":[{"line_number":75,"context_line":"    def neutron(self):"},{"line_number":76,"context_line":"        \u0027\u0027\u0027Rackspace neutron client.\u0027\u0027\u0027"},{"line_number":77,"context_line":"        return self._get_client(\"network\")"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"    def __authenticate(self):"},{"line_number":80,"context_line":"        # current implemenation shown below authenticates using"},{"line_number":81,"context_line":"        # username and password. Need make it work with auth-token"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2Fa2A%3D","line":78,"in_reply_to":"AAAATn%2F%2FbCg%3D","updated":"2013-11-06 17:07:41.000000000","message":"We are just using the cinder client from the OpenStackClients here so it is not really necessary.","commit_id":"8635e9d9c56fdbde162e6d0d3f5d1c4b75b6d936"},{"author":{"_account_id":7253,"name":"Jason Dunsmore","email":"jasondunsmore@gmail.com","username":"jbd"},"change_message_id":"3719623a4a09091877af3271542fe6f90bf21990","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from oslo.config import cfg"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from heat.openstack.common import log as logging"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"logger \u003d logging.getLogger(__name__)"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2Fa1I%3D","line":18,"updated":"2013-11-06 17:16:42.000000000","message":"Don\u0027t forget:\n\n from heat.openstack.common.gettextutils import _","commit_id":"2204cc7ae9aa9cac0df03d96aa2deec41b5e31c5"},{"author":{"_account_id":7395,"name":"Andrew Plunk","email":"andrew.plunk@rackspace.com","username":"aplunk"},"change_message_id":"27c6114ab46795bbe1c0f5c60ae1869f10a2c503","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from oslo.config import cfg"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from heat.openstack.common import log as logging"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"logger \u003d logging.getLogger(__name__)"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2Fayg%3D","line":18,"in_reply_to":"AAAATn%2F%2Fa1I%3D","updated":"2013-11-06 18:57:27.000000000","message":"I don\u0027t think that is actually necessary. I will add it to be explicit, however I see examples in the codebase (like the cloud_dns resource) where we do not import _.","commit_id":"2204cc7ae9aa9cac0df03d96aa2deec41b5e31c5"},{"author":{"_account_id":7253,"name":"Jason Dunsmore","email":"jasondunsmore@gmail.com","username":"jbd"},"change_message_id":"3719623a4a09091877af3271542fe6f90bf21990","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        \u0027\u0027\u0027Rackspace cloud dns client.\u0027\u0027\u0027"},{"line_number":69,"context_line":"        return self._get_client(\"dns\")"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def nova(self, service_type\u003d\"compute\"):"},{"line_number":72,"context_line":"        \u0027\u0027\u0027Rackspace cloudservers client.\u0027\u0027\u0027"},{"line_number":73,"context_line":"        return self._get_client(service_type)"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2Fa0g%3D","line":71,"updated":"2013-11-06 17:16:42.000000000","message":"What other possible values of service_type are there?  Maybe list them in the docstring?","commit_id":"2204cc7ae9aa9cac0df03d96aa2deec41b5e31c5"},{"author":{"_account_id":7395,"name":"Andrew Plunk","email":"andrew.plunk@rackspace.com","username":"aplunk"},"change_message_id":"27c6114ab46795bbe1c0f5c60ae1869f10a2c503","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        \u0027\u0027\u0027Rackspace cloud dns client.\u0027\u0027\u0027"},{"line_number":69,"context_line":"        return self._get_client(\"dns\")"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def nova(self, service_type\u003d\"compute\"):"},{"line_number":72,"context_line":"        \u0027\u0027\u0027Rackspace cloudservers client.\u0027\u0027\u0027"},{"line_number":73,"context_line":"        return self._get_client(service_type)"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2Faqs%3D","line":71,"in_reply_to":"AAAATn%2F%2Fa0g%3D","updated":"2013-11-06 18:57:27.000000000","message":"This was added to maintain compatibility with the OpenStackClients class. It is necessary to do this because when resource.py sets up the nova client, it expects nova to accept the service_type parameter. The following patch makes this clear.","commit_id":"2204cc7ae9aa9cac0df03d96aa2deec41b5e31c5"},{"author":{"_account_id":7256,"name":"Randall Burt","email":"randall.burt@rackspace.com","username":"randall-burt"},"change_message_id":"446d249a38039ada4cce3dcd73b14c5a2c272f78","unresolved":false,"context_lines":[{"line_number":26,"context_line":"try:"},{"line_number":27,"context_line":"    import pyrax"},{"line_number":28,"context_line":"except ImportError:"},{"line_number":29,"context_line":"    logger.info(\u0027pyrax not available\u0027)"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"try:"},{"line_number":32,"context_line":"    from swiftclient import client as swiftclient"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2BkzM%3D","line":29,"updated":"2013-11-22 22:25:46.000000000","message":"Should add:\n  pyrax \u003d None","commit_id":"a27b4b95056e242c319ffa5b3f660aaa27a8616f"}]}