)]}'
{"tempest/common/utils/data_utils.py":[{"author":{"_account_id":4220,"name":"patrick-crews","email":"gleebix@gmail.com","username":"patrick-crews"},"change_message_id":"794dcf0e64cd6fb86e9f7c52fc4858eb16dff0e0","unresolved":false,"context_lines":[{"line_number":37,"context_line":"def rand_url():"},{"line_number":38,"context_line":"    randbits \u003d str(random.randint(1, 0x7fffffff))"},{"line_number":39,"context_line":"    return \u0027https://url-\u0027 + randbits + \u0027.com\u0027"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"def rand_int_id(start\u003d0, end\u003d0x7fffffff):"},{"line_number":43,"context_line":"    return random.randint(start, end)"}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_ed489c8c","line":40,"updated":"2014-07-16 19:00:46.000000000","message":"is return \u0027https://url-%s.com\u0027 % randbits a to-may-to / to-mah-to thing compared to the current code?\n\nFor some reason, the current code makes my spidey-sense tingle, but this is just a stylistic quibble.","commit_id":"fd9eca433bc5c87b221004c7956f7aca137208b0"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"8eb173a5a4fcc3284653f1aa4966e1ed4d9a152b","unresolved":false,"context_lines":[{"line_number":37,"context_line":"def rand_url():"},{"line_number":38,"context_line":"    randbits \u003d str(random.randint(1, 0x7fffffff))"},{"line_number":39,"context_line":"    return \u0027https://url-\u0027 + randbits + \u0027.com\u0027"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"def rand_int_id(start\u003d0, end\u003d0x7fffffff):"},{"line_number":43,"context_line":"    return random.randint(start, end)"}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_e1618312","line":40,"in_reply_to":"baada198_ed489c8c","updated":"2014-07-16 20:51:38.000000000","message":"if you\u0027re referring to the return values, rand_url() actually returns a valid URL, whereas rand_name() does not. keystone is working to validate this data as an actual URL, and if so, will start responding to some requests from tempest with an HTTP 400 if this isn\u0027t fixed.\n\nlance\u0027s commit message explained this much better than mine; this patch was just intended to be a refactor, but can stand alone as well.","commit_id":"fd9eca433bc5c87b221004c7956f7aca137208b0"}]}
