)]}'
{"nova/conf/s3.py":[{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"87bca985687c829072fc998504b91dccd19bad43","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    Any directory path, /tmp is default"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"* Related options:"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    None"},{"line_number":36,"context_line":"\"\"\"),"},{"line_number":37,"context_line":"    cfg.StrOpt(\"s3_host\","},{"line_number":38,"context_line":"               deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_78179854","line":35,"range":{"start_line":33,"start_character":0,"end_line":35,"end_character":8},"updated":"2016-06-28 11:09:12.000000000","message":"Drop this section","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"55b4102d9220610f32f6a16ac68699ceb0f98e1f","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    Any directory path, /tmp is default"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"* Related options:"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    None"},{"line_number":36,"context_line":"\"\"\"),"},{"line_number":37,"context_line":"    cfg.StrOpt(\"s3_host\","},{"line_number":38,"context_line":"               deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_7f3c90bf","line":35,"range":{"start_line":33,"start_character":0,"end_line":35,"end_character":8},"in_reply_to":"3aaa91ec_78179854","updated":"2016-06-28 14:32:30.000000000","message":"Done","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"76cd7f5876587bcc11358d862d31dc2d0db3dad0","unresolved":false,"context_lines":[{"line_number":40,"context_line":"               default\u003d\"$my_ip\","},{"line_number":41,"context_line":"               help\u003d\"\"\""},{"line_number":42,"context_line":"Hostname or IP for OpenStack to use when"},{"line_number":43,"context_line":"accessing the S3 api"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* Possible values:"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_34b89be5","line":43,"range":{"start_line":43,"start_character":17,"end_line":43,"end_character":20},"updated":"2016-06-27 19:54:21.000000000","message":"nit: s/api/API","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"55b4102d9220610f32f6a16ac68699ceb0f98e1f","unresolved":false,"context_lines":[{"line_number":40,"context_line":"               default\u003d\"$my_ip\","},{"line_number":41,"context_line":"               help\u003d\"\"\""},{"line_number":42,"context_line":"Hostname or IP for OpenStack to use when"},{"line_number":43,"context_line":"accessing the S3 api"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* Possible values:"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_df4b0458","line":43,"range":{"start_line":43,"start_character":17,"end_line":43,"end_character":20},"in_reply_to":"3aaa91ec_34b89be5","updated":"2016-06-28 14:32:30.000000000","message":"Done","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"87bca985687c829072fc998504b91dccd19bad43","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"* Related options:"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    None"},{"line_number":52,"context_line":"\"\"\"),"},{"line_number":53,"context_line":"    cfg.IntOpt(\"s3_port\","},{"line_number":54,"context_line":"               deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_f339e9d9","line":51,"range":{"start_line":51,"start_character":4,"end_line":51,"end_character":8},"updated":"2016-06-28 11:09:12.000000000","message":"Because of default value I think that my_ip is related option","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"55b4102d9220610f32f6a16ac68699ceb0f98e1f","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"* Related options:"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    None"},{"line_number":52,"context_line":"\"\"\"),"},{"line_number":53,"context_line":"    cfg.IntOpt(\"s3_port\","},{"line_number":54,"context_line":"               deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_bf46f84c","line":51,"range":{"start_line":51,"start_character":4,"end_line":51,"end_character":8},"in_reply_to":"3aaa91ec_f339e9d9","updated":"2016-06-28 14:32:30.000000000","message":"Done","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"76cd7f5876587bcc11358d862d31dc2d0db3dad0","unresolved":false,"context_lines":[{"line_number":57,"context_line":"               min\u003d1,"},{"line_number":58,"context_line":"               max\u003d65535,"},{"line_number":59,"context_line":"               help\u003d\"\"\""},{"line_number":60,"context_line":"Port used when accessing the S3 api. It should be in the range of"},{"line_number":61,"context_line":"1 - 6535"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"* Possible values:"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_14a0ffa4","line":60,"updated":"2016-06-27 19:54:21.000000000","message":"same as above","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"55b4102d9220610f32f6a16ac68699ceb0f98e1f","unresolved":false,"context_lines":[{"line_number":57,"context_line":"               min\u003d1,"},{"line_number":58,"context_line":"               max\u003d65535,"},{"line_number":59,"context_line":"               help\u003d\"\"\""},{"line_number":60,"context_line":"Port used when accessing the S3 api. It should be in the range of"},{"line_number":61,"context_line":"1 - 6535"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"* Possible values:"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_1f746c9a","line":60,"in_reply_to":"3aaa91ec_14a0ffa4","updated":"2016-06-28 14:32:30.000000000","message":"Done","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"87bca985687c829072fc998504b91dccd19bad43","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    Port number, 3333 is default"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"* Related options:"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    None"},{"line_number":70,"context_line":"\"\"\"),"},{"line_number":71,"context_line":"    cfg.StrOpt(\"s3_access_key\","},{"line_number":72,"context_line":"               deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_5351dd0d","line":69,"range":{"start_line":67,"start_character":0,"end_line":69,"end_character":8},"updated":"2016-06-28 11:09:12.000000000","message":"Drop this section","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"55b4102d9220610f32f6a16ac68699ceb0f98e1f","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    Port number, 3333 is default"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"* Related options:"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    None"},{"line_number":70,"context_line":"\"\"\"),"},{"line_number":71,"context_line":"    cfg.StrOpt(\"s3_access_key\","},{"line_number":72,"context_line":"               deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_ff76c0a1","line":69,"range":{"start_line":67,"start_character":0,"end_line":69,"end_character":8},"in_reply_to":"3aaa91ec_5351dd0d","updated":"2016-06-28 14:32:30.000000000","message":"Done","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"76cd7f5876587bcc11358d862d31dc2d0db3dad0","unresolved":false,"context_lines":[{"line_number":73,"context_line":"               deprecated_reason\u003d\"EC2 API related options are not supported.\","},{"line_number":74,"context_line":"               default\u003d\"notchecked\","},{"line_number":75,"context_line":"               help\u003d\"\"\""},{"line_number":76,"context_line":"Access key to use for S3 server for images"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"* Related options:"},{"line_number":79,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_d4c31737","line":76,"range":{"start_line":76,"start_character":16,"end_line":76,"end_character":17},"updated":"2016-06-27 19:54:21.000000000","message":"Would \"Access key to use S3 server for images\" work ?","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"55b4102d9220610f32f6a16ac68699ceb0f98e1f","unresolved":false,"context_lines":[{"line_number":73,"context_line":"               deprecated_reason\u003d\"EC2 API related options are not supported.\","},{"line_number":74,"context_line":"               default\u003d\"notchecked\","},{"line_number":75,"context_line":"               help\u003d\"\"\""},{"line_number":76,"context_line":"Access key to use for S3 server for images"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"* Related options:"},{"line_number":79,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_3f7168a9","line":76,"range":{"start_line":76,"start_character":16,"end_line":76,"end_character":17},"in_reply_to":"3aaa91ec_d4c31737","updated":"2016-06-28 14:32:30.000000000","message":"Done","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"87bca985687c829072fc998504b91dccd19bad43","unresolved":false,"context_lines":[{"line_number":75,"context_line":"               help\u003d\"\"\""},{"line_number":76,"context_line":"Access key to use for S3 server for images"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"* Related options:"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    None"},{"line_number":81,"context_line":"\"\"\"),"},{"line_number":82,"context_line":"    cfg.StrOpt(\"s3_secret_key\","},{"line_number":83,"context_line":"               deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_1348f55a","line":80,"range":{"start_line":78,"start_character":0,"end_line":80,"end_character":8},"updated":"2016-06-28 11:09:12.000000000","message":"Drop this section","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"55b4102d9220610f32f6a16ac68699ceb0f98e1f","unresolved":false,"context_lines":[{"line_number":75,"context_line":"               help\u003d\"\"\""},{"line_number":76,"context_line":"Access key to use for S3 server for images"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"* Related options:"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    None"},{"line_number":81,"context_line":"\"\"\"),"},{"line_number":82,"context_line":"    cfg.StrOpt(\"s3_secret_key\","},{"line_number":83,"context_line":"               deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_7f63f0dd","line":80,"range":{"start_line":78,"start_character":0,"end_line":80,"end_character":8},"in_reply_to":"3aaa91ec_1348f55a","updated":"2016-06-28 14:32:30.000000000","message":"Done","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"87bca985687c829072fc998504b91dccd19bad43","unresolved":false,"context_lines":[{"line_number":86,"context_line":"               help\u003d\"\"\""},{"line_number":87,"context_line":"Secret key to use for S3 server for images"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"* Related options:"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    None"},{"line_number":92,"context_line":"\"\"\"),"},{"line_number":93,"context_line":"    cfg.BoolOpt(\"s3_use_ssl\","},{"line_number":94,"context_line":"                deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_336251d9","line":91,"range":{"start_line":89,"start_character":0,"end_line":91,"end_character":8},"updated":"2016-06-28 11:09:12.000000000","message":"Drop this section","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"55b4102d9220610f32f6a16ac68699ceb0f98e1f","unresolved":false,"context_lines":[{"line_number":86,"context_line":"               help\u003d\"\"\""},{"line_number":87,"context_line":"Secret key to use for S3 server for images"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"* Related options:"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    None"},{"line_number":92,"context_line":"\"\"\"),"},{"line_number":93,"context_line":"    cfg.BoolOpt(\"s3_use_ssl\","},{"line_number":94,"context_line":"                deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_bf5d5817","line":91,"range":{"start_line":89,"start_character":0,"end_line":91,"end_character":8},"in_reply_to":"3aaa91ec_336251d9","updated":"2016-06-28 14:32:30.000000000","message":"Done","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"87bca985687c829072fc998504b91dccd19bad43","unresolved":false,"context_lines":[{"line_number":97,"context_line":"                help\u003d\"\"\""},{"line_number":98,"context_line":"Whether to use SSL when talking to S3"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"* Possible values:"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"    True, False (default)"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"* Related options:"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    None"},{"line_number":107,"context_line":"\"\"\"),"},{"line_number":108,"context_line":"    cfg.BoolOpt(\"s3_affix_tenant\","},{"line_number":109,"context_line":"                deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_33909195","line":106,"range":{"start_line":100,"start_character":0,"end_line":106,"end_character":8},"updated":"2016-06-28 11:09:12.000000000","message":"Following John\u0027s comment on consistency and obvious choices, it might be good to drop both of these.","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"55b4102d9220610f32f6a16ac68699ceb0f98e1f","unresolved":false,"context_lines":[{"line_number":97,"context_line":"                help\u003d\"\"\""},{"line_number":98,"context_line":"Whether to use SSL when talking to S3"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"* Possible values:"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"    True, False (default)"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"* Related options:"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    None"},{"line_number":107,"context_line":"\"\"\"),"},{"line_number":108,"context_line":"    cfg.BoolOpt(\"s3_affix_tenant\","},{"line_number":109,"context_line":"                deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_1f9bccca","line":106,"range":{"start_line":100,"start_character":0,"end_line":106,"end_character":8},"in_reply_to":"3aaa91ec_33909195","updated":"2016-06-28 14:32:30.000000000","message":"Done","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"87bca985687c829072fc998504b91dccd19bad43","unresolved":false,"context_lines":[{"line_number":113,"context_line":"Whether to affix the tenant id to the"},{"line_number":114,"context_line":"access key when downloading from S3"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"* Possible values:"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"    True, False (default)"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"* Related options:"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    None"},{"line_number":123,"context_line":"\"\"\"),"},{"line_number":124,"context_line":"]"},{"line_number":125,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_73e85917","line":122,"range":{"start_line":116,"start_character":0,"end_line":122,"end_character":8},"updated":"2016-06-28 11:09:12.000000000","message":"As above","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"55b4102d9220610f32f6a16ac68699ceb0f98e1f","unresolved":false,"context_lines":[{"line_number":113,"context_line":"Whether to affix the tenant id to the"},{"line_number":114,"context_line":"access key when downloading from S3"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"* Possible values:"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"    True, False (default)"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"* Related options:"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    None"},{"line_number":123,"context_line":"\"\"\"),"},{"line_number":124,"context_line":"]"},{"line_number":125,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_ff9d20bc","line":122,"range":{"start_line":116,"start_character":0,"end_line":122,"end_character":8},"in_reply_to":"3aaa91ec_73e85917","updated":"2016-06-28 14:32:30.000000000","message":"Done","commit_id":"545781cd3b766571fe44b467178d00d4f4a20799"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"f02953f8ed6ae4ed46d9d4160910953b98920aa4","unresolved":false,"context_lines":[{"line_number":50,"context_line":"               max\u003d65535,"},{"line_number":51,"context_line":"               help\u003d\"\"\""},{"line_number":52,"context_line":"Port used when accessing the S3 API. It should be in the range of"},{"line_number":53,"context_line":"1 - 6535"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"* Possible values:"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_a27b6df3","line":53,"updated":"2016-06-28 20:28:40.000000000","message":"s/6535/65535","commit_id":"74cc8120a0c48a32f038e42824b3109537063438"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"dba981c411345ccc6cede1fcac4c657e78cff1ab","unresolved":false,"context_lines":[{"line_number":50,"context_line":"               max\u003d65535,"},{"line_number":51,"context_line":"               help\u003d\"\"\""},{"line_number":52,"context_line":"Port used when accessing the S3 API. It should be in the range of"},{"line_number":53,"context_line":"1 - 6535"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"* Possible values:"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_68a4c7b8","line":53,"in_reply_to":"3aaa91ec_a27b6df3","updated":"2016-06-28 20:45:03.000000000","message":"Done","commit_id":"74cc8120a0c48a32f038e42824b3109537063438"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"4c330f09234c08ce77ba7aee554218184aca759a","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Possible values:"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* IP address for s3"},{"line_number":44,"context_line":"\"\"\"),"},{"line_number":45,"context_line":"    cfg.IntOpt(\u0027s3_port\u0027,"},{"line_number":46,"context_line":"               deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":6,"id":"1aa78d24_2a4adcac","line":43,"range":{"start_line":43,"start_character":0,"end_line":43,"end_character":19},"updated":"2016-07-14 16:18:57.000000000","message":"Description indicated hostname as possibility as well.\nBut I would drop this section, as with description possible value is quite self-explanatory.","commit_id":"87be86e0c609a033f3affd683edc21c9abf155d8"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"4e0a512f78add86c02102e0843e6246d275822e0","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Possible values:"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* IP address for s3"},{"line_number":44,"context_line":"\"\"\"),"},{"line_number":45,"context_line":"    cfg.IntOpt(\u0027s3_port\u0027,"},{"line_number":46,"context_line":"               deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":6,"id":"1aa78d24_1e2bde95","line":43,"range":{"start_line":43,"start_character":0,"end_line":43,"end_character":19},"in_reply_to":"1aa78d24_2a4adcac","updated":"2016-07-14 17:45:39.000000000","message":"I will add hostname to this. More information is better. Plus sometimes operators looks into headers \"Possible Values\" in this case for what they can use.","commit_id":"87be86e0c609a033f3affd683edc21c9abf155d8"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"4c330f09234c08ce77ba7aee554218184aca759a","unresolved":false,"context_lines":[{"line_number":61,"context_line":"               deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"},{"line_number":62,"context_line":"               default\u003d\u0027notchecked\u0027,"},{"line_number":63,"context_line":"               help\u003d\"\"\""},{"line_number":64,"context_line":"Access key to use S3 server for images"},{"line_number":65,"context_line":"\"\"\"),"},{"line_number":66,"context_line":"    cfg.StrOpt(\u0027s3_secret_key\u0027,"},{"line_number":67,"context_line":"               deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":6,"id":"1aa78d24_6a2ea4fb","line":64,"range":{"start_line":64,"start_character":0,"end_line":64,"end_character":38},"updated":"2016-07-14 16:18:57.000000000","message":"This can be in single line as \"help\u003d\"","commit_id":"87be86e0c609a033f3affd683edc21c9abf155d8"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"4e0a512f78add86c02102e0843e6246d275822e0","unresolved":false,"context_lines":[{"line_number":61,"context_line":"               deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"},{"line_number":62,"context_line":"               default\u003d\u0027notchecked\u0027,"},{"line_number":63,"context_line":"               help\u003d\"\"\""},{"line_number":64,"context_line":"Access key to use S3 server for images"},{"line_number":65,"context_line":"\"\"\"),"},{"line_number":66,"context_line":"    cfg.StrOpt(\u0027s3_secret_key\u0027,"},{"line_number":67,"context_line":"               deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":6,"id":"1aa78d24_fe7532cc","line":64,"range":{"start_line":64,"start_character":0,"end_line":64,"end_character":38},"in_reply_to":"1aa78d24_6a2ea4fb","updated":"2016-07-14 17:45:39.000000000","message":"It could but I prefer it this way just to be consistent in formatting else it looks weird. Please be advised that this formatting does not affect the doc http://docs-draft.openstack.org/77/332977/6/check/gate-nova-docs/99cf19b//doc/build/html/sample_config.html","commit_id":"87be86e0c609a033f3affd683edc21c9abf155d8"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"4c330f09234c08ce77ba7aee554218184aca759a","unresolved":false,"context_lines":[{"line_number":68,"context_line":"               deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"},{"line_number":69,"context_line":"               default\u003d\u0027notchecked\u0027,"},{"line_number":70,"context_line":"               help\u003d\"\"\""},{"line_number":71,"context_line":"Secret key to use for S3 server for images"},{"line_number":72,"context_line":"\"\"\"),"},{"line_number":73,"context_line":"    cfg.BoolOpt(\u0027s3_use_ssl\u0027,"},{"line_number":74,"context_line":"                deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":6,"id":"1aa78d24_ea3a9433","line":71,"updated":"2016-07-14 16:18:57.000000000","message":"Ditto","commit_id":"87be86e0c609a033f3affd683edc21c9abf155d8"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"4c330f09234c08ce77ba7aee554218184aca759a","unresolved":false,"context_lines":[{"line_number":75,"context_line":"                deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"},{"line_number":76,"context_line":"                default\u003dFalse,"},{"line_number":77,"context_line":"                help\u003d\"\"\""},{"line_number":78,"context_line":"Whether to use SSL when talking to S3"},{"line_number":79,"context_line":"\"\"\"),"},{"line_number":80,"context_line":"    cfg.BoolOpt(\u0027s3_affix_tenant\u0027,"},{"line_number":81,"context_line":"                deprecated_for_removal\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":6,"id":"1aa78d24_4a4808cd","line":78,"updated":"2016-07-14 16:18:57.000000000","message":"Ditto","commit_id":"87be86e0c609a033f3affd683edc21c9abf155d8"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"4c330f09234c08ce77ba7aee554218184aca759a","unresolved":false,"context_lines":[{"line_number":83,"context_line":"                default\u003dFalse,"},{"line_number":84,"context_line":"                help\u003d\"\"\""},{"line_number":85,"context_line":"Whether to affix the tenant id to the"},{"line_number":86,"context_line":"access key when downloading from S3"},{"line_number":87,"context_line":"\"\"\"),"},{"line_number":88,"context_line":"]"},{"line_number":89,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1aa78d24_4a1f68b7","line":86,"updated":"2016-07-14 16:18:57.000000000","message":"Ditto","commit_id":"87be86e0c609a033f3affd683edc21c9abf155d8"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"ba94306106851bf6bdd3c2ff907ce951dd34368b","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* IP address or Hostname"},{"line_number":44,"context_line":"\"\"\"),"},{"line_number":45,"context_line":"    cfg.IntOpt(\u0027s3_port\u0027,"},{"line_number":46,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":47,"context_line":"               deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"},{"line_number":48,"context_line":"               default\u003d3333,"}],"source_content_type":"text/x-python","patch_set":7,"id":"dada55a8_2840ba33","line":45,"range":{"start_line":45,"start_character":8,"end_line":45,"end_character":14},"updated":"2016-07-20 19:29:21.000000000","message":"Would be nice to change this to PortOpt. min and max params and possible values section can be dropped afterwards.","commit_id":"4e4fe59b58834873c3c93fe62357485daecc4bfb"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"6a6943e7e23094678159b03248237036bee3124a","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* IP address or Hostname"},{"line_number":44,"context_line":"\"\"\"),"},{"line_number":45,"context_line":"    cfg.IntOpt(\u0027s3_port\u0027,"},{"line_number":46,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":47,"context_line":"               deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"},{"line_number":48,"context_line":"               default\u003d3333,"}],"source_content_type":"text/x-python","patch_set":7,"id":"dada55a8_ece4bfac","line":45,"range":{"start_line":45,"start_character":8,"end_line":45,"end_character":14},"in_reply_to":"dada55a8_2840ba33","updated":"2016-07-21 17:04:44.000000000","message":"Done","commit_id":"4e4fe59b58834873c3c93fe62357485daecc4bfb"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"ba94306106851bf6bdd3c2ff907ce951dd34368b","unresolved":false,"context_lines":[{"line_number":60,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":61,"context_line":"               deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"},{"line_number":62,"context_line":"               default\u003d\u0027notchecked\u0027,"},{"line_number":63,"context_line":"               help\u003d\"\"\""},{"line_number":64,"context_line":"Access key to use S3 server for images"},{"line_number":65,"context_line":"\"\"\"),"},{"line_number":66,"context_line":"    cfg.StrOpt(\u0027s3_secret_key\u0027,"},{"line_number":67,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":68,"context_line":"               deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"dada55a8_486a4eb4","line":65,"range":{"start_line":63,"start_character":0,"end_line":65,"end_character":5},"updated":"2016-07-20 19:29:21.000000000","message":"This can be one-liner to avoid unnecessary blank lines in doc.","commit_id":"4e4fe59b58834873c3c93fe62357485daecc4bfb"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"6a6943e7e23094678159b03248237036bee3124a","unresolved":false,"context_lines":[{"line_number":60,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":61,"context_line":"               deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"},{"line_number":62,"context_line":"               default\u003d\u0027notchecked\u0027,"},{"line_number":63,"context_line":"               help\u003d\"\"\""},{"line_number":64,"context_line":"Access key to use S3 server for images"},{"line_number":65,"context_line":"\"\"\"),"},{"line_number":66,"context_line":"    cfg.StrOpt(\u0027s3_secret_key\u0027,"},{"line_number":67,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":68,"context_line":"               deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"dada55a8_8cd9c3f5","line":65,"range":{"start_line":63,"start_character":0,"end_line":65,"end_character":5},"in_reply_to":"dada55a8_486a4eb4","updated":"2016-07-21 17:04:44.000000000","message":"Done","commit_id":"4e4fe59b58834873c3c93fe62357485daecc4bfb"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"ba94306106851bf6bdd3c2ff907ce951dd34368b","unresolved":false,"context_lines":[{"line_number":67,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":68,"context_line":"               deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"},{"line_number":69,"context_line":"               default\u003d\u0027notchecked\u0027,"},{"line_number":70,"context_line":"               help\u003d\"\"\""},{"line_number":71,"context_line":"Secret key to use for S3 server for images"},{"line_number":72,"context_line":"\"\"\"),"},{"line_number":73,"context_line":"    cfg.BoolOpt(\u0027s3_use_ssl\u0027,"},{"line_number":74,"context_line":"                deprecated_for_removal\u003dTrue,"},{"line_number":75,"context_line":"                deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"dada55a8_28671aea","line":72,"range":{"start_line":70,"start_character":0,"end_line":72,"end_character":5},"updated":"2016-07-20 19:29:21.000000000","message":"Ditto","commit_id":"4e4fe59b58834873c3c93fe62357485daecc4bfb"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"6a6943e7e23094678159b03248237036bee3124a","unresolved":false,"context_lines":[{"line_number":67,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":68,"context_line":"               deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"},{"line_number":69,"context_line":"               default\u003d\u0027notchecked\u0027,"},{"line_number":70,"context_line":"               help\u003d\"\"\""},{"line_number":71,"context_line":"Secret key to use for S3 server for images"},{"line_number":72,"context_line":"\"\"\"),"},{"line_number":73,"context_line":"    cfg.BoolOpt(\u0027s3_use_ssl\u0027,"},{"line_number":74,"context_line":"                deprecated_for_removal\u003dTrue,"},{"line_number":75,"context_line":"                deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"dada55a8_0ccdd330","line":72,"range":{"start_line":70,"start_character":0,"end_line":72,"end_character":5},"in_reply_to":"dada55a8_28671aea","updated":"2016-07-21 17:04:44.000000000","message":"Done","commit_id":"4e4fe59b58834873c3c93fe62357485daecc4bfb"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"ba94306106851bf6bdd3c2ff907ce951dd34368b","unresolved":false,"context_lines":[{"line_number":74,"context_line":"                deprecated_for_removal\u003dTrue,"},{"line_number":75,"context_line":"                deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"},{"line_number":76,"context_line":"                default\u003dFalse,"},{"line_number":77,"context_line":"                help\u003d\"\"\""},{"line_number":78,"context_line":"Whether to use SSL when talking to S3"},{"line_number":79,"context_line":"\"\"\"),"},{"line_number":80,"context_line":"    cfg.BoolOpt(\u0027s3_affix_tenant\u0027,"},{"line_number":81,"context_line":"                deprecated_for_removal\u003dTrue,"},{"line_number":82,"context_line":"                deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"dada55a8_8860c6d0","line":79,"range":{"start_line":77,"start_character":0,"end_line":79,"end_character":5},"updated":"2016-07-20 19:29:21.000000000","message":"Ditto","commit_id":"4e4fe59b58834873c3c93fe62357485daecc4bfb"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"6a6943e7e23094678159b03248237036bee3124a","unresolved":false,"context_lines":[{"line_number":74,"context_line":"                deprecated_for_removal\u003dTrue,"},{"line_number":75,"context_line":"                deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"},{"line_number":76,"context_line":"                default\u003dFalse,"},{"line_number":77,"context_line":"                help\u003d\"\"\""},{"line_number":78,"context_line":"Whether to use SSL when talking to S3"},{"line_number":79,"context_line":"\"\"\"),"},{"line_number":80,"context_line":"    cfg.BoolOpt(\u0027s3_affix_tenant\u0027,"},{"line_number":81,"context_line":"                deprecated_for_removal\u003dTrue,"},{"line_number":82,"context_line":"                deprecated_reason\u003d\u0027EC2 API related options are not supported.\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"dada55a8_4cd34bd5","line":79,"range":{"start_line":77,"start_character":0,"end_line":79,"end_character":5},"in_reply_to":"dada55a8_8860c6d0","updated":"2016-07-21 17:04:44.000000000","message":"Done","commit_id":"4e4fe59b58834873c3c93fe62357485daecc4bfb"}]}