)]}'
{"glance/api/policy.py":[{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"1f46a977e8cf9192c1f146c43c7b49a6d66f8c58","unresolved":false,"context_lines":[{"line_number":200,"context_line":"        self.context \u003d context"},{"line_number":201,"context_line":"        self.policy \u003d policy"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"    def new_image(self, image_id\u003dNone, name\u003dNone, visibility\u003d\u0027private\u0027,"},{"line_number":204,"context_line":"                  min_disk\u003d0, min_ram\u003d0, protected\u003dFalse, owner\u003dNone,"},{"line_number":205,"context_line":"                  disk_format\u003dNone, container_format\u003dNone,"},{"line_number":206,"context_line":"                  extra_properties\u003dNone, tags\u003dNone, **other_args):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMH%2F%2Fx7A%3D","line":203,"updated":"2012-11-25 19:59:49.000000000","message":"I\u0027m not a fan of defining all of these here, especially when this module doesn\u0027t own what it means to create an image. Can you just pass \"*args, **kwargs\" through?","commit_id":"0ddf66a6cf2a1564eb693079527320f14499e43b"},{"author":{"_account_id":616,"name":"Mark Washenberger","email":"mark.washenberger@markwash.net","username":"markwash"},"change_message_id":"c7fff3e5cde8dde7120c5b2df3e083f567b21194","unresolved":false,"context_lines":[{"line_number":200,"context_line":"        self.context \u003d context"},{"line_number":201,"context_line":"        self.policy \u003d policy"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"    def new_image(self, image_id\u003dNone, name\u003dNone, visibility\u003d\u0027private\u0027,"},{"line_number":204,"context_line":"                  min_disk\u003d0, min_ram\u003d0, protected\u003dFalse, owner\u003dNone,"},{"line_number":205,"context_line":"                  disk_format\u003dNone, container_format\u003dNone,"},{"line_number":206,"context_line":"                  extra_properties\u003dNone, tags\u003dNone, **other_args):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMH%2F%2Fxn0%3D","line":203,"in_reply_to":"AAAAMH%2F%2Fx7A%3D","updated":"2012-11-26 07:14:56.000000000","message":"I agree that it is pretty gross.\n\nThe thought here was to duplicate the full list so that positional and keyword arguments would work exactly as expected if you were using the underlying image factory. However, its a real pain--I think we can make it all much cleaner if we establish the convention that the new_image() function only takes keyword arguments. Then we can do **kwargs as you suggest.","commit_id":"0ddf66a6cf2a1564eb693079527320f14499e43b"}]}