)]}'
{"tempest/api/compute/servers/test_multiple_create.py":[{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"c8902b4167433f3d4f37edb6e85f4fc1a7c9f840","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        This is the right way to create_multiple servers and manage to get the"},{"line_number":31,"context_line":"        created servers into the servers list to be cleaned up after all."},{"line_number":32,"context_line":"        \"\"\""},{"line_number":33,"context_line":"        kwargs[\u0027name\u0027] \u003d kwargs.get(\u0027name\u0027, self._generate_name())"},{"line_number":34,"context_line":"        body \u003d self.create_test_server(wait_until\u003dwait_until, **kwargs)"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        return body"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a016987_1337ecff","line":33,"updated":"2015-05-24 15:31:22.000000000","message":"Hi , \nI think there is a problem with this function ,  the variable name.\nIf we set a value for name , this name will be ignored because its not part of kwargs\n\nThe function should be :\nif name:\n# use existing value \nelse:\n# we assume its None .... \n\nAm i  missing something ?\n\nBenny","commit_id":"ad68e29ac64602bdf5cfbc191ac5312ca7c08472"}]}