)]}'
{"nova/conf/xenserver.py":[{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"a1631da62fc4ea01e9df2d9d3f4b43808e8218d3","unresolved":false,"context_lines":[{"line_number":269,"context_line":"              image_property `cache_in_nova\u003dTrue`."},{"line_number":270,"context_line":"    * `none`: turns off caching entirely."},{"line_number":271,"context_line":"\"\"\"),"},{"line_number":272,"context_line":"    cfg.IntOpt(\u0027image_compression_level\u0027,"},{"line_number":273,"context_line":"               min\u003d1,"},{"line_number":274,"context_line":"               max\u003d9,"},{"line_number":275,"context_line":"               help\u003d\"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_6fca5351","line":272,"range":{"start_line":272,"start_character":8,"end_line":272,"end_character":14},"updated":"2016-07-27 21:36:20.000000000","message":"Is \u0027None\u0027 default for all IntOpt\u0027s?","commit_id":"e936dc61150af42c7ae0aa4d2625428db4d08596"},{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"b466c4d9975a52eb82917fcea31e927ea754cd12","unresolved":false,"context_lines":[{"line_number":269,"context_line":"              image_property `cache_in_nova\u003dTrue`."},{"line_number":270,"context_line":"    * `none`: turns off caching entirely."},{"line_number":271,"context_line":"\"\"\"),"},{"line_number":272,"context_line":"    cfg.IntOpt(\u0027image_compression_level\u0027,"},{"line_number":273,"context_line":"               min\u003d1,"},{"line_number":274,"context_line":"               max\u003d9,"},{"line_number":275,"context_line":"               help\u003d\"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_24187d84","line":272,"range":{"start_line":272,"start_character":8,"end_line":272,"end_character":14},"in_reply_to":"bacf61ea_6fca5351","updated":"2016-08-01 15:28:27.000000000","message":"Yes, i think if there is no default value set, none is the default for all opts not only IntOpts.","commit_id":"e936dc61150af42c7ae0aa4d2625428db4d08596"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"a1631da62fc4ea01e9df2d9d3f4b43808e8218d3","unresolved":false,"context_lines":[{"line_number":293,"context_line":"    cfg.IntOpt(\u0027block_device_creation_timeout\u0027,"},{"line_number":294,"context_line":"               default\u003d10,"},{"line_number":295,"context_line":"               min\u003d1,"},{"line_number":296,"context_line":"               help\u003d\u0027Time to wait for a block device to be created\u0027),"},{"line_number":297,"context_line":"    cfg.IntOpt(\u0027max_kernel_ramdisk_size\u0027,"},{"line_number":298,"context_line":"               default\u003d16 * units.Mi,"},{"line_number":299,"context_line":"               help\u003d\"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_ef8e4326","line":296,"range":{"start_line":296,"start_character":21,"end_line":296,"end_character":25},"updated":"2016-07-27 21:36:20.000000000","message":"What are the units of this time?","commit_id":"e936dc61150af42c7ae0aa4d2625428db4d08596"},{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"b466c4d9975a52eb82917fcea31e927ea754cd12","unresolved":false,"context_lines":[{"line_number":293,"context_line":"    cfg.IntOpt(\u0027block_device_creation_timeout\u0027,"},{"line_number":294,"context_line":"               default\u003d10,"},{"line_number":295,"context_line":"               min\u003d1,"},{"line_number":296,"context_line":"               help\u003d\u0027Time to wait for a block device to be created\u0027),"},{"line_number":297,"context_line":"    cfg.IntOpt(\u0027max_kernel_ramdisk_size\u0027,"},{"line_number":298,"context_line":"               default\u003d16 * units.Mi,"},{"line_number":299,"context_line":"               help\u003d\"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_a40c6d43","line":296,"range":{"start_line":296,"start_character":21,"end_line":296,"end_character":25},"in_reply_to":"bacf61ea_ef8e4326","updated":"2016-08-01 15:28:27.000000000","message":"Done","commit_id":"e936dc61150af42c7ae0aa4d2625428db4d08596"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"a1631da62fc4ea01e9df2d9d3f4b43808e8218d3","unresolved":false,"context_lines":[{"line_number":351,"context_line":"Name of network to use for booting iPXE ISOs."},{"line_number":352,"context_line":""},{"line_number":353,"context_line":"An iPXE ISO is a specially crafted ISO which supports iPXE booting."},{"line_number":354,"context_line":"This feature gives a means to roll our own image. To use this"},{"line_number":355,"context_line":"feature the config options `ipxe_network_name`, `ipxe_boot_menu_url`"},{"line_number":356,"context_line":"and `ipxe_mkisofs_cmd` should be configured."},{"line_number":357,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_2f35fb45","line":354,"range":{"start_line":354,"start_character":35,"end_line":354,"end_character":38},"updated":"2016-07-27 21:36:20.000000000","message":"your","commit_id":"e936dc61150af42c7ae0aa4d2625428db4d08596"},{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"b466c4d9975a52eb82917fcea31e927ea754cd12","unresolved":false,"context_lines":[{"line_number":351,"context_line":"Name of network to use for booting iPXE ISOs."},{"line_number":352,"context_line":""},{"line_number":353,"context_line":"An iPXE ISO is a specially crafted ISO which supports iPXE booting."},{"line_number":354,"context_line":"This feature gives a means to roll our own image. To use this"},{"line_number":355,"context_line":"feature the config options `ipxe_network_name`, `ipxe_boot_menu_url`"},{"line_number":356,"context_line":"and `ipxe_mkisofs_cmd` should be configured."},{"line_number":357,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_84096934","line":354,"range":{"start_line":354,"start_character":35,"end_line":354,"end_character":38},"in_reply_to":"bacf61ea_2f35fb45","updated":"2016-08-01 15:28:27.000000000","message":"Done","commit_id":"e936dc61150af42c7ae0aa4d2625428db4d08596"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"a1631da62fc4ea01e9df2d9d3f4b43808e8218d3","unresolved":false,"context_lines":[{"line_number":352,"context_line":""},{"line_number":353,"context_line":"An iPXE ISO is a specially crafted ISO which supports iPXE booting."},{"line_number":354,"context_line":"This feature gives a means to roll our own image. To use this"},{"line_number":355,"context_line":"feature the config options `ipxe_network_name`, `ipxe_boot_menu_url`"},{"line_number":356,"context_line":"and `ipxe_mkisofs_cmd` should be configured."},{"line_number":357,"context_line":""},{"line_number":358,"context_line":"By default this option is not set. Enable this option to"},{"line_number":359,"context_line":"boot an iPXE ISO."}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_ef1123b1","line":356,"range":{"start_line":355,"start_character":27,"end_line":356,"end_character":22},"updated":"2016-07-27 21:36:20.000000000","message":"Shouldn\u0027t we add these options in Related options section?","commit_id":"e936dc61150af42c7ae0aa4d2625428db4d08596"},{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"b466c4d9975a52eb82917fcea31e927ea754cd12","unresolved":false,"context_lines":[{"line_number":352,"context_line":""},{"line_number":353,"context_line":"An iPXE ISO is a specially crafted ISO which supports iPXE booting."},{"line_number":354,"context_line":"This feature gives a means to roll our own image. To use this"},{"line_number":355,"context_line":"feature the config options `ipxe_network_name`, `ipxe_boot_menu_url`"},{"line_number":356,"context_line":"and `ipxe_mkisofs_cmd` should be configured."},{"line_number":357,"context_line":""},{"line_number":358,"context_line":"By default this option is not set. Enable this option to"},{"line_number":359,"context_line":"boot an iPXE ISO."}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_4cec04f3","line":356,"range":{"start_line":355,"start_character":27,"end_line":356,"end_character":22},"in_reply_to":"bacf61ea_ef1123b1","updated":"2016-08-01 15:28:27.000000000","message":"Done","commit_id":"e936dc61150af42c7ae0aa4d2625428db4d08596"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"a1631da62fc4ea01e9df2d9d3f4b43808e8218d3","unresolved":false,"context_lines":[{"line_number":363,"context_line":"URL to the iPXE boot menu."},{"line_number":364,"context_line":""},{"line_number":365,"context_line":"An iPXE ISO is a specially crafted ISO which supports iPXE booting."},{"line_number":366,"context_line":"This feature gives a means to roll our own image. To use this"},{"line_number":367,"context_line":"feature the config options `ipxe_network_name`, `ipxe_boot_menu_url`"},{"line_number":368,"context_line":"and `ipxe_mkisofs_cmd` should be configured."},{"line_number":369,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_0f1117ad","line":366,"range":{"start_line":366,"start_character":35,"end_line":366,"end_character":38},"updated":"2016-07-27 21:36:20.000000000","message":"your","commit_id":"e936dc61150af42c7ae0aa4d2625428db4d08596"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"a1631da62fc4ea01e9df2d9d3f4b43808e8218d3","unresolved":false,"context_lines":[{"line_number":364,"context_line":""},{"line_number":365,"context_line":"An iPXE ISO is a specially crafted ISO which supports iPXE booting."},{"line_number":366,"context_line":"This feature gives a means to roll our own image. To use this"},{"line_number":367,"context_line":"feature the config options `ipxe_network_name`, `ipxe_boot_menu_url`"},{"line_number":368,"context_line":"and `ipxe_mkisofs_cmd` should be configured."},{"line_number":369,"context_line":""},{"line_number":370,"context_line":"By default this option is not set. Enable this option to"},{"line_number":371,"context_line":"boot an iPXE ISO."}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_cf7d3fe9","line":368,"range":{"start_line":367,"start_character":27,"end_line":368,"end_character":22},"updated":"2016-07-27 21:36:20.000000000","message":"Ditto.","commit_id":"e936dc61150af42c7ae0aa4d2625428db4d08596"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"a1631da62fc4ea01e9df2d9d3f4b43808e8218d3","unresolved":false,"context_lines":[{"line_number":377,"context_line":""},{"line_number":378,"context_line":"An iPXE ISO is a specially crafted ISO which supports iPXE booting."},{"line_number":379,"context_line":"This feature gives a means to roll our own image. To use this"},{"line_number":380,"context_line":"feature the config options `ipxe_network_name`, `ipxe_boot_menu_url`"},{"line_number":381,"context_line":"and `ipxe_mkisofs_cmd` should be configured."},{"line_number":382,"context_line":""},{"line_number":383,"context_line":"Note: By default `mkisofs` is not present in the Dom0, so the"},{"line_number":384,"context_line":"package can either be manually added to Dom0 or include the"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_4f6aef1b","line":381,"range":{"start_line":380,"start_character":27,"end_line":381,"end_character":22},"updated":"2016-07-27 21:36:20.000000000","message":"ditto","commit_id":"e936dc61150af42c7ae0aa4d2625428db4d08596"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"3285562b2340d06661a2ab02e2c52476775e0d0c","unresolved":false,"context_lines":[{"line_number":255,"context_line":""},{"line_number":256,"context_line":"The value for this option must be choosen from the choices listed"},{"line_number":257,"context_line":"here. Configuring a value other than these will default to \u0027all\u0027."},{"line_number":258,"context_line":""},{"line_number":259,"context_line":"Possible values:"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"    * `all`: will cache all images."}],"source_content_type":"text/x-python","patch_set":4,"id":"9ad45d7e_83a63f51","line":258,"updated":"2016-08-09 17:06:27.000000000","message":"We should put a warning here, there is nothing that deletes these images.","commit_id":"6fa40c06999fef257ba05db26f29cb41816ebd12"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"3285562b2340d06661a2ab02e2c52476775e0d0c","unresolved":false,"context_lines":[{"line_number":258,"context_line":""},{"line_number":259,"context_line":"Possible values:"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"    * `all`: will cache all images."},{"line_number":262,"context_line":"    * `some`: will only cache images that have the"},{"line_number":263,"context_line":"              image_property `cache_in_nova\u003dTrue`."},{"line_number":264,"context_line":"    * `none`: turns off caching entirely."}],"source_content_type":"text/x-python","patch_set":4,"id":"9ad45d7e_4373a7e7","line":261,"range":{"start_line":261,"start_character":0,"end_line":261,"end_character":4},"updated":"2016-08-09 17:06:27.000000000","message":"Nit: we don\u0027t want this indent here. Same with similar things below.","commit_id":"6fa40c06999fef257ba05db26f29cb41816ebd12"},{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"43c5c36fc18f8726998104178e251e7ea45d7023","unresolved":false,"context_lines":[{"line_number":258,"context_line":""},{"line_number":259,"context_line":"Possible values:"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"    * `all`: will cache all images."},{"line_number":262,"context_line":"    * `some`: will only cache images that have the"},{"line_number":263,"context_line":"              image_property `cache_in_nova\u003dTrue`."},{"line_number":264,"context_line":"    * `none`: turns off caching entirely."}],"source_content_type":"text/x-python","patch_set":4,"id":"9ad45d7e_3a9cc968","line":261,"range":{"start_line":261,"start_character":0,"end_line":261,"end_character":4},"in_reply_to":"9ad45d7e_4373a7e7","updated":"2016-08-09 20:53:38.000000000","message":"Done","commit_id":"6fa40c06999fef257ba05db26f29cb41816ebd12"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"3285562b2340d06661a2ab02e2c52476775e0d0c","unresolved":false,"context_lines":[{"line_number":303,"context_line":""},{"line_number":304,"context_line":"Possible values:"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"    * To use the Local Storage in default XenServer/XCP installations"},{"line_number":307,"context_line":"      set this flag to other-config:i18n-key\u003dlocal-storage."},{"line_number":308,"context_line":"    * To select an SR with a different matching criteria, you could"},{"line_number":309,"context_line":"      set it to other-config:my_favorite_sr\u003dtrue."}],"source_content_type":"text/x-python","patch_set":4,"id":"9ad45d7e_83547f21","line":306,"range":{"start_line":306,"start_character":0,"end_line":306,"end_character":4},"updated":"2016-08-09 17:06:27.000000000","message":"Nit: we should delete this gap here","commit_id":"6fa40c06999fef257ba05db26f29cb41816ebd12"},{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"43c5c36fc18f8726998104178e251e7ea45d7023","unresolved":false,"context_lines":[{"line_number":303,"context_line":""},{"line_number":304,"context_line":"Possible values:"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"    * To use the Local Storage in default XenServer/XCP installations"},{"line_number":307,"context_line":"      set this flag to other-config:i18n-key\u003dlocal-storage."},{"line_number":308,"context_line":"    * To select an SR with a different matching criteria, you could"},{"line_number":309,"context_line":"      set it to other-config:my_favorite_sr\u003dtrue."}],"source_content_type":"text/x-python","patch_set":4,"id":"9ad45d7e_9a897529","line":306,"range":{"start_line":306,"start_character":0,"end_line":306,"end_character":4},"in_reply_to":"9ad45d7e_83547f21","updated":"2016-08-09 20:53:38.000000000","message":"Done","commit_id":"6fa40c06999fef257ba05db26f29cb41816ebd12"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0ba4d663a50cb21e83aa86f64bc601facd1c1cb0","unresolved":false,"context_lines":[{"line_number":276,"context_line":"* `all`: will download all images."},{"line_number":277,"context_line":"* `some`: will only download images that have the image_property"},{"line_number":278,"context_line":"          `bittorrent\u003dtrue\u0027."},{"line_number":279,"context_line":"* `none`: will turnoff downloading images via Bit Torrent."},{"line_number":280,"context_line":"\"\"\"),"},{"line_number":281,"context_line":"    cfg.StrOpt(\u0027ipxe_network_name\u0027,"},{"line_number":282,"context_line":"               help\u003d\"\"\""}],"source_content_type":"text/x-python","patch_set":9,"id":"9ad45d7e_86fde870","line":279,"updated":"2016-08-12 17:28:17.000000000","message":"Nit: I think there is a related option here, but this is a total edge case anyways, so lets not block on that.","commit_id":"b3ad315a7434abf205d6f1285cb6a9e46fb6ed20"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"f486acbe6ceb5295729d4c91235686c3009d4601","unresolved":false,"context_lines":[{"line_number":276,"context_line":"* `all`: will download all images."},{"line_number":277,"context_line":"* `some`: will only download images that have the image_property"},{"line_number":278,"context_line":"          `bittorrent\u003dtrue\u0027."},{"line_number":279,"context_line":"* `none`: will turnoff downloading images via Bit Torrent."},{"line_number":280,"context_line":"\"\"\"),"},{"line_number":281,"context_line":"    cfg.StrOpt(\u0027ipxe_network_name\u0027,"},{"line_number":282,"context_line":"               help\u003d\"\"\""}],"source_content_type":"text/x-python","patch_set":9,"id":"9ad45d7e_467810f9","line":279,"in_reply_to":"9ad45d7e_86fde870","updated":"2016-08-12 17:29:57.000000000","message":"Ah, so technically, it only gets used when bit torrent is selected by here: CONF.xen_api.image_upload_handler","commit_id":"b3ad315a7434abf205d6f1285cb6a9e46fb6ed20"}]}