)]}'
{"tempest/config.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"ef79d9c684066bc0d7389b43b61a4f0b35beaf1f","unresolved":false,"context_lines":[{"line_number":1265,"context_line":"                    \"service\"),"},{"line_number":1266,"context_line":"    cfg.IntOpt(\u0027deploy_timeout\u0027,"},{"line_number":1267,"context_line":"               default\u003d300,"},{"line_number":1268,"context_line":"               help\u003d\"Timeout for Ironic state changes.\")"},{"line_number":1269,"context_line":"]"},{"line_number":1270,"context_line":""},{"line_number":1271,"context_line":"negative_group \u003d cfg.OptGroup(name\u003d\u0027negative\u0027, title\u003d\"Negative Test Options\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a7e1126_b2457818","line":1268,"range":{"start_line":1268,"start_character":29,"end_line":1268,"end_character":32},"updated":"2015-12-16 21:47:41.000000000","message":"You need to use the deprecated_opts param here to list that this option is taking over the previous 4. Otherwise this will break any users with an existing config. We can\u0027t just remove options like this without providing a graceful migration. For example, look at use_dynamic_credentials in the auth group for what\u0027s done there.","commit_id":"09221716cdf57b2716e6419431e4101917d846d7"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"942d2d280ed1526fe6e2b0b5ada68bd6b4dbe3f3","unresolved":false,"context_lines":[{"line_number":1263,"context_line":"                        \u0027publicURL\u0027, \u0027adminURL\u0027, \u0027internalURL\u0027],"},{"line_number":1264,"context_line":"               help\u003d\"The endpoint type to use for the baremetal provisioning \""},{"line_number":1265,"context_line":"                    \"service\"),"},{"line_number":1266,"context_line":"    cfg.IntOpt(\u0027deploy_timeout\u0027,"},{"line_number":1267,"context_line":"               default\u003d300,"},{"line_number":1268,"context_line":"               help\u003d\"Timeout for Ironic state changes.\","},{"line_number":1269,"context_line":"               deprecated_opts\u003d[cfg.DeprecatedOpt(\u0027active_timeout\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_4198181c","line":1266,"range":{"start_line":1266,"start_character":16,"end_line":1266,"end_character":30},"updated":"2016-01-06 19:24:07.000000000","message":"is there any reason not to call this build_timeout, that\u0027s what\u0027s used for practically every other service.","commit_id":"b627ef85cf116d70a1fe1a0153015778e32fa0fc"},{"author":{"_account_id":20009,"name":"Mike Turek","email":"modestposer@gmail.com"},"change_message_id":"49b2b59f7ecdf3b38a02641c62a3e7fe82768f11","unresolved":false,"context_lines":[{"line_number":1263,"context_line":"                        \u0027publicURL\u0027, \u0027adminURL\u0027, \u0027internalURL\u0027],"},{"line_number":1264,"context_line":"               help\u003d\"The endpoint type to use for the baremetal provisioning \""},{"line_number":1265,"context_line":"                    \"service\"),"},{"line_number":1266,"context_line":"    cfg.IntOpt(\u0027deploy_timeout\u0027,"},{"line_number":1267,"context_line":"               default\u003d300,"},{"line_number":1268,"context_line":"               help\u003d\"Timeout for Ironic state changes.\","},{"line_number":1269,"context_line":"               deprecated_opts\u003d[cfg.DeprecatedOpt(\u0027active_timeout\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_6d050d0f","line":1266,"range":{"start_line":1266,"start_character":16,"end_line":1266,"end_character":30},"in_reply_to":"fa69d971_4198181c","updated":"2016-01-06 20:39:28.000000000","message":"build_timeout is probably a better name. deploy_timeout was the first thing that came to mind as it\u0027s used during a deploy. For the sake of consistency, I\u0027ll change it to build_timeout","commit_id":"b627ef85cf116d70a1fe1a0153015778e32fa0fc"}]}