)]}'
{"ironicclient/v1/node_shell.py":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"6f7d080bfb93ce62f47a8ed81fb468da2a8ccc51","unresolved":false,"context_lines":[{"line_number":272,"context_line":"    default\u003d[],"},{"line_number":273,"context_line":"    help\u003d\"Attribute to add, replace, or remove. Can be specified \""},{"line_number":274,"context_line":"         \"multiple times. For \u0027remove\u0027, only \u003cpath\u003e is necessary. \""},{"line_number":275,"context_line":"         \"Separate components of nested attributes with slashes, eg \""},{"line_number":276,"context_line":"         \"driver_info/deploy_kerney\u003duuid.\")"},{"line_number":277,"context_line":"def do_node_update(cc, args):"},{"line_number":278,"context_line":"    \"\"\"Update information about a registered node.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_7a182885","line":275,"updated":"2016-08-15 20:20:44.000000000","message":"I find it hard to interpret \"Separate\" as a verb here, I keep thinking of it as an adjective. How about \n\nFor nested attributes, separate the components with forward slashes. For example,","commit_id":"a1894d3326c2c24b75667a760ab05a77bd11a6c0"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"96a2c673ec348e84ae6cc9e4d2f0fc053ffd6275","unresolved":false,"context_lines":[{"line_number":272,"context_line":"    default\u003d[],"},{"line_number":273,"context_line":"    help\u003d\"Attribute to add, replace, or remove. Can be specified \""},{"line_number":274,"context_line":"         \"multiple times. For \u0027remove\u0027, only \u003cpath\u003e is necessary. \""},{"line_number":275,"context_line":"         \"Separate components of nested attributes with slashes, eg \""},{"line_number":276,"context_line":"         \"driver_info/deploy_kerney\u003duuid.\")"},{"line_number":277,"context_line":"def do_node_update(cc, args):"},{"line_number":278,"context_line":"    \"\"\"Update information about a registered node.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_f7a232ad","line":275,"in_reply_to":"3ac371cc_7a182885","updated":"2016-09-07 16:44:07.000000000","message":"Done.","commit_id":"a1894d3326c2c24b75667a760ab05a77bd11a6c0"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"6f7d080bfb93ce62f47a8ed81fb468da2a8ccc51","unresolved":false,"context_lines":[{"line_number":273,"context_line":"    help\u003d\"Attribute to add, replace, or remove. Can be specified \""},{"line_number":274,"context_line":"         \"multiple times. For \u0027remove\u0027, only \u003cpath\u003e is necessary. \""},{"line_number":275,"context_line":"         \"Separate components of nested attributes with slashes, eg \""},{"line_number":276,"context_line":"         \"driver_info/deploy_kerney\u003duuid.\")"},{"line_number":277,"context_line":"def do_node_update(cc, args):"},{"line_number":278,"context_line":"    \"\"\"Update information about a registered node.\"\"\""},{"line_number":279,"context_line":"    patch \u003d utils.args_array_to_patch(args.op, args.attributes[0])"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_1a049415","line":276,"updated":"2016-08-15 20:20:44.000000000","message":"did you really mean to use \u0027kerney\u0027 instead of eg \u0027kernel\u0027?\n\nI would also suggest putting single quotes around this.","commit_id":"a1894d3326c2c24b75667a760ab05a77bd11a6c0"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"96a2c673ec348e84ae6cc9e4d2f0fc053ffd6275","unresolved":false,"context_lines":[{"line_number":273,"context_line":"    help\u003d\"Attribute to add, replace, or remove. Can be specified \""},{"line_number":274,"context_line":"         \"multiple times. For \u0027remove\u0027, only \u003cpath\u003e is necessary. \""},{"line_number":275,"context_line":"         \"Separate components of nested attributes with slashes, eg \""},{"line_number":276,"context_line":"         \"driver_info/deploy_kerney\u003duuid.\")"},{"line_number":277,"context_line":"def do_node_update(cc, args):"},{"line_number":278,"context_line":"    \"\"\"Update information about a registered node.\"\"\""},{"line_number":279,"context_line":"    patch \u003d utils.args_array_to_patch(args.op, args.attributes[0])"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_97948643","line":276,"in_reply_to":"3ac371cc_1a049415","updated":"2016-09-07 16:44:07.000000000","message":"Hah, no I didn\u0027t! Fixed, and I\u0027ve added single quotes.","commit_id":"a1894d3326c2c24b75667a760ab05a77bd11a6c0"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"03dd9736c04f62ea1b2b1794deb669bf6ecb38a5","unresolved":false,"context_lines":[{"line_number":272,"context_line":"    default\u003d[],"},{"line_number":273,"context_line":"    help\u003d\"Attribute to add, replace, or remove. Can be specified \""},{"line_number":274,"context_line":"         \"multiple times. For \u0027remove\u0027, only \u003cpath\u003e is necessary. \""},{"line_number":275,"context_line":"         \"For nested attributes, separate the components with slashes, eg \""},{"line_number":276,"context_line":"         \"\u0027driver_info/deploy_kernel\u003duuid\u0027.\")"},{"line_number":277,"context_line":"def do_node_update(cc, args):"},{"line_number":278,"context_line":"    \"\"\"Update information about a registered node.\"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"7a8ec9b2_30200df7","line":275,"range":{"start_line":275,"start_character":71,"end_line":275,"end_character":73},"updated":"2016-09-16 20:58:33.000000000","message":"nit: I guess we are supposed to use \"for example\" instead of \"eg\"","commit_id":"b8670a8626d828296ffb4ce1020c07ce893c6689"}]}