)]}'
{"tempest/api/object_storage/test_container_services.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"688121479c1d7ad7534a7b73f6a1bd506d2f3d35","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        return container_name"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def _create_object(self, container_name,"},{"line_number":44,"context_line":"                       object_name\u003ddata_utils.rand_name(name\u003d\u0027TestObject\u0027)):"},{"line_number":45,"context_line":"        # setup object"},{"line_number":46,"context_line":"        data \u003d data_utils.arbitrary_string()"},{"line_number":47,"context_line":"        self.object_client.create_object(container_name,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2BsyA%3D","line":44,"updated":"2014-01-23 02:08:47.000000000","message":"This rand_name() call does not work, object_name is static after one call of _create_object().\n\nHow about changing here like the following?\n\n def _create_object(self, container_name, object_name\u003dNone):\n     if object_name is None:\n         object_name\u003ddata_utils.rand_name(name\u003d\u0027TestObject\u0027)","commit_id":"b574b308372803adbcd3cbf954d9537cda0b7de0"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1233b41f7c4ac6b4ac8a96f2fd91da53e815540f","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        return container_name"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def _create_object(self, container_name,"},{"line_number":44,"context_line":"                       object_name\u003ddata_utils.rand_name(name\u003d\u0027TestObject\u0027)):"},{"line_number":45,"context_line":"        # setup object"},{"line_number":46,"context_line":"        data \u003d data_utils.arbitrary_string()"},{"line_number":47,"context_line":"        self.object_client.create_object(container_name,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2BkzY%3D","line":44,"in_reply_to":"AAAAUH%2F%2Bk4w%3D","updated":"2014-01-24 08:41:32.000000000","message":"I guess the above rand_name() is expected that object_name is different by each call.\nIs that right?\n\nbut if passing it as the argument, the object_name is not changed like the following:\n\n \u003e\u003e\u003e from tempest.common.utils import data_utils\n \u003e\u003e\u003e def _create_object(object_name\u003ddata_utils.rand_name(name\u003d\u0027TestObject\u0027)):\n ...     print object_name\n ...\n \u003e\u003e\u003e _create_object()\n TestObject-tempest-1465937914\n \u003e\u003e\u003e _create_object()\n TestObject-tempest-1465937914\n \u003e\u003e\u003e _create_object()\n TestObject-tempest-1465937914\n\nIs that the expected behavior?","commit_id":"b574b308372803adbcd3cbf954d9537cda0b7de0"},{"author":{"_account_id":8859,"name":"Daisuke Morita","email":"morita.daisuke@ntti3.com","username":"morrisson"},"change_message_id":"8d69ab0ba26641445f0427c41fe80075844de0cd","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        return container_name"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def _create_object(self, container_name,"},{"line_number":44,"context_line":"                       object_name\u003ddata_utils.rand_name(name\u003d\u0027TestObject\u0027)):"},{"line_number":45,"context_line":"        # setup object"},{"line_number":46,"context_line":"        data \u003d data_utils.arbitrary_string()"},{"line_number":47,"context_line":"        self.object_client.create_object(container_name,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2Bksg%3D","line":44,"in_reply_to":"AAAAUH%2F%2BkzY%3D","updated":"2014-01-24 09:02:20.000000000","message":"Oh, I see. Sorry for bothering you. Thanks for your kindness!!","commit_id":"b574b308372803adbcd3cbf954d9537cda0b7de0"},{"author":{"_account_id":8859,"name":"Daisuke Morita","email":"morita.daisuke@ntti3.com","username":"morrisson"},"change_message_id":"5ab1f78fa3844dc3a9f44cde0e0b4a00f949c966","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        return container_name"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def _create_object(self, container_name,"},{"line_number":44,"context_line":"                       object_name\u003ddata_utils.rand_name(name\u003d\u0027TestObject\u0027)):"},{"line_number":45,"context_line":"        # setup object"},{"line_number":46,"context_line":"        data \u003d data_utils.arbitrary_string()"},{"line_number":47,"context_line":"        self.object_client.create_object(container_name,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2Bk4w%3D","line":44,"in_reply_to":"AAAAUH%2F%2BsyA%3D","updated":"2014-01-24 08:36:21.000000000","message":"What do you mean \"not work\" ? Jenkins says this code works and this also works in my testing environment.","commit_id":"b574b308372803adbcd3cbf954d9537cda0b7de0"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"688121479c1d7ad7534a7b73f6a1bd506d2f3d35","unresolved":false,"context_lines":[{"line_number":260,"context_line":"        container_name \u003d self._create_container()"},{"line_number":261,"context_line":"        object_name \u003d self._create_object(container_name)"},{"line_number":262,"context_line":""},{"line_number":263,"context_line":"        params \u003d {\u0027limit\u0027: random.randint(1, 10000)}"},{"line_number":264,"context_line":"        resp, object_list \u003d self.container_client.list_container_contents("},{"line_number":265,"context_line":"            container_name,"},{"line_number":266,"context_line":"            params\u003dparams)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2Bs04%3D","line":263,"updated":"2014-01-23 02:08:47.000000000","message":"The data_utils module contains rand_int_id() already.\nPlease use it like the other tests instead of calling random.randint() directly.","commit_id":"b574b308372803adbcd3cbf954d9537cda0b7de0"},{"author":{"_account_id":8859,"name":"Daisuke Morita","email":"morita.daisuke@ntti3.com","username":"morrisson"},"change_message_id":"5ab1f78fa3844dc3a9f44cde0e0b4a00f949c966","unresolved":false,"context_lines":[{"line_number":260,"context_line":"        container_name \u003d self._create_container()"},{"line_number":261,"context_line":"        object_name \u003d self._create_object(container_name)"},{"line_number":262,"context_line":""},{"line_number":263,"context_line":"        params \u003d {\u0027limit\u0027: random.randint(1, 10000)}"},{"line_number":264,"context_line":"        resp, object_list \u003d self.container_client.list_container_contents("},{"line_number":265,"context_line":"            container_name,"},{"line_number":266,"context_line":"            params\u003dparams)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2Bk18%3D","line":263,"in_reply_to":"AAAAUH%2F%2Bs04%3D","updated":"2014-01-24 08:36:21.000000000","message":"Done","commit_id":"b574b308372803adbcd3cbf954d9537cda0b7de0"}]}