)]}'
{"freezer/tests/freezer_tempest_plugin/tests/scenario/test_backups.py":[{"author":{"_account_id":16768,"name":"Ruslan Aliev","email":"raliev@mirantis.com","username":"raliev"},"change_message_id":"bbe7c56917b3d9bca998e49f026dc8df734a9d0d","unresolved":false,"context_lines":[{"line_number":49,"context_line":"            )"},{"line_number":50,"context_line":"        cls.cli.cli_dir \u003d \u0027\u0027"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        # Without specifying a cli directory, PATH will be used. This allows"},{"line_number":53,"context_line":"        # the test to run on different platforms (e.g OSX)"},{"line_number":54,"context_line":"        cls.cli.cli_dir \u003d \u0027\u0027"},{"line_number":55,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_37b73df6","line":52,"updated":"2016-11-10 16:30:42.000000000","message":"It\u0027s a wrong approach. If you don\u0027t specify cli directory, the value \u0027/usr/bin\u0027[1] will be used as default. It will not work as a cross-platform solution.\n\n[1] https://github.com/openstack/tempest/blob/master/tempest/lib/cli/base.py#L97","commit_id":"b34d2120a5f22a181e56a08000f2ff6c91f6606e"},{"author":{"_account_id":16768,"name":"Ruslan Aliev","email":"raliev@mirantis.com","username":"raliev"},"change_message_id":"bbe7c56917b3d9bca998e49f026dc8df734a9d0d","unresolved":false,"context_lines":[{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        # Without specifying a cli directory, PATH will be used. This allows"},{"line_number":53,"context_line":"        # the test to run on different platforms (e.g OSX)"},{"line_number":54,"context_line":"        cls.cli.cli_dir \u003d \u0027\u0027"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    def delete_job(self, job_id):"},{"line_number":57,"context_line":"        self.cli.freezer_scheduler(action\u003d\u0027job-delete\u0027, flags\u003d\u0027-c test_node -j {}\u0027.format(job_id))"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_17f15901","line":54,"updated":"2016-11-10 16:30:42.000000000","message":"You assign the value of cli_dir twice","commit_id":"b34d2120a5f22a181e56a08000f2ff6c91f6606e"}]}