)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"c611b5f2a03fbeae1aac42a60ca6f28d77644d86","unresolved":false,"context_lines":[{"line_number":11,"context_line":"to use trusts we had to specify it by default. Otherwise, many"},{"line_number":12,"context_line":"installations could be broken. But we don\u0027t need to specify"},{"line_number":13,"context_line":"these defaults if user domain name or id is specified already."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes-Bug: #1620999"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I70cb27fe893dd51b6c8c123bd535c168b7169d70"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5a8bc5a2_6717e2a3","line":14,"updated":"2016-09-22 15:13:55.000000000","message":"I think you can add a line that says in which case the default is specified and what the value is (i.e. \u0027default\u0027).\n\nAlso, please mention deprecation too.","commit_id":"d04ae783be664daa1dc5d801bf9991b12961df5c"}],"glance_store/_drivers/swift/utils.py":[{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"c611b5f2a03fbeae1aac42a60ca6f28d77644d86","unresolved":false,"context_lines":[{"line_number":178,"context_line":"                    av \u003d self.conf.glance_store.swift_store_auth_version"},{"line_number":179,"context_line":"                    reference[\u0027auth_version\u0027] \u003d av"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"                if (reference[\u0027project_domain_id\u0027] is None and"},{"line_number":182,"context_line":"                   reference[\u0027project_domain_name\u0027] is None):"},{"line_number":183,"context_line":"                    reference[\u0027project_domain_id\u0027] \u003d \u0027default\u0027"},{"line_number":184,"context_line":"                    LOG.warn(_LW("}],"source_content_type":"text/x-python","patch_set":2,"id":"5a8bc5a2_e41db010","line":181,"updated":"2016-09-22 15:13:55.000000000","message":"I like what you\u0027ve proposed but I would prefer this sub-block to be in a separate private method that\u0027s deprecated using debtcollector http://docs.openstack.org/developer/debtcollector/examples.html#removing-a-class-classmethod-method-function","commit_id":"d04ae783be664daa1dc5d801bf9991b12961df5c"}]}