)]}'
{"tempest/cmd/javelin.py":[{"author":{"_account_id":8859,"name":"Daisuke Morita","email":"morita.daisuke@ntti3.com","username":"morrisson"},"change_message_id":"2a1c72e1689355658625f6f6b51c45bfab72290f","unresolved":false,"context_lines":[{"line_number":164,"context_line":"            \u0027region\u0027: CONF.object_storage.region or CONF.identity.region,"},{"line_number":165,"context_line":"            \u0027endpoint_type\u0027: CONF.object_storage.endpoint_type,"},{"line_number":166,"context_line":"            \u0027build_interval\u0027: CONF.compute.build_interval,"},{"line_number":167,"context_line":"            \u0027build_timeout\u0027: CONF.compute.build_timeout"},{"line_number":168,"context_line":"        }"},{"line_number":169,"context_line":"        object_storage_params.update(default_params)"},{"line_number":170,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1a930d6b_402c0948","line":167,"updated":"2015-01-22 08:13:44.000000000","message":"Any Swift test does not refer build_* attributes so you can probably drop above two lines.","commit_id":"ac458dc075035a3a7758ca141e062802e1111b2b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"478c28a87c6296cd8e18804d9edc5e088de02435","unresolved":false,"context_lines":[{"line_number":164,"context_line":"            \u0027region\u0027: CONF.object_storage.region or CONF.identity.region,"},{"line_number":165,"context_line":"            \u0027endpoint_type\u0027: CONF.object_storage.endpoint_type,"},{"line_number":166,"context_line":"            \u0027build_interval\u0027: CONF.compute.build_interval,"},{"line_number":167,"context_line":"            \u0027build_timeout\u0027: CONF.compute.build_timeout"},{"line_number":168,"context_line":"        }"},{"line_number":169,"context_line":"        object_storage_params.update(default_params)"},{"line_number":170,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1a930d6b_0095a132","line":167,"in_reply_to":"1a930d6b_402c0948","updated":"2015-01-22 08:19:22.000000000","message":"Thanks for your review, and that is a nice point.\nI will remove it soon.","commit_id":"ac458dc075035a3a7758ca141e062802e1111b2b"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"cd647b572da9e290b70e1ac2cda082f2e2c7de6e","unresolved":false,"context_lines":[{"line_number":164,"context_line":"            \u0027region\u0027: CONF.object_storage.region or CONF.identity.region,"},{"line_number":165,"context_line":"            \u0027endpoint_type\u0027: CONF.object_storage.endpoint_type"},{"line_number":166,"context_line":"        }"},{"line_number":167,"context_line":"        object_storage_params.update(default_params)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"        _creds \u003d tempest.auth.KeystoneV2Credentials("},{"line_number":170,"context_line":"            username\u003duser,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a930d6b_03c4a75d","line":167,"updated":"2015-01-23 02:10:31.000000000","message":"build_timeout is going to be default(60) here.\nwe can define default_params_with_timeout_values and use that instead of default_params","commit_id":"75e23674e6423987788c677eb5bbbe18c6c13fe4"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"dc589f3456fee529a51b10deba9d14658ca5cf23","unresolved":false,"context_lines":[{"line_number":164,"context_line":"            \u0027region\u0027: CONF.object_storage.region or CONF.identity.region,"},{"line_number":165,"context_line":"            \u0027endpoint_type\u0027: CONF.object_storage.endpoint_type"},{"line_number":166,"context_line":"        }"},{"line_number":167,"context_line":"        object_storage_params.update(default_params)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"        _creds \u003d tempest.auth.KeystoneV2Credentials("},{"line_number":170,"context_line":"            username\u003duser,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a930d6b_c78c6a82","line":167,"in_reply_to":"1a930d6b_038f47cd","updated":"2015-01-23 02:30:18.000000000","message":"I see. Thanks :)","commit_id":"75e23674e6423987788c677eb5bbbe18c6c13fe4"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"b15de8d151534a53e170dc66b015edd38423f7e3","unresolved":false,"context_lines":[{"line_number":164,"context_line":"            \u0027region\u0027: CONF.object_storage.region or CONF.identity.region,"},{"line_number":165,"context_line":"            \u0027endpoint_type\u0027: CONF.object_storage.endpoint_type"},{"line_number":166,"context_line":"        }"},{"line_number":167,"context_line":"        object_storage_params.update(default_params)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"        _creds \u003d tempest.auth.KeystoneV2Credentials("},{"line_number":170,"context_line":"            username\u003duser,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a930d6b_038f47cd","line":167,"in_reply_to":"1a930d6b_03c4a75d","updated":"2015-01-23 02:14:17.000000000","message":"Yeah, that was the same as my thinking, but swift test doesn\u0027t refer build_* values as Daisuke\u0027s previous comment.\nthen I have removed them from current patch.","commit_id":"75e23674e6423987788c677eb5bbbe18c6c13fe4"}]}