)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7369,"name":"Sergey Skripnick","email":"sskripnick@mirantis.com","username":"redixin"},"change_message_id":"ea6fed8c607211a93309b829d0c37657eb6a9c09","unresolved":false,"context_lines":[{"line_number":10,"context_line":"very convenient because Tempest may be broken in the master branch and"},{"line_number":11,"context_line":"it would be very useful to use some commit or tag for Tempest."},{"line_number":12,"context_line":"So this commit adds a possibility to checkout the specified commit or"},{"line_number":13,"context_line":"tag after Tempest installation."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Icfbb4221005466139c14d3f4aae34bbb3e013982"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a122d0e_ba6b45f0","line":13,"updated":"2016-04-28 09:25:10.000000000","message":"after?","commit_id":"1bf5bd6411100b1a602b2315696ca59adec62b3c"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"5f52ed6f91878012e05d4050cb723fa7c8ac1d75","unresolved":false,"context_lines":[{"line_number":10,"context_line":"very convenient because Tempest may be broken in the master branch and"},{"line_number":11,"context_line":"it would be very useful to use some commit or tag for Tempest."},{"line_number":12,"context_line":"So this commit adds a possibility to checkout the specified commit or"},{"line_number":13,"context_line":"tag after Tempest installation."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Icfbb4221005466139c14d3f4aae34bbb3e013982"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a122d0e_4075e42f","line":13,"in_reply_to":"1a122d0e_ba6b45f0","updated":"2016-04-28 10:27:20.000000000","message":"Oh, of course before installation. Thanks. Will fix.","commit_id":"1bf5bd6411100b1a602b2315696ca59adec62b3c"}],"rally/verification/tempest/tempest.py":[{"author":{"_account_id":7369,"name":"Sergey Skripnick","email":"sskripnick@mirantis.com","username":"redixin"},"change_message_id":"ea6fed8c607211a93309b829d0c37657eb6a9c09","unresolved":false,"context_lines":[{"line_number":276,"context_line":"                        self._clone()"},{"line_number":277,"context_line":"                    shutil.copytree(self.base_repo, self.path())"},{"line_number":278,"context_line":"                    if self.checkout:"},{"line_number":279,"context_line":"                        cmd \u003d [\"git\", \"checkout\", self.checkout]"},{"line_number":280,"context_line":"                        subprocess.check_call(cmd, cwd\u003dself.path(\"tempest\"))"},{"line_number":281,"context_line":"                if not self._system_wide:"},{"line_number":282,"context_line":"                    self._install_venv()"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_1aa4f948","line":279,"updated":"2016-04-28 09:25:10.000000000","message":"We are not doing git pull anymore, but base repo may be outdated. I guess we need to update base repo before copying and checkouting.\n\nAlso `git clone` may be more efficient then shutil.copy","commit_id":"1bf5bd6411100b1a602b2315696ca59adec62b3c"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"5f52ed6f91878012e05d4050cb723fa7c8ac1d75","unresolved":false,"context_lines":[{"line_number":276,"context_line":"                        self._clone()"},{"line_number":277,"context_line":"                    shutil.copytree(self.base_repo, self.path())"},{"line_number":278,"context_line":"                    if self.checkout:"},{"line_number":279,"context_line":"                        cmd \u003d [\"git\", \"checkout\", self.checkout]"},{"line_number":280,"context_line":"                        subprocess.check_call(cmd, cwd\u003dself.path(\"tempest\"))"},{"line_number":281,"context_line":"                if not self._system_wide:"},{"line_number":282,"context_line":"                    self._install_venv()"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_a0de783b","line":279,"in_reply_to":"1a122d0e_1aa4f948","updated":"2016-04-28 10:27:20.000000000","message":"Sergey, I think we shouldn\u0027t do `git pull` here because a user may want to intentionally install outdated Tempest. If the user will want to install the newest Tempest, he/she can do `git pull` in the local repo and then install it by `rally verify install --source /path/to/repo`.","commit_id":"1bf5bd6411100b1a602b2315696ca59adec62b3c"}]}