)]}'
{"nova/conf/workarounds.py":[{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"835ef91b8cb220c5110a02bfc7fa324e74f00198","unresolved":false,"context_lines":[{"line_number":128,"context_line":""},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"def register_opts(conf):"},{"line_number":131,"context_line":"    conf.register_opts(ALL_OPTS, group\u003dworkarounds_group)"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"def list_opts():"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_d2591800","line":131,"range":{"start_line":131,"start_character":39,"end_line":131,"end_character":56},"updated":"2016-07-29 03:24:09.000000000","message":"Is there any specific reason for not registering group first?\nconf.register_group(workarounds_group)","commit_id":"70049164e885d5fb71921f2dbc814f879e3c0e6e"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"e512b0a0ce356e43845f5737a19d5776b0efef89","unresolved":false,"context_lines":[{"line_number":41,"context_line":"should only be enabled in exceptional circumstances. All options are linked"},{"line_number":42,"context_line":"against bug IDs, where more information on the issue can be found."},{"line_number":43,"context_line":"\"\"\")"},{"line_number":44,"context_line":"ALL_OPTS \u003d ["},{"line_number":45,"context_line":"    cfg.BoolOpt("},{"line_number":46,"context_line":"        \u0027disable_rootwrap\u0027,"},{"line_number":47,"context_line":"        default\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":4,"id":"9ad45d7e_9609b2e0","line":44,"updated":"2016-08-08 10:20:08.000000000","message":"Nit: its tempting to have a gap before ALL_OPTS","commit_id":"94a33a3ded267859b70dfb38e29877657d6740ac"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"b66bc06c7187bda3d0d349b06eb0e4b9f44d1c2e","unresolved":false,"context_lines":[{"line_number":41,"context_line":"should only be enabled in exceptional circumstances. All options are linked"},{"line_number":42,"context_line":"against bug IDs, where more information on the issue can be found."},{"line_number":43,"context_line":"\"\"\")"},{"line_number":44,"context_line":"ALL_OPTS \u003d ["},{"line_number":45,"context_line":"    cfg.BoolOpt("},{"line_number":46,"context_line":"        \u0027disable_rootwrap\u0027,"},{"line_number":47,"context_line":"        default\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":4,"id":"9ad45d7e_c8789992","line":44,"in_reply_to":"9ad45d7e_9609b2e0","updated":"2016-08-08 14:49:46.000000000","message":"+1","commit_id":"94a33a3ded267859b70dfb38e29877657d6740ac"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5eada6d0b2c54e48a9253032bd41a369b6beb63a","unresolved":false,"context_lines":[{"line_number":54,"context_line":""},{"line_number":55,"context_line":"  https://bugs.launchpad.net/nova/+bug/1415106"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Possible values:"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"* True: Use sudo instead of rootwrap"},{"line_number":60,"context_line":"* False: Use rootwrap as usual"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"Interdependencies to other options:"},{"line_number":63,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3ac371cc_cdd653f0","line":60,"range":{"start_line":57,"start_character":0,"end_line":60,"end_character":30},"updated":"2016-08-17 16:42:18.000000000","message":"nit: Drop this (it\u0027s obvious)","commit_id":"94a33a3ded267859b70dfb38e29877657d6740ac"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"e512b0a0ce356e43845f5737a19d5776b0efef89","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"* Any options that affect \u0027rootwrap\u0027 will be ignored."},{"line_number":65,"context_line":"\"\"\"),"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    cfg.BoolOpt("},{"line_number":68,"context_line":"        \u0027disable_libvirt_livesnapshot\u0027,"},{"line_number":69,"context_line":"        default\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":4,"id":"9ad45d7e_166f8293","line":66,"updated":"2016-08-08 10:20:08.000000000","message":"Nit: we don\u0027t really need these gaps.","commit_id":"94a33a3ded267859b70dfb38e29877657d6740ac"}]}