)]}'
{"glance/tests/functional/__init__.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"c490ba30a831a801820087a48cccb40387c1032a","unresolved":false,"context_lines":[{"line_number":603,"context_line":"        self.api_port, api_sock \u003d test_utils.get_unused_port_and_socket()"},{"line_number":604,"context_line":"        self.registry_port, reg_sock \u003d test_utils.get_unused_port_and_socket()"},{"line_number":605,"context_line":""},{"line_number":606,"context_line":"        self.tracecmd \u003d tracecmd_osmap.get(platform.system())"},{"line_number":607,"context_line":""},{"line_number":608,"context_line":"        conf_dir \u003d os.path.join(self.test_dir, \u0027etc\u0027)"},{"line_number":609,"context_line":"        utils.safe_mkdirs(conf_dir)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_68935da4","line":606,"updated":"2016-04-15 15:01:27.000000000","message":"I wonder whether it would be worth moving to here the check to see if strace is available? I was thinking it would de-clutter the loop where your patch does the check.  On the other hand, these are are utilities and not actual tests, so a little clutter is not a big deal.  Plus, your solution protects against some joker deleting/changing the executable permission on strace during the test run.  So feel free to ignore this comment.","commit_id":"1109f826b24671a1c38ddaf61251f39b4312eea6"}]}