)]}'
{"nova/api/metadata/handler.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ae5903874dd911a31fd57898e6aeeb365b442c6a","unresolved":false,"context_lines":[{"line_number":269,"context_line":"                provider_id, instance_address)"},{"line_number":270,"context_line":"            if CONF.api.metadata_cache_expiration \u003e 0:"},{"line_number":271,"context_line":"                self._cache.set(cache_key, (instance_id, tenant_id))"},{"line_number":272,"context_line":"        LOG.debug(\u0027Instance %s with address %s matches provider %s\u0027,"},{"line_number":273,"context_line":"                  instance_id, remote_address, provider_id)"},{"line_number":274,"context_line":"        return self._get_meta_by_instance_id(instance_id, tenant_id,"},{"line_number":275,"context_line":"                                             instance_address)"},{"line_number":276,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"ffb9cba7_1d45a317","line":273,"range":{"start_line":272,"start_character":8,"end_line":273,"end_character":59},"updated":"2019-04-25 15:21:00.000000000","message":"Should we nest this under the else block on L267? Otherwise if we pull the value from the cache, we\u0027re logging that and then logging this as well, which seems sort of redundant.","commit_id":"ed7bce6e2af80bb84ccf86fb6208b7fe272a83a6"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"aaea49be15318c54452124de5ad592f9bca1eec9","unresolved":false,"context_lines":[{"line_number":269,"context_line":"                provider_id, instance_address)"},{"line_number":270,"context_line":"            if CONF.api.metadata_cache_expiration \u003e 0:"},{"line_number":271,"context_line":"                self._cache.set(cache_key, (instance_id, tenant_id))"},{"line_number":272,"context_line":"        LOG.debug(\u0027Instance %s with address %s matches provider %s\u0027,"},{"line_number":273,"context_line":"                  instance_id, remote_address, provider_id)"},{"line_number":274,"context_line":"        return self._get_meta_by_instance_id(instance_id, tenant_id,"},{"line_number":275,"context_line":"                                             instance_address)"},{"line_number":276,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"dfbec78f_7788d21e","line":273,"range":{"start_line":272,"start_character":8,"end_line":273,"end_character":59},"in_reply_to":"ffb9cba7_1d45a317","updated":"2019-05-03 16:51:11.000000000","message":"Done","commit_id":"ed7bce6e2af80bb84ccf86fb6208b7fe272a83a6"}]}