)]}'
{"glance/tests/functional/db/test_migrations.py":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"2194cec8acaedee57d6a2f7670925acf5faf0789","unresolved":true,"context_lines":[{"line_number":64,"context_line":"                continue"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"            # File format should be release_phaseNN_description.py"},{"line_number":67,"context_line":"            release, phasever, rest \u003d version_file.split(\u0027_\u0027, 2)"},{"line_number":68,"context_line":"            # Grab the non-numeric part of phaseNN"},{"line_number":69,"context_line":"            phase \u003d \u0027\u0027.join(x for x in phasever if x.isalpha())"},{"line_number":70,"context_line":"            if phase not in required_phases:"}],"source_content_type":"text/x-python","patch_set":1,"id":"afbf7907_d1f20a42","line":67,"range":{"start_line":67,"start_character":12,"end_line":67,"end_character":64},"updated":"2021-02-25 07:29:34.000000000","message":"To do some testing\nI renamed ussuri_expand01_empty.py to ussuriexpand01_empty.py and I got below error here;\n\n\n      File \"/opt/stack/glance/glance/tests/functional/db/test_migrations.py\", line 67, in test_phase_and_naming\n    release, phasever, rest \u003d version_file.split(\u0027_\u0027, 2)\n\n    ValueError: not enough values to unpack (expected 3, got 2)\n\nI think this needs to be caught and give proper message while failing.\n\nOther than that working as per expectation","commit_id":"610e47fa738dbabfbf0dd47b466fdf94b9d40625"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"75a919b624613dfc85de0bc40c770ef29a1fcfcc","unresolved":true,"context_lines":[{"line_number":64,"context_line":"                continue"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"            # File format should be release_phaseNN_description.py"},{"line_number":67,"context_line":"            release, phasever, rest \u003d version_file.split(\u0027_\u0027, 2)"},{"line_number":68,"context_line":"            # Grab the non-numeric part of phaseNN"},{"line_number":69,"context_line":"            phase \u003d \u0027\u0027.join(x for x in phasever if x.isalpha())"},{"line_number":70,"context_line":"            if phase not in required_phases:"}],"source_content_type":"text/x-python","patch_set":1,"id":"ac5228d8_0b5ffe66","line":67,"range":{"start_line":67,"start_character":12,"end_line":67,"end_character":64},"in_reply_to":"afbf7907_d1f20a42","updated":"2021-02-25 13:54:06.000000000","message":"Ah yep, surely meant to do catch this!","commit_id":"610e47fa738dbabfbf0dd47b466fdf94b9d40625"}]}