)]}'
{"nova/conf/cells.py":[{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"a0e9cc37140c15ef338f3404b31cb27f3246b14c","unresolved":false,"context_lines":[{"line_number":40,"context_line":"Compute deployment in that each cell has its own database server and"},{"line_number":41,"context_line":"message queue broker."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"Possible values:"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* True: Enables the feature"},{"line_number":46,"context_line":"* False: Disables the feature"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_57a9c4dc","line":43,"updated":"2016-07-14 21:13:24.000000000","message":"This should be removed as per https://wiki.openstack.org/wiki/ConfigOptionsConsistency#fix_opt_description.\n\nPlease check under Possible Values.","commit_id":"1bb661b9d8cb373453f00d08ade7424a76a97101"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"a0e9cc37140c15ef338f3404b31cb27f3246b14c","unresolved":false,"context_lines":[{"line_number":69,"context_line":"               default\u003d\u0027nova.cells.manager.CellsManager\u0027,"},{"line_number":70,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":71,"context_line":"               help\u003d\"\"\""},{"line_number":72,"context_line":"DEPRECATED: Manager for cells"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"The nova-cells manager class. This class defines RPC methods that"},{"line_number":75,"context_line":"the local cell may call. This class is NOT used for messages coming"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_775de047","line":72,"updated":"2016-07-14 21:13:24.000000000","message":"DEPRECATED is redundant here. DEPRECATED word will be inserted automatically before text message in sample config file.","commit_id":"1bb661b9d8cb373453f00d08ade7424a76a97101"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"a0e9cc37140c15ef338f3404b31cb27f3246b14c","unresolved":false,"context_lines":[{"line_number":99,"context_line":"its id, leaving this option unset or setting the same name for"},{"line_number":100,"context_line":"two or more cells may cause unexpected behaviour."},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Possible values:"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"* Unique name string"},{"line_number":105,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_d24e626d","line":102,"updated":"2016-07-14 21:13:24.000000000","message":"This can be dropped. Add it only for interesting cases.","commit_id":"1bb661b9d8cb373453f00d08ade7424a76a97101"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"a0e9cc37140c15ef338f3404b31cb27f3246b14c","unresolved":false,"context_lines":[{"line_number":125,"context_line":"\"\"\"),"},{"line_number":126,"context_line":"    cfg.IntOpt(\u0027call_timeout\u0027,"},{"line_number":127,"context_line":"                default\u003d60,"},{"line_number":128,"context_line":"                help\u003d\"\"\""},{"line_number":129,"context_line":"Call timeout"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"Cell messaging module waits for response(s) to be put into the"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_12a74a0c","line":128,"updated":"2016-07-14 21:13:24.000000000","message":"Add min value to the int/float opts where ever  applicable as mentioned in the wiki page.","commit_id":"1bb661b9d8cb373453f00d08ade7424a76a97101"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"a0e9cc37140c15ef338f3404b31cb27f3246b14c","unresolved":false,"context_lines":[{"line_number":153,"context_line":""},{"line_number":154,"context_line":"Possible values:"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"* Float percentage value"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"\"\"\"),"},{"line_number":159,"context_line":"    cfg.StrOpt(\u0027cell_type\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_d2830254","line":156,"updated":"2016-07-14 21:13:24.000000000","message":"Drop this. Add min value to the opt definition as per wiki page.","commit_id":"1bb661b9d8cb373453f00d08ade7424a76a97101"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"a0e9cc37140c15ef338f3404b31cb27f3246b14c","unresolved":false,"context_lines":[{"line_number":167,"context_line":"The top-level cell\u0027s cell_type must be set to ``api``. All other"},{"line_number":168,"context_line":"cells are defined as a ``compute cell`` by default."},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"Possible values:"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"* api: Cell type of top-level cell."},{"line_number":173,"context_line":"* compute: Cell type of all child cells. (Default)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_92dfbad5","line":170,"updated":"2016-07-14 21:13:24.000000000","message":"Drop this. already provided in line 160 and 161.","commit_id":"1bb661b9d8cb373453f00d08ade7424a76a97101"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"a0e9cc37140c15ef338f3404b31cb27f3246b14c","unresolved":false,"context_lines":[{"line_number":289,"context_line":""},{"line_number":290,"context_line":"Possible values:"},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"* \u0027nova.cells.rpc_driver.CellsRPCDriver\u0027 is the default driver"},{"line_number":293,"context_line":"* Otherwise it should be the full Python path to the class to be used"},{"line_number":294,"context_line":""},{"line_number":295,"context_line":"\"\"\"),"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_72768ed9","line":292,"updated":"2016-07-14 21:13:24.000000000","message":"Remove this. default driver is already provided in line 279.","commit_id":"1bb661b9d8cb373453f00d08ade7424a76a97101"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"a0e9cc37140c15ef338f3404b31cb27f3246b14c","unresolved":false,"context_lines":[{"line_number":363,"context_line":""},{"line_number":364,"context_line":"Possible values:"},{"line_number":365,"context_line":""},{"line_number":366,"context_line":"* \u0027nova.cells.scheduler.CellsScheduler\u0027 is the default option"},{"line_number":367,"context_line":"* Otherwise it should be the full Python path to the class to be used"},{"line_number":368,"context_line":""},{"line_number":369,"context_line":"\"\"\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_d232e20e","line":366,"updated":"2016-07-14 21:13:24.000000000","message":"same as above","commit_id":"1bb661b9d8cb373453f00d08ade7424a76a97101"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"a0e9cc37140c15ef338f3404b31cb27f3246b14c","unresolved":false,"context_lines":[{"line_number":421,"context_line":""},{"line_number":422,"context_line":"Possible values:"},{"line_number":423,"context_line":""},{"line_number":424,"context_line":"* \u0027nova.cells.filters.all_filters\u0027 is the default option"},{"line_number":425,"context_line":"* Otherwise it should be the full Python path to the class to be used"},{"line_number":426,"context_line":""},{"line_number":427,"context_line":"\"\"\"),"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_f216fe56","line":424,"updated":"2016-07-14 21:13:24.000000000","message":"drop this. same as above comment.","commit_id":"1bb661b9d8cb373453f00d08ade7424a76a97101"},{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"a0e9cc37140c15ef338f3404b31cb27f3246b14c","unresolved":false,"context_lines":[{"line_number":459,"context_line":""},{"line_number":460,"context_line":"Possible values:"},{"line_number":461,"context_line":""},{"line_number":462,"context_line":"* \u0027nova.cells.weights.all_weighers\u0027 is the default option"},{"line_number":463,"context_line":"* Otherwise it should be the full Python path to the class to be used"},{"line_number":464,"context_line":""},{"line_number":465,"context_line":"\"\"\"),"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_5206f221","line":462,"updated":"2016-07-14 21:13:24.000000000","message":"same as above","commit_id":"1bb661b9d8cb373453f00d08ade7424a76a97101"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"7abff130abb0becc110ba94c2889bcc859a113dd","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# needs:check_deprecation_status"},{"line_number":2,"context_line":"# needs:check_opt_group_and_type"},{"line_number":3,"context_line":"# needs:fix_opt_description_indentation"},{"line_number":4,"context_line":"# needs:fix_opt_registration_consistency"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"# Copyright 2015 OpenStack Foundation"}],"source_content_type":"text/x-python","patch_set":5,"id":"dada55a8_fea8f721","line":4,"updated":"2016-07-27 12:40:23.000000000","message":"I suspect some of the above will not need work, but I am good with just doing this one.","commit_id":"dd2f1566cc9170a6c9d595b9db512163c74719ca"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"c3a265da2c3a9d3960eda7b55b001de39b611fb0","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# needs:check_deprecation_status"},{"line_number":2,"context_line":"# needs:check_opt_group_and_type"},{"line_number":3,"context_line":"# needs:fix_opt_description_indentation"},{"line_number":4,"context_line":"# needs:fix_opt_registration_consistency"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"# Copyright 2015 OpenStack Foundation"}],"source_content_type":"text/x-python","patch_set":5,"id":"bacf61ea_945fda0d","line":4,"in_reply_to":"dada55a8_fea8f721","updated":"2016-07-27 21:06:56.000000000","message":"+1","commit_id":"dd2f1566cc9170a6c9d595b9db512163c74719ca"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"c3a265da2c3a9d3960eda7b55b001de39b611fb0","unresolved":false,"context_lines":[{"line_number":60,"context_line":"* cells: This is the recommended and the default value."},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"\"\"\"),"},{"line_number":63,"context_line":"    cfg.StrOpt(\u0027manager\u0027,"},{"line_number":64,"context_line":"               default\u003d\u0027nova.cells.manager.CellsManager\u0027,"},{"line_number":65,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":66,"context_line":"               help\u003d\"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"bacf61ea_f4e8d618","line":63,"range":{"start_line":63,"start_character":16,"end_line":63,"end_character":23},"updated":"2016-07-27 21:06:56.000000000","message":"This option was deprecated in Mitaka, maybe remove it in followup patch.","commit_id":"dd2f1566cc9170a6c9d595b9db512163c74719ca"},{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"feaa3166d16937fed21a5f6543c3658ad82b846d","unresolved":false,"context_lines":[{"line_number":185,"context_line":""},{"line_number":186,"context_line":"Possible values:"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"* Time in seconds."},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"\"\"\"),"},{"line_number":191,"context_line":"    cfg.IntOpt(\u0027instance_update_sync_database_limit\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"dada55a8_182c1f01","line":188,"range":{"start_line":188,"start_character":2,"end_line":188,"end_character":18},"updated":"2016-07-26 21:36:44.000000000","message":"Can you add min value and after that this line can be removed.","commit_id":"dd2f1566cc9170a6c9d595b9db512163c74719ca"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"7abff130abb0becc110ba94c2889bcc859a113dd","unresolved":false,"context_lines":[{"line_number":185,"context_line":""},{"line_number":186,"context_line":"Possible values:"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"* Time in seconds."},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"\"\"\"),"},{"line_number":191,"context_line":"    cfg.IntOpt(\u0027instance_update_sync_database_limit\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"dada55a8_3e7baff6","line":188,"range":{"start_line":188,"start_character":2,"end_line":188,"end_character":18},"in_reply_to":"dada55a8_182c1f01","updated":"2016-07-27 12:40:23.000000000","message":"I think its good to state the units.\n\nI think really we should say what 0 means and -1 means","commit_id":"dd2f1566cc9170a6c9d595b9db512163c74719ca"},{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"feaa3166d16937fed21a5f6543c3658ad82b846d","unresolved":false,"context_lines":[{"line_number":199,"context_line":""},{"line_number":200,"context_line":"Possible values:"},{"line_number":201,"context_line":""},{"line_number":202,"context_line":"* Number of instances."},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"\"\"\"),"},{"line_number":205,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":5,"id":"dada55a8_f8178ba0","line":202,"range":{"start_line":202,"start_character":2,"end_line":202,"end_character":22},"updated":"2016-07-26 21:36:44.000000000","message":"same as above. Can you add min for all IntOpt\u0027s in the file.","commit_id":"dd2f1566cc9170a6c9d595b9db512163c74719ca"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"7abff130abb0becc110ba94c2889bcc859a113dd","unresolved":false,"context_lines":[{"line_number":199,"context_line":""},{"line_number":200,"context_line":"Possible values:"},{"line_number":201,"context_line":""},{"line_number":202,"context_line":"* Number of instances."},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"\"\"\"),"},{"line_number":205,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":5,"id":"dada55a8_fe7597f9","line":202,"range":{"start_line":202,"start_character":2,"end_line":202,"end_character":22},"in_reply_to":"dada55a8_f8178ba0","updated":"2016-07-27 12:40:23.000000000","message":"Not sure what the min would be for this just yet, as we do allow -1.","commit_id":"dd2f1566cc9170a6c9d595b9db512163c74719ca"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"c3a265da2c3a9d3960eda7b55b001de39b611fb0","unresolved":false,"context_lines":[{"line_number":206,"context_line":""},{"line_number":207,"context_line":"mute_weigher_opts \u003d ["},{"line_number":208,"context_line":"        cfg.FloatOpt(\u0027mute_weight_multiplier\u0027,"},{"line_number":209,"context_line":"                default\u003d-10000.0,"},{"line_number":210,"context_line":"                help\u003d\"\"\""},{"line_number":211,"context_line":"Mute weight multiplier"},{"line_number":212,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"bacf61ea_d40952df","line":209,"updated":"2016-07-27 21:06:56.000000000","message":"Maybe a regex here to check that number is negative?","commit_id":"dd2f1566cc9170a6c9d595b9db512163c74719ca"}]}