)]}'
{"ironicclient/osc/v1/baremetal_node.py":[{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"687a9d9f5682e5cee5f14a7540671a077a5b5a1f","unresolved":false,"context_lines":[{"line_number":1094,"context_line":""},{"line_number":1095,"context_line":"        baremetal_client \u003d self.app.client_manager.baremetal"},{"line_number":1096,"context_line":""},{"line_number":1097,"context_line":"        interfaces \u003d baremetal_client.node.validate(parsed_args.node)._info"},{"line_number":1098,"context_line":"        data \u003d []"},{"line_number":1099,"context_line":"        for key, value in interfaces.items():"},{"line_number":1100,"context_line":"            interface \u003d {\u0027interface\u0027: key}"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa7ab95a_4f5a39f3","line":1097,"range":{"start_line":1097,"start_character":70,"end_line":1097,"end_character":75},"updated":"2016-08-30 15:05:48.000000000","message":"Let\u0027s not use _info directly, we can access the attributes directly, or call to_dict instead.","commit_id":"ce8956caea576073e7d671f0ec3c1c6ba67168a2"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"297ec10212df7ae85e56d2e29ff790ef43614f65","unresolved":false,"context_lines":[{"line_number":1094,"context_line":""},{"line_number":1095,"context_line":"        baremetal_client \u003d self.app.client_manager.baremetal"},{"line_number":1096,"context_line":""},{"line_number":1097,"context_line":"        interfaces \u003d baremetal_client.node.validate(parsed_args.node)._info"},{"line_number":1098,"context_line":"        data \u003d []"},{"line_number":1099,"context_line":"        for key, value in interfaces.items():"},{"line_number":1100,"context_line":"            interface \u003d {\u0027interface\u0027: key}"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa7ab95a_b505d614","line":1097,"range":{"start_line":1097,"start_character":70,"end_line":1097,"end_character":75},"in_reply_to":"fa7ab95a_4f5a39f3","updated":"2016-08-30 15:35:31.000000000","message":"That\u0027s a good point. ._info is already being used in the code, in baremetal_port, chassis, and node, so I\u0027d rather leave this as-is, and change them all (to do the same thing) in a separate patch. (I\u0027m not even sure .validate() should be returning a Resource object but that\u0027s a separate thing).","commit_id":"ce8956caea576073e7d671f0ec3c1c6ba67168a2"}]}