)]}'
{"nova/tests/functional/integrated_helpers.py":[{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"2c5b7c67c8c9780f4a8f0c678d466ad90f23b72d","unresolved":false,"context_lines":[{"line_number":98,"context_line":""},{"line_number":99,"context_line":"        self.addCleanup(nova.tests.unit.image.fake.FakeImageService_reset)"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    def _setup_compute_service(self):"},{"line_number":102,"context_line":"        return self.start_service(\u0027compute\u0027)"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"    def _setup_scheduler_service(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_aad78f4e","line":101,"updated":"2015-03-16 17:20:53.000000000","message":"I\u0027m not sure to have understand why you have added these two new methods since you can do the same work for \u0027cert\u0027, \u0027consoleauth\u0027... I was expected to find a call of them at least two times.\n\nOr you probably want ot use them here: https://review.openstack.org/#/c/155005/3/nova/tests/functional/test_servers.py,cm","commit_id":"86450238d65197e475504268661661ab2f8a93f1"},{"author":{"_account_id":8802,"name":"Vladik Romanovsky","email":"vromanso@redhat.com","username":"vladikr"},"change_message_id":"914e34a89686f1ffc7e8acfbc06f868741189cb3","unresolved":false,"context_lines":[{"line_number":98,"context_line":""},{"line_number":99,"context_line":"        self.addCleanup(nova.tests.unit.image.fake.FakeImageService_reset)"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    def _setup_compute_service(self):"},{"line_number":102,"context_line":"        return self.start_service(\u0027compute\u0027)"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"    def _setup_scheduler_service(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_16936201","line":101,"in_reply_to":"9a80dd14_aad78f4e","updated":"2015-03-16 18:42:30.000000000","message":"Yea, as you already commented :) \nThe reason these are separated is to allow the tests classes to override it and to be able to control the start sequence of these service instead of starting it in the constructor. As it is in:\nhttps://review.openstack.org/#/c/155006","commit_id":"86450238d65197e475504268661661ab2f8a93f1"}]}
