)]}'
{"nova/tests/db/test_migrations.py":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"f6c6f5f4300d079565cf76f0348297aafbbf89e9","unresolved":false,"context_lines":[{"line_number":204,"context_line":"    PASSWD \u003d None"},{"line_number":205,"context_line":"    DATABASE \u003d None"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    TIMEOUT_SCALING_FACTOR \u003d 0  # don\u0027t stop migrations tests on time out"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"    def __init__(self, *args, **kwargs):"},{"line_number":210,"context_line":"        super(BaseMigrationTestCase, self).__init__(*args, **kwargs)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F9o8c%3D","line":207,"updated":"2013-12-09 12:54:54.000000000","message":"I don\u0027t think we should just disable the timeout. How about just allowing it 50% or 100% more time? If the migration tests start taking significantly longer that\u0027s probably something we want the gate to pick up and we can always bump this up in the future if we need to.","commit_id":"df862678e9d16079f1df2863fb0d05d206695f43"},{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"434490460c935b14891a28e177e7ef22175b32d1","unresolved":false,"context_lines":[{"line_number":204,"context_line":"    PASSWD \u003d None"},{"line_number":205,"context_line":"    DATABASE \u003d None"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    TIMEOUT_SCALING_FACTOR \u003d 0  # don\u0027t stop migrations tests on time out"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"    def __init__(self, *args, **kwargs):"},{"line_number":210,"context_line":"        super(BaseMigrationTestCase, self).__init__(*args, **kwargs)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F9oyw%3D","line":207,"in_reply_to":"AAAATn%2F9o8c%3D","updated":"2013-12-09 13:24:03.000000000","message":"I\u0027m not keen on disabling of the timeout either, but the problem is that the time it takes to run migrations tests varies greatly (e.g. for postgresql I\u0027ve seen results in the range from ~50s to ~250s), so it depends strongly on the load of CI nodes.\n\nI\u0027d go for scaling factor of 2 here. Will upload a new patch set soon.","commit_id":"df862678e9d16079f1df2863fb0d05d206695f43"}]}