)]}'
{"openstack/object_store/v1/_proxy.py":[{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"76f87f5620c15e29692a585bf1b424647d44691d","unresolved":false,"context_lines":[{"line_number":226,"context_line":"               comprised of the properties on the Object class."},{"line_number":227,"context_line":""},{"line_number":228,"context_line":"        :rtype: ``None``"},{"line_number":229,"context_line":"        \"\"\""},{"line_number":230,"context_line":"        self._update(_obj.Object, value, path_args\u003d{\"container\": container},"},{"line_number":231,"context_line":"                     **attrs)"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa32b979_d1c3bc87","line":229,"updated":"2015-06-17 20:13:40.000000000","message":"Aren\u0027t you going to have to do something like if container \u003d\u003d None: container \u003d value.container?","commit_id":"94122290caf4d8ef5001b9b8aca94766cf180e3f"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"2cf4e99456312f6a6ba40b3059b7c84195584ff1","unresolved":false,"context_lines":[{"line_number":226,"context_line":"               comprised of the properties on the Object class."},{"line_number":227,"context_line":""},{"line_number":228,"context_line":"        :rtype: ``None``"},{"line_number":229,"context_line":"        \"\"\""},{"line_number":230,"context_line":"        self._update(_obj.Object, value, path_args\u003d{\"container\": container},"},{"line_number":231,"context_line":"                     **attrs)"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa32b979_43659fb2","line":229,"in_reply_to":"fa32b979_b12718c9","updated":"2015-06-17 20:30:33.000000000","message":"Oh, wait, you are right in the case of value being an obj.Object. If it\u0027s not (as in just the name), then container will have to be something, otherwise we should bail out.","commit_id":"94122290caf4d8ef5001b9b8aca94766cf180e3f"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"5d975b6c80857709d5ae3ba893f85f771c0eead1","unresolved":false,"context_lines":[{"line_number":226,"context_line":"               comprised of the properties on the Object class."},{"line_number":227,"context_line":""},{"line_number":228,"context_line":"        :rtype: ``None``"},{"line_number":229,"context_line":"        \"\"\""},{"line_number":230,"context_line":"        self._update(_obj.Object, value, path_args\u003d{\"container\": container},"},{"line_number":231,"context_line":"                     **attrs)"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa32b979_b12718c9","line":229,"in_reply_to":"fa32b979_d1c3bc87","updated":"2015-06-17 20:22:57.000000000","message":"value points to the object\u0027s instance/name, container is either a container\u0027s instance or name, so that particular case wouldn\u0027t be right, but this is still broken since it doesn\u0027t handle the several cases we could encounter.","commit_id":"94122290caf4d8ef5001b9b8aca94766cf180e3f"},{"author":{"_account_id":1112,"name":"Everett Toews","email":"everett.toews@rackspace.com","username":"everett-toews"},"change_message_id":"9d749dc1fc30dd7f0a79dec34b5f5d68d91e878c","unresolved":false,"context_lines":[{"line_number":74,"context_line":"            :class:`~openstack.object_store.v1.container.Container`"},{"line_number":75,"context_line":"        :param dict attrs: Keyword arguments which will be used to create"},{"line_number":76,"context_line":"               a :class:`~openstack.object_store.v1.container.Container`,"},{"line_number":77,"context_line":"               comprised of the properties on the Container class."},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        :rtype: ``None``"},{"line_number":80,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"fa32b979_a2a40808","line":77,"updated":"2015-06-22 16:52:41.000000000","message":"If I look at the Container class, I have no idea what properties on that class are relevant to this method.\n\nPlease include at least one example of using this method in the docstring of how to set arbitrary metadata on a container. \n\nThere needs to be some indication that the user would need to prefix a header with X-Container-Meta- for it to be an arbitrary metadata item as per http://developer.openstack.org/api-ref-objectstorage-v1.html#updateContainerMeta","commit_id":"ac252e148d3b9a9b328b4c2fedaa09f1e2a00857"},{"author":{"_account_id":1112,"name":"Everett Toews","email":"everett.toews@rackspace.com","username":"everett-toews"},"change_message_id":"9d749dc1fc30dd7f0a79dec34b5f5d68d91e878c","unresolved":false,"context_lines":[{"line_number":226,"context_line":"        return self._head(_obj.Object, value,"},{"line_number":227,"context_line":"                          path_args\u003d{\"container\": container_name})"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"    def set_object_metadata(self, value, container\u003dNone, **attrs):"},{"line_number":230,"context_line":"        \"\"\"Set metatdata for an object."},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        :param value: The object to set metadata for. Can be either the name"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa32b979_8277843a","line":229,"updated":"2015-06-22 16:52:41.000000000","message":"Same comment as for set_container_metadata() but for objects.","commit_id":"ac252e148d3b9a9b328b4c2fedaa09f1e2a00857"}],"openstack/tests/unit/object_store/v1/test_obj.py":[{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"76f87f5620c15e29692a585bf1b424647d44691d","unresolved":false,"context_lines":[{"line_number":143,"context_line":""},{"line_number":144,"context_line":"        sot.update(self.sess)"},{"line_number":145,"context_line":""},{"line_number":146,"context_line":"        headers \u003d {\"x-delete-after\": val}"},{"line_number":147,"context_line":"        url \u003d \"/%s/%s\" % (CONTAINER_NAME, OBJECT_NAME)"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"        self.sess.post.assert_called_with(url, service\u003dsot.service, data\u003dNone,"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa32b979_51a80c8d","line":146,"updated":"2015-06-17 20:13:40.000000000","message":"Unrelated to this change, it\u0027d be nice if the proxy object delete method had an after\u003d parameter to set this.","commit_id":"94122290caf4d8ef5001b9b8aca94766cf180e3f"}]}