)]}'
{"rally/verification/tempest/tempest.py":[{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"47b86a3d9b2e1ef49a68d6960a77e8bb960e2082","unresolved":false,"context_lines":[{"line_number":266,"context_line":"        \"\"\"Creates local Tempest repo and virtualenv for deployment.\"\"\""},{"line_number":267,"context_line":"        if not self.is_installed():"},{"line_number":268,"context_line":"            try:"},{"line_number":269,"context_line":"                if not os.path.exists(self.path()):"},{"line_number":270,"context_line":"                    if not self._is_git_repo(self.base_repo):"},{"line_number":271,"context_line":"                        self._clone()"},{"line_number":272,"context_line":"                    shutil.copytree(self.base_repo, self.path())"}],"source_content_type":"text/x-python","patch_set":9,"id":"9a8ffd7b_f7755963","line":269,"updated":"2015-11-28 22:47:52.000000000","message":"this check is redundant(the same as in method `is_installed`)","commit_id":"2a946812311dba1b9e923559ddbd24346a829564"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"5fb689a054f08f78df8ba41ad2dc5a765c3edc59","unresolved":false,"context_lines":[{"line_number":266,"context_line":"        \"\"\"Creates local Tempest repo and virtualenv for deployment.\"\"\""},{"line_number":267,"context_line":"        if not self.is_installed():"},{"line_number":268,"context_line":"            try:"},{"line_number":269,"context_line":"                if not os.path.exists(self.path()):"},{"line_number":270,"context_line":"                    if not self._is_git_repo(self.base_repo):"},{"line_number":271,"context_line":"                        self._clone()"},{"line_number":272,"context_line":"                    shutil.copytree(self.base_repo, self.path())"}],"source_content_type":"text/x-python","patch_set":9,"id":"9a8ffd7b_dc75d5ca","line":269,"in_reply_to":"9a8ffd7b_f7755963","updated":"2015-11-29 11:58:30.000000000","message":"No, it is needed. Consider the following case:\n\n    I would like to run tests with --system-wide-install flag and I execute as follows:\n        \n        $ rally verify start --system-wide-install\n\n    In this case Tempest is cloned, requirements are not installed, tests are run.\n\n    Now I would like to run tests without --system-wide-install flag and \n    I execute the following command:\n\n         $ rally verify start\n\n    In this case without this check Tempest will be cloned again! Why? \n    We have already cloned Tempest. Ideally only venv should be created.\n\nSo this check allows us to create only venv and run tests in it.","commit_id":"2a946812311dba1b9e923559ddbd24346a829564"}]}