)]}'
{"nova/tests/unit/virt/ironic/utils.py":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"3549f03bca06a05103c264c5f728714b9486be08","unresolved":false,"context_lines":[{"line_number":46,"context_line":"                \u0027reservation\u0027: kw.get(\u0027reservation\u0027),"},{"line_number":47,"context_line":"                \u0027maintenance\u0027: kw.get(\u0027maintenance\u0027, False),"},{"line_number":48,"context_line":"                \u0027network_provider\u0027: kw.get(\u0027network_provider\u0027),"},{"line_number":49,"context_line":"                \u0027resource_class\u0027: kw.get(\u0027resource_class\u0027, \u0027\u0027),"},{"line_number":50,"context_line":"                \u0027extra\u0027: kw.get(\u0027extra\u0027, {}),"},{"line_number":51,"context_line":"                \u0027updated_at\u0027: kw.get(\u0027created_at\u0027),"},{"line_number":52,"context_line":"                \u0027created_at\u0027: kw.get(\u0027updated_at\u0027)})()"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_f8fce63f","line":49,"updated":"2016-07-25 23:57:01.000000000","message":"How about defaulting it to None instead of \u0027\u0027 so we can accurately emulate what the client will return? Or is there a difference between the network_provider field and the resource_class with regards to nullability?","commit_id":"f773a56561b0873d6b70b3f8f95503aeb58bc433"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"1d3408719da3808e2348d47cb6fb00e6b1507c7e","unresolved":false,"context_lines":[{"line_number":46,"context_line":"                \u0027reservation\u0027: kw.get(\u0027reservation\u0027),"},{"line_number":47,"context_line":"                \u0027maintenance\u0027: kw.get(\u0027maintenance\u0027, False),"},{"line_number":48,"context_line":"                \u0027network_provider\u0027: kw.get(\u0027network_provider\u0027),"},{"line_number":49,"context_line":"                \u0027resource_class\u0027: kw.get(\u0027resource_class\u0027, \u0027\u0027),"},{"line_number":50,"context_line":"                \u0027extra\u0027: kw.get(\u0027extra\u0027, {}),"},{"line_number":51,"context_line":"                \u0027updated_at\u0027: kw.get(\u0027created_at\u0027),"},{"line_number":52,"context_line":"                \u0027created_at\u0027: kw.get(\u0027updated_at\u0027)})()"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_842470af","line":49,"in_reply_to":"dada55a8_f8fce63f","updated":"2016-07-29 10:55:10.000000000","message":"Yeah, for whatever reason I initially was defaulting resource_class to \u0027\u0027. It defaults to None now, I\u0027ll fix this.","commit_id":"f773a56561b0873d6b70b3f8f95503aeb58bc433"}]}