)]}'
{"nova/tests/unit/objects/test_block_device.py":[{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"0e0603fa4ecb51605fc7ab54d7fc2131e8d06724","unresolved":false,"context_lines":[{"line_number":437,"context_line":""},{"line_number":438,"context_line":"        bdm \u003d objects.BlockDeviceMapping("},{"line_number":439,"context_line":"            self.context, destination_type\u003d\u0027volume\u0027, instance_uuid\u003duuids.inst)"},{"line_number":440,"context_line":"        self.assertNotIn(\u0027attach_mode\u0027, bdm)"},{"line_number":441,"context_line":"        self.assertEqual(\u0027rw\u0027, bdm.attach_mode)"},{"line_number":442,"context_line":""},{"line_number":443,"context_line":"        # Make sure we don\u0027t ignore the attribute if it\u0027s set."},{"line_number":444,"context_line":"        bdm.attach_mode \u003d \u0027ro\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_f14ee8bf","line":441,"range":{"start_line":440,"start_character":8,"end_line":441,"end_character":47},"updated":"2018-05-21 01:55:19.000000000","message":"Sorry, could you help me out here, why are you assert attach_mode not in bdm and then assert the value of bdm.attach_mode?","commit_id":"d50ae83de689276701136436949a026882d0f384"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"d72902b6e9f2ce0bc94b01786259a9361598122b","unresolved":false,"context_lines":[{"line_number":437,"context_line":""},{"line_number":438,"context_line":"        bdm \u003d objects.BlockDeviceMapping("},{"line_number":439,"context_line":"            self.context, destination_type\u003d\u0027volume\u0027, instance_uuid\u003duuids.inst)"},{"line_number":440,"context_line":"        self.assertNotIn(\u0027attach_mode\u0027, bdm)"},{"line_number":441,"context_line":"        self.assertEqual(\u0027rw\u0027, bdm.attach_mode)"},{"line_number":442,"context_line":""},{"line_number":443,"context_line":"        # Make sure we don\u0027t ignore the attribute if it\u0027s set."},{"line_number":444,"context_line":"        bdm.attach_mode \u003d \u0027ro\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_7008892c","line":441,"range":{"start_line":440,"start_character":8,"end_line":441,"end_character":47},"in_reply_to":"5f7c97a3_f14ee8bf","updated":"2018-05-21 16:35:57.000000000","message":"Because bdm.attach_mode will lazy-load the default value via the obj_load_attr() method, so I want to first make sure the field isn\u0027t set before calling bdm.attach_mode to load it.","commit_id":"d50ae83de689276701136436949a026882d0f384"}]}