)]}'
{"networking_ovn/tests/functional/test_impl_idl.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"badf5e849d0b182ad7f4aec852531d6be037ee24","unresolved":false,"context_lines":[{"line_number":28,"context_line":"#                 fallback mechanism when we limit ovsdbapp version to the one"},{"line_number":29,"context_line":"#                 containing event module."},{"line_number":30,"context_line":"try:"},{"line_number":31,"context_line":"    from ovsdbapp.tests.functional import event"},{"line_number":32,"context_line":"except ImportError:"},{"line_number":33,"context_line":"    from ovsdbapp.tests.functional.schema.ovn_southbound \\"},{"line_number":34,"context_line":"        import test_impl_idl as event"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfb3d3c7_6b67b9ea","line":31,"updated":"2019-05-30 19:56:56.000000000","message":"The linked review shows the path as:\n\novsdbapp/tests/functional/schema/ovn_southbound/event.py\n\nSo this looks wrong.\n\nOr should we just set requirements \u003c\u003d0.15.0 ?","commit_id":"debfdcfb775576ae29730f04797f4c7413af2940"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"21e64eed73f1aa55b082448be9669e5c4bf2745c","unresolved":false,"context_lines":[{"line_number":28,"context_line":"#                 fallback mechanism when we limit ovsdbapp version to the one"},{"line_number":29,"context_line":"#                 containing event module."},{"line_number":30,"context_line":"try:"},{"line_number":31,"context_line":"    from ovsdbapp.tests.functional import event"},{"line_number":32,"context_line":"except ImportError:"},{"line_number":33,"context_line":"    from ovsdbapp.tests.functional.schema.ovn_southbound \\"},{"line_number":34,"context_line":"        import test_impl_idl as event"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfb3d3c7_cb16e5a8","line":31,"in_reply_to":"bfb3d3c7_6b67b9ea","updated":"2019-05-30 20:23:31.000000000","message":"you\u0027re right, I tested only with the older version. We can set it once it\u0027s released but imho this will good fine, once I use real paths. Thanks for noticing, you have sharp eyes :)\n\nI would remove this fallback once we bump the requirement for other reason than this changed import.","commit_id":"debfdcfb775576ae29730f04797f4c7413af2940"}]}