)]}'
{"nova/cells/manager.py":[{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"7c2a2f96c697bebb03fff9d287f84c64cdfdff30","unresolved":false,"context_lines":[{"line_number":36,"context_line":"                help\u003d\u0027Cells communication driver to use\u0027),"},{"line_number":37,"context_line":"        cfg.IntOpt(\"instance_update_interval\","},{"line_number":38,"context_line":"                default\u003d60,"},{"line_number":39,"context_line":"                help\u003d\"Number of seconds between cell instance updates\"),"},{"line_number":40,"context_line":"        cfg.IntOpt(\"instance_updated_at_threshold\","},{"line_number":41,"context_line":"                default\u003d3600,"},{"line_number":42,"context_line":"                help\u003d\"Number of seconds after an instance was updated \""}],"source_content_type":"text/x-python","patch_set":14,"id":"AAAAM3%2F%2FyoI%3D","line":39,"updated":"2013-01-04 22:00:03.000000000","message":"So this is also dependent on how often periodic tasks run.  Just because you set this to the 60 doesn\u0027t mean it will *actually* run every 60 seconds.  It\u0027s also not going to work if you set this to anything less than the periodic task interval.  It\u0027s basically only going to run at some multiple of the interval in between periodic task runs.\n\nI wonder if it\u0027s worth clarifying the behavior in the docs here, or just removing the config option completely and just update on every periodic task run, whatever that interval is?  So, either:\n\n1) Remove this option completely\n\n2) Rename this option to \"instance_update_periodic_task_interval\" or something ... where the behavior is how many periodic task ticks to let go by before doing another heal, with a default of 0.\n\nI\u0027m leaning toward #1, but if you have a need for controlling this, then #2 seems like it\u0027s easier to understand than the current behavior.","commit_id":"41019276f5c36090bbd2427625b20b954afbacc9"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"8b1483d73ee8febb517587a73dcf47a3a2b153c2","unresolved":false,"context_lines":[{"line_number":36,"context_line":"                help\u003d\u0027Cells communication driver to use\u0027),"},{"line_number":37,"context_line":"        cfg.IntOpt(\"instance_update_interval\","},{"line_number":38,"context_line":"                default\u003d60,"},{"line_number":39,"context_line":"                help\u003d\"Number of seconds between cell instance updates\"),"},{"line_number":40,"context_line":"        cfg.IntOpt(\"instance_updated_at_threshold\","},{"line_number":41,"context_line":"                default\u003d3600,"},{"line_number":42,"context_line":"                help\u003d\"Number of seconds after an instance was updated \""}],"source_content_type":"text/x-python","patch_set":14,"id":"AAAAM3%2F%2FymE%3D","line":39,"in_reply_to":"AAAAM3%2F%2FyoI%3D","updated":"2013-01-04 22:07:17.000000000","message":"mikal has a patch up that fixes a lot of this problem with tasks... I think I\u0027ll go with #1 and just remove it for now.  I can always re-add it later based on what happens with periodic task refactoring.","commit_id":"41019276f5c36090bbd2427625b20b954afbacc9"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"7c2a2f96c697bebb03fff9d287f84c64cdfdff30","unresolved":false,"context_lines":[{"line_number":43,"context_line":"                        \"or deleted to continue to update cells\"),"},{"line_number":44,"context_line":"        cfg.IntOpt(\"instance_update_num_instances\","},{"line_number":45,"context_line":"                default\u003d1,"},{"line_number":46,"context_line":"                help\u003d\"Number of instances to update per periodic task run\")"},{"line_number":47,"context_line":"]"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"AAAAM3%2F%2FyoA%3D","line":46,"updated":"2013-01-04 22:00:03.000000000","message":"Might be worth documenting how an instance is chosen ... effectively randomly, right?","commit_id":"41019276f5c36090bbd2427625b20b954afbacc9"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"8b1483d73ee8febb517587a73dcf47a3a2b153c2","unresolved":false,"context_lines":[{"line_number":43,"context_line":"                        \"or deleted to continue to update cells\"),"},{"line_number":44,"context_line":"        cfg.IntOpt(\"instance_update_num_instances\","},{"line_number":45,"context_line":"                default\u003d1,"},{"line_number":46,"context_line":"                help\u003d\"Number of instances to update per periodic task run\")"},{"line_number":47,"context_line":"]"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"AAAAM3%2F%2Fyl8%3D","line":46,"in_reply_to":"AAAAM3%2F%2FyoA%3D","updated":"2013-01-04 22:07:17.000000000","message":"Ya, somewhat.  I\u0027ll clarify why it is the way it is.","commit_id":"41019276f5c36090bbd2427625b20b954afbacc9"}]}