)]}'
{"nova/tests/unit/compute/test_compute_api.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"e3b4de20c727d973b05e5043418ce3b04d74de8c","unresolved":false,"context_lines":[{"line_number":4258,"context_line":"        # Tests that bdm validation *always* returns an attachment_id even if"},{"line_number":4259,"context_line":"        # it\u0027s None."},{"line_number":4260,"context_line":"        instance \u003d self._create_instance_obj()"},{"line_number":4261,"context_line":"        del instance.id"},{"line_number":4262,"context_line":"        instance_type \u003d self._create_flavor()"},{"line_number":4263,"context_line":"        volume_id \u003d \u0027e856840e-9f5b-4894-8bde-58c6e29ac1e8\u0027"},{"line_number":4264,"context_line":"        volume_info \u003d {\u0027status\u0027: \u0027available\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_21c68943","line":4261,"range":{"start_line":4261,"start_character":8,"end_line":4261,"end_character":23},"updated":"2018-08-02 18:40:03.000000000","message":"?","commit_id":"8ec87c950f476ace93dd77162c83a2d314f98374"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f0722cdba1701eea714d6b34f16f92b3da37ba46","unresolved":false,"context_lines":[{"line_number":4258,"context_line":"        # Tests that bdm validation *always* returns an attachment_id even if"},{"line_number":4259,"context_line":"        # it\u0027s None."},{"line_number":4260,"context_line":"        instance \u003d self._create_instance_obj()"},{"line_number":4261,"context_line":"        del instance.id"},{"line_number":4262,"context_line":"        instance_type \u003d self._create_flavor()"},{"line_number":4263,"context_line":"        volume_id \u003d \u0027e856840e-9f5b-4894-8bde-58c6e29ac1e8\u0027"},{"line_number":4264,"context_line":"        volume_info \u003d {\u0027status\u0027: \u0027available\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_fd6fdb8f","line":4261,"range":{"start_line":4261,"start_character":8,"end_line":4261,"end_character":23},"in_reply_to":"5f7c97a3_21c68943","updated":"2018-08-03 19:58:22.000000000","message":"That\u0027s for this:\n\nhttps://github.com/openstack/nova/blob/4c37ff72e5446c835a48d569dd5a1416fcd36c71/nova/compute/api.py#L3968\n\nThe tests doing this for _validate_bdm were previously getting by because they mocked out Service.get_minimum_version but I\u0027ve removed that and fixed the tests. Anything going through _validate_bdm won\u0027t be for an existing instance, so instance.id wouldn\u0027t be set yet.","commit_id":"8ec87c950f476ace93dd77162c83a2d314f98374"}]}