)]}'
{"nova/conf/compute.py":[{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"cb60f596814b170ecf1a472c3d0be8401a65d231","unresolved":false,"context_lines":[{"line_number":736,"context_line":"* Any other value"},{"line_number":737,"context_line":"\"\"\"),"},{"line_number":738,"context_line":"    cfg.IntOpt(\u0027bandwidth_poll_interval\u0027,"},{"line_number":739,"context_line":"        default\u003d600,"},{"line_number":740,"context_line":"        help\u003d\"\"\""},{"line_number":741,"context_line":"Interval to pull network bandwidth usage info."},{"line_number":742,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_ebf31572","line":739,"range":{"start_line":739,"start_character":8,"end_line":739,"end_character":20},"updated":"2018-06-21 00:57:04.000000000","message":"The default value is defined here. It is 600 seconds.\n\nIn the sample file, the default value is described\nlike \"bandwidth_poll_interval \u003d 600\".\nSo IMO, it is not worth adding additional help texts.\n\n#\n# Interval to pull network bandwidth usage info.\n#\n# Not supported on all hypervisors. If a hypervisor doesn\u0027t support bandwidth\n# usage, it will not get the info in the usage events.\n#\n# Possible values:\n#\n# * 0: Will run at the default periodic interval.\n# * Any value \u003c 0: Disables the option.\n# * Any positive integer in seconds.\n#  (integer value)\n#bandwidth_poll_interval \u003d 600","commit_id":"2743b72aec5f8a0b7722a5612a8c196b7399582e"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"6050acfd0f69d6903e7f7cf814ab01494733de92","unresolved":false,"context_lines":[{"line_number":943,"context_line":"  is set to False."},{"line_number":944,"context_line":"\"\"\"),"},{"line_number":945,"context_line":"    cfg.IntOpt(\u0027update_resources_interval\u0027,"},{"line_number":946,"context_line":"        default\u003d60,"},{"line_number":947,"context_line":"        help\u003d\"\"\""},{"line_number":948,"context_line":"Interval for updating compute resources."},{"line_number":949,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_9ffadb74","line":946,"range":{"start_line":946,"start_character":8,"end_line":946,"end_character":18},"updated":"2018-06-21 06:09:11.000000000","message":"not sure whether it\u0027s really necessary, \nbecause it indicate we will anyway\nrun at the default periodic interval at default is 60 in oslo","commit_id":"9f02c7189f0b5987c19943bd07acba7678981e08"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"6050acfd0f69d6903e7f7cf814ab01494733de92","unresolved":false,"context_lines":[{"line_number":949,"context_line":""},{"line_number":950,"context_line":"This option specifies how often the update_available_resources"},{"line_number":951,"context_line":"periodic task should run. A number less than 0 means to disable the"},{"line_number":952,"context_line":"task completely. Leaving this at the default of 0 will cause this to"},{"line_number":953,"context_line":"run at the default periodic interval. Setting it to any positive"},{"line_number":954,"context_line":"value will cause it to run at approximately that number of seconds."},{"line_number":955,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_ff098f94","line":952,"range":{"start_line":952,"start_character":30,"end_line":952,"end_character":49},"updated":"2018-06-21 06:09:11.000000000","message":"at least you might want to change it","commit_id":"9f02c7189f0b5987c19943bd07acba7678981e08"},{"author":{"_account_id":25562,"name":"Chen","email":"dstbtgagt@foxmail.com","username":"chenn2"},"change_message_id":"eb13a2312e179f0ef824757e82d65501dd4fa156","unresolved":false,"context_lines":[{"line_number":949,"context_line":""},{"line_number":950,"context_line":"This option specifies how often the update_available_resources"},{"line_number":951,"context_line":"periodic task should run. A number less than 0 means to disable the"},{"line_number":952,"context_line":"task completely. Leaving this at the default of 0 will cause this to"},{"line_number":953,"context_line":"run at the default periodic interval. Setting it to any positive"},{"line_number":954,"context_line":"value will cause it to run at approximately that number of seconds."},{"line_number":955,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_3f7867bf","line":952,"range":{"start_line":952,"start_character":30,"end_line":952,"end_character":49},"in_reply_to":"5f7c97a3_ff098f94","updated":"2018-06-21 06:31:18.000000000","message":"Thats a good point. Ive fixed it and believe it looks better now.","commit_id":"9f02c7189f0b5987c19943bd07acba7678981e08"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"bb47f6bc129dcf7af0921e57a6b4e6af7b364d02","unresolved":false,"context_lines":[{"line_number":793,"context_line":"* Any value \u003c\u003d0 will disable the sync. This is not recommended."},{"line_number":794,"context_line":"\"\"\"),"},{"line_number":795,"context_line":"    cfg.IntOpt(\u0027reclaim_instance_interval\u0027,"},{"line_number":796,"context_line":"        default\u003d0,"},{"line_number":797,"context_line":"        help\u003d\"\"\""},{"line_number":798,"context_line":"Interval for reclaiming deleted instances."},{"line_number":799,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_48a70b52","line":796,"updated":"2018-06-28 18:19:22.000000000","message":"same here","commit_id":"11a776a167e4450d6d741453f6cfd3fa82e6caf1"},{"author":{"_account_id":25562,"name":"Chen","email":"dstbtgagt@foxmail.com","username":"chenn2"},"change_message_id":"bf94fd1e82437bafea795884d11124fd1a7e4b7c","unresolved":false,"context_lines":[{"line_number":793,"context_line":"* Any value \u003c\u003d0 will disable the sync. This is not recommended."},{"line_number":794,"context_line":"\"\"\"),"},{"line_number":795,"context_line":"    cfg.IntOpt(\u0027reclaim_instance_interval\u0027,"},{"line_number":796,"context_line":"        default\u003d0,"},{"line_number":797,"context_line":"        help\u003d\"\"\""},{"line_number":798,"context_line":"Interval for reclaiming deleted instances."},{"line_number":799,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_ec85c20b","line":796,"in_reply_to":"5f7c97a3_48a70b52","updated":"2018-06-29 02:20:27.000000000","message":"L819 also says \"Any value \u003c\u003d0 will disable the option\". Seems different from the case of \u0027update_resources_interval\u0027 option.","commit_id":"11a776a167e4450d6d741453f6cfd3fa82e6caf1"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"bb47f6bc129dcf7af0921e57a6b4e6af7b364d02","unresolved":false,"context_lines":[{"line_number":819,"context_line":"* Any value \u003c\u003d0 will disable the option."},{"line_number":820,"context_line":"\"\"\"),"},{"line_number":821,"context_line":"    cfg.IntOpt(\u0027volume_usage_poll_interval\u0027,"},{"line_number":822,"context_line":"        default\u003d0,"},{"line_number":823,"context_line":"        help\u003d\"\"\""},{"line_number":824,"context_line":"Interval for gathering volume usages."},{"line_number":825,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_28640f21","line":822,"updated":"2018-06-28 18:19:22.000000000","message":"same","commit_id":"11a776a167e4450d6d741453f6cfd3fa82e6caf1"},{"author":{"_account_id":25562,"name":"Chen","email":"dstbtgagt@foxmail.com","username":"chenn2"},"change_message_id":"bf94fd1e82437bafea795884d11124fd1a7e4b7c","unresolved":false,"context_lines":[{"line_number":819,"context_line":"* Any value \u003c\u003d0 will disable the option."},{"line_number":820,"context_line":"\"\"\"),"},{"line_number":821,"context_line":"    cfg.IntOpt(\u0027volume_usage_poll_interval\u0027,"},{"line_number":822,"context_line":"        default\u003d0,"},{"line_number":823,"context_line":"        help\u003d\"\"\""},{"line_number":824,"context_line":"Interval for gathering volume usages."},{"line_number":825,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_0ca27eae","line":822,"in_reply_to":"5f7c97a3_28640f21","updated":"2018-06-29 02:20:27.000000000","message":"L833 says \"Any value \u003c\u003d0 will disable the option\". so here 0 means not running this task at all instead of running at default period?","commit_id":"11a776a167e4450d6d741453f6cfd3fa82e6caf1"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"fa516ad0e5c8f4b71b2ce2b87fb0f547a8e32bc7","unresolved":false,"context_lines":[{"line_number":949,"context_line":""},{"line_number":950,"context_line":"This option specifies how often the update_available_resources"},{"line_number":951,"context_line":"periodic task should run. A number less than 0 means to disable the"},{"line_number":952,"context_line":"task completely. Setting it to 0 will cause it to run at the default"},{"line_number":953,"context_line":"periodic interval. Setting it to any positive value will cause it to"},{"line_number":954,"context_line":"run at approximately that number of seconds."},{"line_number":955,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_cdf55f86","line":952,"range":{"start_line":952,"start_character":16,"end_line":952,"end_character":53},"updated":"2018-06-29 07:32:29.000000000","message":"Is it just me or this seems a little bit reduntant? as we aready write the possible values and what it means below, maybe we should mention the actual default value 60 here? same with the above config ``scheduler_instance_sync_interval``","commit_id":"11a776a167e4450d6d741453f6cfd3fa82e6caf1"},{"author":{"_account_id":25562,"name":"Chen","email":"dstbtgagt@foxmail.com","username":"chenn2"},"change_message_id":"0cccdcd49b1df689e4dd55ddd2b513ab7fc9539c","unresolved":false,"context_lines":[{"line_number":949,"context_line":""},{"line_number":950,"context_line":"This option specifies how often the update_available_resources"},{"line_number":951,"context_line":"periodic task should run. A number less than 0 means to disable the"},{"line_number":952,"context_line":"task completely. Setting it to 0 will cause it to run at the default"},{"line_number":953,"context_line":"periodic interval. Setting it to any positive value will cause it to"},{"line_number":954,"context_line":"run at approximately that number of seconds."},{"line_number":955,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_eee27502","line":952,"range":{"start_line":952,"start_character":16,"end_line":952,"end_character":53},"in_reply_to":"5f7c97a3_cdf55f86","updated":"2018-07-02 01:57:42.000000000","message":"Not sure what is expected here. To me, either setting \"default\u003d60\" or stating that the default interval is 60 sec in the text would be necessary, since otherwise we still don\u0027t have a clue what the actual interval at default is just by reading the docs","commit_id":"11a776a167e4450d6d741453f6cfd3fa82e6caf1"}]}