)]}'
{"nova/conf/cloudpipe.py":[{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"65348cbb9d6bcc386261c3bd9be7ddd29f5505bc","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":"    * m1.tiny (Default)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Related options:"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_2f24c8ad","line":43,"updated":"2016-06-27 20:45:36.000000000","message":"Can you specify the other possible values for this. Like m1.small, m1.medium etc","commit_id":"cc5f5dfc45f28cffd96078e41160087915a4b92d"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"376d34063239339cd028e43a2df99d0f60e13fe4","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":"    * m1.tiny (Default)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Related options:"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_d90b4b2a","line":43,"in_reply_to":"3aaa91ec_2f24c8ad","updated":"2016-06-27 22:33:46.000000000","message":"Done","commit_id":"cc5f5dfc45f28cffd96078e41160087915a4b92d"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"65348cbb9d6bcc386261c3bd9be7ddd29f5505bc","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Possible values:"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    * \u0027nova/cloudpipe/bootscript.template\u0027 (default)"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Related options:"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_6f21b0b4","line":58,"updated":"2016-06-27 20:45:36.000000000","message":"$pybasedir/nova/cloudpipe/bootscript.template is the default value for this option.\n\nwhere pybasedir is a config option specifying the directory where the nova python module is installed.","commit_id":"cc5f5dfc45f28cffd96078e41160087915a4b92d"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"376d34063239339cd028e43a2df99d0f60e13fe4","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Possible values:"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    * \u0027nova/cloudpipe/bootscript.template\u0027 (default)"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Related options:"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_9911c360","line":58,"in_reply_to":"3aaa91ec_6f21b0b4","updated":"2016-06-27 22:33:46.000000000","message":"Done","commit_id":"cc5f5dfc45f28cffd96078e41160087915a4b92d"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"4c92adfc943aa991faeece2d23ea91b429a2f8a9","unresolved":false,"context_lines":[{"line_number":73,"context_line":"Possible values:"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    * Any valid IP address"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    * \u002710.0.0.0\u0027 (default)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"Related options:"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a9d85d2_6654f920","line":76,"updated":"2016-06-21 13:39:10.000000000","message":"nit: redundant new line is redundant","commit_id":"cc5f5dfc45f28cffd96078e41160087915a4b92d"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"376d34063239339cd028e43a2df99d0f60e13fe4","unresolved":false,"context_lines":[{"line_number":73,"context_line":"Possible values:"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    * Any valid IP address"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    * \u002710.0.0.0\u0027 (default)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"Related options:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_79169756","line":76,"in_reply_to":"5a9d85d2_6654f920","updated":"2016-06-27 22:33:46.000000000","message":"Done","commit_id":"cc5f5dfc45f28cffd96078e41160087915a4b92d"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"366b67a47c1ca73f14e947eaf9b6c1374601561b","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        \u0027vpn_flavor\u0027,"},{"line_number":37,"context_line":"        default\u003d\u0027m1.tiny\u0027,"},{"line_number":38,"context_line":"        help\u003d\"\"\""},{"line_number":39,"context_line":"This options helps you specify flavor for vpn instances"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Possible values:"},{"line_number":42,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_c04f9968","line":39,"updated":"2016-06-28 18:21:10.000000000","message":"nit: s/options/option","commit_id":"2fdb0cd56069541aa8f97450047a6c9b337f2d1b"},{"author":{"_account_id":14511,"name":"Maciej Szankin","email":"maciej+review@szankin.pl","username":"mszankin"},"change_message_id":"77b7b53bbe478b04b6618071c5b0c34745b9bf48","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":"    * m1.tiny (Default), m1.small, m1.medium, m1.large, m1.xlarge"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Related options:"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_0b3400f9","line":43,"range":{"start_line":43,"start_character":25,"end_line":43,"end_character":65},"updated":"2016-06-29 11:58:17.000000000","message":"I think that every flavor that is available in deployment is a possibility here. The list you provided is just a default set that can be expanded by adding new flavors.\n\nQuestion to anyone who knows: is the set provided in this patch a default for all new deployments, or can it differ? If it can, the best option would be to just note that possible value is any flavor name.","commit_id":"2fdb0cd56069541aa8f97450047a6c9b337f2d1b"},{"author":{"_account_id":18335,"name":"Pushkar Umaranikar","email":"pushkar.umaranikar@intel.com","username":"pushkarumaranikar"},"change_message_id":"122b4de29a3fdcad6f51bea84d266cd0a8fb0336","unresolved":false,"context_lines":[{"line_number":60,"context_line":"      specifying the directory where the nova python module is"},{"line_number":61,"context_line":"      installed."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Related options:"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    * dmz_net"},{"line_number":66,"context_line":"    * dmz_mask"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_24479f78","line":63,"range":{"start_line":63,"start_character":0,"end_line":63,"end_character":16},"updated":"2016-06-28 14:29:33.000000000","message":"Can we add \"pybasedir\" from path_opts as related option here ?","commit_id":"2fdb0cd56069541aa8f97450047a6c9b337f2d1b"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"366b67a47c1ca73f14e947eaf9b6c1374601561b","unresolved":false,"context_lines":[{"line_number":110,"context_line":"    * Any string value"},{"line_number":111,"context_line":"    * \u0027-vpn\u0027 (Default)"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Related options:"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"    * None"},{"line_number":116,"context_line":"\"\"\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_a0588d57","line":113,"updated":"2016-06-28 18:21:10.000000000","message":"This can be removed if you are submitting a new patch.","commit_id":"2fdb0cd56069541aa8f97450047a6c9b337f2d1b"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"653476e9476121c14cdf1328c2d31898f68a6ffe","unresolved":false,"context_lines":[{"line_number":21,"context_line":"        \u0027vpn_image_id\u0027,"},{"line_number":22,"context_line":"        default\u003d\u00270\u0027,"},{"line_number":23,"context_line":"        help\u003d\"\"\""},{"line_number":24,"context_line":"This option helps you specify image ID to use when starting up"},{"line_number":25,"context_line":"cloudpipe vpn server."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Possible values:"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_b07fa2a9","line":25,"range":{"start_line":24,"start_character":0,"end_line":25,"end_character":21},"updated":"2016-07-01 09:41:21.000000000","message":"I think the original wording of this was better. Rather than simply rewording this, could you add more information?","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"32d0e55f3d06ba75d8be69fb8163762abe8c5dab","unresolved":false,"context_lines":[{"line_number":21,"context_line":"        \u0027vpn_image_id\u0027,"},{"line_number":22,"context_line":"        default\u003d\u00270\u0027,"},{"line_number":23,"context_line":"        help\u003d\"\"\""},{"line_number":24,"context_line":"This option helps you specify image ID to use when starting up"},{"line_number":25,"context_line":"cloudpipe vpn server."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Possible values:"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_d485b7ad","line":25,"range":{"start_line":24,"start_character":0,"end_line":25,"end_character":21},"in_reply_to":"3aaa91ec_b07fa2a9","updated":"2016-07-08 21:27:37.000000000","message":"I have reverted it back to the original wording. I have tried to add more info wherever possible. If you could get me some more info, I would be happy to add it as well.","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"653476e9476121c14cdf1328c2d31898f68a6ffe","unresolved":false,"context_lines":[{"line_number":24,"context_line":"This option helps you specify image ID to use when starting up"},{"line_number":25,"context_line":"cloudpipe vpn server."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Possible values:"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    * 0 (Default)"},{"line_number":30,"context_line":"\"\"\"),"},{"line_number":31,"context_line":"    cfg.StrOpt("},{"line_number":32,"context_line":"        \u0027vpn_flavor\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_932d5091","line":29,"range":{"start_line":27,"start_character":0,"end_line":29,"end_character":17},"updated":"2016-07-01 09:41:21.000000000","message":"More values than 0 are supported. How about:\n\n    * Any valid ID for a Glance image\n\n...assuming this is Glance images we\u0027re talking about? I don\u0027t know, and it really should be detailed above :)","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"32d0e55f3d06ba75d8be69fb8163762abe8c5dab","unresolved":false,"context_lines":[{"line_number":24,"context_line":"This option helps you specify image ID to use when starting up"},{"line_number":25,"context_line":"cloudpipe vpn server."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Possible values:"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    * 0 (Default)"},{"line_number":30,"context_line":"\"\"\"),"},{"line_number":31,"context_line":"    cfg.StrOpt("},{"line_number":32,"context_line":"        \u0027vpn_flavor\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_34935bfc","line":29,"range":{"start_line":27,"start_character":0,"end_line":29,"end_character":17},"in_reply_to":"3aaa91ec_932d5091","updated":"2016-07-08 21:27:37.000000000","message":"Done","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"653476e9476121c14cdf1328c2d31898f68a6ffe","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Possible values:"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    * m1.tiny (Default), m1.small, m1.medium, m1.large, m1.xlarge"},{"line_number":40,"context_line":"    * Any valid flavor names that were added later"},{"line_number":41,"context_line":"\"\"\"),"},{"line_number":42,"context_line":"    cfg.StrOpt("}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_33561cce","line":39,"range":{"start_line":39,"start_character":4,"end_line":39,"end_character":65},"updated":"2016-07-01 09:41:21.000000000","message":"These are flavors defined by DevStack - not Nova. As such, they\u0027re not guaranteed to be present in a deployment. Drop this and reword the below to something like:\n\n    * Any valid flavor name\n\nAlso, remove indentation so this is consistent with other files","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"32d0e55f3d06ba75d8be69fb8163762abe8c5dab","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Possible values:"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    * m1.tiny (Default), m1.small, m1.medium, m1.large, m1.xlarge"},{"line_number":40,"context_line":"    * Any valid flavor names that were added later"},{"line_number":41,"context_line":"\"\"\"),"},{"line_number":42,"context_line":"    cfg.StrOpt("}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_1490dfeb","line":39,"range":{"start_line":39,"start_character":4,"end_line":39,"end_character":65},"in_reply_to":"3aaa91ec_33561cce","updated":"2016-07-08 21:27:37.000000000","message":"Done","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"653476e9476121c14cdf1328c2d31898f68a6ffe","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Possible values:"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    * $pybasedir/nova/cloudpipe/bootscript.template is the default"},{"line_number":52,"context_line":"      value for this option, where pybasedir is a config option"},{"line_number":53,"context_line":"      specifying the directory where the nova python module is"},{"line_number":54,"context_line":"      installed."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Related options:"},{"line_number":57,"context_line":"    * pybasedir - Directory where the nova python module in installed"}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_13e840b2","line":54,"range":{"start_line":51,"start_character":4,"end_line":54,"end_character":16},"updated":"2016-07-01 09:41:21.000000000","message":"We already know this by looking at value for \u0027default\u0027 (which \u0027nova.conf\u0027 will be prepopulated with) :)\nInstead, say something like:\n\n    * Any valid path to a CloudPipe instance boot script template","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"32d0e55f3d06ba75d8be69fb8163762abe8c5dab","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Possible values:"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    * $pybasedir/nova/cloudpipe/bootscript.template is the default"},{"line_number":52,"context_line":"      value for this option, where pybasedir is a config option"},{"line_number":53,"context_line":"      specifying the directory where the nova python module is"},{"line_number":54,"context_line":"      installed."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Related options:"},{"line_number":57,"context_line":"    * pybasedir - Directory where the nova python module in installed"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_74bb2376","line":54,"range":{"start_line":51,"start_character":4,"end_line":54,"end_character":16},"in_reply_to":"3aaa91ec_13e840b2","updated":"2016-07-08 21:27:37.000000000","message":"Done","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"653476e9476121c14cdf1328c2d31898f68a6ffe","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Related options:"},{"line_number":57,"context_line":"    * pybasedir - Directory where the nova python module in installed"},{"line_number":58,"context_line":"    * dmz_net"},{"line_number":59,"context_line":"    * dmz_mask"},{"line_number":60,"context_line":"    * cnt_vpn_clients"},{"line_number":61,"context_line":"\"\"\"),"},{"line_number":62,"context_line":"    cfg.StrOpt("},{"line_number":63,"context_line":"        \u0027dmz_net\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_33fc5c71","line":60,"range":{"start_line":58,"start_character":0,"end_line":60,"end_character":21},"updated":"2016-07-01 09:41:21.000000000","message":"How are these related?","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"32d0e55f3d06ba75d8be69fb8163762abe8c5dab","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Related options:"},{"line_number":57,"context_line":"    * pybasedir - Directory where the nova python module in installed"},{"line_number":58,"context_line":"    * dmz_net"},{"line_number":59,"context_line":"    * dmz_mask"},{"line_number":60,"context_line":"    * cnt_vpn_clients"},{"line_number":61,"context_line":"\"\"\"),"},{"line_number":62,"context_line":"    cfg.StrOpt("},{"line_number":63,"context_line":"        \u0027dmz_net\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_54c0a7e0","line":60,"range":{"start_line":58,"start_character":0,"end_line":60,"end_character":21},"in_reply_to":"3aaa91ec_33fc5c71","updated":"2016-07-08 21:27:37.000000000","message":"Done","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"653476e9476121c14cdf1328c2d31898f68a6ffe","unresolved":false,"context_lines":[{"line_number":63,"context_line":"        \u0027dmz_net\u0027,"},{"line_number":64,"context_line":"        default\u003d\u002710.0.0.0\u0027,"},{"line_number":65,"context_line":"        help\u003d\"\"\""},{"line_number":66,"context_line":"This option allows you to specify network address that needs to"},{"line_number":67,"context_line":"be pushed into openvpn config."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"Possible values:"},{"line_number":70,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_b3f04c92","line":67,"range":{"start_line":66,"start_character":0,"end_line":67,"end_character":30},"updated":"2016-07-01 09:41:21.000000000","message":"ditto: don\u0027t reword - add more info instead","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"653476e9476121c14cdf1328c2d31898f68a6ffe","unresolved":false,"context_lines":[{"line_number":69,"context_line":"Possible values:"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    * Any valid IP address"},{"line_number":72,"context_line":"    * \u002710.0.0.0\u0027 (default)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Related options:"},{"line_number":75,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_d32fd8f3","line":72,"range":{"start_line":72,"start_character":0,"end_line":72,"end_character":26},"updated":"2016-07-01 09:41:21.000000000","message":"Drop this.","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"32d0e55f3d06ba75d8be69fb8163762abe8c5dab","unresolved":false,"context_lines":[{"line_number":69,"context_line":"Possible values:"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    * Any valid IP address"},{"line_number":72,"context_line":"    * \u002710.0.0.0\u0027 (default)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Related options:"},{"line_number":75,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_b4b58b7f","line":72,"range":{"start_line":72,"start_character":0,"end_line":72,"end_character":26},"in_reply_to":"3aaa91ec_d32fd8f3","updated":"2016-07-08 21:27:37.000000000","message":"Done","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"653476e9476121c14cdf1328c2d31898f68a6ffe","unresolved":false,"context_lines":[{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Related options:"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    * boot_script_template"},{"line_number":77,"context_line":"\"\"\"),"},{"line_number":78,"context_line":"    cfg.StrOpt("},{"line_number":79,"context_line":"        \u0027dmz_mask\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_7341c4a8","line":76,"range":{"start_line":76,"start_character":0,"end_line":76,"end_character":26},"updated":"2016-07-01 09:41:21.000000000","message":"related how?","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"32d0e55f3d06ba75d8be69fb8163762abe8c5dab","unresolved":false,"context_lines":[{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Related options:"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    * boot_script_template"},{"line_number":77,"context_line":"\"\"\"),"},{"line_number":78,"context_line":"    cfg.StrOpt("},{"line_number":79,"context_line":"        \u0027dmz_mask\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_94ba0f6d","line":76,"range":{"start_line":76,"start_character":0,"end_line":76,"end_character":26},"in_reply_to":"3aaa91ec_7341c4a8","updated":"2016-07-08 21:27:37.000000000","message":"Done","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"653476e9476121c14cdf1328c2d31898f68a6ffe","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        \u0027dmz_mask\u0027,"},{"line_number":80,"context_line":"        default\u003d\u0027255.255.255.0\u0027,"},{"line_number":81,"context_line":"        help\u003d\"\"\""},{"line_number":82,"context_line":"This option allows you to specify subnet mask that needs to"},{"line_number":83,"context_line":"be pushed into openvpn config."},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Possible values:"},{"line_number":86,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_f3bdb468","line":83,"range":{"start_line":82,"start_character":0,"end_line":83,"end_character":30},"updated":"2016-07-01 09:41:21.000000000","message":"Don\u0027t reword - expand","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"32d0e55f3d06ba75d8be69fb8163762abe8c5dab","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        \u0027dmz_mask\u0027,"},{"line_number":80,"context_line":"        default\u003d\u0027255.255.255.0\u0027,"},{"line_number":81,"context_line":"        help\u003d\"\"\""},{"line_number":82,"context_line":"This option allows you to specify subnet mask that needs to"},{"line_number":83,"context_line":"be pushed into openvpn config."},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Possible values:"},{"line_number":86,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_14a73f50","line":83,"range":{"start_line":82,"start_character":0,"end_line":83,"end_character":30},"in_reply_to":"3aaa91ec_f3bdb468","updated":"2016-07-08 21:27:37.000000000","message":"Done","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"653476e9476121c14cdf1328c2d31898f68a6ffe","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Possible values:"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    * \u0027255.255.255.0\u0027 (default)"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"Related options:"},{"line_number":90,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_13132066","line":87,"range":{"start_line":87,"start_character":0,"end_line":87,"end_character":31},"updated":"2016-07-01 09:41:21.000000000","message":"Is IPv6 supported? If not:\n\n    * Any valid IPv4 netmask\n\nIf so, the paragraph above needs to state as much","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"32d0e55f3d06ba75d8be69fb8163762abe8c5dab","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Possible values:"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    * \u0027255.255.255.0\u0027 (default)"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"Related options:"},{"line_number":90,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_74620318","line":87,"range":{"start_line":87,"start_character":0,"end_line":87,"end_character":31},"in_reply_to":"3aaa91ec_13132066","updated":"2016-07-08 21:27:37.000000000","message":"Done","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"653476e9476121c14cdf1328c2d31898f68a6ffe","unresolved":false,"context_lines":[{"line_number":95,"context_line":"        \u0027vpn_key_suffix\u0027,"},{"line_number":96,"context_line":"        default\u003d\u0027-vpn\u0027,"},{"line_number":97,"context_line":"        help\u003d\"\"\""},{"line_number":98,"context_line":"This option allows you to specify suffix to be added to the vpn"},{"line_number":99,"context_line":"instances, key pair and security groups."},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"Possible values:"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_f39654ee","line":99,"range":{"start_line":98,"start_character":0,"end_line":99,"end_character":40},"updated":"2016-07-01 09:41:21.000000000","message":"Don\u0027t reword - expand.","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"32d0e55f3d06ba75d8be69fb8163762abe8c5dab","unresolved":false,"context_lines":[{"line_number":95,"context_line":"        \u0027vpn_key_suffix\u0027,"},{"line_number":96,"context_line":"        default\u003d\u0027-vpn\u0027,"},{"line_number":97,"context_line":"        help\u003d\"\"\""},{"line_number":98,"context_line":"This option allows you to specify suffix to be added to the vpn"},{"line_number":99,"context_line":"instances, key pair and security groups."},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"Possible values:"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_b46c6b09","line":99,"range":{"start_line":98,"start_character":0,"end_line":99,"end_character":40},"in_reply_to":"3aaa91ec_f39654ee","updated":"2016-07-08 21:27:37.000000000","message":"Done","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"653476e9476121c14cdf1328c2d31898f68a6ffe","unresolved":false,"context_lines":[{"line_number":100,"context_line":""},{"line_number":101,"context_line":"Possible values:"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"    * Any string value"},{"line_number":104,"context_line":"    * \u0027-vpn\u0027 (Default)"},{"line_number":105,"context_line":"\"\"\")"},{"line_number":106,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_f3c81424","line":103,"range":{"start_line":103,"start_character":0,"end_line":103,"end_character":22},"updated":"2016-07-01 09:41:21.000000000","message":"*Any* string value? Surely there are limits on what can be passed in here?","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"32d0e55f3d06ba75d8be69fb8163762abe8c5dab","unresolved":false,"context_lines":[{"line_number":100,"context_line":""},{"line_number":101,"context_line":"Possible values:"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"    * Any string value"},{"line_number":104,"context_line":"    * \u0027-vpn\u0027 (Default)"},{"line_number":105,"context_line":"\"\"\")"},{"line_number":106,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_94716fe3","line":103,"range":{"start_line":103,"start_character":0,"end_line":103,"end_character":22},"in_reply_to":"3aaa91ec_f3c81424","updated":"2016-07-08 21:27:37.000000000","message":"Done","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"653476e9476121c14cdf1328c2d31898f68a6ffe","unresolved":false,"context_lines":[{"line_number":101,"context_line":"Possible values:"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"    * Any string value"},{"line_number":104,"context_line":"    * \u0027-vpn\u0027 (Default)"},{"line_number":105,"context_line":"\"\"\")"},{"line_number":106,"context_line":"]"},{"line_number":107,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_934270a9","line":104,"range":{"start_line":104,"start_character":0,"end_line":104,"end_character":22},"updated":"2016-07-01 09:41:21.000000000","message":"Drop this","commit_id":"97f1f9d1d6f50d34ded2721d67f74371ca3e0574"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"7f09f33349d345c71fee280ba62aa9d21b785de7","unresolved":false,"context_lines":[{"line_number":24,"context_line":"Image ID used when starting up a cloudpipe VPN client."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"An empty instance is created and configured with OpenVPN using"},{"line_number":27,"context_line":"boot_script_template. This instance would be snahpshotted and stored"},{"line_number":28,"context_line":"in glance. ID of the stored image is used in \u0027vpn_image_id\u0027 to"},{"line_number":29,"context_line":"create cloudpipe VPN client."},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1aa78d24_ca2219a3","line":27,"range":{"start_line":27,"start_character":45,"end_line":27,"end_character":57},"updated":"2016-07-10 16:58:28.000000000","message":"snapshotted","commit_id":"37144bf63ce51363be301561dafdd4f99caad960"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"271c8e7ad90d287d0bc17dddb281ed78b3a95165","unresolved":false,"context_lines":[{"line_number":24,"context_line":"Image ID used when starting up a cloudpipe VPN client."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"An empty instance is created and configured with OpenVPN using"},{"line_number":27,"context_line":"boot_script_template. This instance would be snahpshotted and stored"},{"line_number":28,"context_line":"in glance. ID of the stored image is used in \u0027vpn_image_id\u0027 to"},{"line_number":29,"context_line":"create cloudpipe VPN client."},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1aa78d24_6ae03bcd","line":27,"range":{"start_line":27,"start_character":45,"end_line":27,"end_character":57},"in_reply_to":"1aa78d24_ca2219a3","updated":"2016-07-11 15:26:11.000000000","message":"Done","commit_id":"37144bf63ce51363be301561dafdd4f99caad960"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"7f09f33349d345c71fee280ba62aa9d21b785de7","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        help\u003d\"\"\""},{"line_number":66,"context_line":"Network to push into OpenVPN config."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"Configuration file server.conf can be found at /etc/openvpn"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Possible values:"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1aa78d24_2a411d3a","line":68,"range":{"start_line":68,"start_character":0,"end_line":68,"end_character":59},"updated":"2016-07-10 16:58:28.000000000","message":"Please explain the context of this sentence.","commit_id":"37144bf63ce51363be301561dafdd4f99caad960"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"7f09f33349d345c71fee280ba62aa9d21b785de7","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        help\u003d\"\"\""},{"line_number":66,"context_line":"Network to push into OpenVPN config."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"Configuration file server.conf can be found at /etc/openvpn"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Possible values:"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1aa78d24_8a4f7169","line":68,"range":{"start_line":68,"start_character":0,"end_line":68,"end_character":13},"updated":"2016-07-10 16:58:28.000000000","message":"The configuration file","commit_id":"37144bf63ce51363be301561dafdd4f99caad960"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"271c8e7ad90d287d0bc17dddb281ed78b3a95165","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        help\u003d\"\"\""},{"line_number":66,"context_line":"Network to push into OpenVPN config."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"Configuration file server.conf can be found at /etc/openvpn"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Possible values:"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1aa78d24_0a4097a4","line":68,"range":{"start_line":68,"start_character":0,"end_line":68,"end_character":13},"in_reply_to":"1aa78d24_8a4f7169","updated":"2016-07-11 15:26:11.000000000","message":"Please let me know if current change looks good.","commit_id":"37144bf63ce51363be301561dafdd4f99caad960"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"7f09f33349d345c71fee280ba62aa9d21b785de7","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Possible values:"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"* Any valid IPv4/IPV6  netmask"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"Related options:"},{"line_number":90,"context_line":"dmz_net and dmz_mask is pushed into bootscript.template to configure"}],"source_content_type":"text/x-python","patch_set":5,"id":"1aa78d24_2a3abdcd","line":87,"range":{"start_line":87,"start_character":22,"end_line":87,"end_character":23},"updated":"2016-07-10 16:58:28.000000000","message":"remove extra space","commit_id":"37144bf63ce51363be301561dafdd4f99caad960"},{"author":{"_account_id":18339,"name":"Sivasathurappan Radhakrishnan","email":"siva.radhakrishnan@intel.com","username":"sivakrishnan"},"change_message_id":"271c8e7ad90d287d0bc17dddb281ed78b3a95165","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Possible values:"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"* Any valid IPv4/IPV6  netmask"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"Related options:"},{"line_number":90,"context_line":"dmz_net and dmz_mask is pushed into bootscript.template to configure"}],"source_content_type":"text/x-python","patch_set":5,"id":"1aa78d24_ea3a6b33","line":87,"range":{"start_line":87,"start_character":22,"end_line":87,"end_character":23},"in_reply_to":"1aa78d24_2a3abdcd","updated":"2016-07-11 15:26:11.000000000","message":"Done","commit_id":"37144bf63ce51363be301561dafdd4f99caad960"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"7f09f33349d345c71fee280ba62aa9d21b785de7","unresolved":false,"context_lines":[{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Possible values:"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"* Any string value representing vpn key suffix"},{"line_number":105,"context_line":"\"\"\")"},{"line_number":106,"context_line":"]"},{"line_number":107,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1aa78d24_aa4ced76","line":104,"range":{"start_line":104,"start_character":32,"end_line":104,"end_character":35},"updated":"2016-07-10 16:58:28.000000000","message":"the VPN","commit_id":"37144bf63ce51363be301561dafdd4f99caad960"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"7d724ba0c7c750eb771f80d9e996d8860f42794f","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Related options:"},{"line_number":58,"context_line":"Following options are required to configure cloudpipe-managed"},{"line_number":59,"context_line":"OpenVPN server."},{"line_number":60,"context_line":"* dmz_net"},{"line_number":61,"context_line":"* dmz_mask"},{"line_number":62,"context_line":"* cnt_vpn_clients"}],"source_content_type":"text/x-python","patch_set":8,"id":"bacf61ea_6632e45f","line":59,"updated":"2016-08-05 09:12:57.000000000","message":"Nit: I am not over the moon about the formatting in here. But the information is here, so I will not block on that.","commit_id":"850874c955d859f0d1f6001d43b4ea6a453c4ebc"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"7d724ba0c7c750eb771f80d9e996d8860f42794f","unresolved":false,"context_lines":[{"line_number":77,"context_line":"Related options:"},{"line_number":78,"context_line":"dmz_net is pushed into bootscript.template to configure"},{"line_number":79,"context_line":"cloudpipe-managed OpenVPN server"},{"line_number":80,"context_line":"* boot_script_template"},{"line_number":81,"context_line":"\"\"\"),"},{"line_number":82,"context_line":"    cfg.IPOpt("},{"line_number":83,"context_line":"        \u0027dmz_mask\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"bacf61ea_4637e04c","line":80,"updated":"2016-08-05 09:12:57.000000000","message":"As above.","commit_id":"850874c955d859f0d1f6001d43b4ea6a453c4ebc"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"7d724ba0c7c750eb771f80d9e996d8860f42794f","unresolved":false,"context_lines":[{"line_number":93,"context_line":"dmz_net and dmz_mask is pushed into bootscript.template to configure"},{"line_number":94,"context_line":"cloudpipe-managed OpenVPN server"},{"line_number":95,"context_line":"* dmz_net"},{"line_number":96,"context_line":"* boot_script_template"},{"line_number":97,"context_line":"\"\"\"),"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    cfg.StrOpt("}],"source_content_type":"text/x-python","patch_set":8,"id":"bacf61ea_863d582d","line":96,"updated":"2016-08-05 09:12:57.000000000","message":"As above.","commit_id":"850874c955d859f0d1f6001d43b4ea6a453c4ebc"}]}