)]}'
{"heat/db/sqlalchemy/api.py":[{"author":{"_account_id":7385,"name":"Thomas Herve","email":"therve@redhat.com","username":"therve"},"change_message_id":"97130e976bd2b22aa96ed1eb905127bac6233cd3","unresolved":false,"context_lines":[{"line_number":127,"context_line":"    for result in results:"},{"line_number":128,"context_line":"        if context is None or context.tenant_id in ("},{"line_number":129,"context_line":"                result.stack.tenant, result.stack.stack_user_project_id):"},{"line_number":130,"context_line":"            return result"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"    return None"},{"line_number":133,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7adec928_4a589932","line":130,"updated":"2014-05-09 06:21:34.000000000","message":"It\u0027d be nice to have a direct test for that.","commit_id":"63f1cfdaa1d06b524a16c34355016db4e3881114"}],"heat/engine/resources/server.py":[{"author":{"_account_id":7385,"name":"Thomas Herve","email":"therve@redhat.com","username":"therve"},"change_message_id":"97130e976bd2b22aa96ed1eb905127bac6233cd3","unresolved":false,"context_lines":[{"line_number":346,"context_line":"        return self.properties.get(self.CONFIG_DRIVE)"},{"line_number":347,"context_line":""},{"line_number":348,"context_line":"    def _populate_deployments_metadata(self):"},{"line_number":349,"context_line":"        meta \u003d self.metadata_get(True) or {}"},{"line_number":350,"context_line":"        meta[\u0027deployments\u0027] \u003d meta.get(\u0027deployments\u0027, [])"},{"line_number":351,"context_line":"        if self.transport_poll_server_heat():"},{"line_number":352,"context_line":"            meta[\u0027os-collect-config\u0027] \u003d {\u0027heat\u0027: {"}],"source_content_type":"text/x-python","patch_set":2,"id":"7adec928_aa3e7db6","line":349,"updated":"2014-05-09 06:21:34.000000000","message":"I think there is some guarantee that metadata_get always return a dict. If not there should be.","commit_id":"63f1cfdaa1d06b524a16c34355016db4e3881114"},{"author":{"_account_id":7385,"name":"Thomas Herve","email":"therve@redhat.com","username":"therve"},"change_message_id":"97130e976bd2b22aa96ed1eb905127bac6233cd3","unresolved":false,"context_lines":[{"line_number":347,"context_line":""},{"line_number":348,"context_line":"    def _populate_deployments_metadata(self):"},{"line_number":349,"context_line":"        meta \u003d self.metadata_get(True) or {}"},{"line_number":350,"context_line":"        meta[\u0027deployments\u0027] \u003d meta.get(\u0027deployments\u0027, [])"},{"line_number":351,"context_line":"        if self.transport_poll_server_heat():"},{"line_number":352,"context_line":"            meta[\u0027os-collect-config\u0027] \u003d {\u0027heat\u0027: {"},{"line_number":353,"context_line":"                \u0027user_id\u0027: self._get_user_id(),"}],"source_content_type":"text/x-python","patch_set":2,"id":"7adec928_8a41c13b","line":350,"updated":"2014-05-09 06:21:34.000000000","message":"It\u0027d be more elegantly written meta.setdefault(\u0027deployments\u0027, []). OTOH, I don\u0027t think there can be deployments present at creation time, so meta[\u0027deployments\u0027] \u003d [] would do the trick to. Otherwise a test would be nice.","commit_id":"63f1cfdaa1d06b524a16c34355016db4e3881114"}]}